Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ServerRequestFactory::load Not Working Correctly #15286

Closed
Fenikkusu opened this issue Feb 9, 2021 · 1 comment · Fixed by #15380
Closed

ServerRequestFactory::load Not Working Correctly #15286

Fenikkusu opened this issue Feb 9, 2021 · 1 comment · Fixed by #15380
Assignees
@niden
Labels
5.0 The issues we want to solve in the 5.0 release bug A bug report status: medium Medium

Comments

@Fenikkusu
Copy link
Contributor

I don't have time right now to dig into this, but I presume it's an easy fix. If you use the Phalcon\Http\Message\ServerRequestFactory::load method without passing any arguments, the values are coming back as empty (IE: getUploadedFiles). If you pass all of the args manually, it seems to work fine.
@Jeckerson Jeckerson added 5.0 The issues we want to solve in the 5.0 release bug A bug report status: unverified Unverified labels Mar 26, 2021
@Jeckerson Jeckerson added this to the 5.0.0 milestone Mar 26, 2021
@niden niden added status: medium Medium and removed status: unverified Unverified labels Apr 8, 2021
@niden niden removed this from the 5.0.0 milestone Apr 8, 2021
@niden niden mentioned this issue Apr 8, 2021
Merged
5 tasks
@niden
Copy link
Member

niden commented Apr 9, 2021

Fixed in #15380

Thank you @Fenikkusu

@niden niden closed this as completed Apr 9, 2021
@niden niden moved this to Released in Phalcon v5 Aug 25, 2022
@niden niden added this to Phalcon v5 Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@niden niden

Labels
5.0 The issues we want to solve in the 5.0 release bug A bug report status: medium Medium
Projects
Phalcon v5
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

T15286 serverrequestfactory load parameters
3 participants
@niden @Jeckerson @Fenikkusu