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.
Due to the cleanup it would be really nice to have $view->pager instead of $view->query->pager which seems to
be somehow really odd.
Comment | File | Size | Author |
---|---|---|---|
#8 | 1751294-pager-view-8.patch | 14.64 KB | dawehner |
#1 | 1751294-pager-view.patch | 14.65 KB | dawehner |
Comments
Comment #1
dawehner.
Comment #2
aspilicious CreditAttribution: aspilicious commentedComment #4
aspilicious CreditAttribution: aspilicious commenteduse statements should be on top of the file, just behind the namespace stuff
Comment #5
lunaris CreditAttribution: lunaris commentedYou've changed "this" to "view" here (and throughout the class in question); is this correct?
Comment #6
aspilicious CreditAttribution: aspilicious commentedyes thats correct
Comment #7
dawehnerI agree in theory but shouldn't this be done like that in api documention? It would be cool to discuss that.
Comment #8
dawehnerOkay changed that and added a test for init_pager();
Comment #9
tim.plunketthttp://drupalcode.org/project/views.git/commit/15256fd