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.
Updated: Comment #0
Problem/Motivation
In #2087231: Add a PluginBase in the Core namespace with t() as a helper method we added method t() to PluginBase. So, it's no longer needed in LocalTaskDefault
Proposed resolution
Removed the t() method and stop injecting the translation manager.
Remaining tasks
N/A
User interface changes
N/A
API changes
N/A
Related Issues
Follow-up from #2087231: Add a PluginBase in the Core namespace with t() as a helper method.
Comment | File | Size | Author |
---|---|---|---|
#4 | local-task-2095223-4.patch | 3.62 KB | tim.plunkett |
#4 | interdiff.txt | 1.11 KB | tim.plunkett |
#1 | 2095223-1.patch | 2.51 KB | pwolanin |
Comments
Comment #1
pwolanin CreditAttribution: pwolanin commentedSimple patch.
Comment #2
tim.plunkettI double checked all other plugin classes, none of them had yet added their own $this->t() yet, this was the only one.
Comment #3
dawehnerNext time lets better run unit tests ...
Comment #4
tim.plunkettDuh! @dawehner++
Comment #5
neclimdulbam, lets do that.
Comment #6
alexpottCommitted 3210323 and pushed to 8.x. Thanks!