Jackrabbit Content Repository
  1. Jackrabbit Content Repository
  2. JCR-1944

Privilege content representation should be of property type NAME

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.5
    • Fix Version/s: 1.6
    • Component/s: jackrabbit-core, security
    • Labels:
      None

      Description

      the content representation of jcr privileges should reflect that fact that privilege names changed from simple string to JCR name.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        20h 14m 1 angela 21/Jan/09 09:58
        Resolved Resolved Closed Closed
        204d 5h 5m 1 Jukka Zitting 13/Aug/09 16:03
        Jukka Zitting made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Jukka Zitting made changes -
        Workflow jira [ 12450652 ] no-reopen-closed, patch-avail [ 12468158 ]
        Hide
        angela added a comment -

        > I don't think we should merge this to 1.5

        no, rather not.

        > BTW, do you think this should be noted in the 1.6 release notes for
        > people who are already trying out the new security code? I'm not sure
        > how big an impact the changed property type will cause.

        big. that's why i scheduled it for 2.0.0.

        but after all the 283 implementation is still work in progress and Issue #JCR-1588 has not been closed up to now. unless the spec is finalized there may be any kind of changes that break backwards compability.

        listing in in the release notes may still be a good idea.

        Show
        angela added a comment - > I don't think we should merge this to 1.5 no, rather not. > BTW, do you think this should be noted in the 1.6 release notes for > people who are already trying out the new security code? I'm not sure > how big an impact the changed property type will cause. big. that's why i scheduled it for 2.0.0. but after all the 283 implementation is still work in progress and Issue # JCR-1588 has not been closed up to now. unless the spec is finalized there may be any kind of changes that break backwards compability. listing in in the release notes may still be a good idea.
        Jukka Zitting made changes -
        Fix Version/s 2.0.0 [ 12312449 ]
        Issue Type Bug [ 1 ] Improvement [ 4 ]
        Fix Version/s 1.6.0 [ 12313459 ]
        Hide
        Jukka Zitting added a comment -

        Changing this to an improvement issue as the original implementation was in line with previous spec text (and I don't think we should merge this to 1.5).

        BTW, do you think this should be noted in the 1.6 release notes for people who are already trying out the new security code? I'm not sure how big an impact the changed property type will cause.

        Show
        Jukka Zitting added a comment - Changing this to an improvement issue as the original implementation was in line with previous spec text (and I don't think we should merge this to 1.5). BTW, do you think this should be noted in the 1.6 release notes for people who are already trying out the new security code? I'm not sure how big an impact the changed property type will cause.
        angela made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        angela added a comment -

        changed property definition in built-in nodetypes and adjusted the corresponding createValue/setProperty calls to user PropertyType.NAME.

        Show
        angela added a comment - changed property definition in built-in nodetypes and adjusted the corresponding createValue/setProperty calls to user PropertyType.NAME.
        angela made changes -
        Field Original Value New Value
        Assignee angela [ anchela ]
        angela created issue -

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development