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.
C:\xampp\htdocs\views\views_ui\admin.inc
994 $display = &$view->display[$display_id];
2143 $plugin->init($view, $view->display[$display_id], $handler->options['style']['options']);
2 matches in C:\xampp\htdocs\views\views_ui\admin.inc
Comment | File | Size | Author |
---|---|---|---|
#14 | views-1810826-13.patch | 6.08 KB | tim.plunkett |
#12 | settings.png | 10.81 KB | dawehner |
#11 | views-1810826-11.patch | 5.26 KB | tim.plunkett |
#11 | interdiff.txt | 2.3 KB | tim.plunkett |
#10 | 1810826-10.patch | 3.17 KB | fastangel |
Comments
Comment #1
dawehnerOne code was not used at all, so let's remove it.
Comment #2
dawehnerJust realized that this function is not used anymore. The UI for summaries is included in arguments directly.
Comment #3
xjmgrep shows the following references:
That first one is in
views_ui_ajax_forms()
. Do we need to remove that entry for the form there?Comment #4
tim.plunkettYes, it should be removed from views_ui_ajax_forms() as well.
Comment #5
xjmComment #6
dawehnerNice! I like to have bugs turning into actual cleanups.
Comment #7
dawehnerRerole against core.
Comment #8
dawehnerWith an actual patch.
Comment #10
fastangel CreditAttribution: fastangel commentedRerolled.
Comment #11
tim.plunkettThis came up while I was working on #1798574: Refactor Views UI to be a form controller
Comment #12
dawehner@tim
This is not a feature we can drop, see the attached screenshot
Comment #13
dawehnerSorry i was wrong. config-type was used for the override button in d6.
Comment #14
tim.plunkettMissed the calling code. @dawehner and I talked in IRC and this is fine.
Comment #15
dawehnerSo finally!
Comment #16
webchickYay less code! :D
Committed and pushed to 8.x.