ICEfaces
  1. ICEfaces
  2. ICE-7938

Resource requests generated Warning in Glassfish 3.1.2 log

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Minor Minor
          • Resolution: Fixed
          • Affects Version/s: 3.0.1
          • Fix Version/s: 3.2
          • Component/s: ACE-Components
          • Labels:
            None
          • Environment:
            Glassfish 3.1.2, ICEfaces 3.0.1

            Description

            When running ICEfaces 3.0.1 on Glassfish 3.1.2, resource requests generate the following Warning log in the Glassfish log:

            [#|2012-03-21T14:03:29.809-0600|WARNING|glassfish3.1.2|org.apache.catalina.connector.Request|_ThreadID=22;_ThreadName=http-thread-pool-8080(1);|PWC4011: Unable to set request character encoding to UTF-8 from context /showcase, because request parameters have already been read, or ServletRequest.getReader() has already been called|#]

            There is no functional impact from this.

              Activity

              Ascending order - Click to sort in descending order
              Ken Fyten created issue -
              Ken Fyten made changes -
              Field Original Value New Value
              Salesforce Case []
              Assignee Priority P2
              Assignee Arturo Zambrano [ artzambrano ]
              Priority Major [ 3 ] Minor [ 4 ]
              Hide
              Permalink
              Ted Goddard added a comment -

              Can be tested via the following change:

              +++ ace/component/resources/icefaces.ace/META-INCLUDE/faces-config.xml (working copy)
              @@ -21,11 +21,6 @@
              <application>
              <resource-handler>org.icefaces.ace.component.fileentry.FileEntryResourceHandler</resource-handler>
              </application>

              • <factory>
              • <partial-view-context-factory>
              • org.icefaces.ace.context.AutoUpdateCallbackPartialViewContextFactory
              • </partial-view-context-factory>
              • </factory>
                <render-kit>
                <renderer>
                <component-family>javax.faces.Output</component-family>
              Show
              Ted Goddard added a comment - Can be tested via the following change: +++ ace/component/resources/icefaces.ace/META-INCLUDE/faces-config.xml (working copy) @@ -21,11 +21,6 @@ <application> <resource-handler>org.icefaces.ace.component.fileentry.FileEntryResourceHandler</resource-handler> </application> <factory> <partial-view-context-factory> org.icefaces.ace.context.AutoUpdateCallbackPartialViewContextFactory </partial-view-context-factory> </factory> <render-kit> <renderer> <component-family>javax.faces.Output</component-family>
              Ken Fyten made changes -
              Salesforce Case []
              Fix Version/s 3.1.0.RC1 [ 10337 ]
              Assignee Arturo Zambrano [ artzambrano ] Jack van Ooststroom [ jack.van.ooststroom ]
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              I confirmed that the trunk of ICEfaces 3 still exhibits the issue. I also confirmed that with Ted's suggested change the issue seems to go away. We should look into the possibility of removing the AutoUpdateCallbackPartialViewContext class.

              Show
              Jack Van Ooststroom added a comment - I confirmed that the trunk of ICEfaces 3 still exhibits the issue. I also confirmed that with Ted's suggested change the issue seems to go away. We should look into the possibility of removing the AutoUpdateCallbackPartialViewContext class.
              Ken Fyten made changes -
              Salesforce Case []
              Fix Version/s 3.1.0.RC1 [ 10337 ]
              Assignee Priority P2 P1
              Assignee Jack van Ooststroom [ jack.van.ooststroom ] Nils Lundquist [ nils.lundquist ]
              Hide
              Permalink
              Nils Lundquist added a comment -

              Components still using customUpdate:

              DataTable - lazyScrolling
              Accordion - Load Dynamic Tab

              Show
              Nils Lundquist added a comment - Components still using customUpdate: DataTable - lazyScrolling Accordion - Load Dynamic Tab
              Hide
              Permalink
              Nils Lundquist added a comment -

              Note I've thought about removing customUpdate from lazy scrolling in the months since we've removed it from everywhere else. lazyScrolling without customUpdate will require a major core enhancement to allow the insertion of dom elements without updating the container.

              Show
              Nils Lundquist added a comment - Note I've thought about removing customUpdate from lazy scrolling in the months since we've removed it from everywhere else. lazyScrolling without customUpdate will require a major core enhancement to allow the insertion of dom elements without updating the container.
              Ken Fyten made changes -
              Fix Version/s 3.2 [ 10338 ]
              Fix Version/s 3.1 [ 10312 ]
              Ken Fyten made changes -
              Salesforce Case []
              Assignee Priority P1
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #29988 Mon Jul 16 11:25:15 MDT 2012 nils.lundquist ICE-7938 - Altered liveScroll TLD to represent its deprecated status.
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datatable/DataTableMeta.java
              Hide
              Permalink
              Nils Lundquist added a comment -

              Revision #29988
              Committed by nils.lundquist
              Moments ago
              ICE-7938 - Altered liveScroll TLD to represent its deprecated status.

              Show
              Nils Lundquist added a comment - Revision #29988 Committed by nils.lundquist Moments ago ICE-7938 - Altered liveScroll TLD to represent its deprecated status.
              Nils Lundquist made changes -
              Salesforce Case []
              Assignee Nils Lundquist [ nils.lundquist ]
              Hide
              Permalink
              Arturo Zambrano added a comment - - edited

              ace:accordion no longer requires the use of the ice.customUpdate parameter after improvements made as part of ICE-7681.
              http://jira.icesoft.org/browse/ICE-7681?focusedCommentId=42327&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-42327

              Show
              Arturo Zambrano added a comment - - edited ace:accordion no longer requires the use of the ice.customUpdate parameter after improvements made as part of ICE-7681 . http://jira.icesoft.org/browse/ICE-7681?focusedCommentId=42327&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-42327
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #31290 Wed Oct 03 18:10:08 MDT 2012 yip.ng ICE-7938: Resource requests generated Warning in Glassfish 3.1.2 log.
              Files Changed
              Commit graph DEL /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/context/AutoUpdateCallbackPartialViewContext.java
              Commit graph DEL /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/context/AutoUpdateCallbackPartialViewContextFactory.java
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/META-INCLUDE/faces-config.xml
              Hide
              Permalink
              yip.ng added a comment - - edited

              Class removed. faces-config updated. Tested with Glassfish 3.1.2.2. See screenshot-01.png.
              Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\resources\icefaces.ace\META-INCLUDE\faces-config.xml
              Deleting: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\context\AutoUpdateCallbackPartialViewContext.java
              Deleting: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\context\AutoUpdateCallbackPartialViewContextFactory.java
              Completed: At revision: 31290

              Show
              yip.ng added a comment - - edited Class removed. faces-config updated. Tested with Glassfish 3.1.2.2. See screenshot-01.png. Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\resources\icefaces.ace\META-INCLUDE\faces-config.xml Deleting: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\context\AutoUpdateCallbackPartialViewContext.java Deleting: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\context\AutoUpdateCallbackPartialViewContextFactory.java Completed: At revision: 31290
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #31338 Fri Oct 05 16:14:05 MDT 2012 nils.lundquist ICE-8631 / ICE-7938 - Restoring AutoUpdateCallbackPartialViewContext but removed manual check of request parameter map for javax.faces.partial.ajax key , preventing warnings in Glassfish 3.1.2 log.
              Files Changed
              Commit graph ADD /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/context/AutoUpdateCallbackPartialViewContext.java
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/context/CallbackPartialResponseWriter.java
              Commit graph ADD /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/context/AutoUpdateCallbackPartialViewContextFactory.java
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/META-INCLUDE/faces-config.xml
              Migration made changes -
              Assignee yip.ng [ yip.ng ]
              Assignee Priority P2 [ 10011 ]
              Migration made changes -
              Attachment screenshot-01.png [ 14891 ]
              Migration made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]
              Assignee Priority P2 [ 10011 ]

                People

                • Assignee:
                  yip.ng
                  Reporter:
                  Ken Fyten
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: