{"action":"created","issue":{"url":"https://api.github.com/repos/civicrm/civicrm-core/issues/12679","repository_url":"https://api.github.com/repos/civicrm/civicrm-core","labels_url":"https://api.github.com/repos/civicrm/civicrm-core/issues/12679/labels{/name}","comments_url":"https://api.github.com/repos/civicrm/civicrm-core/issues/12679/comments","events_url":"https://api.github.com/repos/civicrm/civicrm-core/issues/12679/events","html_url":"https://github.com/civicrm/civicrm-core/pull/12679","id":351234640,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA4ODgyMjA0","number":12679,"title":"dev/core#334 Fix Contribution Page with Checksum and multiple Payment Processors","user":{"login":"mlutfy","id":254741,"node_id":"MDQ6VXNlcjI1NDc0MQ==","avatar_url":"https://avatars3.githubusercontent.com/u/254741?v=4","gravatar_id":"","url":"https://api.github.com/users/mlutfy","html_url":"https://github.com/mlutfy","followers_url":"https://api.github.com/users/mlutfy/followers","following_url":"https://api.github.com/users/mlutfy/following{/other_user}","gists_url":"https://api.github.com/users/mlutfy/gists{/gist_id}","starred_url":"https://api.github.com/users/mlutfy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mlutfy/subscriptions","organizations_url":"https://api.github.com/users/mlutfy/orgs","repos_url":"https://api.github.com/users/mlutfy/repos","events_url":"https://api.github.com/users/mlutfy/events{/privacy}","received_events_url":"https://api.github.com/users/mlutfy/received_events","type":"User","site_admin":false},"labels":[{"id":246037889,"node_id":"MDU6TGFiZWwyNDYwMzc4ODk=","url":"https://api.github.com/repos/civicrm/civicrm-core/labels/master","name":"master","color":"0052cc","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2018-08-16T14:38:48Z","updated_at":"2018-08-27T05:27:24Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/civicrm/civicrm-core/pulls/12679","html_url":"https://github.com/civicrm/civicrm-core/pull/12679","diff_url":"https://github.com/civicrm/civicrm-core/pull/12679.diff","patch_url":"https://github.com/civicrm/civicrm-core/pull/12679.patch"},"body":"Overview\r\n----------------------------------------\r\n\r\nPublic contribution form and Checksums: billing information not loaded if using multiple processors\r\n\r\nHow to reproduce:\r\n\r\n- Create a Contribution Page\r\n- Associate multiple payment processors\r\n- Create a contact\r\n- Fill in a billing address for the contact\r\n- Generate a contribution page link that includes the checksum (cs) and cid.\r\n\r\nWhat happens:\r\n\r\n- The payment block does not pre-populate the Billing Address\r\n- Note that this only happens when there are multiple processors, because of the ajax request that loads the payment block. This request does not happen when there is only one PP.\r\n\r\nBefore\r\n----------------------------------------\r\n\r\n![core-334-billing-block-empty](https://user-images.githubusercontent.com/254741/44215398-65a39b80-a140-11e8-9494-be97528bf120.gif)\r\n\r\nAfter\r\n----------------------------------------\r\n\r\n![core-334-billing-block-works](https://user-images.githubusercontent.com/254741/44215407-6c321300-a140-11e8-86a9-56c94d2c9e49.gif)\r\n\r\n\r\nTechnical Details\r\n----------------------------------------\r\n\r\nSeems like a copy-paste error in the code. According to the git blame, goes back to 2015.\r\n"},"comment":{"url":"https://api.github.com/repos/civicrm/civicrm-core/issues/comments/416118091","html_url":"https://github.com/civicrm/civicrm-core/pull/12679#issuecomment-416118091","issue_url":"https://api.github.com/repos/civicrm/civicrm-core/issues/12679","id":416118091,"node_id":"MDEyOklzc3VlQ29tbWVudDQxNjExODA5MQ==","user":{"login":"eileenmcnaughton","id":336308,"node_id":"MDQ6VXNlcjMzNjMwOA==","avatar_url":"https://avatars2.githubusercontent.com/u/336308?v=4","gravatar_id":"","url":"https://api.github.com/users/eileenmcnaughton","html_url":"https://github.com/eileenmcnaughton","followers_url":"https://api.github.com/users/eileenmcnaughton/followers","following_url":"https://api.github.com/users/eileenmcnaughton/following{/other_user}","gists_url":"https://api.github.com/users/eileenmcnaughton/gists{/gist_id}","starred_url":"https://api.github.com/users/eileenmcnaughton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/eileenmcnaughton/subscriptions","organizations_url":"https://api.github.com/users/eileenmcnaughton/orgs","repos_url":"https://api.github.com/users/eileenmcnaughton/repos","events_url":"https://api.github.com/users/eileenmcnaughton/events{/privacy}","received_events_url":"https://api.github.com/users/eileenmcnaughton/received_events","type":"User","site_admin":false},"created_at":"2018-08-27T05:27:24Z","updated_at":"2018-08-27T05:27:24Z","author_association":"CONTRIBUTOR","body":"merging based on @jitendrapurohit review. Patch is constrained so testing seems appropriate"}}
{"actor":{"display_login":"eileenmcnaughton"}}