{"action":"opened","issue":{"url":"https://api.github.com/repos/sjka/sample/issues/59","labels_url":"https://api.github.com/repos/sjka/sample/issues/59/labels{/name}","comments_url":"https://api.github.com/repos/sjka/sample/issues/59/comments","events_url":"https://api.github.com/repos/sjka/sample/issues/59/events","html_url":"https://github.com/sjka/sample/issues/59","id":120135299,"number":59,"title":"BaseThingHandler.thingUpdated has no copy of the old thing","user":{"login":"sjka","id":2592443,"avatar_url":"https://avatars.githubusercontent.com/u/2592443?v=3","gravatar_id":"","url":"https://api.github.com/users/sjka","html_url":"https://github.com/sjka","followers_url":"https://api.github.com/users/sjka/followers","following_url":"https://api.github.com/users/sjka/following{/other_user}","gists_url":"https://api.github.com/users/sjka/gists{/gist_id}","starred_url":"https://api.github.com/users/sjka/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sjka/subscriptions","organizations_url":"https://api.github.com/users/sjka/orgs","repos_url":"https://api.github.com/users/sjka/repos","events_url":"https://api.github.com/users/sjka/events{/privacy}","received_events_url":"https://api.github.com/users/sjka/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/sjka/sample/labels/bug","name":"bug","color":"fc2929"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-12-03T10:10:51Z","updated_at":"2015-12-03T10:10:51Z","closed_at":null,"body":"Automaticall migrated from Bugzilla [#468613](https://bugs.eclipse.org/bugs/show_bug.cgi?id=468613)\nstatus NEW severity *normal* in component *Core* for *---*\nReported in version *0.8.0* on platform *PC*\nAssigned to: Project Inbox\n\nOn 2015-05-28 07:44:00 -0400, Andre Fuechsel wrote:\n\n> The class member thing of the BaseThingHandler does not contain the old configuration before thingUpdated was called. The argument thing of this method is actually a reference to the same object that has already set the new configuration. \n> \n> Suggested solution: the argument thing should be a COPY of the old thing with the new configuration so the class member thing still would contain the old configuration and thingUpdated implementations could easily compare the old with the new configuration to see, what's changed.\n\nOn 2015-05-28 10:07:24 -0400, karel goderis wrote:\n\n> Or you can keep as is, and it is up to the ThingHandler to store a local copy of the old configuration?\n\nOn 2015-05-29 02:45:13 -0400, Andre Fuechsel wrote:\n\n> Yes, but storing a copy of the old config by myself looks like a workaround for we.\n\nOn 2015-06-01 04:52:12 -0400, Dennis Nobel wrote:\n\n> I agree with Andre, especially the original thing reference should not be changed, before thingUpdated is called. There is something wrong with the handling of entities in the Core.\n\n"}}