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
              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>
              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.
              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.
              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.
              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
              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

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: