ICEfaces
  1. ICEfaces
  2. ICE-5422

Multiple instances of outputResource do not work correctly together

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.8.2-EE-GA
    • Fix Version/s: 1.8.2-EE-GA_P01, 1.8.3
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      All

      Description

      This issue was previously fixed in ICEfaces 1.8.0 in ICE-3667. Testing on ICEfaces EE 1.8.2 GA this issue seems to have risen from the grave. When clicking on an outputResource in a dataTable the last resource is always shown. Testing the same test case on ICEfaces 1.8.2 open-source does not show this issue.
      1. main.jspx
        3 kB
        Arran Mccullough
      2. TableItem.java
        3 kB
        Arran Mccullough
      3. TestBean.java
        2 kB
        Arran Mccullough

        Activity

        Arran Mccullough created issue -
        Arran Mccullough made changes -
        Field Original Value New Value
        Salesforce Case [5007000000AqfD0]
        Arran Mccullough made changes -
        Attachment Case8980Example.war [ 12223 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #20755 Tue Feb 23 14:25:22 MST 2010 adnan.durrani Fix for ICE-5422 (Multiple instances of outputResource do not work correctly together)
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/outputresource/OutputResource.java
        Adnan Durrani made changes -
        Assignee Adnan Durrani [ adnan.durrani ]
        Adnan Durrani made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        Adnan Durrani added a comment -

        Trunk:
        Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\outputresource\OutputResource.java
        Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\outputresource\OutputResource.java
        Completed: At revision: 20755

        EE branch
        Modified: D:\work\development\head\svn\ossrepo\icefaces-ee\branches\icefaces-ee-1.8.2\icefaces\component\src\com\icesoft\faces\component\outputresource\OutputResource.java
        Sending content: D:\work\development\head\svn\ossrepo\icefaces-ee\branches\icefaces-ee-1.8.2\icefaces\component\src\com\icesoft\faces\component\outputresource\OutputResource.java
        Completed: At revision: 20756

        Show
        Adnan Durrani added a comment - Trunk: Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\outputresource\OutputResource.java Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\outputresource\OutputResource.java Completed: At revision: 20755 EE branch Modified: D:\work\development\head\svn\ossrepo\icefaces-ee\branches\icefaces-ee-1.8.2\icefaces\component\src\com\icesoft\faces\component\outputresource\OutputResource.java Sending content: D:\work\development\head\svn\ossrepo\icefaces-ee\branches\icefaces-ee-1.8.2\icefaces\component\src\com\icesoft\faces\component\outputresource\OutputResource.java Completed: At revision: 20756
        Adnan Durrani made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Fix Version/s 1.8.2-EE-GA_P01 [ 10220 ]
        Fix Version/s 1.8.3 [ 10211 ]
        Resolution Fixed [ 1 ]
        Hide
        Joanne Bai added a comment -

        Use test case ICE-3667 to verify:

        • Problem is reproducible on ICEfaces-ee-1.8.2 revision 20744
        • Tested successfully on ICEfaces-ee-1.8.2 revision 20758

        Tested with Tomcat 6 + FF3.5 and IE8

        Show
        Joanne Bai added a comment - Use test case ICE-3667 to verify: Problem is reproducible on ICEfaces-ee-1.8.2 revision 20744 Tested successfully on ICEfaces-ee-1.8.2 revision 20758 Tested with Tomcat 6 + FF3.5 and IE8
        Hide
        Arran Mccullough added a comment -

        Reopening the issue due to an issue when dynamically updating the table.

        Show
        Arran Mccullough added a comment - Reopening the issue due to an issue when dynamically updating the table.
        Arran Mccullough made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Arran Mccullough made changes -
        Attachment Case8980Example.war [ 12223 ]
        Hide
        Arran Mccullough added a comment -

        Adding new test case that shows the issue when adding a new row to the table. When a new row is added all of the outputResource components use the last value.

        Example data to add:
        File Name: logo.jpg
        Mime Type: image/jpeg

        File Name: address-thumb.png
        Mime Type: image/png

        File Name: auction-thumb.png
        Mime Type: image/png

        File Name: mobility-thumb.png
        Mime Type: image/png

        File Name: timezone-thumb.png
        Mime Type: image/png

        Show
        Arran Mccullough added a comment - Adding new test case that shows the issue when adding a new row to the table. When a new row is added all of the outputResource components use the last value. Example data to add: File Name: logo.jpg Mime Type: image/jpeg File Name: address-thumb.png Mime Type: image/png File Name: auction-thumb.png Mime Type: image/png File Name: mobility-thumb.png Mime Type: image/png File Name: timezone-thumb.png Mime Type: image/png
        Arran Mccullough made changes -
        Attachment Case8980Example.war [ 12229 ]
        Arran Mccullough made changes -
        Attachment main.jspx [ 12230 ]
        Attachment TableItem.java [ 12231 ]
        Attachment TestBean.java [ 12232 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #20804 Thu Feb 25 20:19:22 MST 2010 adnan.durrani Fix for ICE-5422:Reopen (Multiple instances of outputResource do not work correctly together)
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/outputresource/OutputResource.java
        Hide
        Adnan Durrani added a comment -

        Trunk
        Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\outputresource\OutputResource.java
        Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\outputresource\OutputResource.java
        Completed: At revision: 20804

        EE
        Modified: D:\work\development\head\svn\ossrepo\icefaces-ee\branches\icefaces-ee-1.8.2\icefaces\component\src\com\icesoft\faces\component\outputresource\OutputResource.java
        Sending content: D:\work\development\head\svn\ossrepo\icefaces-ee\branches\icefaces-ee-1.8.2\icefaces\component\src\com\icesoft\faces\component\outputresource\OutputResource.java
        Completed: At revision: 20805

        Show
        Adnan Durrani added a comment - Trunk Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\outputresource\OutputResource.java Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\outputresource\OutputResource.java Completed: At revision: 20804 EE Modified: D:\work\development\head\svn\ossrepo\icefaces-ee\branches\icefaces-ee-1.8.2\icefaces\component\src\com\icesoft\faces\component\outputresource\OutputResource.java Sending content: D:\work\development\head\svn\ossrepo\icefaces-ee\branches\icefaces-ee-1.8.2\icefaces\component\src\com\icesoft\faces\component\outputresource\OutputResource.java Completed: At revision: 20805
        Adnan Durrani made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Joanne Bai made changes -
        Assignee Adnan Durrani [ adnan.durrani ] Joanne Bai [ joanne_bai ]
        Arran Mccullough made changes -
        Attachment Case8980Example.war [ 12239 ]
        Hide
        Joanne Bai added a comment -

        Thanks, Arran!

        Verified with success on ICEfaces-ee-1.8.2 revision 20818 using the attached example.
        Verified on Tomcat 6 + IE8 and FF3.5

        Test app has been checked into repo\qa\trunk\Regression\ICE-5422

        Show
        Joanne Bai added a comment - Thanks, Arran! Verified with success on ICEfaces-ee-1.8.2 revision 20818 using the attached example. Verified on Tomcat 6 + IE8 and FF3.5 Test app has been checked into repo\qa\trunk\Regression\ ICE-5422
        Joanne Bai made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Joanne Bai
            Reporter:
            Arran Mccullough
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: