Uploaded image for project: 'Jackrabbit Oak'
  1. Jackrabbit Oak
  2. OAK-55

Provide reasonable way to set property on NodeStateEditor

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 0.3
    • core
    • None

    Description

      currently properties are add/modified on the NodeStateEditor using: void setProperty(PropertyState state);

      the consequence of that is that every single usage of that method needs to know the details
      of how to create a property state. IMO that's really bad practice and should be refactored.

      i would suggest to change it to something like

      • setProperty(String name, Scalar value)
      • setProperty(String name, List<Scalar> values)

      Attachments

        Activity

          People

            angela Angela Schreiber
            angela Angela Schreiber
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: