ICEfaces
  1. ICEfaces
  2. ICE-5275

compat: showcase rowSelector demo is not working under IE

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-Alpha1
    • Fix Version/s: 2.0-Alpha2, 2.0.0
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      rowSelector

      Description

      compat: showcase rowSelector demo is not working under IE

        Issue Links

          Activity

          Adnan Durrani created issue -
          Adnan Durrani made changes -
          Field Original Value New Value
          Assignee Adnan Durrani [ adnan.durrani ]
          Ken Fyten made changes -
          Salesforce Case []
          Fix Version/s 2.0-Alpha2 [ 10214 ]
          Assignee Priority P1
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #20256 Fri Jan 15 10:49:03 MST 2010 adnan.durrani Fix for ICE-5275 (compat: showcase rowSelector demo is not working under IE)
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/core/src/main/resources/META-INF/resources/ice-extras.js
          Hide
          Judy Guglielmin added a comment -

          required for ICEfaces-2.0 A2 release

          Show
          Judy Guglielmin added a comment - required for ICEfaces-2.0 A2 release
          Judy Guglielmin made changes -
          Link This issue blocks ICE-4458 [ ICE-4458 ]
          Adnan Durrani made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          Hide
          Adnan Durrani added a comment -

          fixed.

          Modified: D:\work\development\head\svn\ossrepo\icefaces\scratchpads\glimmer\compat\core\src\main\resources\META-INF\resources\ice-extras.js
          Sending content: D:\work\development\head\svn\ossrepo\icefaces\scratchpads\glimmer\compat\core\src\main\resources\META-INF\resources\ice-extras.js
          Completed: At revision: 20256

          Show
          Adnan Durrani added a comment - fixed. Modified: D:\work\development\head\svn\ossrepo\icefaces\scratchpads\glimmer\compat\core\src\main\resources\META-INF\resources\ice-extras.js Sending content: D:\work\development\head\svn\ossrepo\icefaces\scratchpads\glimmer\compat\core\src\main\resources\META-INF\resources\ice-extras.js Completed: At revision: 20256
          Adnan Durrani made changes -
          Status In Progress [ 3 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          Adnan Durrani added a comment -

          The problem was related to an empty id of event target. I looked at the history and found out that the id was removed intentionally to fix the following JIRA:
          http://jira.icefaces.org/browse/ICE-2874

          In addition to that a null form was being passed to the iceSubmitPartial() call, where resetHiddenFieldsFor(form); was throwing an exception due to the null form.

          To fix this we are setting an id on the target element before submit and to preserve the fix of ICE-2874 we are resetting it back to empty string. As well as we are passing a valid form element to the iceSubmitPartial() call.

          Show
          Adnan Durrani added a comment - The problem was related to an empty id of event target. I looked at the history and found out that the id was removed intentionally to fix the following JIRA: http://jira.icefaces.org/browse/ICE-2874 In addition to that a null form was being passed to the iceSubmitPartial() call, where resetHiddenFieldsFor(form); was throwing an exception due to the null form. To fix this we are setting an id on the target element before submit and to preserve the fix of ICE-2874 we are resetting it back to empty string. As well as we are passing a valid form element to the iceSubmitPartial() call.
          Mandeep Hayher made changes -
          Assignee Adnan Durrani [ adnan.durrani ] Mandeep Hayher [ mandeep.hayher ]
          Hide
          Mandeep Hayher added a comment -

          Tested successfully on ICEfaces 2.0.0-Alpha2 pre-release bundle using IE7 & IE6 browser.

          Show
          Mandeep Hayher added a comment - Tested successfully on ICEfaces 2.0.0-Alpha2 pre-release bundle using IE7 & IE6 browser.
          Ken Fyten made changes -
          Fix Version/s 2.0.0 [ 10230 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Assignee Priority P1

            People

            • Assignee:
              Mandeep Hayher
              Reporter:
              Adnan Durrani
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: