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
Ran successful test creating a new view.
Add access modifier "public", reroll, retest.
@jfm2k1 - Please refer to https://drupal.org/node/1856630#comment-7450696 when reviewing patches related to the drupal.org/node/1856630 Meta issue.
This function is only used internally in argument plugins. Probably should be protected in stead of public.
tagging
reviewed #9 views.module-summary_name_field-2002982-8.patch gtg
Perfect.
Committed and pushed to 8.x. Thanks!
Moving to the Views queue for the change notice.
We'll create a single change notice in #1856630: [Change notice] [META] Rename Views methods to core standards.
Automatically closed -- issue fixed for 2 weeks with no activity.
Comments
Comment #1
chr.fritschComment #2
chr.fritschComment #3
chr.fritschComment #4
munizjor CreditAttribution: munizjor commentedRan successful test creating a new view.
Comment #5
SpartyDan CreditAttribution: SpartyDan commentedAdd access modifier "public", reroll, retest.
@jfm2k1 - Please refer to https://drupal.org/node/1856630#comment-7450696 when reviewing patches related to the drupal.org/node/1856630 Meta issue.
Comment #6
aspilicious CreditAttribution: aspilicious commentedThis function is only used internally in argument plugins. Probably should be protected in stead of public.
Comment #7
danylevskyiComment #8
danylevskyiComment #9
danylevskyiComment #10
podaroktagging
Comment #11
somepal CreditAttribution: somepal commentedreviewed #9 views.module-summary_name_field-2002982-8.patch
gtg
Comment #12
dawehnerPerfect.
Comment #13
webchickCommitted and pushed to 8.x. Thanks!
Moving to the Views queue for the change notice.
Comment #14
xjmWe'll create a single change notice in #1856630: [Change notice] [META] Rename Views methods to core standards.