Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Problem/Motivation
#1808810: Remove handling of moved tables removed the code that handles moved fields/tables.
However, the code within views plugins that responds to them is still leftover.
It is stale code that will never run.
Proposed resolution
Remove actualField and actualTable, both the public properties and the 3 lines for each in \Drupal\views\Plugin\views\HandlerBase::init()
Remaining tasks
User interface changes
N/A
API changes
These are public properties, so backwards compatibility is questionable here.
Data model changes
N/A
Comment | File | Size | Author |
---|---|---|---|
#2 | remove-2659986-2.patch | 1.37 KB | lluvigne |
Comments
Comment #2
lluvigneHi,
i've removed this public properties and the code relating to this properties on \Drupal\views\Plugin\views\HandlerBase::init().
Regards!
Comment #3
tim.plunkettPerfect, thanks!
Comment #4
alexpottStale code just adds confusion and complexity. Views is complex enough. Committed f2fe78a and pushed to 8.1.x. Thanks!