Renaming the check_empty_arguments method to something more descriptive. #9345

Merged
merged 1 commit into from Feb 20, 2013

2 participants

@wangjohn

This is in response to comments from @richardkmichael on #9332.

I'm renaming the function which checks to see whether or not there are arguments to the query method. The function is now called has_arguments? so that it's easier to tell that it's just checking to see if the args are blank or not.

@wangjohn wangjohn Renaming the check_empty_arguments method to something more descriptive.
The function is now called has_arguments? so that it's easier to tell
that it's just checking to see if the args are blank or not.
7b30cec
@rafaelfranca rafaelfranca merged commit 24b7a76 into rails:master Feb 20, 2013
@wangjohn wangjohn deleted the wangjohn:change_name_of_query_method_argument_checker_for_clarity branch Mar 2, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment