ICEfaces
  1. ICEfaces
  2. ICE-5835

panelConfirmation not working (patch)

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-Beta1
    • Fix Version/s: 2.0-Beta2, 2.0.0
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      Icefaces 2.0 alpha 3 on Glassfish 2.1
      Firefox 3.6.3

      Description

      In a JSF 2 project on Glassfish 2.1 (which works fine using class-loader="false" and useMyFaces property set to true), panelConfirmation pops up as expected, but original action is never triggered when clicking on accept button.

      I had this error on firefox error console:

      Ice.Focus.setFocus is undefined
      localhost:8080/Myapp/faces/javax.faces.resource/icefaces-compat.js
      (line 13166)

      I commented the line, as Ice.Focus doesn't seem to be a defined class:
              //Ice.Focus.setFocus(this.srcComp.id);

      ... and it now works fine

      There is another use of Ice.Focus in same file, I guess it should be removed to, but I don't have an existing example to test that.

      Here is the code of my page:

                              <ice:panelConfirmation id="deleteConfirmation" message="Are you sure?" autoCentre="true" acceptLabel="Delete" cancelLabel="Cancel" />
                              <ice:commandLink value="Delete" action="#{appController.actionDelete(app)}" panelConfirmation="deleteConfirmation" />

        Issue Links

          Activity

          Hide
          Mark Collette added a comment -

          ICE-5881 will provide the new API for panelConfirmation to use, for the complete fix.

          Show
          Mark Collette added a comment - ICE-5881 will provide the new API for panelConfirmation to use, for the complete fix.
          Hide
          Mark Collette added a comment -

          Temporarily, until the blocker task is completed, we should do as the issue reporter suggests, and comment out the offending code.

          Show
          Mark Collette added a comment - Temporarily, until the blocker task is completed, we should do as the issue reporter suggests, and comment out the offending code.
          Hide
          Mark Collette added a comment -

          Duplicated the issue, and then found all uses of Ice.Focus.setFocus and added code to see if that function exists before invoking it.

          Ran into some problems after getting the newest code, for running the compat component-showcase, but cleared that up.

          Subversion 22160

          Show
          Mark Collette added a comment - Duplicated the issue, and then found all uses of Ice.Focus.setFocus and added code to see if that function exists before invoking it. Ran into some problems after getting the newest code, for running the compat component-showcase, but cleared that up. Subversion 22160

            People

            • Assignee:
              Mark Collette
              Reporter:
              Yannick Majoros
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: