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

Deleting a user breaks content created by that user

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: High High
    • Resolution: Fixed
    • Affects Version/s: 1.3.2, 1.4.2
    • Fix Version/s: Customer request
    • Component/s: Platform stack
    • Labels:
      None

      Description

      When a user who has created content is deleted then the created content is broken as the system still tries to load the already deleted user object.

      Steps to reproduce:
      1. Open the backend and create a new user "usr-001".
      2. Login with "usr-001" and create a new article "Article #001".
      3. Logout from "usr-001" and delete it with another account.
      4. Try to edit "Article #001".
      5. When "Article #001" is loaded, you will get the following error message:

        Could not load the user with id '/api/ezp/v2/user/users/52'

      Expected behaviour:
      • Non existent users should not prevent content from being loaded, displayed and edited.
      Stack trace

      "ErrorMessage": {
          "_media-type": "application/vnd.ez.api.ErrorMessage+json",
          "errorCode": 404,
          "errorMessage": "Not Found",
          "errorDescription": "Could not find 'Content' with identifier'array (
              'id' => '52',
              'languages'=> NULL,
              'versionNo' => NULL)'",
          "trace": "#0 vendor/ezsystems/ezpublish-kernel/eZ/Publish/Core/Repository/UserService.php(514):eZ\\Publish\\Core\\Repository\\ContentService->internalLoadContent('52')
              #1 vendor/ezsystems/ezpublish-kernel/eZ/Publish/Core/SignalSlot/UserService.php(242): eZ\\Publish\\Core\\Repository\\UserService->loadUser('52')
              #2 app/cache/dev/appDevDebugProjectContainer.php(41365): eZ\\Publish\\Core\\SignalSlot\\UserService->loadUser('52')
              #3 vendor/ezsystems/ezpublish-kernel/eZ/Publish/Core/REST/Server/Controller/User.php(179)"
      }

        Issue Links

          Activity

          Show
          Stéphane Diot added a comment - PR: https://github.com/ezsystems/PlatformUIBundle/pull/694
          Hide
          Rui Silva (Inactive) added a comment -

          PR tested and approved by QA for master.
          Sending back to dev so PR can be merged.
          Please send back to QA after merge is completed so that QA can properly inform Support.

          Show
          Rui Silva (Inactive) added a comment - PR tested and approved by QA for master. Sending back to dev so PR can be merged. Please send back to QA after merge is completed so that QA can properly inform Support.
          Show
          Stéphane Diot added a comment - Merged in 1.5 : https://github.com/ezsystems/PlatformUIBundle/commit/ace520aadcebdc6fadee7a14ab479f784fb3b536
          Hide
          Stéphane Diot added a comment -

          [~rui.silva@ez.no] was it approved for 1.5 ?

          Show
          Stéphane Diot added a comment - [~rui.silva@ez.no] was it approved for 1.5 ?
          Hide
          Rui Silva (Inactive) added a comment -

          Not yet.
          As per agreed on the recent procedures, a PR is evaluated and tested by QA on master version, then sent back to be merged. After the merge is done and is back with QA, a new evaluation set is made for the versions it has been merged to.

          Show
          Rui Silva (Inactive) added a comment - Not yet. As per agreed on the recent procedures, a PR is evaluated and tested by QA on master version, then sent back to be merged. After the merge is done and is back with QA, a new evaluation set is made for the versions it has been merged to.
          Hide
          Rui Silva (Inactive) added a comment -

          Tested and approved by QA for 1.5.

          Show
          Rui Silva (Inactive) added a comment - Tested and approved by QA for 1.5.

            People

            • Assignee:
              Unassigned
              Reporter:
              Peter Halasz
            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: