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 #1856630: [Change notice] [META] Rename Views methods to core standards
Comment | File | Size | Author |
---|---|---|---|
#12 | views-valueForm-2002932-12.patch | 10.71 KB | vito_a |
#7 | views-valueForm-2002932-7.patch | 9.7 KB | zschmid |
#4 | views-valueForm-2002932-4.patch | 12.23 KB | zschmid |
#2 | vdc-2002932.patch | 10.17 KB | jastraat |
Comments
Comment #1
jastraat CreditAttribution: jastraat commentedComment #2
jastraat CreditAttribution: jastraat commentedComment #3
aspilicious CreditAttribution: aspilicious commentedThese should all be protected. ==> protected function valueForm()
Comment #4
zschmid CreditAttribution: zschmid commentedupdated patch
Comment #5
zschmid CreditAttribution: zschmid commentedComment #6
aspilicious CreditAttribution: aspilicious commentedThis shouldn't be here :)
protected
Comment #7
zschmid CreditAttribution: zschmid commentedCrap. Sorry about those extra files, must have forgotten to switch branches before this. Anywayss, i think it needed to be rerolled anyways after the core commits yesterday/today. Should be okay now.
Comment #8
zschmid CreditAttribution: zschmid commentedComment #10
E.Kurko CreditAttribution: E.Kurko commentedI'll check. CodeSprintUA
Comment #11
E.Kurko CreditAttribution: E.Kurko commentedAssigned to me
Comment #12
vito_a CreditAttribution: vito_a commentedRe-rolling the #7.
Comment #13
vito_a CreditAttribution: vito_a commentedComment #14
vito_a CreditAttribution: vito_a commentedComment #15
dawehnerPerfect
Comment #16
webchickCommitted and pushed to 8.x. Thanks!
Moving to the Views queue for the change notice.
Comment #17
xjmWe'll create a single change notice in #1856630: [Change notice] [META] Rename Views methods to core standards.