{"action":"created","issue":{"url":"https://api.github.com/repos/nasa/mct/issues/341","labels_url":"https://api.github.com/repos/nasa/mct/issues/341/labels{/name}","comments_url":"https://api.github.com/repos/nasa/mct/issues/341/comments","events_url":"https://api.github.com/repos/nasa/mct/issues/341/events","html_url":"https://github.com/nasa/mct/pull/341","id":99544546,"number":341,"title":"Removing Objects > Import menu option when there is no valid component selected","user":{"login":"brianwyu","id":4096238,"avatar_url":"https://avatars.githubusercontent.com/u/4096238?v=3","gravatar_id":"","url":"https://api.github.com/users/brianwyu","html_url":"https://github.com/brianwyu","followers_url":"https://api.github.com/users/brianwyu/followers","following_url":"https://api.github.com/users/brianwyu/following{/other_user}","gists_url":"https://api.github.com/users/brianwyu/gists{/gist_id}","starred_url":"https://api.github.com/users/brianwyu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brianwyu/subscriptions","organizations_url":"https://api.github.com/users/brianwyu/orgs","repos_url":"https://api.github.com/users/brianwyu/repos","events_url":"https://api.github.com/users/brianwyu/events{/privacy}","received_events_url":"https://api.github.com/users/brianwyu/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"VWoeltjen","id":1685764,"avatar_url":"https://avatars.githubusercontent.com/u/1685764?v=3","gravatar_id":"","url":"https://api.github.com/users/VWoeltjen","html_url":"https://github.com/VWoeltjen","followers_url":"https://api.github.com/users/VWoeltjen/followers","following_url":"https://api.github.com/users/VWoeltjen/following{/other_user}","gists_url":"https://api.github.com/users/VWoeltjen/gists{/gist_id}","starred_url":"https://api.github.com/users/VWoeltjen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/VWoeltjen/subscriptions","organizations_url":"https://api.github.com/users/VWoeltjen/orgs","repos_url":"https://api.github.com/users/VWoeltjen/repos","events_url":"https://api.github.com/users/VWoeltjen/events{/privacy}","received_events_url":"https://api.github.com/users/VWoeltjen/received_events","type":"User","site_admin":false},"milestone":null,"comments":1,"created_at":"2015-08-06T23:23:49Z","updated_at":"2015-08-06T23:36:37Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/nasa/mct/pulls/341","html_url":"https://github.com/nasa/mct/pull/341","diff_url":"https://github.com/nasa/mct/pull/341.diff","patch_url":"https://github.com/nasa/mct/pull/341.patch"},"body":"Removing Objects > Import menu option when there is no valid component selected in the Browse tree. This option shows when the user is admin. The regular user does not see this option.\r\n\r\nReference Jira: ChillPlots - MPCS-7522.\r\n\r\n- Handles null cases.\r\n- ThisMenu canHandle set to false.\r\n\r\nWrote or re-wrote unit tests (Y/N): Y\r\n Ran entire suite of unit tests successfully (Y/N): Y\r\n Verified bug fix with the described replication steps or verified the implementation meets the requirement cited (Y/N): Y\r\n Code coverage requirements satisfied (new code=80%, changed code=at least as high as existing) (Y/N): N\r\n Code review date and attendees: On pull request\r\n Describe potential impact on other areas of the code: menu\r\n Describe any test prerequisites for this fix (e.g., restart the database):\r\n For bugs marked \"Regression - Yes\", describe the cause of the regression:\r\n Did you modify any pom.xml file? If yes, did you thoroughly document your changes and reasons? N"},"comment":{"url":"https://api.github.com/repos/nasa/mct/issues/comments/128539530","html_url":"https://github.com/nasa/mct/pull/341#issuecomment-128539530","issue_url":"https://api.github.com/repos/nasa/mct/issues/341","id":128539530,"user":{"login":"brianwyu","id":4096238,"avatar_url":"https://avatars.githubusercontent.com/u/4096238?v=3","gravatar_id":"","url":"https://api.github.com/users/brianwyu","html_url":"https://github.com/brianwyu","followers_url":"https://api.github.com/users/brianwyu/followers","following_url":"https://api.github.com/users/brianwyu/following{/other_user}","gists_url":"https://api.github.com/users/brianwyu/gists{/gist_id}","starred_url":"https://api.github.com/users/brianwyu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brianwyu/subscriptions","organizations_url":"https://api.github.com/users/brianwyu/orgs","repos_url":"https://api.github.com/users/brianwyu/repos","events_url":"https://api.github.com/users/brianwyu/events{/privacy}","received_events_url":"https://api.github.com/users/brianwyu/received_events","type":"User","site_admin":false},"created_at":"2015-08-06T23:36:37Z","updated_at":"2015-08-06T23:36:37Z","body":"When first loading MCT, the admin user can access the menu Objects > Import menu. When executing the Import commands this causes unexpected null exceptions (Import Dictionary/Data scenarios where you open the dialog and X out.) and irregular behaviors (importing MCT Objects to what ever is first in the Browse tree.)\r\n\r\nThe fix removes this option of always showing, but rather what is selected in the Browse tree.\r\nNote that a right-click Import on a valid browse element already has the correct behavior of showing the Import menu and menu items. And the fix for the main menu for Import follows suit."}}