Part of meta-issue #1856630: [Change notice] [META] Rename Views methods to core standards
| Comment | File | Size | Author |
|---|---|---|---|
| #16 | 2002500-rename-exposed-form-validate-16.patch | 2.76 KB | fmizzell |
| #12 | 2002500-rename-exposed-form-validate-12.patch | 2.74 KB | fmizzell |
| #10 | 2002500-10.patch | 2.74 KB | jibran |
| #3 | 2002500-rename-exposed-form-validate-3.patch | 2.73 KB | fmizzell |
| #2 | 2002500-rename-exposed-form-validate.patch | 2.71 KB | fmizzell |
Comments
Comment #1
fmizzell commentedComment #2
fmizzell commentedComment #3
fmizzell commentedComment #4
fmizzell commentedthe last patch adds the public modifiers to the methods
Comment #6
oenie commentedThe code looks good, but there seems to be something wrong with the BlockRenderOrderTest (very recently added)
I would mark this as blocked by http://drupal.org/node/1987952
Comment #7
oenie commented#3: 2002500-rename-exposed-form-validate-3.patch queued for re-testing.
Comment #8
oenie commentedLooks good to me now !
Comment #9
alexpottNeeds a re-roll
Comment #10
jibranreroll
Comment #12
fmizzell commentedComment #14
fmizzell commented#12: 2002500-rename-exposed-form-validate-12.patch queued for re-testing.
Comment #16
fmizzell commentedthe patch applied just fine for me, but here goes a "reroll" anyways
Comment #17
aspilicious commentedComment #18
alexpottCommitted d7ef279 and pushed to 8.x. Thanks!