ICEfaces
  1. ICEfaces
  2. ICE-10298

Consolidate common class between core, ace and mobi

    Details

    • Assignee Priority:
      P2

      Description

      While working on bridgeIt issue #44 I came across a a duplicate classes:

      svn\ossrepo\icefaces4\trunk\icefaces\core\src\main\java\org\icefaces\util\UserAgentInfo.java
      svn\ossrepo\icefaces4\trunk\icefaces\mobi\component\src\org\icefaces\mobi\util\UserAgentInfo.java

      while adding detection for iOS8.

      So dug little deeper and also noticed 13 common classes between the packages :

      svn\ossrepo\icefaces4\trunk\icefaces\ace\component\src\org\icefaces\ace\util\
      and
      svn\ossrepo\icefaces4\trunk\icefaces\mobi\component\src\org\icefaces\mobi\util\

      Interestingly enough some of these duplicate class have started to diverge. For example mobi has a outputStrem memory patch for Utilts.java while ace Utils.java doesn't.

      It would be a good time before final to move any common source the core and merge the divergent changes.

        Activity

        Patrick Corless created issue -
        Patrick Corless made changes -
        Field Original Value New Value
        Assignee Patrick Corless [ patrick.corless ]
        Patrick Corless made changes -
        Assignee Patrick Corless [ patrick.corless ] Ken Fyten [ ken.fyten ]
        Ken Fyten made changes -
        Assignee Ken Fyten [ ken.fyten ] Arturo Zambrano [ artzambrano ]
        Fix Version/s 4.0 [ 11382 ]
        Assignee Priority P2 [ 10011 ]
        Arturo Zambrano made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Patrick Corless
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: