{"action":"created","issue":{"url":"https://api.github.com/repos/Slicer/Slicer/issues/313","labels_url":"https://api.github.com/repos/Slicer/Slicer/issues/313/labels{/name}","comments_url":"https://api.github.com/repos/Slicer/Slicer/issues/313/comments","events_url":"https://api.github.com/repos/Slicer/Slicer/issues/313/events","html_url":"https://github.com/Slicer/Slicer/pull/313","id":98999825,"number":313,"title":"ENH: check for data that will get deleted on scene view restore","user":{"login":"naucoin","id":289054,"avatar_url":"https://avatars.githubusercontent.com/u/289054?v=3","gravatar_id":"","url":"https://api.github.com/users/naucoin","html_url":"https://github.com/naucoin","followers_url":"https://api.github.com/users/naucoin/followers","following_url":"https://api.github.com/users/naucoin/following{/other_user}","gists_url":"https://api.github.com/users/naucoin/gists{/gist_id}","starred_url":"https://api.github.com/users/naucoin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/naucoin/subscriptions","organizations_url":"https://api.github.com/users/naucoin/orgs","repos_url":"https://api.github.com/users/naucoin/repos","events_url":"https://api.github.com/users/naucoin/events{/privacy}","received_events_url":"https://api.github.com/users/naucoin/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2015-08-04T15:45:57Z","updated_at":"2015-08-05T18:03:50Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Slicer/Slicer/pulls/313","html_url":"https://github.com/Slicer/Slicer/pull/313","diff_url":"https://github.com/Slicer/Slicer/pull/313.diff","patch_url":"https://github.com/Slicer/Slicer/pull/313.patch"},"body":"Added a flag to the scene view node restore call that will cancel if\r\nany nodes come up as being in the main Slicer scene but not in the\r\nscene view scene about to be restored.\r\nAdded a method to add nodes to the scene view scene that are in the\r\nmain Slicer scene but not in the scene view.\r\nUpdated the GUI for restoring scenes via the tool bar and the Scene\r\nViews module to pop up message boxes to give the user options to\r\ndiscard the data, save it to the scene view, or cancel the restore.\r\nUpdated the test to include a failure case when removing nodes\r\nis not allowed.\r\nUnify scene view node to use SafeDownCast instead of a mix of\r\ndynamic_cast and SafeDownCast.\r\n\r\nIssue #3956"},"comment":{"url":"https://api.github.com/repos/Slicer/Slicer/issues/comments/128093359","html_url":"https://github.com/Slicer/Slicer/pull/313#issuecomment-128093359","issue_url":"https://api.github.com/repos/Slicer/Slicer/issues/313","id":128093359,"user":{"login":"jcfr","id":219043,"avatar_url":"https://avatars.githubusercontent.com/u/219043?v=3","gravatar_id":"","url":"https://api.github.com/users/jcfr","html_url":"https://github.com/jcfr","followers_url":"https://api.github.com/users/jcfr/followers","following_url":"https://api.github.com/users/jcfr/following{/other_user}","gists_url":"https://api.github.com/users/jcfr/gists{/gist_id}","starred_url":"https://api.github.com/users/jcfr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jcfr/subscriptions","organizations_url":"https://api.github.com/users/jcfr/orgs","repos_url":"https://api.github.com/users/jcfr/repos","events_url":"https://api.github.com/users/jcfr/events{/privacy}","received_events_url":"https://api.github.com/users/jcfr/received_events","type":"User","site_admin":false},"created_at":"2015-08-05T18:03:50Z","updated_at":"2015-08-05T18:03:50Z","body":"Looking good :+1: \r\n\r\nThat said, since clicking on \"Restore scene view\" does not check if added data are already present in other scene view. I would prefer the sentence in the middle to look like this:\r\n\r\n```\r\nIf you don't save, data not already saved to file or added in an other scene views will be permanently lost.\r\n```\r\n\r\nMinor tweak. I suggest we keep only the question in bold. "}}