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

[API] When moving an object to an hidden container, the moved object doesn't have the hidden by superior status

    Details

      Description

      In PlatformUI, that uses Rest API and Public API, when I try to move an object to an hidden container, the moved object does not inherit the hidden status

      The following steps may be used in PlatformUI:

      Scenario: Move one object to an hidden location
      Given an "Older News" folder exists
      And "Older News" is hidden
      And a "News Flash" article exists
      And "News Flash" is not hidden
      When I move the "News Flash" as a child of "Older News"
      Then the "News Flash" is successfully moved with message "'News Flash' has been successfully moved under 'Older News'"
      And I see "News Flash" as a child of "Older News"
      And "News Flash" is hidden by superior

      In last step, the article should be "hidden by superior" and it's not.

        Issue Links

          Activity

          Paulo Nunes (Inactive) created issue -
          Paulo Nunes (Inactive) made changes -
          Field Original Value New Value
          Status Open [ 1 ] Confirmed [ 10037 ]
          Paulo Nunes (Inactive) made changes -
          Component/s Platform/Repository & Services (Public API impl) [ 10309 ]
          Paulo Nunes (Inactive) made changes -
          Description In PlatformUI, that uses Rest API, when I try to move an object to an hidden container, the moved object does not inherit the hidden status

          The following steps may be used in PlatformUI:

          *Scenario: Move one object to an hidden location*
          Given an "Older News" folder exists
          And "Older News" is hidden
          And a "News Flash" article exists
          And "News Flash" is not hidden
          When I move the "News Flash" as a child of "Older News"
          Then the "News Flash" is successfully moved with message "'News Flash' has been successfully moved under 'Older News'"
          And I see "News Flash" as a child of "Older News"
          And "News Flash" is hidden

          In last step, the article should be hidden and it's not.
          In PlatformUI, that uses Rest API and Public API, when I try to move an object to an hidden container, the moved object does not inherit the hidden status

          The following steps may be used in PlatformUI:

          *Scenario: Move one object to an hidden location*
          Given an "Older News" folder exists
          And "Older News" is hidden
          And a "News Flash" article exists
          And "News Flash" is not hidden
          When I move the "News Flash" as a child of "Older News"
          Then the "News Flash" is successfully moved with message "'News Flash' has been successfully moved under 'Older News'"
          And I see "News Flash" as a child of "Older News"
          And "News Flash" is hidden

          In last step, the article should be hidden and it's not.
          Paulo Nunes (Inactive) made changes -
          Description In PlatformUI, that uses Rest API and Public API, when I try to move an object to an hidden container, the moved object does not inherit the hidden status

          The following steps may be used in PlatformUI:

          *Scenario: Move one object to an hidden location*
          Given an "Older News" folder exists
          And "Older News" is hidden
          And a "News Flash" article exists
          And "News Flash" is not hidden
          When I move the "News Flash" as a child of "Older News"
          Then the "News Flash" is successfully moved with message "'News Flash' has been successfully moved under 'Older News'"
          And I see "News Flash" as a child of "Older News"
          And "News Flash" is hidden

          In last step, the article should be hidden and it's not.
          In PlatformUI, that uses Rest API and Public API, when I try to copy or move an object to an hidden container, the moved object does not inherit the hidden status

          The following steps may be used in PlatformUI:

          *Scenario: Move one object to an hidden location*
          Given an "Older News" folder exists
          And "Older News" is hidden
          And a "News Flash" article exists
          And "News Flash" is not hidden
          When I move the "News Flash" as a child of "Older News"
          Then the "News Flash" is successfully moved with message "'News Flash' has been successfully moved under 'Older News'"
          And I see "News Flash" as a child of "Older News"
          And "News Flash" is hidden

          In last step, the article should be hidden and it's not.
          Paulo Nunes (Inactive) made changes -
          Summary When moving an object to an hidden container, the moved object is not inhering the hidden status When copying or moving an object to an hidden container, the copied/moved object is not inhering the hidden status
          Pedro Resende (Inactive) made changes -
          Link This issue relates to EZP-24814 [ EZP-24814 ]
          Hide
          Damien Pobel (Inactive) added a comment -

          It seems like you are mixing Hidden and Invisible. If you move a Location A under a Hidden Location, the Location A should be invisible, not hidden. In our domain, hidden != invisible.
          Please read https://doc.ez.no/eZ-Publish/Technical-manual/5.x/Concepts-and-basics/Content-management/Node-visibility

          Show
          Damien Pobel (Inactive) added a comment - It seems like you are mixing Hidden and Invisible. If you move a Location A under a Hidden Location, the Location A should be invisible, not hidden. In our domain, hidden != invisible. Please read https://doc.ez.no/eZ-Publish/Technical-manual/5.x/Concepts-and-basics/Content-management/Node-visibility
          Hide
          Paulo Nunes (Inactive) added a comment -

          Hi,
          The issue was opened under the assumption:
          (from https://doc.ez.no/eZ-Publish/Technical-manual/5.x/Concepts-and-basics/Content-management/Node-visibility)

          Once a node is hidden, all its descendants will automatically be marked "Hidden by superior" and thus the descendants will also become hidden

          Show
          Paulo Nunes (Inactive) added a comment - Hi, The issue was opened under the assumption: (from https://doc.ez.no/eZ-Publish/Technical-manual/5.x/Concepts-and-basics/Content-management/Node-visibility ) Once a node is hidden, all its descendants will automatically be marked "Hidden by superior" and thus the descendants will also become hidden
          Hide
          André Rømcke added a comment -

          Ok, that explains it, but here is the truth:

          Hidden by superior !== Hidden
          Hidden by superior === Invisible

          Show
          André Rømcke added a comment - Ok, that explains it, but here is the truth: Hidden by superior !== Hidden Hidden by superior === Invisible
          Hide
          Paulo Nunes (Inactive) added a comment - - edited

          Ok. Yes, it was "hidden by superior" what i mean. So yes, by that prism, then the expected is invisible.
          I'll retry the test scenario in a master and will see if this is still a bug.

          Show
          Paulo Nunes (Inactive) added a comment - - edited Ok. Yes, it was "hidden by superior" what i mean. So yes, by that prism, then the expected is invisible. I'll retry the test scenario in a master and will see if this is still a bug.
          Paulo Nunes (Inactive) made changes -
          Description In PlatformUI, that uses Rest API and Public API, when I try to copy or move an object to an hidden container, the moved object does not inherit the hidden status

          The following steps may be used in PlatformUI:

          *Scenario: Move one object to an hidden location*
          Given an "Older News" folder exists
          And "Older News" is hidden
          And a "News Flash" article exists
          And "News Flash" is not hidden
          When I move the "News Flash" as a child of "Older News"
          Then the "News Flash" is successfully moved with message "'News Flash' has been successfully moved under 'Older News'"
          And I see "News Flash" as a child of "Older News"
          And "News Flash" is hidden

          In last step, the article should be hidden and it's not.
          In PlatformUI, that uses Rest API and Public API, when I try to copy or move an object to an hidden container, the moved object does not inherit the hidden status

          The following steps may be used in PlatformUI:

          *Scenario: Move one object to an hidden location*
          Given an "Older News" folder exists
          And "Older News" is hidden
          And a "News Flash" article exists
          And "News Flash" is not hidden
          When I move the "News Flash" as a child of "Older News"
          Then the "News Flash" is successfully moved with message "'News Flash' has been successfully moved under 'Older News'"
          And I see "News Flash" as a child of "Older News"
          And "News Flash" is hidden by superior

          In last step, the article should be hidden by superior and it's not.
          Hide
          Paulo Nunes (Inactive) added a comment - - edited

          Tested the scenario again on latest master:

          Scenario Visibility status of Object A after the action Expected visibility status Test result
          Object A Visible, Object B Hidden => Move object A to a location under Object B Visible Hidden by superior Not Ok
          Object A Visible, Object B Hidden => Copy object A to a location under Object B Hidden by superior Hidden by superior Ok
          Object B Hidden => Create object A as child of Object B Hidden by superior Hidden by superior Ok
          Show
          Paulo Nunes (Inactive) added a comment - - edited Tested the scenario again on latest master: Scenario Visibility status of Object A after the action Expected visibility status Test result Object A Visible, Object B Hidden => Move object A to a location under Object B Visible Hidden by superior Not Ok Object A Visible, Object B Hidden => Copy object A to a location under Object B Hidden by superior Hidden by superior Ok Object B Hidden => Create object A as child of Object B Hidden by superior Hidden by superior Ok
          Paulo Nunes (Inactive) made changes -
          Description In PlatformUI, that uses Rest API and Public API, when I try to copy or move an object to an hidden container, the moved object does not inherit the hidden status

          The following steps may be used in PlatformUI:

          *Scenario: Move one object to an hidden location*
          Given an "Older News" folder exists
          And "Older News" is hidden
          And a "News Flash" article exists
          And "News Flash" is not hidden
          When I move the "News Flash" as a child of "Older News"
          Then the "News Flash" is successfully moved with message "'News Flash' has been successfully moved under 'Older News'"
          And I see "News Flash" as a child of "Older News"
          And "News Flash" is hidden by superior

          In last step, the article should be hidden by superior and it's not.
          In PlatformUI, that uses Rest API and Public API, when I try to copy or move an object to an hidden container, the moved object does not inherit the hidden status

          The following steps may be used in PlatformUI:

          *Scenario: Move one object to an hidden location*
          Given an "Older News" folder exists
          And "Older News" is hidden
          And a "News Flash" article exists
          And "News Flash" is not hidden
          When I move the "News Flash" as a child of "Older News"
          Then the "News Flash" is successfully moved with message "'News Flash' has been successfully moved under 'Older News'"
          And I see "News Flash" as a child of "Older News"
          And "News Flash" is hidden by superior

          In last step, the article should be "hidden by superior" and it's not.
          Paulo Nunes (Inactive) made changes -
          Summary When copying or moving an object to an hidden container, the copied/moved object is not inhering the hidden status When moving an object to an hidden container, the moved object is not inhering the hidden status
          Paulo Nunes (Inactive) made changes -
          Description In PlatformUI, that uses Rest API and Public API, when I try to copy or move an object to an hidden container, the moved object does not inherit the hidden status

          The following steps may be used in PlatformUI:

          *Scenario: Move one object to an hidden location*
          Given an "Older News" folder exists
          And "Older News" is hidden
          And a "News Flash" article exists
          And "News Flash" is not hidden
          When I move the "News Flash" as a child of "Older News"
          Then the "News Flash" is successfully moved with message "'News Flash' has been successfully moved under 'Older News'"
          And I see "News Flash" as a child of "Older News"
          And "News Flash" is hidden by superior

          In last step, the article should be "hidden by superior" and it's not.
          In PlatformUI, that uses Rest API and Public API, when I try to move an object to an hidden container, the moved object does not inherit the hidden status

          The following steps may be used in PlatformUI:

          *Scenario: Move one object to an hidden location*
          Given an "Older News" folder exists
          And "Older News" is hidden
          And a "News Flash" article exists
          And "News Flash" is not hidden
          When I move the "News Flash" as a child of "Older News"
          Then the "News Flash" is successfully moved with message "'News Flash' has been successfully moved under 'Older News'"
          And I see "News Flash" as a child of "Older News"
          And "News Flash" is hidden by superior

          In last step, the article should be "hidden by superior" and it's not.
          Hide
          André Rømcke added a comment -

          Clear, thanks Paulo Nunes

          Show
          André Rømcke added a comment - Clear, thanks Paulo Nunes
          André Rømcke made changes -
          Status Confirmed [ 10037 ] InputQ [ 10001 ]
          Hide
          Damien Pobel (Inactive) added a comment -

          > Ok. Yes, it was "hidden by superior" what i mean. So yes, by that prism, then the expected is invisible.

          actually since you are testing the behaviour of the Public API you should use hidden and invisible terms everywhere. "Hidden by superior" is more a UI/end user term.

          Show
          Damien Pobel (Inactive) added a comment - > Ok. Yes, it was "hidden by superior" what i mean. So yes, by that prism, then the expected is invisible. actually since you are testing the behaviour of the Public API you should use hidden and invisible terms everywhere. "Hidden by superior" is more a UI/end user term.
          Hide
          Paulo Nunes (Inactive) added a comment -

          [~damien.pobel@ez.no]: I used the terms used in UI, since the tests were done in there.

          Show
          Paulo Nunes (Inactive) added a comment - [~damien.pobel@ez.no] : I used the terms used in UI, since the tests were done in there.
          Paulo Nunes (Inactive) made changes -
          Summary When moving an object to an hidden container, the moved object is not inhering the hidden status When moving an object to an hidden container, the moved object is not inheriting the hidden status
          Bertrand Dunogier made changes -
          Priority High [ 3 ] Critical [ 2 ]
          Bertrand Dunogier made changes -
          Summary When moving an object to an hidden container, the moved object is not inheriting the hidden status When moving an object to an hidden container, the moved object doesn't have the hidden by superior status
          Hide
          Bertrand Dunogier added a comment -

          Did anyone try to clear the SPI (ezpublish/cache/*/stash) cache to see if it changes anything ?

          Show
          Bertrand Dunogier added a comment - Did anyone try to clear the SPI (ezpublish/cache/*/stash) cache to see if it changes anything ?
          André Rømcke made changes -
          Status InputQ [ 10001 ] Development [ 3 ]
          Assignee André Rømcke [ andre.romcke@ez.no ]
          Hide
          André Rømcke added a comment -
          Show
          André Rømcke added a comment - Issue is in DB layer. PR: https://github.com/ezsystems/ezpublish-kernel/pull/1528
          André Rømcke made changes -
          Status Development [ 3 ] Development Review [ 10006 ]
          Show
          André Rømcke added a comment - Merged in: https://github.com/ezsystems/ezpublish-kernel/commit/960eeb66c004d03ddf4a5c2f64d3d001426cad20
          André Rømcke made changes -
          Status Development Review [ 10006 ] Documentation Review done [ 10011 ]
          Affects Version/s 5.4.5 [ 14490 ]
          Affects Version/s 5.3.7 [ 14379 ]
          Fix Version/s 5.3.8 [ 14382 ]
          Fix Version/s 5.4.6 [ 14493 ]
          Fix Version/s 2015.12 [ 14489 ]
          Assignee André Rømcke [ andre.romcke@ez.no ]
          André Rømcke made changes -
          Summary When moving an object to an hidden container, the moved object doesn't have the hidden by superior status [API] When moving an object to an hidden container, the moved object doesn't have the hidden by superior status
          Rui Silva (Inactive) made changes -
          Status Documentation Review done [ 10011 ] QA [ 10008 ]
          Bertrand Dunogier made changes -
          Affects Version/s 2015.12 [ 14489 ]
          Hide
          Rui Silva (Inactive) added a comment -

          Tested and approved by QA for 5.3, 5.4 and master.

          Show
          Rui Silva (Inactive) added a comment - Tested and approved by QA for 5.3, 5.4 and master.
          Rui Silva (Inactive) made changes -
          Assignee Rui Silva [ rui.silva@ez.no ]
          Status QA [ 10008 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          Rui Silva (Inactive) made changes -
          Resolution Fixed [ 1 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          Assignee Rui Silva [ rui.silva@ez.no ]
          Rui Silva (Inactive) made changes -
          Status Reopened [ 4 ] Confirmed [ 10037 ]
          Rui Silva (Inactive) made changes -
          Status Confirmed [ 10037 ] InputQ [ 10001 ]
          Rui Silva (Inactive) made changes -
          Status InputQ [ 10001 ] Development Review done [ 10028 ]
          Assignee Rui Silva [ rui.silva@ez.no ]
          Rui Silva (Inactive) made changes -
          Status Development Review done [ 10028 ] Documentation Review done [ 10011 ]
          Rui Silva (Inactive) made changes -
          Status Documentation Review done [ 10011 ] QA [ 10008 ]
          Rui Silva (Inactive) made changes -
          Flagged Impediment [ 10000 ]
          Bertrand Dunogier made changes -
          Fix Version/s 2015.12 [ 14489 ]
          Rui Silva (Inactive) made changes -
          Status QA [ 10008 ] InputQ [ 10001 ]
          Assignee Rui Silva [ rui.silva@ez.no ]
          André Rømcke made changes -
          Status InputQ [ 10001 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          Bertrand Dunogier made changes -
          Resolution Fixed [ 1 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          Bertrand Dunogier made changes -
          Fix Version/s 2015.12.1 [ 14503 ]
          Bertrand Dunogier made changes -
          Status Reopened [ 4 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          André Rømcke made changes -
          Remote Link This issue links to "Page (eZ Documentation)" [ 16165 ]
          Dominika Kurek made changes -
          Remote Link This issue links to "Page (eZ Documentation)" [ 16409 ]
          Dominika Kurek made changes -
          Remote Link This issue links to "Page (eZ Documentation)" [ 16409 ] This issue links to "Page (eZ Documentation)" [ 16409 ]
          Sarah Haïm-Lubczanski (Inactive) made changes -
          Remote Link This issue links to "Page (eZ Documentation)" [ 16409 ] This issue links to "Page (eZ Documentation)" [ 16409 ]
          Sarah Haïm-Lubczanski (Inactive) made changes -
          Remote Link This issue links to "Page (eZ Documentation)" [ 16409 ] This issue links to "Page (eZ Documentation)" [ 16409 ]
          Dominika Kurek made changes -
          Remote Link This issue links to "Page (eZ Documentation)" [ 16409 ] This issue links to "Page (eZ Documentation)" [ 16409 ]
          Dominika Kurek made changes -
          Remote Link This issue links to "Page (eZ Documentation)" [ 16409 ] This issue links to "Page (eZ Documentation)" [ 16409 ]
          Dominika Kurek made changes -
          Remote Link This issue links to "Page (eZ Documentation)" [ 16409 ] This issue links to "Page (eZ Documentation)" [ 16409 ]
          Dominika Kurek made changes -
          Remote Link This issue links to "Page (eZ Documentation)" [ 16409 ] This issue links to "Page (eZ Documentation)" [ 16409 ]
          Dominika Kurek made changes -
          Remote Link This issue links to "Page (eZ Documentation)" [ 16409 ] This issue links to "Page (eZ Documentation)" [ 16409 ]
          Alex Schuster made changes -
          Workflow EZ* Development Workflow [ 85944 ] EZEE Development Workflow [ 124870 ]
          Petar Španja made changes -
          Link This issue relates to EZP-30147 [ EZP-30147 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Confirmed Confirmed
          7s 1 Paulo Nunes 02/Jun/15 6:02 PM
          InputQ InputQ Development Development
          50d 16h 1m 1 André Rømcke 03/Dec/15 10:23 AM
          Development Development Development Review Development Review
          45m 14s 1 André Rømcke 03/Dec/15 11:08 AM
          Development Review Development Review Documentation Review done Documentation Review done
          11d 5h 1m 1 André Rømcke 14/Dec/15 4:10 PM
          QA QA Closed Closed
          18h 37m 1 rui.silva@ez.no 15/Dec/15 12:11 PM
          Reopened Reopened Confirmed Confirmed
          5s 1 rui.silva@ez.no 15/Dec/15 12:20 PM
          Confirmed Confirmed InputQ InputQ
          133d 1h 19m 2 rui.silva@ez.no 15/Dec/15 12:20 PM
          InputQ InputQ Development Review done Development Review done
          11s 1 rui.silva@ez.no 15/Dec/15 12:21 PM
          Development Review done Development Review done Documentation Review done Documentation Review done
          7s 1 rui.silva@ez.no 15/Dec/15 12:21 PM
          Documentation Review done Documentation Review done QA QA
          1h 23m 2 rui.silva@ez.no 15/Dec/15 12:21 PM
          QA QA InputQ InputQ
          6d 23h 15m 1 rui.silva@ez.no 22/Dec/15 11:36 AM
          InputQ InputQ Closed Closed
          1d 3h 6m 1 André Rømcke 23/Dec/15 2:43 PM
          Closed Closed Reopened Reopened
          27d 2h 2 Bertrand Dunogier 19/Jan/16 4:35 PM
          Reopened Reopened Closed Closed
          2m 36s 1 Bertrand Dunogier 19/Jan/16 4:37 PM

            People

            • Assignee:
              Unassigned
              Reporter:
              Paulo Nunes (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: