It would make a good project for someone who is new to the Drupal contribution process. It's preferred over Newbie.
Related to the Views in Drupal Core initiative.
Part of meta-issue #1856630: [Change notice] [META] Rename Views methods to core standards
renamed operator_submit to operatorSubmit
changed functions to public functions
The last submitted patch, 2003462-02-rename-operator_submit-to-operatorSubmit.patch, failed testing.
#2: 2003462-02-rename-operator_submit-to-operatorSubmit.patch queued for re-testing.
Thanks for working on this. You need to add the access modifier (public/private/protected) before the function name. Take a look at the parent issue (http://drupal.org/node/1856630), comments #28, #34, #35 for more details.
You can see an example patch here: http://drupal.org/node/2001672
Added access modifier.
Looks good.
Reviewed patch based on the following criteria: http://drupal.org/node/1856630#comment-7450696
Committed a39c996 and pushed to 8.x. Thanks!
Automatically closed -- issue fixed for 2 weeks with no activity.
Comments
Comment #1
SpartyDan commentedrenamed operator_submit to operatorSubmit
Comment #2
SpartyDan commentedchanged functions to public functions
Comment #4
SpartyDan commented#2: 2003462-02-rename-operator_submit-to-operatorSubmit.patch queued for re-testing.
Comment #5
elvis2 commentedThanks for working on this. You need to add the access modifier (public/private/protected) before the function name. Take a look at the parent issue (http://drupal.org/node/1856630), comments #28, #34, #35 for more details.
You can see an example patch here: http://drupal.org/node/2001672
Comment #6
tvlooy commentedAdded access modifier.
Comment #7
tvlooy commentedComment #8
elvis2 commentedLooks good.
Reviewed patch based on the following criteria: http://drupal.org/node/1856630#comment-7450696
Comment #9
alexpottCommitted a39c996 and pushed to 8.x. Thanks!