ICEmobile
  1. ICEmobile
  2. MOBI-134

Refactor Blackberry Container for ease of reuse

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.0 Beta
    • Fix Version/s: 1.0 Beta
    • Component/s: None
    • Labels:
      None
    • Environment:
      Blackberry Container

      Description

      In porting the code from the container project into one for debugging MOBI-128, I found the codebase contained numerous references to ICEmobileContainer. Some of these references were to publicly exposed methods on the main class that provided functions necessary to interact with the Document, and some were convenience Logging methods, since Blackberry logging calls makes for poor looking code.

      It would be better if the publicly exposed methods were defined in an interface so that people doing this type of work in their projects don't have to rename everything, as well as being used as a measure for how appropriate the names and features exposed actually are. Also, I moved all the logging convenience methods into a Static Logger class.

      There remain a few very rare variables tied to ICEmobileContainer.java, which relate to the HOME_URL, etc, which we wanted to have visible at the top of the main class. ICEmobileContainer implements the ContainerController interface, and passes itself into the constructor of all the scripts and menus that use the methods. Anyone wishing to redo the Container main application entry point now only has to implement the methods of the interface (or copy them) and the script and menu classes wont have to be changed everywhere.

        Activity

        Greg Dick created issue -
        Greg Dick made changes -
        Field Original Value New Value
        Assignee Steve Maryka [ steve.maryka ] Greg Dick [ greg.dick ]
        Greg Dick made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Greg Dick made changes -
        Fix Version/s 1.0 Beta [ 10290 ]
        Migration made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Greg Dick
            Reporter:
            Greg Dick
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: