ICEfaces
  1. ICEfaces
  2. ICE-10291

remove dependency of mobi components on ace-component.js, instead using only the support for ace-ajax.js

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.0.BETA
    • Fix Version/s: 4.0
    • Labels:
      None
    • Environment:
      jsf 2.2 mobi ace

      Description

      The current mobi components are dependent on ace js for ace:ajax support. The dependency should only be for the minimum js required (which is just what is needed for ace:ajax).

        Issue Links

          Activity

          Judy Guglielmin created issue -
          Ken Fyten made changes -
          Field Original Value New Value
          Link This issue depends on ICE-10290 [ ICE-10290 ]
          Judy Guglielmin made changes -
          Assignee Judy Guglielmin [ judy.guglielmin ]
          Hide
          Judy Guglielmin added a comment -

          after serious study of the resources (js in particular) for the ace and mobi components. It has been noted that:-
          1) ice.ace.ab(.....) which is what ace:ajax is dependent upon is present in core.js, using ice.ace.jq (which is defined in util.js).
          this would mean that core.js has a dependency to util.js which has a dependency to jquery.js and all 3 are required for ace:ajax. (kind of a waste as jq really is not that necessary for this task).
          2) There are other utilities in util.js and core.js which are not for ace:ajax. This means either these 2 files are small enough that they can be combined as a resource dependency, or they should be split into non-jquery dependent utilties and dependent jquery utilities.....another discussion.
          3). mobi project has the following entry in it's resource-dependency.xml:-
          Upon testing for 10291 (where I simply use the ace util/util.js for the ace:ajax support), I noticed the following in the mobile project resource-dependency.xml:-
          <resource name="core/bridgeit.js" library="icefaces.mobi">
          <requires>
          <resource name="util/ace-jquery.js" library="icefaces.ace"/>
          </requires>
          </resource>
          according to Philip this is not true as bridgeit has no dependency.

          so....in order to remove the dependency of ace-jquery to flipswitch, dateSpinner and timeSpinner (the current mobi components which use ace:ajax):-
          1) ace:ajax needs to have better definition for it's own resource dependencies. Unless util & core are redone, that would mean that core.js should bring in util.js as well as jquery.js. (defined with dependencies).
          2) flipswitch should not need to define ace:ajax's resource dependencies, since if ace:ajax is on the page, it will do that definition itself (correct?).
          3) remove the bridgeit resource dependency on the ace-jquery.js library.

          Show
          Judy Guglielmin added a comment - after serious study of the resources (js in particular) for the ace and mobi components. It has been noted that:- 1) ice.ace.ab(.....) which is what ace:ajax is dependent upon is present in core.js, using ice.ace.jq (which is defined in util.js). this would mean that core.js has a dependency to util.js which has a dependency to jquery.js and all 3 are required for ace:ajax. (kind of a waste as jq really is not that necessary for this task). 2) There are other utilities in util.js and core.js which are not for ace:ajax. This means either these 2 files are small enough that they can be combined as a resource dependency, or they should be split into non-jquery dependent utilties and dependent jquery utilities.....another discussion. 3). mobi project has the following entry in it's resource-dependency.xml:- Upon testing for 10291 (where I simply use the ace util/util.js for the ace:ajax support), I noticed the following in the mobile project resource-dependency.xml:- <resource name="core/bridgeit.js" library="icefaces.mobi"> <requires> <resource name="util/ace-jquery.js" library="icefaces.ace"/> </requires> </resource> according to Philip this is not true as bridgeit has no dependency. so....in order to remove the dependency of ace-jquery to flipswitch, dateSpinner and timeSpinner (the current mobi components which use ace:ajax):- 1) ace:ajax needs to have better definition for it's own resource dependencies. Unless util & core are redone, that would mean that core.js should bring in util.js as well as jquery.js. (defined with dependencies). 2) flipswitch should not need to define ace:ajax's resource dependencies, since if ace:ajax is on the page, it will do that definition itself (correct?). 3) remove the bridgeit resource dependency on the ace-jquery.js library.
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #42703 Tue Oct 07 13:16:48 MDT 2014 judy.guglielmin ICE-10304 ICE-10291 cannot separate the two jiras for initial reorg of component.js. For now the unused js is commented out and refactored. Will wait until QA gets a chance to test to remove
          Files Changed
          Commit graph ADD /icefaces4/trunk/icefaces/mobi/component/resources/org.icefaces.component.dataview/dataviewcontroller.js
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/augmentedreality/AugmentedRealityMeta.java
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/flipswitch/FlipSwitchMeta.java
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/fetchcontact/FetchContactMeta.java
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/resources/org.icefaces.component.util/component.js
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/cloudpush/CloudPushMeta.java
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/camera/CameraMeta.java
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/sms/SmsMeta.java
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/camcorder/CamcorderMeta.java
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/viewmanager/ViewManagerMeta.java
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/thumbnail/ThumbnailMeta.java
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/scan/ScanMeta.java
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/dataview/DataViewMeta.java
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/timespinner/TimeSpinnerMeta.java
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/resources/icefaces.mobi/META-INCLUDE/resource-dependency.xml
          Commit graph ADD /icefaces4/trunk/icefaces/mobi/component/resources/org.icefaces.component.viewmanager/viewmanager.js
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/datespinner/DateSpinnerMeta.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #42709 Tue Oct 07 20:47:48 MDT 2014 art.zambrano ICE-10291 added ace:ajax support to mobi:dataView and the "select" ajax event; removed activation mode attribute and related code
          Files Changed
          Commit graph DEL /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/dataview/ActivationMode.java
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/resources/org.icefaces.component.dataview/dataviewcontroller.js
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/dataview/DataView.java
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/dataview/DataViewRenderer.java
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/dataview/DataViewMeta.java
          Hide
          Arturo Zambrano added a comment -

          r42709: added ace:ajax support to mobi:dataView and the "select" ajax event; removed activation mode attribute and related code.

          Show
          Arturo Zambrano added a comment - r42709: added ace:ajax support to mobi:dataView and the "select" ajax event; removed activation mode attribute and related code.
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #42713 Wed Oct 08 12:15:33 MDT 2014 art.zambrano ICE-10291 updated TLD documentation to remove ActivationMode stuff and explain the use of ace:ajax
          Files Changed
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/dataview/DataViewDetailsMeta.java
          Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/dataview/DataViewMeta.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #42714 Wed Oct 08 12:28:57 MDT 2014 art.zambrano ICE-10291 updated dataView demo to use ajax instead of the activationMode attribute
          Files Changed
          Commit graph MODIFY /icefaces4/trunk/icefaces/samples/showcase/showcase/src/main/webapp/resources/examples/mobi/dataview/dataview-example.xhtml
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #42717 Wed Oct 08 14:16:33 MDT 2014 judy.guglielmin ICE-10291 added ace namespace as support for ace:ajax tag
          Files Changed
          Commit graph MODIFY /icefaces4/trunk/icefaces/samples/showcase/showcase/src/main/webapp/resources/examples/mobi/dataview/dataview-example.xhtml
          Hide
          Judy Guglielmin added a comment -

          done with exception of removing jQuery from ace:ajax (not sure if that will be required at this time).

          Show
          Judy Guglielmin added a comment - done with exception of removing jQuery from ace:ajax (not sure if that will be required at this time).
          Judy Guglielmin made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Ken Fyten made changes -
          Fix Version/s 4.0 [ 11382 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

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

              Dates

              • Created:
                Updated:
                Resolved: