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

[BUG]: Cannot fetch a PATCH request payload #16188

Closed
danielcosta-94 opened this issue Oct 31, 2022 · 1 comment · Fixed by #16190
Closed

[BUG]: Cannot fetch a PATCH request payload #16188

danielcosta-94 opened this issue Oct 31, 2022 · 1 comment · Fixed by #16190
Assignees
@niden
Labels
5.0 The issues we want to solve in the 5.0 release new feature request Planned Feature or New Feature Request

Comments

@danielcosta-94
Copy link

Questions? Forum: https://phalcon.io/forum or Discord: https://phalcon.io/discord

Describe the bug
Cannot fetch a PATCH request payload (sanitized or not), because we don't have equivalent like a POST request getPost or getFilteredPost

To Reproduce
Check available methods in class Request.php in namespace Phalcon\Http

Expected behavior
To have the equivalent methods getPatch and getFilteredPatch

Details

  • Phalcon version: (5.0.4)
  • PHP Version: (8.1.11)
  • Operating System: Linux Ubuntu 20.04.5 LTS
  • Installation type: installing via package manager
  • Zephir version (if any): (0.16.3)
  • Server: Nginx
@danielcosta-94 danielcosta-94 added bug A bug report status: unverified Unverified labels Oct 31, 2022
@niden niden self-assigned this Oct 31, 2022
@niden niden added new feature request Planned Feature or New Feature Request 5.0 The issues we want to solve in the 5.0 release and removed bug A bug report status: unverified Unverified labels Oct 31, 2022
niden added a commit to niden/cphalcon that referenced this issue Nov 1, 2022
@niden
[phalcon#16188] - adding getPatch(), getFilteredPatch() and hasPatch(…
f40d171 
…) methods
niden added a commit to niden/cphalcon that referenced this issue Nov 1, 2022
@niden
[phalcon#16188] - adding more tests!
89665e8
niden added a commit to niden/cphalcon that referenced this issue Nov 1, 2022
@niden
[phalcon#16188] - updated changelog
7c6c291
@niden niden mentioned this issue Nov 1, 2022
Merged
5 tasks
@niden niden linked a pull request Nov 1, 2022 that will close this issue
T16188 cannot fetch patch payload #16190
Merged
5 tasks
@niden
Copy link
Sponsor Member

niden commented Nov 1, 2022

Resolved in #16190

Thank you @danielcosta-94

@niden niden closed this as completed Nov 1, 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 new feature request Planned Feature or New Feature Request
Projects
Phalcon v5
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

T16188 cannot fetch patch payload niden/cphalcon
2 participants
@niden @danielcosta-94