ICEfaces
  1. ICEfaces
  2. ICE-5276

compat showcase get NavigationBean working in Window scope of JSF2

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 2.0-Alpha2
    • Fix Version/s: 2.0.0
    • Component/s: Sample Apps
    • Labels:
      None
    • Environment:
      jsf2.0, ICEfaces2.0 compat
    • Affects:
      Sample App./Tutorial

      Description

      currenlty NavigationBean is in session scope. When changing to View scope, any submit on the example of components (facelet includes) causes view to go to startup view

        Activity

        Judy Guglielmin created issue -
        Hide
        Judy Guglielmin added a comment -

        improve application by using view scope for navigation within the panels/application

        Show
        Judy Guglielmin added a comment - improve application by using view scope for navigation within the panels/application
        Judy Guglielmin made changes -
        Field Original Value New Value
        Link This issue blocks ICE-4458 [ ICE-4458 ]
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s 2.0-Beta [ 10231 ]
        Hide
        Ken Fyten added a comment -

        Let's use Window scope instead.

        Show
        Ken Fyten added a comment - Let's use Window scope instead.
        Ken Fyten made changes -
        Salesforce Case []
        Affects [Sample App./Tutorial]
        Security Private [ 10001 ]
        Assignee Judy Guglielmin [ judy.guglielmin ]
        Ken Fyten made changes -
        Summary compat showcase get NavigationBean working in view scope of jsf2 compat showcase get NavigationBean working in Window scope of JSF2
        Salesforce Case []
        Ken Fyten made changes -
        Fix Version/s 2.0-Beta2 [ 10242 ]
        Fix Version/s 2.0-Beta1 [ 10231 ]
        Hide
        Judy Guglielmin added a comment -

        Tried Windows Scope by annotating the NavigationBean with the following:-

        @ManagedBean(name = "WindowScopedCounter")
        @CustomScoped(value = "#

        {window}")

        see the attached screenshot to see that the content-panel is not included at all. If clicking on the collapsible-panels that contain the sub-menus you get the following:-
        @ManagedBean(name = "WindowScopedCounter")
        @CustomScoped(value = "#{window}

        ")

        note that the beans that are in Session scope (style and language) are there and working properly. The view-scoped backing beans are not.
        Will try placing the content backing beans in windows scope as well (they are all currently in view scope).

        Show
        Judy Guglielmin added a comment - Tried Windows Scope by annotating the NavigationBean with the following:- @ManagedBean(name = "WindowScopedCounter") @CustomScoped(value = "# {window}") see the attached screenshot to see that the content-panel is not included at all. If clicking on the collapsible-panels that contain the sub-menus you get the following:- @ManagedBean(name = "WindowScopedCounter") @CustomScoped(value = "#{window} ") note that the beans that are in Session scope (style and language) are there and working properly. The view-scoped backing beans are not. Will try placing the content backing beans in windows scope as well (they are all currently in view scope).
        Hide
        Judy Guglielmin added a comment -

        when NavigationBean is in Windows Scope

        Show
        Judy Guglielmin added a comment - when NavigationBean is in Windows Scope
        Judy Guglielmin made changes -
        Attachment screenshot-1.jpg [ 12455 ]
        Hide
        Judy Guglielmin added a comment -

        when in windows scope (see attached screenshot) and click on any of the collapsible menu's you get the following;_
        Jul 19, 2010 3:31:44 PM com.sun.faces.renderkit.RenderKitUtils renderUnhandledMessages
        INFO: WARNING: FacesMessage(s) have been enqueued, but may not have been displayed.
        sourceId=navigationMenuForm:extendedCompClpsbl[severity=(ERROR 2), summary=(javax.el.PropertyNotFoundException: /WEB-INF/includes/content/navigation.xhtml @31,114 expanded="#

        {navigation.extendedRender}": Target Unreachable, identifier 'navigation' resolved to null), detail=(javax.el.PropertyNotFoundException: /WEB-INF/includes/content/navigation.xhtml @31,114 expanded="#{navigation.extendedRender}

        ": Target Unreachable, identifier 'navigation' resolved to null)]

        (basically the el resolving "navigation" named bean resolves to 'null')

        Show
        Judy Guglielmin added a comment - when in windows scope (see attached screenshot) and click on any of the collapsible menu's you get the following;_ Jul 19, 2010 3:31:44 PM com.sun.faces.renderkit.RenderKitUtils renderUnhandledMessages INFO: WARNING: FacesMessage(s) have been enqueued, but may not have been displayed. sourceId=navigationMenuForm:extendedCompClpsbl[severity=(ERROR 2), summary=(javax.el.PropertyNotFoundException: /WEB-INF/includes/content/navigation.xhtml @31,114 expanded="# {navigation.extendedRender}": Target Unreachable, identifier 'navigation' resolved to null), detail=(javax.el.PropertyNotFoundException: /WEB-INF/includes/content/navigation.xhtml @31,114 expanded="#{navigation.extendedRender} ": Target Unreachable, identifier 'navigation' resolved to null)] (basically the el resolving "navigation" named bean resolves to 'null')
        Judy Guglielmin made changes -
        Salesforce Case []
        Hide
        Judy Guglielmin added a comment -

        Ted said to assign to you. Not sure when Windows scope is supposed to resolve the el in conjunction with facelets composition .

        Show
        Judy Guglielmin added a comment - Ted said to assign to you. Not sure when Windows scope is supposed to resolve the el in conjunction with facelets composition .
        Judy Guglielmin made changes -
        Assignee Judy Guglielmin [ judy.guglielmin ] Ken Fyten [ ken.fyten ]
        Hide
        Ted Goddard added a comment -

        Please indicate if there is any functional impact.

        Show
        Ted Goddard added a comment - Please indicate if there is any functional impact.
        Ken Fyten made changes -
        Assignee Ken Fyten [ ken.fyten ] Judy Guglielmin [ judy.guglielmin ]
        Hide
        Ken Fyten added a comment -

        This requires the ui:include fix that's included in Mojarra JSF 2.1.

        Show
        Ken Fyten added a comment - This requires the ui:include fix that's included in Mojarra JSF 2.1.
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s 2.0.0 [ 10230 ]
        Fix Version/s 2.0-Beta2 [ 10242 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #22686 Tue Oct 19 10:28:32 MDT 2010 judy.guglielmin ICE-5589 update tlddoc properties, ICE-5276 update tlddoc properties. Sample xhtml for tes app for pushbuttonSam was missing
        Files Changed
        Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/pushbutton/PushButtonMeta.java
        Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/checkboxbutton/CheckboxButtonMeta.java
        Commit graph ADD /icefaces/scratchpads/sparkle/showcase/test/web/buttons/pushbuttonSam.xhtml
        Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/pushbutton/PushButtonRenderer.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #22694 Tue Oct 19 23:52:49 MDT 2010 judy.guglielmin ICE-5589 update tlddoc properties, ICE-5276 update tlddoc properties. Use jsf value for yui label in pushbutton and jsf label for yui arialabel. fix checkboxButton rime for labelPosition="left".
        Files Changed
        Commit graph DEL /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/pushbutton/skins/rime/button-skin.css
        Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/pushbutton/PushButtonMeta.java
        Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/checkboxbutton/CheckboxButtonMeta.java
        Commit graph MODIFY /icefaces/scratchpads/sparkle/component/resources/org.icefaces.component.pushbutton/pushbutton.js
        Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/web/buttons/pushbuttonSam.xhtml
        Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/pushbutton/skins/rime/button.css
        Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/pushbutton/PushButtonRenderer.java
        Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/checkboxbutton/CheckboxButtonRenderer.java
        Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/checkboxbutton/skins/rime/checkboxButton.css
        Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/web/buttons/pushbuttonRime.xhtml
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #22698 Wed Oct 20 11:23:44 MDT 2010 judy.guglielmin ICE-5589 update tlddoc properties, ICE-5276 update tlddoc properties. finish standard style and styleClass support for both buttons. Test more on value vs label on pushbutton.
        Files Changed
        Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/checkboxbutton/CheckboxButtonMeta.java
        Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/pushbutton/skins/rime/bullet.gif
        Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/web/buttons/pushbuttonSam.xhtml
        Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/pushbutton/PushButtonRenderer.java
        Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/checkboxbutton/CheckboxButtonRenderer.java
        Commit graph ADD /icefaces/scratchpads/sparkle/showcase/test/web/buttons/immediateTest.xhtml
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #22700 Wed Oct 20 13:08:48 MDT 2010 judy.guglielmin ICE-5589, ICE-5276 remove System.out statements
        Files Changed
        Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/pushbutton/PushButtonRenderer.java
        Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/checkboxbutton/CheckboxButtonRenderer.java
        Ken Fyten made changes -
        Link This issue blocks ICE-4458 [ ICE-4458 ]
        Ken Fyten made changes -
        Assignee Priority P2
        Hide
        Ken Fyten added a comment -

        Since we are demonstrating Window scope with the ace-showcase and possibly future samples, and we are currently blocked from adopting it for compat-showcase due to the JSF issue (until JSF 2.1, anyway), we won't bother changing the scope being used in compat-showcase. It is currently using Session scope.

        Show
        Ken Fyten added a comment - Since we are demonstrating Window scope with the ace-showcase and possibly future samples, and we are currently blocked from adopting it for compat-showcase due to the JSF issue (until JSF 2.1, anyway), we won't bother changing the scope being used in compat-showcase. It is currently using Session scope.
        Ken Fyten made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Priority P2
        Resolution Won't Fix [ 2 ]
        Assignee Judy Guglielmin [ judy.guglielmin ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Judy Guglielmin
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: