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

Content still visible in ezobjectrelationlist after sending it to trash

    Details

      Description

      The customer observed, that removing content from ezobjectrelationlist doesn't affect this list - position is still visible, but clicking on it doesn't redirect to the content.

      Steps to reproduce
      1. Create a new content type that contains ezobjectrelationlist FieldType, e.g. "menu".
      2. Create new content using "menu" content type, name it "test" and select some contents (e.g. articles).
      3. Publish this new content.
      4. Select this content and go to one of the articles that is on relation list.
      5. Send this article to the trash.
      6. Go back to "test". Note, that deleted article still appears on the list and clicking on it doesn't redirect to this article.

      In v2.0.1 attempt of reproducing this issue ends up with an exception:

      Type error: Argument 1 passed to EzSystems\EzPlatformAdminUi\UI\Service\PathService::loadPathLocations() must be an instance of eZ\Publish\API\Repository\Values\Content\Location, null given, called in /vendor/ezsystems/ezplatform-admin-ui/src/bundle/Controller/ContentViewController.php on line 101 [in vendor/ezsystems/ezplatform-admin-ui/src/lib/UI/Service/PathService.php:38]

      Note: removing an article from the trash doesn't cause any difference.

        Issue Links

          Activity

          Hide
          Adam Wójs added a comment -
          Show
          Adam Wójs added a comment - PR (for v1): https://github.com/ezsystems/PlatformUIBundle/pull/955
          Hide
          Sylvain Guittard added a comment -

          Thanks Adam Wójs for this fix. Do you think you can provide a fix for v2?

          Show
          Sylvain Guittard added a comment - Thanks Adam Wójs for this fix. Do you think you can provide a fix for v2?
          Hide
          Adam Wójs added a comment -

          Yes, of course

          Show
          Adam Wójs added a comment - Yes, of course
          Hide
          Sylvain Guittard added a comment -

          Cool.

          Show
          Sylvain Guittard added a comment - Cool.
          Hide
          Adam Wójs added a comment -

          Sylvain Guittard FYI: After discussion with Sławomir Uchto on Slack the v2 part of this issue will be processed as EZP-28865.

          Show
          Adam Wójs added a comment - Sylvain Guittard FYI: After discussion with Sławomir Uchto on Slack the v2 part of this issue will be processed as EZP-28865 .
          Hide
          Sylvain Guittard added a comment -

          Thanks

          Show
          Sylvain Guittard added a comment - Thanks
          Show
          Adam Wójs added a comment - Merged: https://github.com/ezsystems/PlatformUIBundle/commit/42f3f3446be71c214d2caab8bd26ffc2d6e2ca21
          Hide
          Michał Szołtysek added a comment -

          QA Approved on eZ Platform v1.7.7 with patch & v1.13.2-rc1.

          Show
          Michał Szołtysek added a comment - QA Approved on eZ Platform v1.7.7 with patch & v1.13.2-rc1.

            People

            • Assignee:
              Unassigned
              Reporter:
              Konrad Oboza
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: