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

Deprecate RepositoryService.getPropertyInfo method

    Details

      Description

      I would like to deprecate and ultimately remove the RepositoryService.getPropertyInfo method and extend RepositoryService.getItemInfos to take over that functionality. getItemInfos would thus change to

      /**

      • Method used to 'batch-read' from the persistent storage. It returns the
      • <code>ItemInfo</code> for the given <code>ItemId</code> as the first
      • element in the <code>Iterator</code>. In addition the iterator may contain
      • arbitrary <code>ItemInfo</code>s.
        */
        public Iterator<? extends ItemInfo> getItemInfos(SessionInfo sessionInfo, ItemId itemId) throws ItemNotFoundException, RepositoryException;
      1. JCR-3054.patch
        32 kB
        Michael Dürig

        Activity

        Jukka Zitting made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Angela D committed 1181840 (1 file)
        Reviews: none

        JCR-3054 - Deprecate RepositoryService.getPropertyInfo method (fix outdated javadoc links left by refactoring RepositoryService#getItemInfos)

        Angela D committed 1179197 (1 file)
        Reviews: none

        JCR-3054: Deprecate RepositoryService.getPropertyInfo method - fixing javadoc and wrong indention

        Michael Dürig made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Michael Dürig added a comment -

        Fixed at revision 1159182

        Show
        Michael Dürig added a comment - Fixed at revision 1159182
        Hide
        angela added a comment -

        i am fine with deprecating the method.

        the only thing that i was thinking of was the question of the same-name node + property
        feature which is allowed since jcr 283. this used to be a problem in the jcr-remoting
        since on the server-side there - up to now - no possibility to distinguish them path-wise.
        (see JCR-1616)... but since the spi-ItemID itself is sufficient to avoid any ambiguity,
        the proposed solution should not do any harm (au contraire).

        Show
        angela added a comment - i am fine with deprecating the method. the only thing that i was thinking of was the question of the same-name node + property feature which is allowed since jcr 283. this used to be a problem in the jcr-remoting since on the server-side there - up to now - no possibility to distinguish them path-wise. (see JCR-1616 )... but since the spi-ItemID itself is sufficient to avoid any ambiguity, the proposed solution should not do any harm (au contraire).
        Michael Dürig made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Fix Version/s 2.3.0 [ 12315901 ]
        Hide
        Michael Dürig added a comment -

        tentative patch

        Show
        Michael Dürig added a comment - tentative patch
        Michael Dürig made changes -
        Affects Version/s 2.3.0 [ 12315901 ]
        Michael Dürig made changes -
        Field Original Value New Value
        Attachment JCR-3054.patch [ 12490534 ]
        Hide
        Michael Dürig added a comment -

        Tentative patch

        Show
        Michael Dürig added a comment - Tentative patch
        Michael Dürig created issue -

          People

          • Assignee:
            Michael Dürig
            Reporter:
            Michael Dürig
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development