{"action":"created","issue":{"url":"https://api.github.com/repos/zendframework/zend-inputfilter/issues/172","repository_url":"https://api.github.com/repos/zendframework/zend-inputfilter","labels_url":"https://api.github.com/repos/zendframework/zend-inputfilter/issues/172/labels{/name}","comments_url":"https://api.github.com/repos/zendframework/zend-inputfilter/issues/172/comments","events_url":"https://api.github.com/repos/zendframework/zend-inputfilter/issues/172/events","html_url":"https://github.com/zendframework/zend-inputfilter/pull/172","id":351394528,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA5MDA0NDUy","number":172,"title":"Add file input compatibility with PSR-7","user":{"login":"alextech","id":3820828,"node_id":"MDQ6VXNlcjM4MjA4Mjg=","avatar_url":"https://avatars1.githubusercontent.com/u/3820828?v=4","gravatar_id":"","url":"https://api.github.com/users/alextech","html_url":"https://github.com/alextech","followers_url":"https://api.github.com/users/alextech/followers","following_url":"https://api.github.com/users/alextech/following{/other_user}","gists_url":"https://api.github.com/users/alextech/gists{/gist_id}","starred_url":"https://api.github.com/users/alextech/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alextech/subscriptions","organizations_url":"https://api.github.com/users/alextech/orgs","repos_url":"https://api.github.com/users/alextech/repos","events_url":"https://api.github.com/users/alextech/events{/privacy}","received_events_url":"https://api.github.com/users/alextech/received_events","type":"User","site_admin":false},"labels":[{"id":209128153,"node_id":"MDU6TGFiZWwyMDkxMjgxNTM=","url":"https://api.github.com/repos/zendframework/zend-inputfilter/labels/enhancement","name":"enhancement","color":"84b6eb","default":true}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/zendframework/zend-inputfilter/milestones/20","html_url":"https://github.com/zendframework/zend-inputfilter/milestone/20","labels_url":"https://api.github.com/repos/zendframework/zend-inputfilter/milestones/20/labels","id":2958770,"node_id":"MDk6TWlsZXN0b25lMjk1ODc3MA==","number":20,"title":"2.9.0","description":"next feature release","creator":{"login":"froschdesign","id":103362,"node_id":"MDQ6VXNlcjEwMzM2Mg==","avatar_url":"https://avatars2.githubusercontent.com/u/103362?v=4","gravatar_id":"","url":"https://api.github.com/users/froschdesign","html_url":"https://github.com/froschdesign","followers_url":"https://api.github.com/users/froschdesign/followers","following_url":"https://api.github.com/users/froschdesign/following{/other_user}","gists_url":"https://api.github.com/users/froschdesign/gists{/gist_id}","starred_url":"https://api.github.com/users/froschdesign/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/froschdesign/subscriptions","organizations_url":"https://api.github.com/users/froschdesign/orgs","repos_url":"https://api.github.com/users/froschdesign/repos","events_url":"https://api.github.com/users/froschdesign/events{/privacy}","received_events_url":"https://api.github.com/users/froschdesign/received_events","type":"User","site_admin":false},"open_issues":0,"closed_issues":2,"state":"closed","created_at":"2017-12-05T09:02:49Z","updated_at":"2018-12-17T21:31:58Z","due_on":null,"closed_at":"2018-12-17T21:31:58Z"},"comments":7,"created_at":"2018-08-16T22:45:37Z","updated_at":"2018-12-21T09:15:45Z","closed_at":"2018-12-17T21:28:56Z","author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/zendframework/zend-inputfilter/pulls/172","html_url":"https://github.com/zendframework/zend-inputfilter/pull/172","diff_url":"https://github.com/zendframework/zend-inputfilter/pull/172.diff","patch_url":"https://github.com/zendframework/zend-inputfilter/pull/172.patch"},"body":"Fixes #145 - PSR-7 compatibility for validating and filtering forms with inputs.\r\n\r\nThis patch allows validating forms submitted via a PSR-7 payload, and, specifically file uploads.\r\n\r\n**Prerequisite PRs**:\r\n- [x] Validation: https://github.com/zendframework/zend-validator/pull/237\r\n- [x] Filter: https://github.com/zendframework/zend-filter/pull/70\r\n\r\nI could not separate PSR vs HTTP $_FilES into completely separate subclasses because current implementation is stateful, so cannot simply pass calls from current `FileInput` to appropriate implementation - private protected properties that hold values and options get lost. `FileInput` is hardcoded in other repositories, eg `Zend\\Form` as a input specification so has to remain main entry point for forms to be used in both MVC and Expressive. Consequently, did decorator-ish style approach."},"comment":{"url":"https://api.github.com/repos/zendframework/zend-inputfilter/issues/comments/449325256","html_url":"https://github.com/zendframework/zend-inputfilter/pull/172#issuecomment-449325256","issue_url":"https://api.github.com/repos/zendframework/zend-inputfilter/issues/172","id":449325256,"node_id":"MDEyOklzc3VlQ29tbWVudDQ0OTMyNTI1Ng==","user":{"login":"Slamdunk","id":152236,"node_id":"MDQ6VXNlcjE1MjIzNg==","avatar_url":"https://avatars0.githubusercontent.com/u/152236?v=4","gravatar_id":"","url":"https://api.github.com/users/Slamdunk","html_url":"https://github.com/Slamdunk","followers_url":"https://api.github.com/users/Slamdunk/followers","following_url":"https://api.github.com/users/Slamdunk/following{/other_user}","gists_url":"https://api.github.com/users/Slamdunk/gists{/gist_id}","starred_url":"https://api.github.com/users/Slamdunk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Slamdunk/subscriptions","organizations_url":"https://api.github.com/users/Slamdunk/orgs","repos_url":"https://api.github.com/users/Slamdunk/repos","events_url":"https://api.github.com/users/Slamdunk/events{/privacy}","received_events_url":"https://api.github.com/users/Slamdunk/received_events","type":"User","site_admin":false},"created_at":"2018-12-21T09:15:45Z","updated_at":"2018-12-21T09:15:45Z","author_association":"CONTRIBUTOR","body":"Like https://github.com/zendframework/zend-filter/pull/70 this PR introduced a BC Break. Hope to isolate the issue and post a PR soon."}}
{"actor":{"display_login":"Slamdunk"}}