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.
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
Use typed access to request parameters #5267
Conversation
This was a big source of mixed datatypes in many places
@@ -35,6 +42,7 @@ public static function params() { | |||
* @param mixed $default default value, if no parameter is given | |||
* @param bool $specialchars special characters | |||
* @return mixed value of the parameter | |||
* @deprecated use typed versions instead |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Making the previous method deprecated.
Still used in one location: FreshRSS_entry_Controller->readAction()
@ColonelMoutarde would you feel like proofreading a bit here? |
Let's merge to avoid too many PR conflicts |
Fix FreshRSS#5293 Regression from FreshRSS#5267
$shortcuts = Minz_Request::param('shortcuts'); | ||
if (false !== Minz_Request::param('load_default_shortcuts')) { | ||
$shortcuts = Minz_Request::paramArray('shortcuts'); | ||
if (!Minz_Request::paramBoolean('load_default_shortcuts')) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Negation bug fixed in #5294
This was a big source of mixed datatypes in many places
Contributes to #4112