ICEfaces
  1. ICEfaces
  2. ICE-7317

ace:dataExporter inside ui:Repeat issue

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 2.1-Beta
          • Fix Version/s: 3.0
          • Component/s: ACE-Components
          • Labels:
            None
          • Environment:
            Grimlock code rev# 25870, all major browsers (IE8, IE7, Firefox 6, Firefox 3.6, Gchrome14), tomcat6 server.
          • Assignee Priority:
            P1

            Description

            The ace:dataExporter does not function when it is placed inside an ui:Repeat.
            The "target" attribute of the ace:dataExporter is being used to identify the ace:dataTable containing the data to be exported, and this table is within the same form, but outside the ui:Repeat containing the dataExporter.
            When trying to export data, a Server Internal Error happens (screen shots attached), and the below error is thrown in the server log.

            4-Oct-2011 4:13:01 PM com.sun.faces.lifecycle.InvokeApplicationPhase execute
            WARNING: Cannot find component "carTable" in view.
            javax.faces.FacesException: Cannot find component "carTable" in view.
                    at org.icefaces.ace.component.export.DataExporter.processAction(DataExpo
            rter.java:104)
                    at javax.faces.event.ActionEvent.processListener(ActionEvent.java:88)
                    at javax.faces.component.UIComponentBase.broadcast(UIComponentBase.java:
            769)
                    at javax.faces.component.UICommand.broadcast(UICommand.java:300)
                    at javax.faces.component.UIData.broadcast(UIData.java:1093)
                    at javax.faces.component.UIViewRoot.broadcastEvents(UIViewRoot.java:794)

                    at javax.faces.component.UIViewRoot.processApplication(UIViewRoot.java:1
            259)
                    at com.sun.faces.lifecycle.InvokeApplicationPhase.execute(InvokeApplicat
            ionPhase.java:81)
                    at com.sun.faces.lifecycle.Phase.doPhase(Phase.java:101)
                    at com.sun.faces.lifecycle.LifecycleImpl.execute(LifecycleImpl.java:118)

                    at javax.faces.webapp.FacesServlet.service(FacesServlet.java:593)
                    at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(Appl
            icationFilterChain.java:290)
                    at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationF
            ilterChain.java:206)
            1. dataExporter-in-UI-Repeat.png
              105 kB
            2. IE7.png
              71 kB
            3. IE8.png
              216 kB

              Issue Links

              depends on

              Improvement - An improvement or enhancement to an existing feature or task. ICE-7381 ace:dataExporter, avoid serving file by hijacking the faces response

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Carmen Cristurean created issue -
                Carmen Cristurean made changes -
                Field Original Value New Value
                Attachment dataExporter-in-UI-Repeat.png [ 13628 ]
                Hide
                Permalink
                Carmen Cristurean added a comment -

                To reproduce the issue check out test app from:
                http://server.ice:8888/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/dataExporter

                load the page: http://localhost:8080/dataExporter/dataExporterRepeat.jsf

                and click on any of the 'Export File' buttons.

                Show
                Carmen Cristurean added a comment - To reproduce the issue check out test app from: http://server.ice:8888/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/dataExporter load the page: http://localhost:8080/dataExporter/dataExporterRepeat.jsf and click on any of the 'Export File' buttons.
                Ken Fyten made changes -
                Salesforce Case []
                Fix Version/s 2.1 [ 10241 ]
                Assignee Priority P2
                Assignee Mark Collette [ mark.collette ]
                Ken Fyten made changes -
                Salesforce Case []
                Assignee Mark Collette [ mark.collette ] Arturo Zambrano [ artzambrano ]
                Hide
                Permalink
                Arturo Zambrano added a comment -

                I managed to make the data exporter find the table component and sucessfully export the table. However, there's a problem with the file delivery. It's something similar to what we've seen before with the client expecting a JSF response and complaining that the response contains no XML, or if the exported file is in XML format, it complains that the response body cannot be parsed.

                Show
                Arturo Zambrano added a comment - I managed to make the data exporter find the table component and sucessfully export the table. However, there's a problem with the file delivery. It's something similar to what we've seen before with the client expecting a JSF response and complaining that the response contains no XML, or if the exported file is in XML format, it complains that the response body cannot be parsed.
                Ken Fyten made changes -
                Salesforce Case []
                Affects Version/s 2.1-Beta [ 10291 ]
                Affects Version/s 2.1 [ 10241 ]
                Arturo Zambrano made changes -
                Link This issue depends on ICE-7381 [ ICE-7381 ]
                Hide
                Permalink
                Arturo Zambrano added a comment -

                This issue was fixed by changes committed as part of ICE-7381.

                Show
                Arturo Zambrano added a comment - This issue was fixed by changes committed as part of ICE-7381 .
                Arturo Zambrano made changes -
                Status Open [ 1 ] Resolved [ 5 ]
                Resolution Fixed [ 1 ]
                Ken Fyten made changes -
                Fix Version/s 2.1-Beta2 [ 10294 ]
                Ken Fyten made changes -
                Security Private [ 10001 ]
                Repository Revision Date User Message
                ICEsoft Public SVN Repository #26844 Tue Dec 13 08:58:40 MST 2011 art.zambrano ICE-7545 fixed by using two different methods for finding the table; made sure this doesn't break ICE-7317
                Files Changed
                Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/dataexporter/DataExporter.java
                Hide
                Permalink
                Carmen Cristurean added a comment -

                Fix has been tested with icefaces3 rev 27114, in all major browsers.

                Note: a JS error occurs in IE8 and a Runtime error in IE7, when exporting file, then changing export type to any other type and exporting again.

                Message: 'ice.ace.DataExporters.panelForm:panelRepeat:0:dataExporter4' is null or not an object
                Line: 36569817
                Char: 296
                Code: 0
                URI: http://localhost:8080/dataExporter/dataExporterRepeat.jsf

                Show
                Carmen Cristurean added a comment - Fix has been tested with icefaces3 rev 27114, in all major browsers. Note: a JS error occurs in IE8 and a Runtime error in IE7, when exporting file, then changing export type to any other type and exporting again. Message: 'ice.ace.DataExporters.panelForm:panelRepeat:0:dataExporter4' is null or not an object Line: 36569817 Char: 296 Code: 0 URI: http://localhost:8080/dataExporter/dataExporterRepeat.jsf
                Ken Fyten made changes -
                Resolution Fixed [ 1 ]
                Status Resolved [ 5 ] Reopened [ 4 ]
                Assignee Priority P2 P1
                Hide
                Permalink
                Carmen Cristurean added a comment -

                Attached screen shots for the recent errors.

                Show
                Carmen Cristurean added a comment - Attached screen shots for the recent errors.
                Carmen Cristurean made changes -
                Attachment IE7.png [ 13955 ]
                Attachment IE8.png [ 13956 ]
                Repository Revision Date User Message
                ICEsoft Public SVN Repository #27125 Tue Jan 10 08:39:22 MST 2012 art.zambrano ICE-7640 fixed problem by only showing one popup window if the export button is clicked multiple times; simplified popup messages; this fix also solves error mentioned in ICE-7317
                Files Changed
                Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/dataexporter/dataexporter.js
                Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/dataexporter/DataExporterRenderer.java
                Ken Fyten made changes -
                Salesforce Case []
                Fix Version/s 2.1-Beta2 [ 10294 ]
                Hide
                Permalink
                Arturo Zambrano added a comment -

                Fixed at revision 27125 with a fix related to ICE-7640.

                Show
                Arturo Zambrano added a comment - Fixed at revision 27125 with a fix related to ICE-7640.
                Arturo Zambrano made changes -
                Status Reopened [ 4 ] Resolved [ 5 ]
                Resolution Fixed [ 1 ]
                Hide
                Permalink
                Carmen Cristurean added a comment -

                Verified fix with code revision 27126, in IE7 (icepc13), IE8, FF3.6, Chrome15.

                Show
                Carmen Cristurean added a comment - Verified fix with code revision 27126, in IE7 (icepc13), IE8, FF3.6, Chrome15.
                Ken Fyten made changes -
                Status Resolved [ 5 ] Closed [ 6 ]

                  People

                  • Assignee:
                    Arturo Zambrano
                    Reporter:
                    Carmen Cristurean
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    0 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: