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

        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.
        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.
        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.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development