Rename Views method execute_count_query() to executeCountQuery() [#2002490]

Comment	File	Size	Author
#4	views-executeCountQuery-2002490-4.patch	1.99 KB	jeroent
#2	Rename-Views-method-execute_count_query-to-executeCountQuery-issue-number-2002490-comment-7450112.patch	1.97 KB	themic8

Comment #1

themic8 commented 24 May 2013 at 18:15

Assigned:

Unassigned

» themic8

Working on for DrupalCon Portland.

Log in or register to post comments

Comment #2

themic8 commented 24 May 2013 at 19:11

Assigned:	themic8	» Unassigned
Status:	Active	» Needs review

Status	File	Size
new	Rename-Views-method-execute_count_query-to-executeCountQuery-issue-number-2002490-comment-7450112.patch	1.97 KB

Log in or register to post comments

Comment #3

oenie commented 24 May 2013 at 22:56

Status:

Needs review

» Needs work

+++ b/core/modules/views/lib/Drupal/views/Plugin/views/pager/None.phpundefined
@@ -75,7 +75,7 @@ function get_items_per_page() {
+  function executeCountQuery(&$count_query) {

+++ b/core/modules/views/lib/Drupal/views/Plugin/views/pager/PagerPluginBase.phpundefined
@@ -177,7 +177,7 @@ function use_count_query() {
+  function executeCountQuery(&$count_query) {

Add public access modifier in front of the functions to adher to the new OOP standards.