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 |
---|---|---|---|
#2 | 2003464-02-rename-operator_options-to-operatorOptions.patch | 5.3 KB | munizjor |
#1 | 2003464-01-rename-operator_options-to-operatorOptions.patch | 5.26 KB | SpartyDan |
Comments
Comment #1
SpartyDan CreditAttribution: SpartyDan commentedPatch to rename operator_options() to operatorOptions()
Comment #2
munizjor CreditAttribution: munizjor commentedUpdated patch and changed 'functions' to 'public function'.
Comment #3
jmolivas CreditAttribution: jmolivas commentedComment #5
SpartyDan CreditAttribution: SpartyDan commented#2: 2003464-02-rename-operator_options-to-operatorOptions.patch queued for re-testing.
Comment #6
elvis2 CreditAttribution: elvis2 commentedLooks good. Reviewed patch based on: http://drupal.org/node/1856630#comment-7450696
Comment #7
aspilicious CreditAttribution: aspilicious commentedSmells like protected to me. But I'm not 100% sure...
Comment #8
dawehnerIt feels like exposed form plugins could potentially use that information so I would rather go with public.
Comment #9
alexpottCommitted 86b2e89 and pushed to 8.x. Thanks!