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 |
---|---|---|---|
#8 | views_rename_uses_tokens-2002938-8.patch | 741 bytes | nathangervais |
#2 | views-rename_uses_tokens_function-2002938-2.patch | 734 bytes | vuzzbox |
Comments
Comment #1
vuzzbox CreditAttribution: vuzzbox commentedWorking on this at 2013 DrupalCon "getting involved sprint." Wish me luck.
Comment #2
vuzzbox CreditAttribution: vuzzbox commentedrenamed function uses_tokens to usesTokens
Comment #3
vuzzbox CreditAttribution: vuzzbox commentedReady for review.
Comment #5
aspilicious CreditAttribution: aspilicious commentedStrange seams like a random fail...
Can you please prepend the function with public "public function usesTokens".
Thnx!
Comment #6
vuzzbox CreditAttribution: vuzzbox commentedI think all tests would be failing randomly because of an issue in a block positioning test not related to this (or many other issues). that test is failing randomly, but a patch was posted to fix it. See this issue: http://drupal.org/node/1987952#comment-7454266
Once that is fixed, this should pass.
Comment #7
aspilicious CreditAttribution: aspilicious commentedyeah but you still need tyo prepend the function with "public".
Views has realy old code and adding the correct type is needed in this issue.
Comment #8
nathangervais CreditAttribution: nathangervais commentedRerolled Patch with public modifier.
Comment #9
dawehnerThis function is not used at all. I have the feeling that this is just wrong.
Comment #10
aspilicious CreditAttribution: aspilicious commentedShould we remove it?
Comment #11
heddnShould we convert it for now and create a follow-up to decide if it should be removed.
Comment #12
vuzzbox CreditAttribution: vuzzbox commentedI noticed this, too, no instances of this function being called.
Is it possible that it's being used by contrib modules? Is it possible to determine which contrib modules might be calling any given core function?
Comment #13
dawehnerLet's not remove code for now, as there might be contrib modules requiring this.
Comment #14
alexpottCommitted ad6858d and pushed to 8.x. Thanks!
So can so do a scan of contrib to see if this is used...