-
Notifications
You must be signed in to change notification settings - Fork 108
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
actionpack: Add missing methods to ActionController::Parameters #603
actionpack: Add missing methods to ActionController::Parameters #603
Conversation
@tk0miya Thanks for your contribution! Please follow the instructions below for each change. Available commandsYou can use the following commands by commenting on this PR.
|
- [::Kernel public] def to_s: () -> ::String
+ [::ActionController::Parameters public] def to_s: () -> ::String
- -
+ [::ActionController::Parameters public] def keys: () -> ::Array[untyped]
- -
+ [::ActionController::Parameters public] def key?: () -> bool
- -
+ [::ActionController::Parameters public] def has_key?: () -> bool
- -
+ [::ActionController::Parameters public] def values: () -> ::Array[untyped]
- -
+ [::ActionController::Parameters public] def has_value?: () -> bool
- -
+ [::ActionController::Parameters public] def empty?: () -> bool
- -
+ [::ActionController::Parameters public] def include?: (untyped) -> bool
- -
+ [::ActionController::Parameters public] def as_json: () -> ::String
- -
+ [::ActionController::Parameters public] def each_key: () { (untyped) -> untyped } -> ::Hash[untyped, untyped] | () -> ::Enumerator[untyped, self] |
96bce67
to
63e74e8
Compare
Some methods for ActionController::Parameters has been missing because they're defined via delegation. This adds them manually. refs: https://github.com/rails/rails/blob/v6.0.6.1/actionpack/lib/action_controller/metal/strong_parameters.rb#L214-L215
63e74e8
to
89576fd
Compare
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.
APPROVE
/merge |
@ksss Thank you for reviewing! |
Some methods for ActionController::Parameters has been missing because they're defined via delegation.
This adds them manually.
refs: https://github.com/rails/rails/blob/v6.0.6.1/actionpack/lib/action_controller/metal/strong_parameters.rb#L214-L215