Updated: Comment #6
Problem/Motivation
In ViewExecutable
several methods dealing with Views handlers are using ...Item()
instead. Especially as ViewExecutable::getItemsPerPage()
and ViewExecutable::setItemsPerPage()
are used in a total different context this may be very confusing.
Proposed resolution
Renaming the following methods to use "Handler" instead of "Item":
- addItem() => addHandler()
- generateItemId() => generateHandlerId()
- getItems() => getHandlers()
- getItem() => getHandler()
- removeItem() => removeHandler()
- setItemOption() => setHandlerOption()
Remaining tasks
-
User interface changes
-
API changes
Public methods of \Drupal\views\ViewExecutable
are renamed:
- addItem() => addHandler()
- generateItemId() => generateHandlerId()
- getItems() => getHandlers()
- getItem() => getHandler()
- removeItem() => removeHandler()
- setItemOption() => setHandlerOption()
Developers dealing with handlers associated to a view need to rename their method calls.
The AddItem form is now called AddHandler.