ICEfaces
  1. ICEfaces
  2. ICE-9453

Clarify the recommended usage of the various Ajax Push API in ICEfaces 1.8

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: EE-1.8.2.GA_P03
    • Fix Version/s: EE-1.8.2.GA_P07
    • Component/s: Documentation
    • Labels:
      None
    • Environment:
      Ajax Push Documentation
    • Assignee Priority:
      P1
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.)

      Description

      The current 1.8 documentation on using Ajax Push APIs (http://res.icesoft.org/docs/v1_8_2/htmlguide/devguide/AdvancedTopics2.html#1075177) covers the 3 available approaches:

      * PersistentFacesState - the low-level API not generally recommended for direct use by developers (requires creation and use of their own thread)
      * RenderManager API - the initial public API
      * SessionRenderer API - the later, simpler API

      While the documentation encourages using the RenderManager and SessionManager APIs rather than the PersistentFacesState directly, it doesn't expressly declare not to use the PersistentFaceState. From a support perspective, it's desirable that we more clearly define the recommend usage and best practices.

        Activity

        Deryk Sinotte created issue -
        Ken Fyten made changes -
        Field Original Value New Value
        Assignee Deryk Sinotte [ deryk.sinotte ]
        Fix Version/s EE-1.8.2.GA_P07 [ 10672 ]
        Affects Version/s EE-1.8.2.GA_P03 [ 10251 ]
        Affects Version/s EE-1.8.2.GA_P07 [ 10672 ]
        Priority Major [ 3 ] Minor [ 4 ]
        Affects Documentation (User Guide, Ref. Guide, etc.) [ 10003 ]
        Assignee Priority P1 [ 10010 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #37042 Mon Jul 22 14:30:11 MDT 2013 deryk.sinotte ICE-9453: added API note to class and relevant methods to highlight that using PersistentFacesState directly is not the supported strategy and included a link to the online docs
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/webapp/xmlhttp/PersistentFacesState.java
        Hide
        Deryk Sinotte added a comment -

        Added API notes to the class and relevant methods explaining that it's not the supported mechanism for doing Ajax Push and link them to the online documentation and the supported API classes (SessionRenderer and RenderManager).

        Show
        Deryk Sinotte added a comment - Added API notes to the class and relevant methods explaining that it's not the supported mechanism for doing Ajax Push and link them to the online documentation and the supported API classes (SessionRenderer and RenderManager).
        Hide
        Deryk Sinotte added a comment -

        Added the change to the P07 tag as well us updating the online document to include a note that aligns with the new JavaDoc, stating that the PersistentFacesState is not the supported API for using Ajax Push.

        Show
        Deryk Sinotte added a comment - Added the change to the P07 tag as well us updating the online document to include a note that aligns with the new JavaDoc, stating that the PersistentFacesState is not the supported API for using Ajax Push.
        Deryk Sinotte made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Deryk Sinotte
            Reporter:
            Deryk Sinotte
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: