Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Part of meta-issue #2052421: [META] Rename Views properties to core standards
Comment | File | Size | Author |
---|---|---|---|
#45 | 2066451-45.patch | 6.21 KB | ankit_rathore |
| |||
#44 | 2066451-nr-bot.txt | 144 bytes | needs-review-queue-bot |
#38 | 2066451-38.patch | 5.83 KB | Hardik_Patel_12 |
#31 | 2066451-31.patch | 5.68 KB | anmolgoyal74 |
Comments
Comment #1
jlindsey15 CreditAttribution: jlindsey15 commentedComment #2
jlindsey15 CreditAttribution: jlindsey15 commentedComment #4
jlindsey15 CreditAttribution: jlindsey15 commentedHmm I guess I have to change it in all the subclasses too...
Comment #5
jlindsey15 CreditAttribution: jlindsey15 commentedHmm I guess I have to change it in all the subclasses too...
EDIT: Sorry about the double post, not sure why that happened.
Comment #6
dawehnerThis also should be protected/public ....
It feels odd that the uses of these variables aren't changed.
Comment #7
dawehner.
Comment #8
id.medionComment #9
InternetDevels CreditAttribution: InternetDevels commentedRerolled patch for FilterPluginBase, other changes will be implemented in related issues.
Comment #10
mpv CreditAttribution: mpv commentedRerolled this patch for latest HEAD.
Comment #11
ohthehugemanatee CreditAttribution: ohthehugemanatee commentedRerolled for latest HEAD. @dawehner can we get a review?
Comment #14
LinL CreditAttribution: LinL commentedPatch no longer applies. Tagging for reroll.
Comment #15
alimac CreditAttribution: alimac commentedRerolled, and added reference from #2183415: In BooleanOperator Rename Views properties to core standards which also makes changes to FilterPluginBase.
Comment #16
alimac CreditAttribution: alimac commentedComment #17
trzcinski.t@gmail.com CreditAttribution: trzcinski.t@gmail.com commentedHi,
this patch no longer applies. I am tagging with Needs reroll and start doing reroll.
Comment #18
trzcinski.t@gmail.com CreditAttribution: trzcinski.t@gmail.com commentedAttaching the rerolled patch. It applies for the latest dev currently.
Comment #19
trzcinski.t@gmail.com CreditAttribution: trzcinski.t@gmail.com commentedRemoving Needs reroll tag. Sorry - I forgot in the first post.
Comment #20
alexpott#2052421: [META] Rename Views properties to core standards has been postponed.
Comment #25
tobiberlinI am a first time sprinter on Vienna 2017 and will work on this know...
Comment #26
tobiberlinI was told not to work on this issue so I am stopping
Comment #28
dawehnerThe parent is not longer postponed
Comment #29
anmolgoyal74 CreditAttribution: anmolgoyal74 at OpenSense Labs for DrupalFit commentedComment #31
anmolgoyal74 CreditAttribution: anmolgoyal74 at OpenSense Labs for DrupalFit commentedComment #33
anmolgoyal74 CreditAttribution: anmolgoyal74 at OpenSense Labs for DrupalFit commentedThe same patch #31 works fine with 8.7.x
Comment #37
andypostThere's better cleaned patch in duplicated issue
Comment #38
Hardik_Patel_12 CreditAttribution: Hardik_Patel_12 at QED42 for Drupal India Association commentedPatch for 9.1.x branch , kindly review.
Comment #44
needs-review-queue-bot CreditAttribution: needs-review-queue-bot as a volunteer commentedThe Needs Review Queue Bot tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.
Consult the Drupal Contributor Guide to find step-by-step guides for working with issues.
Comment #45
ankit_rathore CreditAttribution: ankit_rathore at OpenSense Labs for DrupalFit commentedRe-rolled patch #38 against 10.1.x-dev
Please review it.
Comment #46
quietone CreditAttribution: quietone at PreviousNext commentedThis is a coding standard fix and they are now done by sniff, not file. See #3346468: [meta] Fix Drupal.NamingConventions.ValidVariableName.LowerCamelName