{"action":"created","issue":{"url":"https://api.github.com/repos/neos/neos-development-collection/issues/51","labels_url":"https://api.github.com/repos/neos/neos-development-collection/issues/51/labels{/name}","comments_url":"https://api.github.com/repos/neos/neos-development-collection/issues/51/comments","events_url":"https://api.github.com/repos/neos/neos-development-collection/issues/51/events","html_url":"https://github.com/neos/neos-development-collection/pull/51","id":105351201,"number":51,"title":"[!!!][TASK] Use identifier instead of node path for PluginView reference","user":{"login":"aertmann","id":903567,"avatar_url":"https://avatars.githubusercontent.com/u/903567?v=3","gravatar_id":"","url":"https://api.github.com/users/aertmann","html_url":"https://github.com/aertmann","followers_url":"https://api.github.com/users/aertmann/followers","following_url":"https://api.github.com/users/aertmann/following{/other_user}","gists_url":"https://api.github.com/users/aertmann/gists{/gist_id}","starred_url":"https://api.github.com/users/aertmann/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aertmann/subscriptions","organizations_url":"https://api.github.com/users/aertmann/orgs","repos_url":"https://api.github.com/users/aertmann/repos","events_url":"https://api.github.com/users/aertmann/events{/privacy}","received_events_url":"https://api.github.com/users/aertmann/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/neos/neos-development-collection/labels/Neos","name":"Neos","color":"009800"},{"url":"https://api.github.com/repos/neos/neos-development-collection/labels/PHP","name":"PHP","color":"fbca04"},{"url":"https://api.github.com/repos/neos/neos-development-collection/labels/Task","name":"Task","color":"0052cc"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":13,"created_at":"2015-09-08T10:42:33Z","updated_at":"2015-11-05T08:00:50Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/neos/neos-development-collection/pulls/51","html_url":"https://github.com/neos/neos-development-collection/pull/51","diff_url":"https://github.com/neos/neos-development-collection/pull/51.diff","patch_url":"https://github.com/neos/neos-development-collection/pull/51.patch"},"body":"Replaces the usage of node path for referencing the master plugin in PluginView.\r\nInstead the identifier is used, like with node linking/reference properties.\r\nThis is done to make it more bulletproof in case the master plugin is moved.\r\n\r\nRun the following node migration to adjust existing PluginViews::\r\n\r\n  ./flow node:migrate 20150907194505\r\n\r\nResolves: NEOS-1501"},"comment":{"url":"https://api.github.com/repos/neos/neos-development-collection/issues/comments/153982091","html_url":"https://github.com/neos/neos-development-collection/pull/51#issuecomment-153982091","issue_url":"https://api.github.com/repos/neos/neos-development-collection/issues/51","id":153982091,"user":{"login":"robertlemke","id":95582,"avatar_url":"https://avatars.githubusercontent.com/u/95582?v=3","gravatar_id":"","url":"https://api.github.com/users/robertlemke","html_url":"https://github.com/robertlemke","followers_url":"https://api.github.com/users/robertlemke/followers","following_url":"https://api.github.com/users/robertlemke/following{/other_user}","gists_url":"https://api.github.com/users/robertlemke/gists{/gist_id}","starred_url":"https://api.github.com/users/robertlemke/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/robertlemke/subscriptions","organizations_url":"https://api.github.com/users/robertlemke/orgs","repos_url":"https://api.github.com/users/robertlemke/repos","events_url":"https://api.github.com/users/robertlemke/events{/privacy}","received_events_url":"https://api.github.com/users/robertlemke/received_events","type":"User","site_admin":false},"created_at":"2015-11-05T08:00:50Z","updated_at":"2015-11-05T08:00:50Z","body":"In the past we said that having to execute a Doctrine Migration doesn't meant it's a breaking change. So if [!!!] really only means: look at this, you'll need to do something during deployment, then I'm fine with merging it. If it means: you need to adjust your code or some existing functionality will break, then don't.\r\n\r\nIf it is breaking we need to clearly state: what are the advantages of merging this now? Does it justify the hassle of all users having to go through a major upgrade? What happens, from the user's perspective, if we don't merge this for the next version?\r\n\r\nCan you make something out of my reply? Or is it just riddles? ;-)"}}