ICEfaces
  1. ICEfaces
  2. ICE-6183

ACE component making request to external site

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-Beta2
    • Fix Version/s: 2.0.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      ACE

      Description

      Running the ACE showcase application, you can see during the initial page load that a request is being made to the Yahoo's external API site:

      http://yui.yahooapis.com/combo?3.1.1/build/oop/oop-min.js&3.1.1/build/event-custom/event-custom-min.js&3.1.1/build/attribute/attribute-base-min.js&3.1.1/build/base/base-base-min.js&3.1.1/build/event/event-base-min.js&3.1.1/build/dom/dom-min.js&3.1.1/build/node/node-min.js&3.1.1/build/event/event-delegate-min.js

      This won't be acceptable for most intranet type deployments and could also constitute a security risk.

        Issue Links

          Activity

          Hide
          Arturo Zambrano added a comment -

          Fixed by finally adding the 'bootstrap: false' flag to the YUI() function in component.js and by adding the required @ResourceDepenency's to AnimationBahavior and DateTimeEntry. The required resources were needed by anim-min.js, so they have to be loaded BEFORE loading anim-min.js.

          Show
          Arturo Zambrano added a comment - Fixed by finally adding the 'bootstrap: false' flag to the YUI() function in component.js and by adding the required @ResourceDepenency's to AnimationBahavior and DateTimeEntry. The required resources were needed by anim-min.js, so they have to be loaded BEFORE loading anim-min.js.
          Hide
          Mark Collette added a comment -

          #2 and #3 should take into account ICE-6194. #1 can be finished on it's own, but #2 and #3 should be coordinated with, or be done after, ICE-6194.

          Show
          Mark Collette added a comment - #2 and #3 should take into account ICE-6194 . #1 can be finished on it's own, but #2 and #3 should be coordinated with, or be done after, ICE-6194 .
          Hide
          Mark Collette added a comment -

          Basically, what remains, is to test this both with MandatoryResourceComponent enabled and disabled.

          Show
          Mark Collette added a comment - Basically, what remains, is to test this both with MandatoryResourceComponent enabled and disabled.
          Hide
          Arturo Zambrano added a comment -

          I tested the following pages with @MandatoryResourceComponent enabled and disabled (commented out from all renderer classes).

          1 http://localhost:8080/ace-showcase/main.jsf
          2 http://localhost:8080/ace-showcase/dateTimeEntry.jsf
          3 http://localhost:8080/ace-showcase/tabSetServer.jsf
          4 http://localhost:8080/ace-showcase/pushButton.jsf
          5 http://localhost:8080/ace-test/calendar.jsf
          6 http://localhost:8080/ace-test/slider.jsf
          7 http://localhost:8080/ace-test/buttons/checkboxSam.jsf
          8 http://localhost:8080/ace-test/buttons/menubutton.jsf

          I confirmed, by comparing the source code of both versions, the difference in the number of resources loaded (with @MandatoryResourceComponent there are more resources in the page). In both cases, the pages worked normally, there were no warnings in the console (except for slider, but because of another issue).

          So, it's safe to say that the 'bootstrap: false' flag does not affect the @MandatoryResourceComponent functionality.

          Show
          Arturo Zambrano added a comment - I tested the following pages with @MandatoryResourceComponent enabled and disabled (commented out from all renderer classes). 1 http://localhost:8080/ace-showcase/main.jsf 2 http://localhost:8080/ace-showcase/dateTimeEntry.jsf 3 http://localhost:8080/ace-showcase/tabSetServer.jsf 4 http://localhost:8080/ace-showcase/pushButton.jsf 5 http://localhost:8080/ace-test/calendar.jsf 6 http://localhost:8080/ace-test/slider.jsf 7 http://localhost:8080/ace-test/buttons/checkboxSam.jsf 8 http://localhost:8080/ace-test/buttons/menubutton.jsf I confirmed, by comparing the source code of both versions, the difference in the number of resources loaded (with @MandatoryResourceComponent there are more resources in the page). In both cases, the pages worked normally, there were no warnings in the console (except for slider, but because of another issue). So, it's safe to say that the 'bootstrap: false' flag does not affect the @MandatoryResourceComponent functionality.

            People

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

              Dates

              • Created:
                Updated:
                Resolved: