{"action":"created","issue":{"url":"https://api.github.com/repos/orika-mapper/orika/issues/142","repository_url":"https://api.github.com/repos/orika-mapper/orika","labels_url":"https://api.github.com/repos/orika-mapper/orika/issues/142/labels{/name}","comments_url":"https://api.github.com/repos/orika-mapper/orika/issues/142/comments","events_url":"https://api.github.com/repos/orika-mapper/orika/issues/142/events","html_url":"https://github.com/orika-mapper/orika/pull/142","id":178512728,"number":142,"title":"Allow user customized MappingStrategies","user":{"login":"oillio","id":205051,"avatar_url":"https://avatars.githubusercontent.com/u/205051?v=3","gravatar_id":"","url":"https://api.github.com/users/oillio","html_url":"https://github.com/oillio","followers_url":"https://api.github.com/users/oillio/followers","following_url":"https://api.github.com/users/oillio/following{/other_user}","gists_url":"https://api.github.com/users/oillio/gists{/gist_id}","starred_url":"https://api.github.com/users/oillio/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/oillio/subscriptions","organizations_url":"https://api.github.com/users/oillio/orgs","repos_url":"https://api.github.com/users/oillio/repos","events_url":"https://api.github.com/users/oillio/events{/privacy}","received_events_url":"https://api.github.com/users/oillio/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-09-22T03:22:59Z","updated_at":"2016-12-09T07:53:14Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/orika-mapper/orika/pulls/142","html_url":"https://github.com/orika-mapper/orika/pull/142","diff_url":"https://github.com/orika-mapper/orika/pull/142.diff","patch_url":"https://github.com/orika-mapper/orika/pull/142.patch"},"body":"I ran into a situation where I needed to write my own MappingStrategy.  I was able to implement this in the current release, but it is not very clean.  It required extending DefaultMapperFactory and MapperFacadeImpl.  It would be nice if I could define and inject my own MappingStrategy without so much trouble.\n\nI wrote up a quick implementation of my line of thinking.  Each MappingStrategy type has a factory, which knows if its strategy can be applied to the requested mapping, and it can create the strategy for the mapping.\nWhen a strategy is required, the system will query each registered factory (user defined first, then default factories).  The first factory that can take the job will produce the strategy.\n\nIf this looks like a feature you would be interested in adding, and this implementation is acceptable, I can clean it up, add documentation and testing.\n"},"comment":{"url":"https://api.github.com/repos/orika-mapper/orika/issues/comments/265954536","html_url":"https://github.com/orika-mapper/orika/pull/142#issuecomment-265954536","issue_url":"https://api.github.com/repos/orika-mapper/orika/issues/142","id":265954536,"user":{"login":"andreabertagnolli","id":8570990,"avatar_url":"https://avatars.githubusercontent.com/u/8570990?v=3","gravatar_id":"","url":"https://api.github.com/users/andreabertagnolli","html_url":"https://github.com/andreabertagnolli","followers_url":"https://api.github.com/users/andreabertagnolli/followers","following_url":"https://api.github.com/users/andreabertagnolli/following{/other_user}","gists_url":"https://api.github.com/users/andreabertagnolli/gists{/gist_id}","starred_url":"https://api.github.com/users/andreabertagnolli/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andreabertagnolli/subscriptions","organizations_url":"https://api.github.com/users/andreabertagnolli/orgs","repos_url":"https://api.github.com/users/andreabertagnolli/repos","events_url":"https://api.github.com/users/andreabertagnolli/events{/privacy}","received_events_url":"https://api.github.com/users/andreabertagnolli/received_events","type":"User","site_admin":false},"created_at":"2016-12-09T07:53:14Z","updated_at":"2016-12-09T07:53:14Z","body":"Hi, can you resolve conflicts and try to merge again? thanks ;)"}}
{"actor":{"display_login":"andreabertagnolli"}}