diff --git a/handlers/views_handler_argument.inc b/handlers/views_handler_argument.inc index 05644cf..27cfdf7 100644 --- a/handlers/views_handler_argument.inc +++ b/handlers/views_handler_argument.inc @@ -45,7 +45,7 @@ class views_handler_argument extends views_handler { } } - function init(&$view, &$options) { + function init(&$view, $options) { parent::init($view, $options); } diff --git a/handlers/views_handler_argument_many_to_one.inc b/handlers/views_handler_argument_many_to_one.inc index cc887d4..ee6f2f4 100644 --- a/handlers/views_handler_argument_many_to_one.inc +++ b/handlers/views_handler_argument_many_to_one.inc @@ -7,13 +7,13 @@ * - numeric: If true, the field will be considered numeric. Probably should * always be set TRUE as views_handler_argument_string has many to one * capabilities. - * - zero is null: If true, a 0 will be handled as empty, so for example + * - zero is null: If true, a 0 will be handled as empty, so for example * a default argument can be provided or a summary can be shown. * * @ingroup views_argument_handlers */ class views_handler_argument_many_to_one extends views_handler_argument { - function init(&$view, &$options) { + function init(&$view, $options) { parent::init($view, $options); $this->helper = new views_many_to_one_helper($this); diff --git a/handlers/views_handler_argument_string.inc b/handlers/views_handler_argument_string.inc index ae7dd1e..8dee6b9 100644 --- a/handlers/views_handler_argument_string.inc +++ b/handlers/views_handler_argument_string.inc @@ -7,7 +7,7 @@ * @ingroup views_argument_handlers */ class views_handler_argument_string extends views_handler_argument { - function init(&$view, &$options) { + function init(&$view, $options) { parent::init($view, $options); if (!empty($this->definition['many to one'])) { $this->helper = new views_many_to_one_helper($this); diff --git a/handlers/views_handler_field.inc b/handlers/views_handler_field.inc index 06c6f94..250702d 100644 --- a/handlers/views_handler_field.inc +++ b/handlers/views_handler_field.inc @@ -454,7 +454,7 @@ If you would like to have the characters %5B and %5D please use the html entity * @param $values * An array of all objects returned from the query. */ - function pre_render($values) { } + function pre_render(&$values) { } /** * Render the field. diff --git a/handlers/views_handler_filter.inc b/handlers/views_handler_filter.inc index 49b93e5..2718042 100644 --- a/handlers/views_handler_filter.inc +++ b/handlers/views_handler_filter.inc @@ -101,7 +101,7 @@ class views_handler_filter extends views_handler { /** * Simple validate handler */ - function options_validate(&$form, &$form_state) { + function options_validate($form, &$form_state) { $this->operator_validate($form, $form_state); $this->value_validate($form, $form_state); if (!empty($this->options['exposed'])) { @@ -113,7 +113,7 @@ class views_handler_filter extends views_handler { /** * Simple submit handler */ - function options_submit(&$form, &$form_state) { + function options_submit($form, &$form_state) { unset($form_state['values']['expose_button']); // don't store this. $this->operator_submit($form, $form_state); $this->value_submit($form, $form_state); diff --git a/handlers/views_handler_filter_boolean_operator.inc b/handlers/views_handler_filter_boolean_operator.inc index c2ec624..ec9e415 100644 --- a/handlers/views_handler_filter_boolean_operator.inc +++ b/handlers/views_handler_filter_boolean_operator.inc @@ -108,7 +108,7 @@ class views_handler_filter_boolean_operator extends views_handler_filter { } } - function value_validate(&$form, &$form_state) { + function value_validate($form, &$form_state) { if ($form_state['values']['options']['value'] == 'All' && empty($form_state['values']['options']['expose']['optional'])) { form_set_error('value', t('You must select a value unless this is an optional exposed filter.')); } diff --git a/handlers/views_handler_filter_date.inc b/handlers/views_handler_filter_date.inc index 356ec78..ab25862 100644 --- a/handlers/views_handler_filter_date.inc +++ b/handlers/views_handler_filter_date.inc @@ -31,7 +31,7 @@ class views_handler_filter_date extends views_handler_filter_numeric { parent::value_form($form, $form_state); } - function options_validate(&$form, &$form_state) { + function options_validate($form, &$form_state) { parent::options_validate($form, $form_state); if (!empty($form_state['values']['options']['expose']['optional'])) { @@ -42,7 +42,7 @@ class views_handler_filter_date extends views_handler_filter_numeric { $this->validate_valid_time($form['value'], $form_state['values']['options']['operator'], $form_state['values']['options']['value']); } - function exposed_validate(&$form, &$form_state) { + function exposed_validate($form, &$form_state) { if (empty($this->options['exposed'])) { return; } diff --git a/includes/base.inc b/includes/base.inc index 7522d9f..126bde3 100644 --- a/includes/base.inc +++ b/includes/base.inc @@ -42,7 +42,7 @@ class views_object { * Set default options on this object. Called by the constructor in a * complex chain to deal with backward compatibility. */ - function options() { } + function options(&$options) { } /** * Set default options. diff --git a/includes/handlers.inc b/includes/handlers.inc index 2ca1c2d..00103c9 100644 --- a/includes/handlers.inc +++ b/includes/handlers.inc @@ -325,7 +325,7 @@ class views_handler extends views_object { /** * Validate the exposed filter form */ - function exposed_validate(&$form, &$form_state) { } + function exposed_validate($form, &$form_state) { } /** * Submit the exposed filter form @@ -514,7 +514,7 @@ class views_many_to_one_helper { $this->handler = &$handler; } - function option_definition(&$options) { + public static function option_definition(&$options) { $options['reduce_duplicates'] = array('default' => FALSE); } diff --git a/includes/view.inc b/includes/view.inc index c58b43f..df246bf 100644 --- a/includes/view.inc +++ b/includes/view.inc @@ -91,7 +91,7 @@ class view extends views_db_object { * Note: In PHP5 this should be static, but PHP4 doesn't support static * methods. */ - function db_objects() { + public static function db_objects() { return array('display'); } @@ -1339,7 +1339,7 @@ class view extends views_db_object { * If TRUE, reset this entry in the load cache. * @return A view object or NULL if it was not available. */ - function &load($arg, $reset = FALSE) { + public static function &load($arg, $reset = FALSE) { static $cache = array(); // We want a NULL value to return TRUE here, so we can't use isset() or empty(). @@ -1392,7 +1392,7 @@ class view extends views_db_object { * that would be very slow. Buiding the views externally from unified queries is * much faster. */ - function load_views() { + public static function load_views() { $result = db_query("SELECT DISTINCT v.* FROM {views_view} v"); $views = array(); $vids = array(); diff --git a/modules/comment/views_handler_field_comment.inc b/modules/comment/views_handler_field_comment.inc index 6755b14..95edf0e 100644 --- a/modules/comment/views_handler_field_comment.inc +++ b/modules/comment/views_handler_field_comment.inc @@ -6,7 +6,7 @@ class views_handler_field_comment extends views_handler_field { /** * Override init function to provide generic option to link to comment. */ - function init(&$view, &$options) { + function init(&$view, $options) { parent::init($view, $options); if (!empty($this->options['link_to_comment'])) { $this->additional_fields['cid'] = 'cid'; diff --git a/modules/comment/views_handler_field_comment_username.inc b/modules/comment/views_handler_field_comment_username.inc index a67612a..7b0a92f 100644 --- a/modules/comment/views_handler_field_comment_username.inc +++ b/modules/comment/views_handler_field_comment_username.inc @@ -6,7 +6,7 @@ class views_handler_field_comment_username extends views_handler_field { /** * Override init function to add uid and homepage fields. */ - function init(&$view, &$data) { + function init(&$view, $data) { parent::init($view, $data); $this->additional_fields['uid'] = 'uid'; $this->additional_fields['homepage'] = 'homepage'; diff --git a/modules/node/views_handler_filter_node_access.inc b/modules/node/views_handler_filter_node_access.inc index f787aee..010a286 100644 --- a/modules/node/views_handler_filter_node_access.inc +++ b/modules/node/views_handler_filter_node_access.inc @@ -4,7 +4,7 @@ */ class views_handler_filter_node_access extends views_handler_filter { function admin_summary() { } - function operator_form() { } + function operator_form(&$form, &$form_state) { } function can_expose() { return FALSE; } diff --git a/modules/node/views_handler_filter_node_status.inc b/modules/node/views_handler_filter_node_status.inc index 7047caa..a91b5c6 100644 --- a/modules/node/views_handler_filter_node_status.inc +++ b/modules/node/views_handler_filter_node_status.inc @@ -4,7 +4,7 @@ */ class views_handler_filter_node_status extends views_handler_filter { function admin_summary() { } - function operator_form() { } + function operator_form(&$form, &$form_state) { } function query() { $table = $this->ensure_my_table(); diff --git a/modules/system/views_handler_field_file.inc b/modules/system/views_handler_field_file.inc index 0c5f4b2..82b06cd 100644 --- a/modules/system/views_handler_field_file.inc +++ b/modules/system/views_handler_field_file.inc @@ -6,7 +6,7 @@ class views_handler_field_file extends views_handler_field { /** * Constructor to provide additional field to add. */ - function init(&$view, &$options) { + function init(&$view, $options) { parent::init($view, $options); if (!empty($options['link_to_file'])) { $this->additional_fields['filepath'] = 'filepath'; diff --git a/modules/taxonomy/views_handler_field_term_node_tid.inc b/modules/taxonomy/views_handler_field_term_node_tid.inc index bc8fc92..8782939 100644 --- a/modules/taxonomy/views_handler_field_term_node_tid.inc +++ b/modules/taxonomy/views_handler_field_term_node_tid.inc @@ -66,7 +66,7 @@ class views_handler_field_term_node_tid extends views_handler_field_prerender_li $this->add_additional_fields(); } - function pre_render($values) { + function pre_render(&$values) { $this->field_alias = $this->aliases['vid']; $vids = array(); foreach ($values as $result) { diff --git a/modules/taxonomy/views_handler_filter_term_node_tid.inc b/modules/taxonomy/views_handler_filter_term_node_tid.inc index ebcf10c..3b9f3b1 100644 --- a/modules/taxonomy/views_handler_filter_term_node_tid.inc +++ b/modules/taxonomy/views_handler_filter_term_node_tid.inc @@ -173,7 +173,7 @@ class views_handler_filter_term_node_tid extends views_handler_filter_many_to_on } } - function value_validate(&$form, &$form_state) { + function value_validate($form, &$form_state) { // We only validate if they've chosen the text field style. if ($this->options['type'] != 'textfield') { return; @@ -208,7 +208,7 @@ class views_handler_filter_term_node_tid extends views_handler_filter_many_to_on return $rc; } - function exposed_validate(&$form, &$form_state) { + function exposed_validate($form, &$form_state) { if (empty($this->options['exposed'])) { return; } diff --git a/modules/upload/views_handler_field_upload_description.inc b/modules/upload/views_handler_field_upload_description.inc index 964925c..2b0ac18 100644 --- a/modules/upload/views_handler_field_upload_description.inc +++ b/modules/upload/views_handler_field_upload_description.inc @@ -4,7 +4,7 @@ * Field handler to provide a list of roles. */ class views_handler_field_upload_description extends views_handler_field { - function init(&$view, &$options) { + function init(&$view, $options) { parent::init($view, $options); if (!empty($options['link_to_file'])) { $this->additional_fields['fid'] = 'fid'; diff --git a/modules/user/views_handler_field_user.inc b/modules/user/views_handler_field_user.inc index de8c6e6..0c62556 100644 --- a/modules/user/views_handler_field_user.inc +++ b/modules/user/views_handler_field_user.inc @@ -7,7 +7,7 @@ class views_handler_field_user extends views_handler_field { /** * Override init function to provide generic option to link to user. */ - function init(&$view, &$data) { + function init(&$view, $data) { parent::init($view, $data); if (!empty($this->options['link_to_user'])) { $this->additional_fields['uid'] = 'uid'; diff --git a/modules/user/views_handler_field_user_name.inc b/modules/user/views_handler_field_user_name.inc index 42ff1c8..deb814a 100644 --- a/modules/user/views_handler_field_user_name.inc +++ b/modules/user/views_handler_field_user_name.inc @@ -6,7 +6,7 @@ class views_handler_field_user_name extends views_handler_field_user { /** * Add uid in the query so we can test for anonymous if needed. */ - function init(&$view, &$data) { + function init(&$view, $data) { parent::init($view, $data); if (!empty($this->options['overwrite_anonymous'])) { $this->additional_fields['uid'] = 'uid'; diff --git a/modules/user/views_handler_field_user_roles.inc b/modules/user/views_handler_field_user_roles.inc index c664630..e677f11 100644 --- a/modules/user/views_handler_field_user_roles.inc +++ b/modules/user/views_handler_field_user_roles.inc @@ -13,7 +13,7 @@ class views_handler_field_user_roles extends views_handler_field_prerender_list $this->field_alias = $this->aliases['uid']; } - function pre_render($values) { + function pre_render(&$values) { $uids = array(); $this->items = array(); diff --git a/modules/user/views_handler_filter_user_name.inc b/modules/user/views_handler_filter_user_name.inc index 8bf1a88..e978175 100644 --- a/modules/user/views_handler_filter_user_name.inc +++ b/modules/user/views_handler_filter_user_name.inc @@ -35,7 +35,7 @@ class views_handler_filter_user_name extends views_handler_filter_in_operator { } } - function value_validate(&$form, &$form_state) { + function value_validate($form, &$form_state) { $values = drupal_explode_tags($form_state['values']['options']['value']); $uids = $this->validate_user_strings($form['value'], $values); @@ -114,7 +114,7 @@ class views_handler_filter_user_name extends views_handler_filter_in_operator { return $uids; } - function value_submit() { + function value_submit($form, &$form_state) { // prevent array filter from removing our anonymous user. } diff --git a/plugins/views_plugin_row.inc b/plugins/views_plugin_row.inc index 06d0930..2fe3ae4 100644 --- a/plugins/views_plugin_row.inc +++ b/plugins/views_plugin_row.inc @@ -98,13 +98,13 @@ class views_plugin_row extends views_plugin { /** * Validate the options form. */ - function options_validate($form, &$form_state) { } + function options_validate(&$form, &$form_state) { } /** * Perform any necessary changes to the form values prior to storage. * There is no need for this function to actually store the data. */ - function options_submit($form, &$form_state) { } + function options_submit(&$form, &$form_state) { } function query() { if (isset($this->base_table) && isset($this->options['relationship']) && isset($this->view->relationship[$this->options['relationship']])) { diff --git a/plugins/views_plugin_row_fields.inc b/plugins/views_plugin_row_fields.inc index 491f5d2..bd66b42 100644 --- a/plugins/views_plugin_row_fields.inc +++ b/plugins/views_plugin_row_fields.inc @@ -60,7 +60,7 @@ class views_plugin_row_fields extends views_plugin_row { * Perform any necessary changes to the form values prior to storage. * There is no need for this function to actually store the data. */ - function options_submit($form, &$form_state) { + function options_submit(&$form, &$form_state) { $form_state['values']['row_options']['inline'] = array_filter($form_state['values']['row_options']['inline']); } }