{"action":"created","issue":{"url":"https://api.github.com/repos/apache/incubator-brooklyn/issues/704","labels_url":"https://api.github.com/repos/apache/incubator-brooklyn/issues/704/labels{/name}","comments_url":"https://api.github.com/repos/apache/incubator-brooklyn/issues/704/comments","events_url":"https://api.github.com/repos/apache/incubator-brooklyn/issues/704/events","html_url":"https://github.com/apache/incubator-brooklyn/pull/704","id":89966298,"number":704,"title":"clean up how locations are rendered in the js gui, and expand \"delete\"","user":{"login":"ahgittin","id":496540,"avatar_url":"https://avatars.githubusercontent.com/u/496540?v=3","gravatar_id":"","url":"https://api.github.com/users/ahgittin","html_url":"https://github.com/ahgittin","followers_url":"https://api.github.com/users/ahgittin/followers","following_url":"https://api.github.com/users/ahgittin/following{/other_user}","gists_url":"https://api.github.com/users/ahgittin/gists{/gist_id}","starred_url":"https://api.github.com/users/ahgittin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ahgittin/subscriptions","organizations_url":"https://api.github.com/users/ahgittin/orgs","repos_url":"https://api.github.com/users/ahgittin/repos","events_url":"https://api.github.com/users/ahgittin/events{/privacy}","received_events_url":"https://api.github.com/users/ahgittin/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-06-21T21:58:09Z","updated_at":"2015-06-21T22:49:19Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/incubator-brooklyn/pulls/704","html_url":"https://github.com/apache/incubator-brooklyn/pull/704","diff_url":"https://github.com/apache/incubator-brooklyn/pull/704.diff","patch_url":"https://github.com/apache/incubator-brooklyn/pull/704.patch"},"body":"there was (is) confusion between catalog name and location displayName and a named location.\r\nit deserves a major sort-out, i think treating location defs as catalog items and location instances as entities;\r\nand only using the catalog api, not a dedicated locations api (or if there is, it delegates to catalog),\r\nalso removing all the LocationManager blah.\r\n\r\nbut until then we needed something which avoided confusion where catalog item names for locations\r\naren't rendered, and magic display names from location-metadata were being showed so it looked\r\nlike nothing was being added, though the location worked.\r\n\r\nfor locations, a catalog item name now does nothing, but the location's name (i.e. human-readable ID aka catalog item ID, not the display name)\r\nis shown in the catalog accordion list (the \"IdentifierName\"), better info (but not the full yaml yet) is shown in the detail,\r\nand versions are at least handled without bugs, even if only one version for location is really supported in the gui.\r\n\r\nalso adds delete for entity and apps and policies, and fixes delete for locations.\r\n\r\n@nakomis this should help prevent the user-confusion which you experienced"},"comment":{"url":"https://api.github.com/repos/apache/incubator-brooklyn/issues/comments/113965692","html_url":"https://github.com/apache/incubator-brooklyn/pull/704#issuecomment-113965692","issue_url":"https://api.github.com/repos/apache/incubator-brooklyn/issues/704","id":113965692,"user":{"login":"asfbot","id":5243949,"avatar_url":"https://avatars.githubusercontent.com/u/5243949?v=3","gravatar_id":"","url":"https://api.github.com/users/asfbot","html_url":"https://github.com/asfbot","followers_url":"https://api.github.com/users/asfbot/followers","following_url":"https://api.github.com/users/asfbot/following{/other_user}","gists_url":"https://api.github.com/users/asfbot/gists{/gist_id}","starred_url":"https://api.github.com/users/asfbot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/asfbot/subscriptions","organizations_url":"https://api.github.com/users/asfbot/orgs","repos_url":"https://api.github.com/users/asfbot/repos","events_url":"https://api.github.com/users/asfbot/events{/privacy}","received_events_url":"https://api.github.com/users/asfbot/received_events","type":"User","site_admin":false},"created_at":"2015-06-21T22:49:18Z","updated_at":"2015-06-21T22:49:18Z","body":"[incubator-brooklyn-pull-requests #1368](https://builds.apache.org/job/incubator-brooklyn-pull-requests/1368/) SUCCESS\nThis pull request looks good"}}