{"action":"created","issue":{"url":"https://api.github.com/repos/silverstripe/silverstripe-cms/issues/1237","labels_url":"https://api.github.com/repos/silverstripe/silverstripe-cms/issues/1237/labels{/name}","comments_url":"https://api.github.com/repos/silverstripe/silverstripe-cms/issues/1237/comments","events_url":"https://api.github.com/repos/silverstripe/silverstripe-cms/issues/1237/events","html_url":"https://github.com/silverstripe/silverstripe-cms/issues/1237","id":92270651,"number":1237,"title":"Issues with new \"archived\" state","user":{"login":"sweddell","id":75909,"avatar_url":"https://avatars.githubusercontent.com/u/75909?v=3","gravatar_id":"","url":"https://api.github.com/users/sweddell","html_url":"https://github.com/sweddell","followers_url":"https://api.github.com/users/sweddell/followers","following_url":"https://api.github.com/users/sweddell/following{/other_user}","gists_url":"https://api.github.com/users/sweddell/gists{/gist_id}","starred_url":"https://api.github.com/users/sweddell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sweddell/subscriptions","organizations_url":"https://api.github.com/users/sweddell/orgs","repos_url":"https://api.github.com/users/sweddell/repos","events_url":"https://api.github.com/users/sweddell/events{/privacy}","received_events_url":"https://api.github.com/users/sweddell/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/silverstripe/silverstripe-cms/labels/blocker","name":"blocker","color":"eb6420"},{"url":"https://api.github.com/repos/silverstripe/silverstripe-cms/labels/frontend","name":"frontend","color":"0052cc"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/silverstripe/silverstripe-cms/milestones/18","html_url":"https://github.com/silverstripe/silverstripe-cms/milestones/3.2.0%20beta%202","labels_url":"https://api.github.com/repos/silverstripe/silverstripe-cms/milestones/18/labels","id":1171985,"number":18,"title":"3.2.0 beta 2","description":"","creator":{"login":"tractorcow","id":936064,"avatar_url":"https://avatars.githubusercontent.com/u/936064?v=3","gravatar_id":"","url":"https://api.github.com/users/tractorcow","html_url":"https://github.com/tractorcow","followers_url":"https://api.github.com/users/tractorcow/followers","following_url":"https://api.github.com/users/tractorcow/following{/other_user}","gists_url":"https://api.github.com/users/tractorcow/gists{/gist_id}","starred_url":"https://api.github.com/users/tractorcow/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tractorcow/subscriptions","organizations_url":"https://api.github.com/users/tractorcow/orgs","repos_url":"https://api.github.com/users/tractorcow/repos","events_url":"https://api.github.com/users/tractorcow/events{/privacy}","received_events_url":"https://api.github.com/users/tractorcow/received_events","type":"User","site_admin":false},"open_issues":3,"closed_issues":1,"state":"open","created_at":"2015-06-19T08:16:14Z","updated_at":"2015-07-07T13:28:13Z","due_on":null,"closed_at":null},"comments":7,"created_at":"2015-07-01T05:08:03Z","updated_at":"2015-07-08T06:03:15Z","closed_at":null,"body":"Giving the new 3.2 beta a test drive and have found issues with how the \"archived\" state is treated. A common usecase for us is the need to restore an entire section when a user archives a page, either knowingly or unsuspectingly, and there were subpages in the tree that needed to be moved before archiving, or the whole lot needed to remain in the case of an unscheduled archive. \r\nArchiving a page with children,\r\n![screenshot-2015-07-01 13-25-14](https://cloud.githubusercontent.com/assets/75909/8447319/f28dd018-1ff9-11e5-837a-c77edfcf939e.png)\r\nResults in the children pages being archived, as expected.\r\n![screenshot-2015-07-01 13-25-55](https://cloud.githubusercontent.com/assets/75909/8447328/236be468-1ffa-11e5-8e3e-605a30a7b2d7.png)\r\nThere needs to be a \"restore\" action in the Multi-selection dropdown, especially if a very large section was archived and needs to be returned to its former glory.\r\n![screenshot-2015-07-01 13-28-38](https://cloud.githubusercontent.com/assets/75909/8447431/0af3938e-1ffc-11e5-8fe0-0373805c2dd4.png)\r\nRestoring the subpages individually only gives an option to restore to the top level, which is expected if the parent is archived, but not if the page is moved to an active page first.\r\n![screenshot-2015-07-01 14-42-54](https://cloud.githubusercontent.com/assets/75909/8447675/8d761856-1fff-11e5-8224-e13cb58bd196.png)\r\nHowever, if the parent page that was archived is restored, then the expected \"restore draft\" option is available, as a child of the same restored parent or another parent in the tree if it is moved.\r\n\r\n![screenshot-2015-07-01 14-48-58](https://cloud.githubusercontent.com/assets/75909/8447725/6ce5c676-2000-11e5-91cb-fcd65786a0ea.png)\r\n![screenshot-2015-07-01 14-51-11](https://cloud.githubusercontent.com/assets/75909/8447731/b66fab7c-2000-11e5-8741-c9ec17873f89.png)\r\nOn restoring, or the tree refreshing, the page returns to the previous parent. It is probably a good idea to restrict the ability to move the page unless restored first. I understand what is going on here and the message about the move not taking effect until the page is saved relates to the reason the page moves back, but a CMS user won't understand that and will think that it should stay in the place they moved it to the same as if the moved a published or draft page.\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/silverstripe/silverstripe-cms/issues/comments/119448329","html_url":"https://github.com/silverstripe/silverstripe-cms/issues/1237#issuecomment-119448329","issue_url":"https://api.github.com/repos/silverstripe/silverstripe-cms/issues/1237","id":119448329,"user":{"login":"clarkepaul","id":555033,"avatar_url":"https://avatars.githubusercontent.com/u/555033?v=3","gravatar_id":"","url":"https://api.github.com/users/clarkepaul","html_url":"https://github.com/clarkepaul","followers_url":"https://api.github.com/users/clarkepaul/followers","following_url":"https://api.github.com/users/clarkepaul/following{/other_user}","gists_url":"https://api.github.com/users/clarkepaul/gists{/gist_id}","starred_url":"https://api.github.com/users/clarkepaul/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/clarkepaul/subscriptions","organizations_url":"https://api.github.com/users/clarkepaul/orgs","repos_url":"https://api.github.com/users/clarkepaul/repos","events_url":"https://api.github.com/users/clarkepaul/events{/privacy}","received_events_url":"https://api.github.com/users/clarkepaul/received_events","type":"User","site_admin":false},"created_at":"2015-07-08T06:03:15Z","updated_at":"2015-07-08T06:03:15Z","body":"We were aware of these issues stated by @sweddell when going about our creation of the archive functionality and ideally if we had the time we would have included it in our work. But because our work was directed by user testing we did, we had a different aim and a set amount of time to do the work so we didn't try and fix all of the existing issues. \r\n\r\nAdjusting the message which notifies the user about the child pages should be an easy addition that could be added sooner than the other things (nice suggestion).\r\n\r\nWe discussed giving the user options to restore to the old location and/or select a new location (if it wasn't available).\r\n\r\nThe bug of being able to move archived pages or newly restored page and it not saving is a pre-existing bug (might be logged somewhere else?)."}}