Wookie
  1. Wookie
  2. WOOKIE-187

Don't create local.*.properties files automatically

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.9.1
    • Component/s: None
    • Labels:
      None

      Description

      At present the code automatically creates a local.*.properties file for each properties file. This means that:

      a) when a new property is added, such as in the fix for WOOKIE-145 or the addition of a new feature wookie breaks for all existing users as they will not have this property in their local.*.properties file

      b) the properties files have all the possible properties at the time of creation, but not ones that are added later (see a)

      Instead we should load the default properties then load the local properties, if they exist.

      We should also move the default properties files out of the java src tree, where they are hidden from view, and put them in root.

        Activity

        Ross Gardler created issue -
        Hide
        Ross Gardler added a comment -

        I've fixed this for widgetserver.properties, we still need to do it for the other property files though.

        Show
        Ross Gardler added a comment - I've fixed this for widgetserver.properties, we still need to do it for the other property files though.
        Ross Gardler made changes -
        Field Original Value New Value
        Status Ready To Review [ 10006 ] Open [ 1 ]
        Scott Wilson made changes -
        Fix Version/s 0.9.1 [ 12315418 ]
        Hide
        Scott Wilson added a comment -

        Implemented for opensocial properties and features.properties is no longer used, so this is now ready to close.

        Show
        Scott Wilson added a comment - Implemented for opensocial properties and features.properties is no longer used, so this is now ready to close.
        Scott Wilson made changes -
        Status Open [ 1 ] Closed [ 6 ]
        Hide
        Paul Sharples added a comment -

        Verified, they are no longer automatically created but if you manually create one, the contextlistener will load the local settings too.

        Show
        Paul Sharples added a comment - Verified, they are no longer automatically created but if you manually create one, the contextlistener will load the local settings too.
        Paul Sharples made changes -
        Status Closed [ 6 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Reviewable Reviewable Open Open
        1h 29m 1 Ross Gardler 25/Mar/11 00:54
        Open Open Closed Closed
        150d 11h 22m 1 Scott Wilson 22/Aug/11 13:16
        Closed Closed Resolved Resolved
        15d 1h 6m 1 Paul Sharples 06/Sep/11 14:23

          People

          • Assignee:
            Unassigned
            Reporter:
            Ross Gardler
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development