{"action":"created","issue":{"url":"https://api.github.com/repos/zzzeek/sqlalchemy/issues/329","repository_url":"https://api.github.com/repos/zzzeek/sqlalchemy","labels_url":"https://api.github.com/repos/zzzeek/sqlalchemy/issues/329/labels{/name}","comments_url":"https://api.github.com/repos/zzzeek/sqlalchemy/issues/329/comments","events_url":"https://api.github.com/repos/zzzeek/sqlalchemy/issues/329/events","html_url":"https://github.com/zzzeek/sqlalchemy/pull/329","id":196606461,"number":329,"title":"Multiple Join Paths in automap","user":{"login":"Martronic-SA","id":6492010,"avatar_url":"https://avatars.githubusercontent.com/u/6492010?v=3","gravatar_id":"","url":"https://api.github.com/users/Martronic-SA","html_url":"https://github.com/Martronic-SA","followers_url":"https://api.github.com/users/Martronic-SA/followers","following_url":"https://api.github.com/users/Martronic-SA/following{/other_user}","gists_url":"https://api.github.com/users/Martronic-SA/gists{/gist_id}","starred_url":"https://api.github.com/users/Martronic-SA/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Martronic-SA/subscriptions","organizations_url":"https://api.github.com/users/Martronic-SA/orgs","repos_url":"https://api.github.com/users/Martronic-SA/repos","events_url":"https://api.github.com/users/Martronic-SA/events{/privacy}","received_events_url":"https://api.github.com/users/Martronic-SA/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-12-20T07:44:01Z","updated_at":"2016-12-20T14:19:48Z","closed_at":"2016-12-20T14:19:48Z","pull_request":{"url":"https://api.github.com/repos/zzzeek/sqlalchemy/pulls/329","html_url":"https://github.com/zzzeek/sqlalchemy/pull/329","diff_url":"https://github.com/zzzeek/sqlalchemy/pull/329.diff","patch_url":"https://github.com/zzzeek/sqlalchemy/pull/329.patch"},"body":"I make a pull request because bitbucket sqlachemy seems broken and there is no issue here.\r\nThe way automap names collections relationships makes impossible to have more than one foreign key path between to tables.\r\nThis can issue in having an erratic behavior as the <referred_cls.__name__.lower() + \"_collection\"> name can to one or another foreign key path.\r\nSo my solution is to add the name of the foreign constraint table to collection name, so there is no more confusion."},"comment":{"url":"https://api.github.com/repos/zzzeek/sqlalchemy/issues/comments/268254228","html_url":"https://github.com/zzzeek/sqlalchemy/pull/329#issuecomment-268254228","issue_url":"https://api.github.com/repos/zzzeek/sqlalchemy/issues/329","id":268254228,"user":{"login":"zzzeek","id":128223,"avatar_url":"https://avatars.githubusercontent.com/u/128223?v=3","gravatar_id":"","url":"https://api.github.com/users/zzzeek","html_url":"https://github.com/zzzeek","followers_url":"https://api.github.com/users/zzzeek/followers","following_url":"https://api.github.com/users/zzzeek/following{/other_user}","gists_url":"https://api.github.com/users/zzzeek/gists{/gist_id}","starred_url":"https://api.github.com/users/zzzeek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zzzeek/subscriptions","organizations_url":"https://api.github.com/users/zzzeek/orgs","repos_url":"https://api.github.com/users/zzzeek/repos","events_url":"https://api.github.com/users/zzzeek/events{/privacy}","received_events_url":"https://api.github.com/users/zzzeek/received_events","type":"User","site_admin":false},"created_at":"2016-12-20T14:19:48Z","updated_at":"2016-12-20T14:19:48Z","body":"unfortunately we can't change defaults because that would break existing applications relying upon them.   Can you please submit a PR that provides additional documentation, illustrating an appropriate naming convention to use as a drop in for the case where multiple FKs between two tables exist (e.g. similar to http://docs.sqlalchemy.org/en/rel_1_1/orm/extensions/automap.html#handling-simple-naming-conflicts)."}}
{"actor":{"display_login":"zzzeek"}}