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

Wrong Path Format in PreviewLocationProvider

    Details

      Description

      The function loadMainLocation in PreviewLocationProvider returns a wrong path format:

      • returns "/1/2//x" instead of "/1/2/x/"

        Activity

        Hide
        Yannick Roger (Inactive) added a comment -

        Could you please provide steps to reproduce for this issue. Our quality insurance team will need it to test it.

        Show
        Yannick Roger (Inactive) added a comment - Could you please provide steps to reproduce for this issue. Our quality insurance team will need it to test it.
        Hide
        Bernhard Rößler added a comment -

        If the Methode returns "/1/2//x" the "Criterion/Subtree" returns a InvalidArgumentException.

        Show
        Bernhard Rößler added a comment - If the Methode returns "/1/2//x" the "Criterion/Subtree" returns a InvalidArgumentException.
        Show
        André Rømcke added a comment - Merged in https://github.com/ezsystems/ezpublish-kernel/commit/83fe1ab22583f1d7f5f11f2a0f8d4b4c55f7f7af
        Hide
        Rui Silva (Inactive) added a comment -

        Bernhard Rößler, I am from eZ's Quality Assurance team, and part of my job to test PR's for bug fixes includes reproducing the issue reported.
        Given that I have to reproduce a specific scenario in which the bug occurs, could you provide us with detailed steps to reproduce the issue itself (what you did and how, and what you saw happening that should not), so that we can verify afterwards that the proposed PR solves it?
        Thank you.

        Show
        Rui Silva (Inactive) added a comment - Bernhard Rößler , I am from eZ's Quality Assurance team, and part of my job to test PR's for bug fixes includes reproducing the issue reported. Given that I have to reproduce a specific scenario in which the bug occurs, could you provide us with detailed steps to reproduce the issue itself (what you did and how, and what you saw happening that should not), so that we can verify afterwards that the proposed PR solves it? Thank you.
        Hide
        Siegfried Zach added a comment -

        The problem occurs in this situation:

        • When rendering a preview and for this preview a subtree query is performed, the preview fails.

        Sorry, that this probably not the info you can use for testing. The problem came for us in a customized code part - so we even not sure if a testcase can be performed on a standard eZ Intallation.

        So e.g. this szenario should fail:

        • You have an article, the article displays objects from it's subtree (e.g. a list of "subarticles")
        • The preview should now fail for such an article, bec. the subtree query fails
          but not sure if you can mirror this scenario.
        Show
        Siegfried Zach added a comment - The problem occurs in this situation: When rendering a preview and for this preview a subtree query is performed, the preview fails. Sorry, that this probably not the info you can use for testing. The problem came for us in a customized code part - so we even not sure if a testcase can be performed on a standard eZ Intallation. So e.g. this szenario should fail: You have an article, the article displays objects from it's subtree (e.g. a list of "subarticles") The preview should now fail for such an article, bec. the subtree query fails but not sure if you can mirror this scenario.
        Hide
        Rui Silva (Inactive) added a comment -

        Hello Siegfried Zach. Thank you for the feedback.
        QA team followed your proposed scenario to test the issue and it could not be reproduced.
        Sanity tests were executed however on the given scope to check for potential errors, and none was found.
        Approved by QA for master.

        Show
        Rui Silva (Inactive) added a comment - Hello Siegfried Zach . Thank you for the feedback. QA team followed your proposed scenario to test the issue and it could not be reproduced. Sanity tests were executed however on the given scope to check for potential errors, and none was found. Approved by QA for master.

          People

          • Assignee:
            Unassigned
            Reporter:
            Bernhard Rößler
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: