diff --git a/core/modules/views/lib/Drupal/views/ManyToOneHelper.php b/core/modules/views/lib/Drupal/views/ManyToOneHelper.php index 08c8cd9..b391528 100644 --- a/core/modules/views/lib/Drupal/views/ManyToOneHelper.php +++ b/core/modules/views/lib/Drupal/views/ManyToOneHelper.php @@ -125,8 +125,8 @@ public function summaryJoin() { // shortcuts $options = $this->handler->options; - $view = &$this->handler->view; - $query = &$this->handler->query; + $view = $this->handler->view; + $query = $this->handler->query; if (!empty($options['require_value'])) { $join->type = 'INNER'; diff --git a/core/modules/views/lib/Drupal/views/Plugin/views/HandlerBase.php b/core/modules/views/lib/Drupal/views/Plugin/views/HandlerBase.php index 763622f..5d267e4 100644 --- a/core/modules/views/lib/Drupal/views/Plugin/views/HandlerBase.php +++ b/core/modules/views/lib/Drupal/views/Plugin/views/HandlerBase.php @@ -353,9 +353,7 @@ public function buildGroupByForm(&$form, &$form_state) { * There is no need for this function to actually store the data. */ public function submitGroupByForm(&$form, &$form_state) { - $item =& $form_state['handler']->options; - - $item['group_type'] = $form_state['values']['options']['group_type']; + $form_state['handler']->options['group_type'] = $form_state['values']['options']['group_type']; } /** diff --git a/core/modules/views/lib/Drupal/views/Plugin/views/argument/ArgumentPluginBase.php b/core/modules/views/lib/Drupal/views/Plugin/views/argument/ArgumentPluginBase.php index 0b0f992..9d152a0 100644 --- a/core/modules/views/lib/Drupal/views/Plugin/views/argument/ArgumentPluginBase.php +++ b/core/modules/views/lib/Drupal/views/Plugin/views/argument/ArgumentPluginBase.php @@ -430,7 +430,7 @@ public function submitOptionsForm(&$form, &$form_state) { } // Clear out the content of title if it's not enabled. - $options =& $form_state['values']['options']; + $options = &$form_state['values']['options']; if (empty($options['title_enable'])) { $options['title'] = ''; } diff --git a/core/modules/views/lib/Drupal/views/Plugin/views/relationship/GroupwiseMax.php b/core/modules/views/lib/Drupal/views/Plugin/views/relationship/GroupwiseMax.php index 823af0d..18cd4fd 100644 --- a/core/modules/views/lib/Drupal/views/Plugin/views/relationship/GroupwiseMax.php +++ b/core/modules/views/lib/Drupal/views/Plugin/views/relationship/GroupwiseMax.php @@ -242,7 +242,7 @@ protected function leftQuery($options) { // Workaround until http://drupal.org/node/844910 is fixed: // Remove all fields from the SELECT except the base id. - $fields =& $subquery->getFields(); + $fields = &$subquery->getFields(); foreach (array_keys($fields) as $field_name) { // The base id for this subquery is stored in our definition. if ($field_name != $this->definition['field']) { @@ -252,7 +252,7 @@ protected function leftQuery($options) { // Make every alias in the subquery safe within the outer query by // appending a namespace to it, '_inner' by default. - $tables =& $subquery->getTables(); + $tables = &$subquery->getTables(); foreach (array_keys($tables) as $table_name) { $tables[$table_name]['alias'] .= $this->subquery_namespace; // Namespace the join on every table. @@ -266,13 +266,13 @@ protected function leftQuery($options) { $fields[$field_name]['alias'] .= $this->subquery_namespace; } // Namespace conditions. - $where =& $subquery->conditions(); + $where = &$subquery->conditions(); $this->alterSubqueryCondition($subquery, $where); // Not sure why, but our sort order clause doesn't have a table. // TODO: the call to add_item() above to add the sort handler is probably // wrong -- needs attention from someone who understands it. // In the meantime, this works, but with a leap of faith... - $orders =& $subquery->getOrderBy(); + $orders = &$subquery->getOrderBy(); foreach ($orders as $order_key => $order) { // But if we're using a whole view, we don't know what we have! if ($options['subquery_view']) { @@ -314,7 +314,7 @@ protected function alterSubqueryCondition(AlterableInterface $query, &$condition $condition['field'] = $this->conditionNamespace($condition['field']); } elseif (is_object($condition['field'])) { - $sub_conditions =& $condition['field']->conditions(); + $sub_conditions = &$condition['field']->conditions(); $this->alterSubqueryCondition($query, $sub_conditions); } } diff --git a/core/modules/views/lib/Drupal/views/Plugin/views/wizard/WizardPluginBase.php b/core/modules/views/lib/Drupal/views/Plugin/views/wizard/WizardPluginBase.php index e000b92..debb036 100644 --- a/core/modules/views/lib/Drupal/views/Plugin/views/wizard/WizardPluginBase.php +++ b/core/modules/views/lib/Drupal/views/Plugin/views/wizard/WizardPluginBase.php @@ -510,7 +510,7 @@ public static function getSelected($form_state, $parents, $default_value, $eleme * The display ID (e.g. 'page' or 'block'). */ protected function buildFormStyle(array &$form, array &$form_state, $type) { - $style_form =& $form['displays'][$type]['options']['style']; + $style_form = &$form['displays'][$type]['options']['style']; $style = $style_form['style_plugin']['#default_value']; $style_plugin = Views::pluginManager('style')->createInstance($style); if (isset($style_plugin) && $style_plugin->usesRowPlugin()) { diff --git a/core/modules/views/lib/Drupal/views/Tests/Handler/FieldWebTest.php b/core/modules/views/lib/Drupal/views/Tests/Handler/FieldWebTest.php index 6d73636..7bf0e89 100644 --- a/core/modules/views/lib/Drupal/views/Tests/Handler/FieldWebTest.php +++ b/core/modules/views/lib/Drupal/views/Tests/Handler/FieldWebTest.php @@ -206,7 +206,7 @@ public function testAlterUrl() { // Get the expected start of the path string. $base = ($absolute ? $base_url . '/' : base_path()) . $script_path; $absolute_string = $absolute ? 'absolute' : NULL; - $alter =& $id_field->options['alter']; + $alter = &$id_field->options['alter']; $alter['path'] = 'node/123'; $expected_result = url('node/123', array('absolute' => $absolute)); diff --git a/core/modules/views/lib/Drupal/views/ViewExecutable.php b/core/modules/views/lib/Drupal/views/ViewExecutable.php index fe11e11..ad16546 100644 --- a/core/modules/views/lib/Drupal/views/ViewExecutable.php +++ b/core/modules/views/lib/Drupal/views/ViewExecutable.php @@ -914,7 +914,7 @@ protected function _buildArguments() { // Iterate through each argument and process. foreach ($this->argument as $id => $arg) { $position++; - $argument = &$this->argument[$id]; + $argument = $this->argument[$id]; if ($argument->broken()) { continue; @@ -2072,15 +2072,6 @@ public function getItem($display_id, $type, $id) { } /** - * Sets the build array used by the view. - * - * @param array $element - */ - public function setElement(&$element) { - $this->element =& $element; - } - - /** * Sets the configuration of a handler instance on a given display. * * @param string $display_id diff --git a/core/modules/views/views.module b/core/modules/views/views.module index f8f9ff5..f62b96a 100644 --- a/core/modules/views/views.module +++ b/core/modules/views/views.module @@ -298,7 +298,7 @@ function views_preprocess_comment(&$variables) { // The view data is added to the comment in // \Drupal\views\Plugin\views\row\EntityRow::preRender(). if (!empty($variables['comment']->view) && $variables['comment']->view->storage->id()) { - $variables['view'] = &$variables['comment']->view; + $variables['view'] = $variables['comment']->view; } } @@ -1371,7 +1371,7 @@ function views_pre_render_views_form_views_form($element) { /** * Form builder for the exposed widgets form. * - * Be sure that $view and $display are references. + * Be sure that $display is a reference. */ function views_exposed_form($form, &$form_state) { // Don't show the form when batch operations are in progress. @@ -1385,7 +1385,7 @@ function views_exposed_form($form, &$form_state) { // Make sure that we validate because this form might be submitted // multiple times per page. $form_state['must_validate'] = TRUE; - $view = &$form_state['view']; + $view = $form_state['view']; $display = &$form_state['display']; $form_state['input'] = $view->getExposedInput(); @@ -1530,8 +1530,8 @@ function views_exposed_form_cache($views_name, $display_name, $form_output = NUL */ function views_query_views_alter(AlterableInterface $query) { $substitutions = $query->getMetaData('views_substitutions'); - $tables =& $query->getTables(); - $where =& $query->conditions(); + $tables = &$query->getTables(); + $where = &$query->conditions(); // Replaces substitions in tables. foreach ($tables as $table_name => $table_metadata) { @@ -1556,7 +1556,7 @@ function _views_query_tag_alter_condition(AlterableInterface $query, &$condition $condition['field'] = str_replace(array_keys($substitutions), array_values($substitutions), $condition['field']); } elseif (is_object($condition['field'])) { - $sub_conditions =& $condition['field']->conditions(); + $sub_conditions = &$condition['field']->conditions(); _views_query_tag_alter_condition($query, $sub_conditions, $substitutions); } // $condition['value'] is a subquery so alter the subquery recursive. diff --git a/core/modules/views/views.theme.inc b/core/modules/views/views.theme.inc index 65f8ac4..084b87d 100644 --- a/core/modules/views/views.theme.inc +++ b/core/modules/views/views.theme.inc @@ -180,7 +180,7 @@ function template_preprocess_views_view_fields(&$variables) { $empty = $field->isValueEmpty($field_output, $field->options['empty_zero']); if (empty($field->options['exclude']) && (!$empty || (empty($field->options['hide_empty']) && empty($variables['options']['hide_empty'])))) { $object = new stdClass(); - $object->handler = &$view->field[$id]; + $object->handler = $view->field[$id]; $object->inline = !empty($variables['options']['inline'][$id]); $object->element_type = $object->handler->elementType(TRUE, !$variables['options']['default_field_elements'], $object->inline); @@ -958,11 +958,10 @@ function template_preprocess_views_view_list(&$variables) { * An associative array containing: * - view: A ViewExecutable object. * - rows: The raw row data. -*/ + */ function template_preprocess_views_view_rss(&$variables) { - $view = $variables['view']; + $view = $variables['view']; $items = $variables['rows']; - $style = $view->style_plugin; $config = \Drupal::config('system.site'); diff --git a/core/modules/views_ui/admin.inc b/core/modules/views_ui/admin.inc index 8631f25..aac0e39 100644 --- a/core/modules/views_ui/admin.inc +++ b/core/modules/views_ui/admin.inc @@ -343,7 +343,7 @@ function views_ui_pre_render_move_argument_options($form) { * the current display. */ function views_ui_standard_display_dropdown(&$form, &$form_state, $section) { - $view = &$form_state['view']; + $view = $form_state['view']; $display_id = $form_state['display_id']; $executable = $view->getExecutable(); $displays = $executable->displayHandlers; diff --git a/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/AddItem.php b/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/AddItem.php index ed6c1e2..b45e113 100644 --- a/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/AddItem.php +++ b/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/AddItem.php @@ -49,7 +49,7 @@ public function getFormId() { * {@inheritdoc} */ public function buildForm(array $form, array &$form_state) { - $view = &$form_state['view']; + $view = $form_state['view']; $display_id = $form_state['display_id']; $type = $form_state['type']; diff --git a/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/Analyze.php b/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/Analyze.php index df8d83c..6214e9b 100644 --- a/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/Analyze.php +++ b/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/Analyze.php @@ -34,7 +34,7 @@ public function getFormId() { * {@inheritdoc} */ public function buildForm(array $form, array &$form_state) { - $view = &$form_state['view']; + $view = $form_state['view']; $form['#title'] = $this->t('View analysis'); $form['#section'] = 'analyze'; diff --git a/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/ConfigItem.php b/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/ConfigItem.php index 6728002..0185bf7 100644 --- a/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/ConfigItem.php +++ b/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/ConfigItem.php @@ -51,7 +51,7 @@ public function getFormId() { * {@inheritdoc} */ public function buildForm(array $form, array &$form_state) { - $view = &$form_state['view']; + $view = $form_state['view']; $display_id = $form_state['display_id']; $type = $form_state['type']; $id = $form_state['id']; @@ -158,7 +158,7 @@ public function buildForm(array $form, array &$form_state) { // Get form from the handler. $handler->buildOptionsForm($form['options'], $form_state); - $form_state['handler'] = &$handler; + $form_state['handler'] = $handler; } $name = NULL; diff --git a/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/ConfigItemExtra.php b/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/ConfigItemExtra.php index f097a26..b36bb75 100644 --- a/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/ConfigItemExtra.php +++ b/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/ConfigItemExtra.php @@ -50,7 +50,7 @@ public function getFormId() { * {@inheritdoc} */ public function buildForm(array $form, array &$form_state) { - $view = &$form_state['view']; + $view = $form_state['view']; $display_id = $form_state['display_id']; $type = $form_state['type']; $id = $form_state['id']; @@ -81,7 +81,7 @@ public function buildForm(array $form, array &$form_state) { // Get form from the handler. $handler->buildExtraOptionsForm($form['options'], $form_state); - $form_state['handler'] = &$handler; + $form_state['handler'] = $handler; } $view->getStandardButtons($form, $form_state, 'views_ui_config_item_extra_form'); diff --git a/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/ConfigItemGroup.php b/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/ConfigItemGroup.php index c4ddbf2..c4f552b 100644 --- a/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/ConfigItemGroup.php +++ b/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/ConfigItemGroup.php @@ -51,7 +51,7 @@ public function getFormId() { * {@inheritdoc} */ public function buildForm(array $form, array &$form_state) { - $view = &$form_state['view']; + $view = $form_state['view']; $display_id = $form_state['display_id']; $type = $form_state['type']; $id = $form_state['id']; @@ -84,7 +84,7 @@ public function buildForm(array $form, array &$form_state) { $form['#title'] = $this->t('Configure aggregation settings for @type %item', array('@type' => $types[$type]['lstitle'], '%item' => $handler->adminLabel())); $handler->buildGroupByForm($form['options'], $form_state); - $form_state['handler'] = &$handler; + $form_state['handler'] = $handler; } $view->getStandardButtons($form, $form_state, 'views_ui_config_item_group_form'); diff --git a/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/Display.php b/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/Display.php index 9bafab1..5b85f37 100644 --- a/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/Display.php +++ b/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/Display.php @@ -59,13 +59,12 @@ public function getFormId() { * {@inheritdoc} */ public function buildForm(array $form, array &$form_state) { - $view = &$form_state['view']; + $view = $form_state['view']; $display_id = $form_state['display_id']; $section = $form_state['section']; $executable = $view->getExecutable(); $executable->setDisplay($display_id); - $display = &$executable->display[$display_id]; // Get form from the handler. $form['options'] = array( diff --git a/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/EditDetails.php b/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/EditDetails.php index ba41b62..090ed91 100644 --- a/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/EditDetails.php +++ b/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/EditDetails.php @@ -33,7 +33,7 @@ public function getFormId() { * {@inheritdoc} */ public function buildForm(array $form, array &$form_state) { - $view = &$form_state['view']; + $view = $form_state['view']; $form['#title'] = $this->t('View name and description'); $form['#section'] = 'details'; diff --git a/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/Rearrange.php b/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/Rearrange.php index 6beed55..dff2c6f 100644 --- a/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/Rearrange.php +++ b/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/Rearrange.php @@ -48,7 +48,7 @@ public function getFormId() { * {@inheritdoc} */ public function buildForm(array $form, array &$form_state) { - $view = &$form_state['view']; + $view = $form_state['view']; $display_id = $form_state['display_id']; $type = $form_state['type']; diff --git a/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/RearrangeFilter.php b/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/RearrangeFilter.php index 71b0694..219522e 100644 --- a/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/RearrangeFilter.php +++ b/core/modules/views_ui/lib/Drupal/views_ui/Form/Ajax/RearrangeFilter.php @@ -34,7 +34,7 @@ public function getFormId() { * {@inheritdoc} */ public function buildForm(array $form, array &$form_state) { - $view = &$form_state['view']; + $view = $form_state['view']; $display_id = $form_state['display_id']; $type = 'filter';