Rename Views method process_summary_arguments() to processSummaryArguments()

Patch passed all argument and argument validator tests on my localhost...EXCEPT for the date argument handler test. However, that test fails even with the patch not applied, so I think that must be different issue.

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 the public modifier to all the functions in the patch.

...and marking as needing review. :)

Looks fine to me !

And thanks elvis2 for helping out ... paying it forward rocks ! :)

Needs a re-roll

curl http://drupal.org/files/views-2003356-processSummaryArguments.patch | git a
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  4061  100  4061    0     0   7609      0 --:--:-- --:--:-- --:--:-- 11407
error: patch failed: core/modules/views/lib/Drupal/views/Plugin/views/argument/ArgumentPluginBase.php:754
error: core/modules/views/lib/Drupal/views/Plugin/views/argument/ArgumentPluginBase.php: patch does not apply

reroll
conflict

++<<<<<<< HEAD
 +      if (isset($this->validator) || $this->validator = $this->getPlugin('argument_validator')) {
 +        $this->validator->process_summary_arguments($args);
++=======
+       if (isset($this->validator) || $this->validator = $this->get_plugin('argument_validator')) {
+         $this->validator->processSummaryArguments($args);
++>>>>>>> 4

resolved like this

 -      if (isset($this->validator) || $this->validator = $this->get_plugin('argument_validator')) {
 +      if (isset($this->validator) || $this->validator = $this->getPlugin('argument_validator')) {
-         $this->validator->process_summary_arguments($args);
+         $this->validator->processSummaryArguments($args);

#9: 2003356-9.patch queued for re-testing.

We are working today with this issue during Code Sprint UA

Re-rolled. Please review.

Looks good.

agree :)

Committed da83b1a and pushed to 8.x. Thanks!