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

User can not be properly created from inside storage

    Details

    • Sprint:
      Aconcagua Sprint 1, Aconcagua Sprint 4

      Description

      When field definition is added to the ContentType, all Content instances of that ContentType are updated by adding a field with default value defined in field definition.

      As user can be created only through UserService (or User storage handler) and not through external storage, field of the ezuser type added in such a way will not have all data properly created. Most important, it will not be possible to update such a user so that it becomes functional.

      Some options here:

      1. do not allow adding ezuser field to the ContentType that has existing Content instances
      2. handle it as a special case and create non-functional user that can be later updated (Legacy Storage way)
      3. something else?

        Issue Links

          Activity

          Petar Spanja (Inactive) created issue -
          Petar Spanja (Inactive) made changes -
          Field Original Value New Value
          Link This issue discovered while testing EZP-20936 [ EZP-20936 ]
          André Rømcke made changes -
          Status Open [ 1 ] Backlog [ 10000 ]
          André Rømcke made changes -
          Rank Ranked higher
          Hide
          André Rømcke added a comment -

          I think I'm leaning towards #1. However, how is legacy handling this?

          Show
          André Rømcke added a comment - I think I'm leaning towards #1. However, how is legacy handling this?
          André Rømcke made changes -
          Assignee Petar Spanja [ petar.spanja@ez.no ]
          André Rømcke made changes -
          Sprint Aconcagua Sprint 1 [ 10 ]
          André Rømcke made changes -
          Rank Ranked lower
          André Rømcke made changes -
          Remaining Estimate 1 day [ 28800 ]
          André Rømcke made changes -
          Original Estimate 1 day [ 28800 ]
          André Rømcke made changes -
          Remaining Estimate 1 day [ 28800 ] 6 hours [ 21600 ]
          Hide
          Petar Spanja (Inactive) added a comment -

          @André That would be option #2; LS creates empty (non functional) user that can be later updated to become functional.

          Show
          Petar Spanja (Inactive) added a comment - @André That would be option #2; LS creates empty (non functional) user that can be later updated to become functional.
          Petar Spanja (Inactive) logged work - 18/Jun/13 2:14 PM
          • Time Spent:
            1 hour
             

            fixing

          Petar Spanja (Inactive) made changes -
          Status Backlog [ 10000 ] Development [ 3 ]
          Show
          Petar Spanja (Inactive) added a comment - Pull request: https://github.com/ezsystems/ezpublish-kernel/pull/422
          Petar Spanja (Inactive) logged work - 19/Jun/13 2:14 PM - edited
          • Time Spent:
            4 hours
             

            fixing and testing

          Petar Spanja (Inactive) logged work - 20/Jun/13 1:27 PM
          • Time Spent:
            40 minutes
             

            fix by review

          Petar Spanja (Inactive) made changes -
          Remaining Estimate 6 hours [ 21600 ] 5 hours [ 18000 ]
          Time Spent 1 hour [ 3600 ]
          Worklog Id 35475 [ 35475 ]
          Petar Spanja (Inactive) made changes -
          Remaining Estimate 5 hours [ 18000 ] 1 hour [ 3600 ]
          Time Spent 1 hour [ 3600 ] 5 hours [ 18000 ]
          Worklog Id 35476 [ 35476 ]
          Petar Spanja (Inactive) made changes -
          Worklog Id 35476 [ 35476 ]
          Petar Spanja (Inactive) made changes -
          Status Development [ 3 ] Development review [ 10006 ]
          Show
          Petar Spanja (Inactive) added a comment - Fixed in ezpublish-kernel/master: https://github.com/ezsystems/ezpublish-kernel/commit/e1142b7e3c2d6d558473efb0c7e5da50b98eb33b
          Petar Spanja (Inactive) made changes -
          Status Development review [ 10006 ] Development Review done [ 10028 ]
          Petar Spanja (Inactive) made changes -
          Status Development Review done [ 10028 ] Documentation done [ 10011 ]
          Marcos Loureiro (Inactive) made changes -
          Status Documentation done [ 10011 ] QA [ 10008 ]
          Assignee Petar Spanja [ petar.spanja@ez.no ] Marcos Loureiro [ marcos.loureiro@ez.no ]
          Marcos Loureiro (Inactive) logged work - 28/Jun/13 10:05 AM
          • Time Spent:
            5 hours, 30 minutes
             

            Testing

          Marcos Loureiro (Inactive) made changes -
          Link This issue testing discovered EZP-21149 [ EZP-21149 ]
          Marcos Loureiro (Inactive) made changes -
          Status QA [ 10008 ] InputQ [ 10001 ]
          Assignee Marcos Loureiro [ marcos.loureiro@ez.no ]
          Marcos Loureiro (Inactive) made changes -
          Remaining Estimate 1 hour [ 3600 ] 0 minutes [ 0 ]
          Time Spent 5 hours [ 18000 ] 1 day, 2 hours, 30 minutes [ 37800 ]
          Worklog Id 35912 [ 35912 ]
          Petar Spanja (Inactive) made changes -
          Time Spent 1 day, 2 hours, 30 minutes [ 37800 ] 1 day, 3 hours, 10 minutes [ 40200 ]
          Worklog Id 36146 [ 36146 ]
          André Rømcke made changes -
          Assignee Petar Spanja [ petar.spanja@ez.no ]
          André Rømcke made changes -
          Sprint Aconcagua Sprint 1 [ 10 ] Aconcagua Sprint 1, Aconcagua Sprint 4 [ 10, 13 ]
          Petar Spanja (Inactive) made changes -
          Status InputQ [ 10001 ] Development [ 3 ]
          Petar Spanja (Inactive) logged work - 23/Jul/13 4:04 PM
          • Time Spent:
            3 hours, 15 minutes
             

            fixing by QA review

          Show
          Petar Spanja (Inactive) added a comment - New pull request: https://github.com/ezsystems/ezpublish-kernel/pull/466
          Petar Spanja (Inactive) logged work - 24/Jul/13 4:04 PM
          • Time Spent:
            3 hours
             

            fixing by QA review

          Petar Spanja (Inactive) logged work - 01/Aug/13 5:16 PM
          • Time Spent:
            30 minutes
             

            changes by PR review (onlyEmptyInstance)

          Petar Spanja (Inactive) made changes -
          Time Spent 1 day, 3 hours, 10 minutes [ 40200 ] 1 day, 6 hours, 25 minutes [ 51900 ]
          Worklog Id 37612 [ 37612 ]
          Petar Spanja (Inactive) made changes -
          Time Spent 1 day, 6 hours, 25 minutes [ 51900 ] 2 days, 1 hour, 25 minutes [ 62700 ]
          Worklog Id 37613 [ 37613 ]
          Petar Spanja (Inactive) made changes -
          Time Spent 2 days, 1 hour, 25 minutes [ 62700 ] 2 days, 1 hour, 55 minutes [ 64500 ]
          Worklog Id 37629 [ 37629 ]
          Petar Spanja (Inactive) made changes -
          Status Development [ 3 ] Development review [ 10006 ]
          André Rømcke made changes -
          Status Development review [ 10006 ] Development Review done [ 10028 ]
          André Rømcke made changes -
          Status Development Review done [ 10028 ] Development Acceptance Done [ 10030 ]
          Patrick Allaert (Inactive) made changes -
          Time Spent 2 days, 1 hour, 55 minutes [ 64500 ] 2 days, 2 hours, 15 minutes [ 65700 ]
          Worklog Id 37754 [ 37754 ]
          Patrick Allaert (Inactive) logged work - 07/Aug/13 3:13 PM
          • Time Spent:
            20 minutes
             

            PR review

          Show
          André Rømcke added a comment - Merged in https://github.com/ezsystems/ezpublish-kernel/commit/02adc861b17607bf67482d85d950f6d43e95c02d
          André Rømcke made changes -
          Status Development Acceptance Done [ 10030 ] Documentation done [ 10011 ]
          André Rømcke made changes -
          Affects Version/s 5.1 [ 11280 ]
          André Rømcke made changes -
          Fix Version/s 5.2 [ 11281 ]
          Marcos Loureiro (Inactive) made changes -
          Status Documentation done [ 10011 ] QA [ 10008 ]
          Assignee Petar Spanja [ petar.spanja@ez.no ] Marcos Loureiro [ marcos.loureiro@ez.no ]
          Hide
          Marcos Loureiro (Inactive) added a comment -

          Added PR with integration test for the case of create a new content type.
          https://github.com/ezsystems/ezpublish-kernel/pull/487

          Show
          Marcos Loureiro (Inactive) added a comment - Added PR with integration test for the case of create a new content type. https://github.com/ezsystems/ezpublish-kernel/pull/487
          Marcos Loureiro (Inactive) made changes -
          Time Spent 2 days, 2 hours, 15 minutes [ 65700 ] 2 days, 6 hours, 30 minutes [ 81000 ]
          Worklog Id 37939 [ 37939 ]
          Marcos Loureiro (Inactive) logged work - 13/Aug/13 5:56 PM
          • Time Spent:
            4 hours, 15 minutes
             

            Testing
            Reviewing
            Creating integration test

          Marcos Loureiro (Inactive) made changes -
          Time Spent 2 days, 6 hours, 30 minutes [ 81000 ] 2 days, 7 hours, 30 minutes [ 84600 ]
          Worklog Id 39018 [ 39018 ]
          Marcos Loureiro (Inactive) logged work - 09/Sep/13 2:08 PM
          • Time Spent:
            1 hour
             

            re-testing
            changing PR as suggested

          Petar Spanja (Inactive) logged work - 09/Sep/13 2:16 PM
          • Time Spent:
            15 minutes
             

            fixes by QA review

          Marcos Loureiro (Inactive) made changes -
          Time Spent 2 days, 7 hours, 30 minutes [ 84600 ] 3 days, 45 minutes [ 89100 ]
          Worklog Id 39028 [ 39028 ]
          Hide
          Petar Spanja (Inactive) added a comment -

          @Marcos Loureiro

          The comment is not correct, it was copied form the test directly below. I'll fix this.

          Show
          Petar Spanja (Inactive) added a comment - @ Marcos Loureiro The comment is not correct, it was copied form the test directly below. I'll fix this.
          Show
          Petar Spanja (Inactive) added a comment - Fixed in https://github.com/ezsystems/ezpublish-kernel/commit/fe7d2e8b59b60eb9cc4a846eb6cc691d5589d679 .
          Marcos Loureiro (Inactive) logged work - 09/Sep/13 7:00 PM
          • Time Spent:
            1 hour, 15 minutes
             

            Rebasing
            Fixing wrong rebase

          Petar Spanja (Inactive) logged work - 10/Sep/13 2:26 PM
          • Time Spent:
            15 minutes
             

            QA sync

          Marcos Loureiro (Inactive) made changes -
          Time Spent 3 days, 45 minutes [ 89100 ] 3 days, 1 hour, 15 minutes [ 90900 ]
          Worklog Id 39302 [ 39302 ]
          Hide
          Marcos Loureiro (Inactive) added a comment -

          QA Approved

          Show
          Marcos Loureiro (Inactive) added a comment - QA Approved
          Marcos Loureiro (Inactive) made changes -
          Assignee Marcos Loureiro [ marcos.loureiro@ez.no ]
          Status QA [ 10008 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          Marcos Loureiro (Inactive) logged work - 11/Sep/13 1:02 PM
          • Time Spent:
            30 minutes
             

            rebasing, fixing PR

          Petar Spanja (Inactive) made changes -
          Resolution Fixed [ 1 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          Petar Spanja (Inactive) made changes -
          Time Spent 3 days, 1 hour, 15 minutes [ 90900 ] 3 days, 1 hour, 30 minutes [ 91800 ]
          Worklog Id 40731 [ 40731 ]
          Petar Spanja (Inactive) made changes -
          Status Reopened [ 4 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          Petar Spanja (Inactive) made changes -
          Resolution Fixed [ 1 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          Petar Spanja (Inactive) made changes -
          Time Spent 3 days, 1 hour, 30 minutes [ 91800 ] 3 days, 1 hour, 45 minutes [ 92700 ]
          Worklog Id 40735 [ 40735 ]
          Petar Spanja (Inactive) made changes -
          Status Reopened [ 4 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          André Rømcke made changes -
          Workflow eZ Engineering Scrumban Workflow [ 56075 ] EZ* Development Workflow [ 83920 ]
          Alex Schuster made changes -
          Workflow EZ* Development Workflow [ 83920 ] EZEE Development Workflow [ 122543 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Backlog Backlog
          19h 48m 1 André Rømcke 12/Jun/13 2:03 PM
          Backlog Backlog Development Development
          6d 55m 1 Petar Spanja (Inactive) 18/Jun/13 2:59 PM
          Development Review done Development Review done Documentation Review done Documentation Review done
          5s 1 Petar Spanja (Inactive) 25/Jun/13 9:59 AM
          QA QA InputQ InputQ
          7h 52m 1 Marcos Loureiro (Inactive) 28/Jun/13 5:56 PM
          InputQ InputQ Development Development
          24d 3h 1 Petar Spanja (Inactive) 22/Jul/13 8:56 PM
          Development Development Development Review Development Review
          15d 16h 2m 2 Petar Spanja (Inactive) 04/Aug/13 3:25 PM
          Development Review Development Review Development Review done Development Review done
          4d 16h 16m 2 André Rømcke 05/Aug/13 10:15 AM
          Development Review done Development Review done Removed Status Removed Status
          10s 1 André Rømcke 05/Aug/13 10:15 AM
          Removed Status Removed Status Documentation Review done Documentation Review done
          7d 1h 42m 1 André Rømcke 12/Aug/13 11:58 AM
          Documentation Review done Documentation Review done QA QA
          3d 6h 2m 2 Marcos Loureiro (Inactive) 12/Aug/13 5:56 PM
          QA QA Closed Closed
          29d 18h 7m 1 Marcos Loureiro (Inactive) 11/Sep/13 12:03 PM
          Closed Closed Reopened Reopened
          22d 24m 2 Petar Spanja (Inactive) 03/Oct/13 12:27 PM
          Reopened Reopened Closed Closed
          2s 2 Petar Spanja (Inactive) 03/Oct/13 12:27 PM

            People

            • Assignee:
              Unassigned
              Reporter:
              Petar Spanja (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 1 day Original Estimate - 1 day
                1d
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 3 days, 1 hour, 45 minutes
                3d 1h 45m

                  Agile