ICEfaces
  1. ICEfaces
  2. ICE-10710

Showcase - Setting mandatoryResourceConfiguration does not pre-load resources

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-3.3.0.GA_P03
    • Fix Version/s: 4.1, EE-3.3.0.GA_P04
    • Component/s: Framework, Sample Apps
    • Labels:
      None
    • Environment:
      ICEfaces EE 3.3

      Description

      When setting the mandatoryResourceConfiguration context param in the Showcase demo web.xml, the resource tied to the component is not pre-loaded on the page.

      For example when using the following:

      <context-param>
              <param-name>org.icefaces.mandatoryResourceConfiguration</param-name>
              <param-value>fileEntry</param-value>
          </context-param>

      The fileEntry.js.jsf resource is not in the head tag.

        Activity

        Arran Mccullough created issue -
        Ken Fyten made changes -
        Field Original Value New Value
        Fix Version/s EE-3.3.0.GA_P04 [ 12270 ]
        Environment All ICEfaces EE 3.3
        Ken Fyten made changes -
        Assignee Mircea Toma [ mircea.toma ]
        Assignee Priority P1 [ 10010 ]
        Component/s Framework [ 10013 ]
        Hide
        Mircea Toma added a comment -

        It seems that the _ mandatoryResourceConfiguration_ defined in icecore:config will override what's defined in the web.xml. In the case of showcase application the dataTable is configured as mandatory resource in the main template which will block any mandatory resource defined in the context parameter.

        Show
        Mircea Toma added a comment - It seems that the _ mandatoryResourceConfiguration_ defined in icecore:config will override what's defined in the web.xml . In the case of showcase application the dataTable is configured as mandatory resource in the main template which will block any mandatory resource defined in the context parameter.
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #45009 Wed Jun 10 17:44:18 MDT 2015 mircea.toma ICE-10710 Modified EnvUtils.getMandatoryResourceConfig method to join the sets of mandatory resources defined by the context parameter and icecore:config tag.
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/core/src/main/java/org/icefaces/util/EnvUtils.java
        Hide
        Mircea Toma added a comment -

        Modified EnvUtils.getMandatoryResourceConfig method to join the sets of mandatory resources defined by the context parameter and icecore:config tag.

        Show
        Mircea Toma added a comment - Modified EnvUtils.getMandatoryResourceConfig method to join the sets of mandatory resources defined by the context parameter and icecore:config tag.
        Mircea Toma made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Mircea Toma made changes -
        Fix Version/s 4.1 [ 11375 ]
        Hide
        Carmen Cristurean added a comment -

        Verified on ICEfaces4 Trunk and 3.3 maintenance branch revision 45009 in IE11, Chrome43, Firefox34.

        Show
        Carmen Cristurean added a comment - Verified on ICEfaces4 Trunk and 3.3 maintenance branch revision 45009 in IE11, Chrome43, Firefox34.
        Hide
        Carmen Cristurean added a comment -

        Added QA test application:
        http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Manual/ICE-10710 (for ICEfaces4 trunk)
        http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Manual/ICE-10710 (for 3.3 maintenance branch)

        Testing on the 3.3 maintenance branch fails when using an <icecore:config mandatoryResource="..."> tag on the page, as the corresponding resources are not loaded (all browsers).
        This can be reproduced with the QA test app for 3.3 maintenance branch, on the /test.jsf demo page.

        Show
        Carmen Cristurean added a comment - Added QA test application: http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Manual/ICE-10710 (for ICEfaces4 trunk) http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Manual/ICE-10710 (for 3.3 maintenance branch) Testing on the 3.3 maintenance branch fails when using an <icecore:config mandatoryResource="..."> tag on the page, as the corresponding resources are not loaded (all browsers). This can be reproduced with the QA test app for 3.3 maintenance branch, on the /test.jsf demo page.
        Ken Fyten made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Hide
        Mircea Toma added a comment - - edited

        Tested with success the 3.3 maintenance branch. Both ace:slideEntry and ace:dateTimeEntry require "util/ace-components.js" and "util/ace-jquery.js" files which are indeed referenced after navigating to the second page. When using icecore:config tag the mandatory resources are read and enforced only when the page is parsed (unlike the context parameter which is read at application load time).

        Show
        Mircea Toma added a comment - - edited Tested with success the 3.3 maintenance branch. Both ace:slideEntry and ace:dateTimeEntry require "util/ace-components.js" and "util/ace-jquery.js" files which are indeed referenced after navigating to the second page. When using icecore:config tag the mandatory resources are read and enforced only when the page is parsed (unlike the context parameter which is read at application load time).
        Carmen Cristurean made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Arran Mccullough
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: