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

          Yannick Majoros created issue -
          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.
          Mark Collette made changes -
          Field Original Value New Value
          Link This issue depends on ICE-5881 [ ICE-5881 ]
          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.
          Mark Collette made changes -
          Salesforce Case []
          Component/s Components [ 10012 ]
          Fix Version/s 2.0-Beta2 [ 10242 ]
          Affects Version/s 2.0-Beta1 [ 10231 ]
          Assignee Mark Collette [ mark.collette ]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #22160 Wed Aug 11 14:12:45 MDT 2010 mark.collette ICE-5835 : panelConfirmation not working
          Files Changed
          Commit graph MODIFY /icefaces2/trunk/icefaces/compat/core/src/main/javascript/extras/extras.js
          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
          Mark Collette made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Ken Fyten made changes -
          Fix Version/s 2.0.0 [ 10230 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

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

              Dates

              • Created:
                Updated:
                Resolved: