Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-21630

Visibility in LocationService::loadLocationChildren()

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: High High
    • Resolution: Unresolved
    • Affects Version/s: 2013.06
    • Fix Version/s: None
    • Labels:

      Description

      loadLocationChildren() does not use "visibility"!

      It should according to the description : "Loads children which are readable by the current user of a location object sorted by sortField and sortOrder"

        Activity

        Hide
        Jérôme Vieilledent (Inactive) added a comment -

        Hi

        Visibility is not considered as a permission. This is why it's not taken into account. However, we might want to be able to add an optional criterion to this method.

        Show
        Jérôme Vieilledent (Inactive) added a comment - Hi Visibility is not considered as a permission. This is why it's not taken into account. However, we might want to be able to add an optional criterion to this method.
        Hide
        Gaetano Giunta (Inactive) added a comment -

        +1 for only loading visible children by default and allow developer override via parameters

        Show
        Gaetano Giunta (Inactive) added a comment - +1 for only loading visible children by default and allow developer override via parameters
        Hide
        Nicolas Lescure added a comment -

        Thanks Gaetano. I felt alone on this one.

        Show
        Nicolas Lescure added a comment - Thanks Gaetano. I felt alone on this one.
        Hide
        Douglas Hammond added a comment -

        Another solution can be found at http://share.ez.no/forums/ez-publish-5-platform/automatic-sort/comment85732

        I Believe this would be the proper way to go.

        Show
        Douglas Hammond added a comment - Another solution can be found at http://share.ez.no/forums/ez-publish-5-platform/automatic-sort/comment85732 I Believe this would be the proper way to go.

          People

          • Assignee:
            Unassigned
            Reporter:
            Nicolas Lescure
          • Votes:
            2 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated: