diff --git a/drush/views.drush.inc b/drush/views.drush.inc index 4384339..42aff0f 100644 --- a/drush/views.drush.inc +++ b/drush/views.drush.inc @@ -124,7 +124,7 @@ function views_revert_views() { continue; } if ($view->type == dt('Overridden')) { - $overridden[$view->name] = $view->name; + $overridden[$view->storage->name] = $view->storage->name; } } @@ -220,14 +220,14 @@ function views_revert_view($view) { // Revert the view. $view->delete(); // Clear its cache. - views_temp_store()->delete($view->name); + views_temp_store()->delete($view->storage->name); // Give feedback. - $message = dt("Reverted the view '@viewname'", array('@viewname' => $view->name)); + $message = dt("Reverted the view '@viewname'", array('@viewname' => $view->storage->name)); drush_log($message, 'success'); // Reverted one more view. } else { - drush_set_error(dt("The view '@viewname' is not overridden.", array('@viewname' => $view->name))); + drush_set_error(dt("The view '@viewname' is not overridden.", array('@viewname' => $view->storage->name))); } } @@ -319,7 +319,7 @@ function drush_views_list() { foreach ($views as $id => $view) { // if options were specified, check that first // mismatch push the loop to the next view - if ($with_tags && !in_array($view->tag, $tags)) { + if ($with_tags && !in_array($view->storage->tag, $tags)) { continue; } if ($with_status && !$view->disabled == $status) { @@ -328,21 +328,21 @@ function drush_views_list() { if ($with_type && strtolower($view->type) !== $type[0]) { continue; } - if ($with_name && !stristr($view->name, $name[0])) { + if ($with_name && !stristr($view->storage->name, $name[0])) { continue; } $row = array(); // each row entry should be in the same order as the header - $row[] = $view->name; - $row[] = $view->description; + $row[] = $view->storage->name; + $row[] = $view->storage->description; $row[] = $view->type; - $row[] = $view->disabled ? dt('Disabled') : dt('Enabled'); - $row[] = $view->tag; + $row[] = $view->storage->disabled ? dt('Disabled') : dt('Enabled'); + $row[] = $view->storage->tag; // place the row in the appropiate array, // so we can have disabled views at the bottom - if ($view->disabled) { + if ($view->storage->disabled) { $disabled_views[] = $row; } else{ diff --git a/includes/admin.inc b/includes/admin.inc index 0bab69f..9122059 100644 --- a/includes/admin.inc +++ b/includes/admin.inc @@ -131,7 +131,7 @@ function views_ui_preview($view, $display_id, $args = array()) { } // Make view links come back to preview. - $view->override_path = 'admin/structure/views/nojs/preview/' . $view->name . '/' . $display_id; + $view->override_path = 'admin/structure/views/nojs/preview/' . $view->storage->name . '/' . $display_id; // Also override the current path so we get the pager. $original_path = current_path(); @@ -712,7 +712,7 @@ function views_ui_add_form_store_edit_submit($form, &$form_state) { $destination = drupal_get_destination(); $query->remove('destination'); } - $form_state['redirect'] = array('admin/structure/views/view/' . $view->name, array('query' => $destination)); + $form_state['redirect'] = array('admin/structure/views/view/' . $view->storage->name, array('query' => $destination)); } /** @@ -793,24 +793,24 @@ function theme_views_ui_view_info($variables) { /** * Page to delete a view. */ -function views_ui_break_lock_confirm($form, &$form_state, $view) { +function views_ui_break_lock_confirm($form, &$form_state, ViewExecutable $view) { $form_state['view'] = &$view; $form = array(); if (empty($view->locked)) { - $form['message']['#markup'] = t('There is no lock on view %name to break.', array('%name' => $view->name)); + $form['message']['#markup'] = t('There is no lock on view %name to break.', array('%name' => $view->storage->name)); return $form; } $cancel = drupal_container()->get('request')->query->get('cancel'); if (empty($cancel)) { - $cancel = 'admin/structure/views/view/' . $view->name . '/edit'; + $cancel = 'admin/structure/views/view/' . $view->storage->name . '/edit'; } $account = user_load($view->locked->ownerId); return confirm_form($form, t('Are you sure you want to break the lock on view %name?', - array('%name' => $view->name)), + array('%name' => $view->storage->name)), $cancel, t('By breaking this lock, any unsaved changes made by !user will be lost!', array('!user' => theme('username', array('account' => $account)))), t('Break lock'), @@ -821,8 +821,8 @@ function views_ui_break_lock_confirm($form, &$form_state, $view) { * Submit handler to break_lock a view. */ function views_ui_break_lock_confirm_submit(&$form, &$form_state) { - UserTempStore::clearAll('view', $form_state['view']->name); - $form_state['redirect'] = 'admin/structure/views/view/' . $form_state['view']->name . '/edit'; + UserTempStore::clearAll('view', $form_state['view']->storage->name); + $form_state['redirect'] = 'admin/structure/views/view/' . $form_state['view']->storage->name . '/edit'; drupal_set_message(t('The lock has been broken and you may now edit this view.')); } @@ -912,7 +912,7 @@ function views_ui_build_preview($view, $display_id, $render = TRUE) { * * @see views_ui_ajax_get_form() */ -function views_ui_edit_form($form, &$form_state, $view, $display_id = NULL) { +function views_ui_edit_form($form, &$form_state, ViewExecutable $view, $display_id = NULL) { // Do not allow the form to be cached, because $form_state['view'] can become // stale between page requests. // See views_ui_ajax_get_form() for how this affects #ajax. @@ -973,7 +973,7 @@ function views_ui_edit_form($form, &$form_state, $view, $display_id = NULL) { $form['locked'] = array( '#theme_wrappers' => array('container'), '#attributes' => array('class' => array('view-locked', 'messages', 'warning')), - '#markup' => t('This view is being edited by user !user, and is therefore locked from editing by others. This lock is !age old. Click here to break this lock.', array('!user' => theme('username', array('account' => user_load($view->locked->ownerId))), '!age' => format_interval(REQUEST_TIME - $view->locked->updated), '!break' => url('admin/structure/views/view/' . $view->name . '/break-lock'))), + '#markup' => t('This view is being edited by user !user, and is therefore locked from editing by others. This lock is !age old. Click here to break this lock.', array('!user' => theme('username', array('account' => user_load($view->locked->ownerId))), '!age' => format_interval(REQUEST_TIME - $view->locked->updated), '!break' => url('admin/structure/views/view/' . $view->storage->name . '/break-lock'))), ); } if (isset($view->vid) && $view->vid == 'new') { @@ -1097,7 +1097,7 @@ function views_ui_edit_form($form, &$form_state, $view, $display_id = NULL) { /** * Provide the preview formulas and the preview output, too. */ -function views_ui_preview_form($form, &$form_state, $view, $display_id = 'default') { +function views_ui_preview_form($form, &$form_state, ViewExecutable $view, $display_id = 'default') { $form_state['no_cache'] = TRUE; $form_state['view'] = $view; @@ -1129,7 +1129,7 @@ function views_ui_preview_form($form, &$form_state, $view, $display_id = 'defaul '#id' => 'preview-submit', '#submit' => array('views_ui_edit_form_submit_preview'), '#ajax' => array( - 'path' => 'admin/structure/views/view/' . $view->name . '/preview/' . $display_id . '/ajax', + 'path' => 'admin/structure/views/view/' . $view->storage->name . '/preview/' . $display_id . '/ajax', 'wrapper' => 'views-preview-wrapper', 'event' => 'click', 'progress' => array('type' => 'throbber'), @@ -1141,7 +1141,7 @@ function views_ui_preview_form($form, &$form_state, $view, $display_id = 'defaul // we may need to split Preview into a separate form. '#process' => array_merge(array('views_ui_default_button'), element_info_property('submit', '#process', array())), ); - $form['#action'] = url('admin/structure/views/view/' . $view->name .'/preview/' . $display_id); + $form['#action'] = url('admin/structure/views/view/' . $view->storage->name .'/preview/' . $display_id); return $form; } @@ -1166,21 +1166,21 @@ function views_ui_render_display_top($view, $display_id) { '#links' => array( 'edit-details' => array( 'title' => t('edit view name/description'), - 'href' => "admin/structure/views/nojs/edit-details/$view->name", + 'href' => "admin/structure/views/nojs/edit-details/{$view->storage->name}", 'attributes' => array('class' => array('views-ajax-link')), ), 'analyze' => array( 'title' => t('analyze view'), - 'href' => "admin/structure/views/nojs/analyze/$view->name/$display_id", + 'href' => "admin/structure/views/nojs/analyze/{$view->storage->name}/$display_id", 'attributes' => array('class' => array('views-ajax-link')), ), 'clone' => array( 'title' => t('clone view'), - 'href' => "admin/structure/views/view/$view->name/clone", + 'href' => "admin/structure/views/view/{$view->storage->name}/clone", ), 'reorder' => array( 'title' => t('reorder displays'), - 'href' => "admin/structure/views/nojs/reorder-displays/$view->name/$display_id", + 'href' => "admin/structure/views/nojs/reorder-displays/{$view->storage->name}/$display_id", 'attributes' => array('class' => array('views-ajax-link')), ), ), @@ -1193,14 +1193,14 @@ function views_ui_render_display_top($view, $display_id) { if ($view->type == t('Overridden')) { $element['extra_actions']['#links']['revert'] = array( 'title' => t('revert view'), - 'href' => "admin/structure/views/view/$view->name/revert", - 'query' => array('destination' => "admin/structure/views/view/$view->name"), + 'href' => "admin/structure/views/view/{$view->storage->name}/revert", + 'query' => array('destination' => "admin/structure/views/view/{$view->storage->name}"), ); } else { $element['extra_actions']['#links']['delete'] = array( 'title' => t('delete view'), - 'href' => "admin/structure/views/view/$view->name/delete", + 'href' => "admin/structure/views/view/{$view->storage->name}/delete", ); } } @@ -1216,7 +1216,7 @@ function views_ui_render_display_top($view, $display_id) { } // Buttons for adding a new display. - foreach (views_fetch_plugin_names('display', NULL, array($view->base_table)) as $type => $label) { + foreach (views_fetch_plugin_names('display', NULL, array($view->storage->base_table)) as $type => $label) { $element['add_display'][$type] = array( '#type' => 'submit', '#value' => t('Add !display', array('!display' => $label)), @@ -1250,7 +1250,7 @@ function views_ui_edit_form_submit_add_display($form, &$form_state) { views_ui_cache_set($view); // Redirect to the new display's edit page. - $form_state['redirect'] = 'admin/structure/views/view/' . $view->name . '/edit/' . $display_id; + $form_state['redirect'] = 'admin/structure/views/view/' . $view->storage->name . '/edit/' . $display_id; } /** @@ -1272,7 +1272,7 @@ function views_ui_edit_form_submit_duplicate_display($form, &$form_state) { views_ui_cache_set($view); // Redirect to the new display's edit page. - $form_state['redirect'] = 'admin/structure/views/view/' . $view->name . '/edit/' . $new_display_id; + $form_state['redirect'] = 'admin/structure/views/view/' . $view->storage->name . '/edit/' . $new_display_id; } /** @@ -1288,7 +1288,7 @@ function views_ui_edit_form_submit_delete_display($form, &$form_state) { // Redirect to the top-level edit page. The first remaining display will // become the active display. - $form_state['redirect'] = 'admin/structure/views/view/' . $view->name; + $form_state['redirect'] = 'admin/structure/views/view/' . $view->storage->name; } /** @@ -1303,7 +1303,7 @@ function views_ui_edit_form_submit_undo_delete_display($form, &$form_state) { views_ui_cache_set($form_state['view']); // Redirect to the top-level edit page. - $form_state['redirect'] = 'admin/structure/views/view/' . $form_state['view']->name . '/edit/' . $id; + $form_state['redirect'] = 'admin/structure/views/view/' . $form_state['view']->storage->name . '/edit/' . $id; } /** @@ -1318,7 +1318,7 @@ function views_ui_edit_form_submit_enable_display($form, &$form_state) { views_ui_cache_set($form_state['view']); // Redirect to the top-level edit page. - $form_state['redirect'] = 'admin/structure/views/view/' . $form_state['view']->name . '/edit/' . $id; + $form_state['redirect'] = 'admin/structure/views/view/' . $form_state['view']->storage->name . '/edit/' . $id; } /** @@ -1332,7 +1332,7 @@ function views_ui_edit_form_submit_disable_display($form, &$form_state) { views_ui_cache_set($form_state['view']); // Redirect to the top-level edit page. - $form_state['redirect'] = 'admin/structure/views/view/' . $form_state['view']->name . '/edit/' . $id; + $form_state['redirect'] = 'admin/structure/views/view/' . $form_state['view']->storage->name . '/edit/' . $id; } /** @@ -1340,7 +1340,7 @@ function views_ui_edit_form_submit_disable_display($form, &$form_state) { */ function views_ui_edit_form_submit_preview($form, &$form_state) { // Rebuild the form with a pristine $view object. - $form_state['build_info']['args'][0] = views_ui_cache_load($form_state['view']->name); + $form_state['build_info']['args'][0] = views_ui_cache_load($form_state['view']->storage->name); $form_state['show_preview'] = TRUE; $form_state['rebuild'] = TRUE; } @@ -1392,12 +1392,12 @@ function views_ui_edit_page_display_tabs($view, $display_id = NULL) { $tabs = array(); // Create a tab for each display. - foreach ($view->display as $id => $display) { + foreach ($view->storage->display as $id => $display) { $tabs[$id] = array( '#theme' => 'menu_local_task', '#link' => array( 'title' => views_ui_get_display_label($view, $id), - 'href' => 'admin/structure/views/view/' . $view->name . '/edit/' . $id, + 'href' => 'admin/structure/views/view/' . $view->storage->name . '/edit/' . $id, 'localized_options' => array(), ), ); @@ -1416,7 +1416,7 @@ function views_ui_edit_page_display_tabs($view, $display_id = NULL) { // Mark the display tab as red to show validation errors. $view->validate(); - foreach ($view->display as $id => $display) { + foreach ($view->storage->display as $id => $display) { if (!empty($view->display_errors[$id])) { // Always show the tab. $tabs[$id]['#access'] = TRUE; @@ -1437,7 +1437,7 @@ function views_ui_show_default_display($view) { $advanced_mode = config('views.settings')->get('ui.show.master_display'); // For other users, show the default display only if there are no others, and // hide it if there's at least one "real" display. - $additional_displays = (count($view->display) == 1); + $additional_displays = (count($view->displayHandlers) == 1); return $advanced_mode || $additional_displays; } @@ -1447,7 +1447,7 @@ function views_ui_show_default_display($view) { */ function views_ui_get_display_tab($view, $display_id) { $build = array(); - $display = $view->display[$display_id]; + $display = $view->displayHandlers[$display_id]; // If the plugin doesn't exist, display an error message instead of an edit // page. if (empty($display)) { @@ -1457,7 +1457,7 @@ function views_ui_get_display_tab($view, $display_id) { } // Build the content of the edit page. else { - $build['details'] = views_ui_get_display_tab_details($view, $display); + $build['details'] = views_ui_get_display_tab_details($view, $display->display); } // In AJAX context, views_ui_regenerate_tab() returns this outside of form // context, so hook_form_views_ui_edit_form_alter() is insufficient. @@ -1672,7 +1672,7 @@ function views_ui_get_display_tab_details($view, $display) { * This function might be more logical as a method on an object, if a suitable * object emerges out of refactoring. */ -function views_ui_edit_form_get_build_from_option($id, $option, $view, $display) { +function views_ui_edit_form_get_build_from_option($id, $option, ViewExecutable $view, $display) { $option_build = array(); $option_build['#theme'] = 'views_ui_display_tab_setting'; @@ -1877,12 +1877,6 @@ function views_ui_import_page($form, &$form_state) { '#title' => t('Replace an existing view if one exists with the same name'), ); - $form['bypass_validation'] = array( - '#type' => 'checkbox', - '#title' => t('Bypass view validation'), - '#description' => t('Bypass the validation of plugins and handlers when importing this view.'), - ); - $form['view'] = array( '#type' => 'textarea', '#title' => t('Paste view code here'), @@ -1893,121 +1887,17 @@ function views_ui_import_page($form, &$form_state) { '#type' => 'submit', '#value' => t('Import'), '#submit' => array('views_ui_import_submit'), - '#validate' => array('views_ui_import_validate'), ); return $form; } /** - * Validate handler to import a view. - */ -function views_ui_import_validate($form, &$form_state) { - $view = ''; - // Be forgiving if someone pastes views code that starts with 'api_version) || $view->api_version < 2) { - form_error($form['view'], t('That view is not compatible with this version of Views. - If you have a view from views1 you have to go to a drupal6 installation and import it there.')); - } - elseif (version_compare($view->api_version, views_api_version(), '>')) { - form_error($form['view'], t('That view is created for the version @import_version of views, but you only have @api_version', array( - '@import_version' => $view->api_version, - '@api_version' => views_api_version()))); - } - - // View name must be alphanumeric or underscores, no other punctuation. - if (!empty($form_state['values']['name']) && preg_match('/[^a-zA-Z0-9_]/', $form_state['values']['name'])) { - form_error($form['name'], t('View name must be alphanumeric or underscores only.')); - } - - if ($form_state['values']['name']) { - $view->name = $form_state['values']['name']; - } - - $test = views_get_view($view->name); - if (!$form_state['values']['name_override']) { - if ($test && $test->type != t('Default')) { - form_set_error('', t('A view by that name already exists; please choose a different name')); - } - } - else { - if ($test->vid) { - $view->vid = $test->vid; - } - } - - // Make sure base table gets set properly if it got moved. - $view->update(); - - $view->initDisplay(); - - $broken = FALSE; - - // Bypass the validation of view pluigns/handlers if option is checked. - if (!$form_state['values']['bypass_validation']) { - // Make sure that all plugins and handlers needed by this view actually exist. - foreach ($view->displayHandlers as $id => $display) { - if (empty($display) || !empty($display->broken)) { - drupal_set_message(t('Display plugin @plugin is not available.', array('@plugin' => $display->getPluginId())), 'error'); - $broken = TRUE; - continue; - } - - $plugin = views_get_plugin('style', $display->getOption('style_plugin')); - if (!$plugin) { - drupal_set_message(t('Style plugin @plugin is not available.', array('@plugin' => $display->getOption('style_plugin'))), 'error'); - $broken = TRUE; - } - elseif ($plugin->usesRowPlugin()) { - $plugin = views_get_plugin('row', $display->getOption('row_plugin')); - if (!$plugin) { - drupal_set_message(t('Row plugin @plugin is not available.', array('@plugin' => $display->getOption('row_plugin'))), 'error'); - $broken = TRUE; - } - } - - foreach (ViewExecutable::viewsHandlerTypes() as $type => $info) { - $handlers = $display->getHandlers($type); - if ($handlers) { - foreach ($handlers as $id => $handler) { - if ($handler->broken()) { - drupal_set_message(t('@type handler @table.@field is not available.', array( - '@type' => $info['stitle'], - '@table' => $handler->table, - '@field' => $handler->field, - )), 'error'); - $broken = TRUE; - } - } - } - } - } - } - - if ($broken) { - form_set_error('', t('Unable to import view.')); - } - - $form_state['view'] = &$view; -} - -/** * Submit handler for view import. */ function views_ui_import_submit($form, &$form_state) { // Store in cache and then go to edit. views_ui_cache_set($form_state['view']); - $form_state['redirect'] = 'admin/structure/views/view/' . $form_state['view']->name . '/edit'; + $form_state['redirect'] = 'admin/structure/views/view/' . $form_state['view']->storage->name . '/edit'; } /** @@ -2032,17 +1922,17 @@ function views_ui_edit_view_form_validate($form, &$form_state) { */ function views_ui_edit_view_form_submit($form, &$form_state) { // Go through and remove displayed scheduled for removal. - foreach ($form_state['view']->display as $id => $display) { + foreach ($form_state['view']->displayHandlers as $id => $display) { if (!empty($display->deleted)) { - unset($form_state['view']->display[$id]); + unset($form_state['view']->displayHandlers[$id]); } } // Rename display ids if needed. - foreach ($form_state['view']->display as $id => $display) { - if (!empty($display['new_id'])) { - $form_state['view']->display[$id]['id'] = $display['new_id']; + foreach ($form_state['view']->displayHandlers as $id => $display) { + if (!empty($display->display['new_id'])) { + $form_state['view']->displayHandlers[$id]['id'] = $display['new_id']; // Redirect the user to the renamed display to be sure that the page itself exists and doesn't throw errors. - $form_state['redirect'] = 'admin/structure/views/view/' . $form_state['view']->name . '/edit/' . $display['new_id']; + $form_state['redirect'] = 'admin/structure/views/view/' . $form_state['view']->storage->name . '/edit/' . $display['new_id']; } } @@ -2052,15 +1942,15 @@ function views_ui_edit_view_form_submit($form, &$form_state) { $destination = $query->get('destination'); if (!empty($destination)) { // Find out the first display which has a changed path and redirect to this url. - $old_view = views_get_view($form_state['view']->name); - foreach ($old_view->display as $id => $display) { + $old_view = views_get_view($form_state['view']->storage->name); + foreach ($old_view->displayHandlers as $id => $display) { // Only check for displays with a path. - if (!isset($display['display_options']['path'])) { + if (!isset($display->display['display_options']['path'])) { continue; } - $old_path = $display['display_options']['path']; - if (($display['display_plugin'] == 'page') && ($old_path == $destination) && ($old_path != $form_state['view']->display[$id]['display_options']['path'])) { - $destination = $form_state['view']->display[$id]['display_options']['path']; + $old_path = $display->display['display_options']['path']; + if (($display->display['display_plugin'] == 'page') && ($old_path == $destination) && ($old_path != $form_state['view']->display[$id]->display['display_options']['path'])) { + $destination = $form_state['view']->displayHandlers[$id]->display['display_options']['path']; $query->remove('destination'); } } @@ -2071,7 +1961,7 @@ function views_ui_edit_view_form_submit($form, &$form_state) { drupal_set_message(t('The view %name has been saved.', array('%name' => $form_state['view']->getHumanName()))); // Remove this view from cache so we can edit it properly. - views_temp_store()->delete($form_state['view']->name); + views_temp_store()->delete($form_state['view']->storage->name); } /** @@ -2079,7 +1969,7 @@ function views_ui_edit_view_form_submit($form, &$form_state) { */ function views_ui_edit_view_form_cancel($form, &$form_state) { // Remove this view from cache so edits will be lost. - views_temp_store()->delete($form_state['view']->name); + views_temp_store()->delete($form_state['view']->storage->name); if (empty($form['view']->vid)) { // I seem to have to drupal_goto here because I can't get fapi to // honor the redirect target. Not sure what I screwed up here. @@ -2094,13 +1984,13 @@ function views_ui_edit_view_form_delete($form, &$form_state) { $request->remove('destination'); } // Redirect to the delete confirm page - $form_state['redirect'] = array('admin/structure/views/view/' . $form_state['view']->name . '/delete', array('query' => drupal_get_destination() + array('cancel' => 'admin/structure/views/view/' . $form_state['view']->name . '/edit'))); + $form_state['redirect'] = array('admin/structure/views/view/' . $form_state['view']->storage->name . '/delete', array('query' => drupal_get_destination() + array('cancel' => 'admin/structure/views/view/' . $form_state['view']->storage->name . '/edit'))); } /** * Add information about a section to a display. */ -function views_ui_edit_form_get_bucket($type, $view, $display) { +function views_ui_edit_form_get_bucket($type, ViewExecutable $view, $display) { $build = array( '#theme_wrappers' => array('views_ui_display_tab_bucket'), ); @@ -2115,7 +2005,7 @@ function views_ui_edit_form_get_bucket($type, $view, $display) { // to get the right one. switch ($type) { case 'filter': - $rearrange_url = "admin/structure/views/nojs/rearrange-$type/$view->name/{$display['id']}/$type"; + $rearrange_url = "admin/structure/views/nojs/rearrange-$type/{$view->storage->name}/{$display['id']}/$type"; $rearrange_text = t('And/Or, Rearrange'); // TODO: Add another class to have another symbol for filter rearrange. $class = 'icon compact rearrange'; @@ -2135,7 +2025,7 @@ function views_ui_edit_form_get_bucket($type, $view, $display) { } default: - $rearrange_url = "admin/structure/views/nojs/rearrange/$view->name/{$display['id']}/$type"; + $rearrange_url = "admin/structure/views/nojs/rearrange/{$view->storage->name}/{$display['id']}/$type"; $rearrange_text = t('Rearrange'); $class = 'icon compact rearrange'; } @@ -2145,7 +2035,7 @@ function views_ui_edit_form_get_bucket($type, $view, $display) { $count_handlers = count($view->displayHandlers[$display['id']]->getHandlers($type)); $actions['add'] = array( 'title' => t('Add'), - 'href' => "admin/structure/views/nojs/add-item/$view->name/{$display['id']}/$type", + 'href' => "admin/structure/views/nojs/add-item/{$view->storage->name}/{$display['id']}/$type", 'attributes' => array('class' => array('icon compact add', 'views-ajax-link'), 'title' => t('Add'), 'id' => 'views-add-' . $type), 'html' => TRUE, ); @@ -2180,7 +2070,7 @@ function views_ui_edit_form_get_bucket($type, $view, $display) { // If there's an options form for the bucket, link to it. if (!empty($types[$type]['options'])) { - $build['#title'] = l($build['#title'], "admin/structure/views/nojs/config-type/$view->name/{$display['id']}/$type", array('attributes' => array('class' => array('views-ajax-link'), 'id' => 'views-title-' . $type))); + $build['#title'] = l($build['#title'], "admin/structure/views/nojs/config-type/{$view->storage->name}/{$display['id']}/$type", array('attributes' => array('class' => array('views-ajax-link'), 'id' => 'views-title-' . $type))); } static $relationships = NULL; @@ -2217,7 +2107,7 @@ function views_ui_edit_form_get_bucket($type, $view, $display) { if (empty($handler)) { $build['fields'][$id]['#class'][] = 'broken'; $field_name = t('Broken/missing handler: @table > @field', array('@table' => $field['table'], '@field' => $field['field'])); - $build['fields'][$id]['#link'] = l($field_name, "admin/structure/views/nojs/config-item/$view->name/{$display['id']}/$type/$id", array('attributes' => array('class' => array('views-ajax-link')), 'html' => TRUE)); + $build['fields'][$id]['#link'] = l($field_name, "admin/structure/views/nojs/config-item/{$view->storage->name}/{$display['id']}/$type/$id", array('attributes' => array('class' => array('views-ajax-link')), 'html' => TRUE)); continue; } @@ -2232,15 +2122,15 @@ function views_ui_edit_form_get_bucket($type, $view, $display) { if (!empty($field['exclude'])) { $link_attributes['class'][] = 'views-field-excluded'; } - $build['fields'][$id]['#link'] = l($link_text, "admin/structure/views/nojs/config-item/$view->name/{$display['id']}/$type/$id", array('attributes' => $link_attributes, 'html' => TRUE)); + $build['fields'][$id]['#link'] = l($link_text, "admin/structure/views/nojs/config-item/{$view->storage->name}/{$display['id']}/$type/$id", array('attributes' => $link_attributes, 'html' => TRUE)); $build['fields'][$id]['#class'][] = drupal_clean_css_identifier($display['id']. '-' . $type . '-' . $id); if ($view->displayHandlers[$display['id']]->useGroupBy() && $handler->usesGroupBy()) { - $build['fields'][$id]['#settings_links'][] = l('' . t('Aggregation settings') . '', "admin/structure/views/nojs/config-item-group/$view->name/{$display['id']}/$type/$id", array('attributes' => array('class' => 'views-button-configure views-ajax-link', 'title' => t('Aggregation settings')), 'html' => TRUE)); + $build['fields'][$id]['#settings_links'][] = l('' . t('Aggregation settings') . '', "admin/structure/views/nojs/config-item-group/{$view->storage->name}/{$display['id']}/$type/$id", array('attributes' => array('class' => 'views-button-configure views-ajax-link', 'title' => t('Aggregation settings')), 'html' => TRUE)); } if ($handler->hasExtraOptions()) { - $build['fields'][$id]['#settings_links'][] = l('' . t('Settings') . '', "admin/structure/views/nojs/config-item-extra/$view->name/{$display['id']}/$type/$id", array('attributes' => array('class' => array('views-button-configure', 'views-ajax-link'), 'title' => t('Settings')), 'html' => TRUE)); + $build['fields'][$id]['#settings_links'][] = l('' . t('Settings') . '', "admin/structure/views/nojs/config-item-extra/{$view->storage->name}/{$display['id']}/$type/$id", array('attributes' => array('class' => array('views-button-configure', 'views-ajax-link'), 'title' => t('Settings')), 'html' => TRUE)); } if ($grouping) { @@ -2285,7 +2175,7 @@ function views_ui_edit_form_get_bucket($type, $view, $display) { /** * Regenerate the current tab for AJAX updates. */ -function views_ui_regenerate_tab(&$view, &$output, $display_id) { +function views_ui_regenerate_tab(ViewExecutable $view, &$output, $display_id) { if (!$view->setDisplay('default')) { return; } @@ -2506,7 +2396,7 @@ function views_ui_standard_cancel($form, &$form_state) { views_ui_cache_set($form_state['view']); } - $form_state['redirect'] = 'admin/structure/views/view/' . $form_state['view']->name . '/edit'; + $form_state['redirect'] = 'admin/structure/views/view/' . $form_state['view']->storage->name . '/edit'; } /** @@ -2674,11 +2564,11 @@ function views_ui_ajax_forms($key = NULL) { * is the same as another. Since the arguments differ slightly * we do a lot of spiffy concatenation here. */ -function views_ui_build_identifier($key, $view, $display_id, $args) { +function views_ui_build_identifier($key, ViewExecutable $view, $display_id, $args) { $form = views_ui_ajax_forms($key); // Automatically remove the single-form cache if it exists and // does not match the key. - $identifier = implode('-', array($key, $view->name, $display_id)); + $identifier = implode('-', array($key, $view->storage->name, $display_id)); foreach ($form['args'] as $id) { $arg = (!empty($args)) ? array_shift($args) : NULL; @@ -2691,7 +2581,7 @@ function views_ui_build_identifier($key, $view, $display_id, $args) { * Build up a $form_state object suitable for use with drupal_build_form * based on known information about a form. */ -function views_ui_build_form_state($js, $key, &$view, $display_id, $args) { +function views_ui_build_form_state($js, $key, ViewExecutable $view, $display_id, $args) { $form = views_ui_ajax_forms($key); // Build up form state $form_state = array( @@ -2717,7 +2607,7 @@ function views_ui_build_form_state($js, $key, &$view, $display_id, $args) { function views_ui_build_form_url($form_state) { $form = views_ui_ajax_forms($form_state['form_key']); $ajax = empty($form_state['ajax']) ? 'nojs' : 'ajax'; - $name = $form_state['view']->name; + $name = $form_state['view']->storage->name; $url = "admin/structure/views/$ajax/$form_state[form_key]/$name/$form_state[display_id]"; foreach ($form['args'] as $arg) { $url .= '/' . $form_state[$arg]; @@ -2729,7 +2619,7 @@ function views_ui_build_form_url($form_state) { * Add another form to the stack; clicking 'apply' will go to this form * rather than closing the ajax popup. */ -function views_ui_add_form_to_stack($key, &$view, $display_id, $args, $top = FALSE, $rebuild_keys = FALSE) { +function views_ui_add_form_to_stack($key, ViewExecutable $view, $display_id, $args, $top = FALSE, $rebuild_keys = FALSE) { if (empty($view->stack)) { $view->stack = array(); } @@ -2773,7 +2663,7 @@ function views_ui_add_form_to_stack($key, &$view, $display_id, $args, $top = FAL * We do this for consistency and to make it easy to chain forms * together. */ -function views_ui_ajax_form($js, $key, $view, $display_id = '') { +function views_ui_ajax_form($js, $key, ViewExecutable $view, $display_id = '') { $form = views_ui_ajax_forms($key); if (empty($form)) { return MENU_NOT_FOUND; @@ -2834,7 +2724,7 @@ function views_ui_ajax_form($js, $key, $view, $display_id = '') { } elseif (!$js) { // if nothing on the stack, non-js forms just go back to the main view editor. - return drupal_goto("admin/structure/views/view/$view->name/edit"); + return drupal_goto("admin/structure/views/view/{$view->storage->name}/edit"); } else { $output = array(); @@ -2895,7 +2785,7 @@ function views_ui_analyze_view_form($form, &$form_state) { * Submit handler for views_ui_analyze_view_form */ function views_ui_analyze_view_form_submit($form, &$form_state) { - $form_state['redirect'] = 'admin/structure/views/view/' . $form_state['view']->name . '/edit'; + $form_state['redirect'] = 'admin/structure/views/view/' . $form_state['view']->storage->name . '/edit'; } /** @@ -2954,7 +2844,7 @@ function views_ui_reorder_displays_form($form, &$form_state) { 'limit' => 0, ); - $form['#action'] = url('admin/structure/views/nojs/reorder-displays/' . $view->name . '/' . $display_id); + $form['#action'] = url('admin/structure/views/nojs/reorder-displays/' . $view->storage->name . '/' . $display_id); views_ui_standard_form_buttons($form, $form_state, 'views_ui_reorder_displays_form'); @@ -3014,7 +2904,7 @@ function views_ui_reorder_displays_form_submit($form, &$form_state) { // Store in cache views_ui_cache_set($form_state['view']); - $form_state['redirect'] = array('admin/structure/views/view/' . $form_state['view']->name . '/edit', array('fragment' => 'views-tab-default')); + $form_state['redirect'] = array('admin/structure/views/view/' . $form_state['view']->storage->name . '/edit', array('fragment' => 'views-tab-default')); } /** @@ -3098,14 +2988,14 @@ function views_ui_edit_details_form($form, &$form_state) { '#type' => 'textfield', '#title' => t('View tag'), '#description' => t('Optionally, enter a comma delimited list of tags for this view to use in filtering and sorting views on the administrative page.'), - '#default_value' => $view->tag, + '#default_value' => $view->storage->tag, '#autocomplete_path' => 'admin/views/ajax/autocomplete/tag', ); $form['details']['description'] = array( '#type' => 'textfield', '#title' => t('View description'), '#description' => t('This description will appear on the Views administrative UI to tell you what the view is about.'), - '#default_value' => $view->description, + '#default_value' => $view->storage->description, ); views_ui_standard_form_buttons($form, $form_state, 'views_ui_edit_details_form'); @@ -3139,14 +3029,14 @@ function views_ui_edit_display_form($form, &$form_state) { if (!$view->setDisplay($display_id)) { views_ajax_error(t('Invalid display id @display', array('@display' => $display_id))); } - $display = &$view->display[$display_id]; + $display = &$view->displayHandlers[$display_id]; // Get form from the handler. $form['options'] = array( '#theme_wrappers' => array('container'), '#attributes' => array('class' => array('scroll')), ); - $view->displayHandlers[$display['id']]->buildOptionsForm($form['options'], $form_state); + $display->buildOptionsForm($form['options'], $form_state); // The handler options form sets $form['#title'], which we need on the entire // $form instead of just the ['options'] section. @@ -3521,8 +3411,8 @@ function views_ui_rearrange_filter_form($form, &$form_state) { if (!$view->setDisplay($display_id)) { views_ajax_render(t('Invalid display id @display', array('@display' => $display_id))); } - $display = &$view->display[$display_id]; - $form['#title'] = check_plain($view->display[$display_id]['display_title']) . ': '; + $display = $view->displayHandlers[$display_id]; + $form['#title'] = check_plain($display->display['display_title']) . ': '; $form['#title'] .= t('Rearrange @type', array('@type' => $types[$type]['ltitle'])); $form['#section'] = $display_id . 'rearrange-item'; @@ -4939,8 +4829,8 @@ function views_ui_autocomplete_tag($string = '') { // get matches from default views: $views = views_get_all_views(); foreach ($views as $view) { - if (!empty($view->tag) && strpos($view->tag, $string) === 0) { - $matches[$view->tag] = $view->tag; + if (!empty($view->storage->tag) && strpos($view->storage->tag, $string) === 0) { + $matches[$view->storage->tag] = $view->storage->tag; if (count($matches) >= 10) { break; } @@ -5202,7 +5092,7 @@ function theme_views_ui_style_plugin_table($variables) { * @todo Remove this function once editing the display title is possible. */ function views_ui_get_display_label($view, $display_id, $check_changed = TRUE) { - $title = $display_id == 'default' ? t('Master') : $view->display[$display_id]['display_title']; + $title = $display_id == 'default' ? t('Master') : $view->storage->display[$display_id]['display_title']; $title = views_ui_truncate($title, 25); if ($check_changed && !empty($view->changed_display[$display_id])) { @@ -5296,7 +5186,7 @@ function views_ui_field_list() { && $data = $data[$item['field']][$type]) { // The final check that we have a fieldapi field now. if (isset($data['field_name'])) { - $fields[$data['field_name']][$view->name] = $view->name; + $fields[$data['field_name']][$view->storage->name] = $view->storage->name; } } } diff --git a/lib/Drupal/views/Plugin/views/HandlerBase.php b/lib/Drupal/views/Plugin/views/HandlerBase.php index 907b839..b61de06 100644 --- a/lib/Drupal/views/Plugin/views/HandlerBase.php +++ b/lib/Drupal/views/Plugin/views/HandlerBase.php @@ -87,7 +87,7 @@ public function __construct(array $configuration, $plugin_id, DiscoveryInterface * The item from the database; the actual contents of this will vary * based upon the type of handler. */ - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { $this->setOptionDefaults($this->options, $this->defineOptions()); $this->view = &$view; $display_id = $this->view->current_display; diff --git a/lib/Drupal/views/Plugin/views/access/AccessPluginBase.php b/lib/Drupal/views/Plugin/views/access/AccessPluginBase.php index 31a92fb..738d712 100644 --- a/lib/Drupal/views/Plugin/views/access/AccessPluginBase.php +++ b/lib/Drupal/views/Plugin/views/access/AccessPluginBase.php @@ -8,6 +8,7 @@ namespace Drupal\views\Plugin\views\access; use Drupal\views\Plugin\views\PluginBase; +use Drupal\views\ViewExecutable; /** * @defgroup views_access_plugins Views access plugins @@ -33,7 +34,7 @@ * @param $display * The display handler. */ - public function init(&$view, &$display) { + public function init(ViewExecutable $view, &$display) { $this->setOptionDefaults($this->options, $this->defineOptions()); $this->view = &$view; $this->displayHandler = &$display; diff --git a/lib/Drupal/views/Plugin/views/area/AreaPluginBase.php b/lib/Drupal/views/Plugin/views/area/AreaPluginBase.php index 94caeb3..db0383e 100644 --- a/lib/Drupal/views/Plugin/views/area/AreaPluginBase.php +++ b/lib/Drupal/views/Plugin/views/area/AreaPluginBase.php @@ -7,6 +7,7 @@ namespace Drupal\views\Plugin\views\area; +use Drupal\views\ViewExecutable; use Drupal\views\Plugin\views\PluginBase; use Drupal\views\Plugin\views\HandlerBase; @@ -30,7 +31,7 @@ * Make sure that no result area handlers are set to be shown when the result * is empty. */ - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { $this->setOptionDefaults($this->options, $this->defineOptions()); parent::init($view, $options); if ($this->definition['plugin_type'] == 'empty') { diff --git a/lib/Drupal/views/Plugin/views/area/Result.php b/lib/Drupal/views/Plugin/views/area/Result.php index 6bf0732..bea7527 100644 --- a/lib/Drupal/views/Plugin/views/area/Result.php +++ b/lib/Drupal/views/Plugin/views/area/Result.php @@ -73,7 +73,7 @@ function render($empty = FALSE) { // @TODO: Maybe use a possible is views empty functionality. // Not every view has total_rows set, use view->result instead. $total = isset($this->view->total_rows) ? $this->view->total_rows : count($this->view->result); - $name = check_plain($this->view->human_name); + $name = check_plain($this->view->storage->human_name); if ($per_page === 0) { $page_count = 1; $start = 1; diff --git a/lib/Drupal/views/Plugin/views/area/View.php b/lib/Drupal/views/Plugin/views/area/View.php index 0c589b8..018996c 100644 --- a/lib/Drupal/views/Plugin/views/area/View.php +++ b/lib/Drupal/views/Plugin/views/area/View.php @@ -35,7 +35,7 @@ protected function defineOptions() { public function buildOptionsForm(&$form, &$form_state) { parent::buildOptionsForm($form, $form_state); - $view_display = $this->view->name . ':' . $this->view->current_display; + $view_display = $this->view->storage->name . ':' . $this->view->current_display; $options = array('' => t('-Select-')); $options += views_get_views_as_options(FALSE, 'all', $view_display, FALSE, TRUE); diff --git a/lib/Drupal/views/Plugin/views/argument/ArgumentPluginBase.php b/lib/Drupal/views/Plugin/views/argument/ArgumentPluginBase.php index b8a1fcf..4676b72 100644 --- a/lib/Drupal/views/Plugin/views/argument/ArgumentPluginBase.php +++ b/lib/Drupal/views/Plugin/views/argument/ArgumentPluginBase.php @@ -301,7 +301,6 @@ public function buildOptionsForm(&$form, &$form_state) { ), ); - $validate_types = array('none' => t('- Basic validation -')); $plugins = views_get_plugin_definitions('argument_validator'); foreach ($plugins as $id => $info) { if (!empty($info['no_ui'])) { @@ -946,17 +945,8 @@ public function validateArgument($arg) { return $this->argument_validated = TRUE; } - if ($this->options['validate']['type'] == 'none') { - return $this->argument_validated = $this->validate_argument_basic($arg); - } - $plugin = $this->get_plugin('argument_validator'); - if ($plugin) { - return $this->argument_validated = $plugin->validate_argument($arg); - } - - // If the plugin isn't found, fall back to the basic validation path: - return $this->argument_validated = $this->validate_argument_basic($arg); + return $this->argument_validated = $plugin->validate_argument($arg); } /** @@ -985,25 +975,6 @@ function validate_argument($arg) { } /** - * Provide a basic argument validation. - * - * This can be overridden for more complex types; the basic - * validator only checks to see if the argument is not NULL - * or is numeric if the definition says it's numeric. - */ - function validate_argument_basic($arg) { - if (!isset($arg) || $arg === '') { - return FALSE; - } - - if (!empty($this->definition['numeric']) && !isset($this->options['break_phrase']) && !is_numeric($arg)) { - return FALSE; - } - - return TRUE; - } - - /** * Set the input for this argument * * @return TRUE if it successfully validates; FALSE if it does not. diff --git a/lib/Drupal/views/Plugin/views/argument/Date.php b/lib/Drupal/views/Plugin/views/argument/Date.php index e136b2c..183011a 100644 --- a/lib/Drupal/views/Plugin/views/argument/Date.php +++ b/lib/Drupal/views/Plugin/views/argument/Date.php @@ -51,7 +51,7 @@ function default_argument_form(&$form, &$form_state) { */ function get_default_argument($raw = FALSE) { if (!$raw && $this->options['default_argument_type'] == 'date') { - return date($this->arg_format, REQUEST_TIME); + return date($this->definition['format'], REQUEST_TIME); } elseif (!$raw && in_array($this->options['default_argument_type'], array('node_created', 'node_changed'))) { foreach (range(1, 3) as $i) { @@ -69,10 +69,10 @@ function get_default_argument($raw = FALSE) { return parent::get_default_argument(); } elseif ($this->options['default_argument_type'] == 'node_created') { - return date($this->arg_format, $node->created); + return date($this->definition['format'], $node->created); } elseif ($this->options['default_argument_type'] == 'node_changed') { - return date($this->arg_format, $node->changed); + return date($this->definition['format'], $node->changed); } } diff --git a/lib/Drupal/views/Plugin/views/argument/ManyToOne.php b/lib/Drupal/views/Plugin/views/argument/ManyToOne.php index fe0073f..9ca5100 100644 --- a/lib/Drupal/views/Plugin/views/argument/ManyToOne.php +++ b/lib/Drupal/views/Plugin/views/argument/ManyToOne.php @@ -7,6 +7,7 @@ namespace Drupal\views\Plugin\views\argument; +use Drupal\views\ViewExecutable; use Drupal\Core\Annotation\Plugin; use Drupal\views\ManyToOneHelper; @@ -29,7 +30,7 @@ */ class ManyToOne extends ArgumentPluginBase { - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { parent::init($view, $options); $this->helper = new ManyToOneHelper($this); diff --git a/lib/Drupal/views/Plugin/views/argument/Null.php b/lib/Drupal/views/Plugin/views/argument/Null.php index ce998cc..6097b00 100644 --- a/lib/Drupal/views/Plugin/views/argument/Null.php +++ b/lib/Drupal/views/Plugin/views/argument/Null.php @@ -60,14 +60,6 @@ function default_actions($which = NULL) { return $actions; } - function validate_argument_basic($arg) { - if (!empty($this->options['must_not_be'])) { - return !isset($arg); - } - - return parent::validate_argument_basic($arg); - } - /** * Override the behavior of query() to prevent the query * from being changed in any way. diff --git a/lib/Drupal/views/Plugin/views/argument/String.php b/lib/Drupal/views/Plugin/views/argument/String.php index 27be1ca..2f3646a 100644 --- a/lib/Drupal/views/Plugin/views/argument/String.php +++ b/lib/Drupal/views/Plugin/views/argument/String.php @@ -7,6 +7,7 @@ namespace Drupal\views\Plugin\views\argument; +use Drupal\views\ViewExecutable; use Drupal\views\ManyToOneHelper; use Drupal\Core\Annotation\Plugin; @@ -22,7 +23,7 @@ */ class String extends ArgumentPluginBase { - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { parent::init($view, $options); if (!empty($this->definition['many to one'])) { $this->helper = new ManyToOneHelper($this); diff --git a/lib/Drupal/views/Plugin/views/argument_default/ArgumentDefaultPluginBase.php b/lib/Drupal/views/Plugin/views/argument_default/ArgumentDefaultPluginBase.php index e0cf9fe..5465f0e 100644 --- a/lib/Drupal/views/Plugin/views/argument_default/ArgumentDefaultPluginBase.php +++ b/lib/Drupal/views/Plugin/views/argument_default/ArgumentDefaultPluginBase.php @@ -7,6 +7,7 @@ namespace Drupal\views\Plugin\views\argument_default; +use Drupal\views\ViewExecutable; use Drupal\views\Plugin\views\PluginBase; /** @@ -33,7 +34,7 @@ function get_argument() { } * Initialize this plugin with the view and the argument * it is linked to. */ - public function init(&$view, &$argument, $options) { + public function init(ViewExecutable $view, &$argument, $options) { $this->setOptionDefaults($this->options, $this->defineOptions()); $this->view = &$view; $this->argument = &$argument; diff --git a/lib/Drupal/views/Plugin/views/argument_validator/ArgumentValidatorPluginBase.php b/lib/Drupal/views/Plugin/views/argument_validator/ArgumentValidatorPluginBase.php index 3a9aa4a..c14be5f 100644 --- a/lib/Drupal/views/Plugin/views/argument_validator/ArgumentValidatorPluginBase.php +++ b/lib/Drupal/views/Plugin/views/argument_validator/ArgumentValidatorPluginBase.php @@ -7,6 +7,7 @@ namespace Drupal\views\Plugin\views\argument_validator; +use Drupal\views\ViewExecutable; use Drupal\views\Plugin\views\PluginBase; /** @@ -26,7 +27,7 @@ * Initialize this plugin with the view and the argument * it is linked to. */ - public function init(&$view, &$argument, $options) { + public function init(ViewExecutable $view, &$argument, $options) { $this->setOptionDefaults($this->options, $this->defineOptions()); $this->view = &$view; $this->argument = &$argument; diff --git a/lib/Drupal/views/Plugin/views/argument_validator/None.php b/lib/Drupal/views/Plugin/views/argument_validator/None.php new file mode 100644 index 0000000..f15fc11 --- /dev/null +++ b/lib/Drupal/views/Plugin/views/argument_validator/None.php @@ -0,0 +1,41 @@ +argument->options['must_not_be'])) { + return !isset($argument); + } + + if (!isset($argument) || $argument === '') { + return FALSE; + } + + if (!empty($this->argument->definition['numeric']) && !isset($this->argument->options['break_phrase']) && !is_numeric($arg)) { + return FALSE; + } + + return TRUE; + } + +} diff --git a/lib/Drupal/views/Plugin/views/cache/CachePluginBase.php b/lib/Drupal/views/Plugin/views/cache/CachePluginBase.php index 0ac749c..a9041af 100644 --- a/lib/Drupal/views/Plugin/views/cache/CachePluginBase.php +++ b/lib/Drupal/views/Plugin/views/cache/CachePluginBase.php @@ -7,6 +7,7 @@ namespace Drupal\views\Plugin\views\cache; +use Drupal\views\ViewExecutable; use Drupal\views\Plugin\views\PluginBase; use Drupal\Core\Database\Query\Select; @@ -62,7 +63,7 @@ * @param $display * The display handler. */ - public function init(&$view, &$display) { + public function init(ViewExecutable $view, &$display) { $this->setOptionDefaults($this->options, $this->defineOptions()); $this->view = &$view; $this->displayHandler = &$display; @@ -177,7 +178,7 @@ function cache_get($type) { * to be sure that we catch everything. Maybe that's a bad idea. */ function cache_flush() { - cache($this->table)->deletePrefix($this->view->name . ':'); + cache($this->table)->deletePrefix($this->view->storage->name . ':'); } /** @@ -302,7 +303,7 @@ function get_results_key() { } } - $this->_results_key = $this->view->name . ':' . $this->displayHandler->display['id'] . ':results:' . md5(serialize($key_data)); + $this->_results_key = $this->view->storage->name . ':' . $this->displayHandler->display['id'] . ':results:' . md5(serialize($key_data)); } return $this->_results_key; @@ -320,7 +321,7 @@ function get_output_key() { 'base_url' => $GLOBALS['base_url'], ); - $this->_output_key = $this->view->name . ':' . $this->displayHandler->display['id'] . ':output:' . md5(serialize($key_data)); + $this->_output_key = $this->view->storage->name . ':' . $this->displayHandler->display['id'] . ':output:' . md5(serialize($key_data)); } return $this->_output_key; diff --git a/lib/Drupal/views/Plugin/views/display/DisplayPluginBase.php b/lib/Drupal/views/Plugin/views/display/DisplayPluginBase.php index 0ca5f5d..5817cf1 100644 --- a/lib/Drupal/views/Plugin/views/display/DisplayPluginBase.php +++ b/lib/Drupal/views/Plugin/views/display/DisplayPluginBase.php @@ -85,7 +85,7 @@ */ protected $usesAttachments = FALSE; - public function init(&$view, &$display, $options = NULL) { + public function init(ViewExecutable $view, &$display, $options = NULL) { $this->setOptionDefaults($this->options, $this->defineOptions()); $this->view = &$view; $this->display = &$display; @@ -769,7 +769,7 @@ public function getPlugin($type = 'style', $name = NULL) { break; case 'query': - $views_data = views_fetch_data($this->view->base_table); + $views_data = views_fetch_data($this->view->storage->base_table); $name = !empty($views_data['table']['base']['query class']) ? $views_data['table']['base']['query class'] : 'views_query'; default: $option_name = $type; @@ -806,10 +806,10 @@ public function getPlugin($type = 'style', $name = NULL) { $display_id = $this->isDefaulted($option_name) ? $this->display['id'] : 'default'; if (!isset($this->base_field)) { - $views_data = views_fetch_data($this->view->base_table); - $this->view->base_field = !empty($views_data['table']['base']['field']) ? $views_data['table']['base']['field'] : ''; + $views_data = views_fetch_data($this->view->storage->base_table); + $this->view->storage->base_field = !empty($views_data['table']['base']['field']) ? $views_data['table']['base']['field'] : ''; } - $plugin->init($this->view->base_table, $this->view->base_field, $options); + $plugin->init($this->view->storage->base_table, $this->view->storage->base_field, $options); } $cache[$type][$name] = $plugin; } @@ -980,7 +980,7 @@ public function optionLink($text, $section, $class = '', $title = '') { $title = $text; } - return l($text, 'admin/structure/views/nojs/display/' . $this->view->name . '/' . $this->display['id'] . '/' . $section, array('attributes' => array('class' => 'views-ajax-link ' . $class, 'title' => $title, 'id' => drupal_html_id('views-' . $this->display['id'] . '-' . $section)), 'html' => TRUE)); + return l($text, 'admin/structure/views/nojs/display/' . $this->view->storage->name . '/' . $this->display['id'] . '/' . $section, array('attributes' => array('class' => 'views-ajax-link ' . $class, 'title' => $title, 'id' => drupal_html_id('views-' . $this->display['id'] . '-' . $section)), 'html' => TRUE)); } /** @@ -1478,7 +1478,7 @@ public function buildOptionsForm(&$form, &$form_state) { $access = $this->getOption('access'); $form['access']['type'] = array( '#type' => 'radios', - '#options' => views_fetch_plugin_names('access', NULL, array($this->view->base_table)), + '#options' => views_fetch_plugin_names('access', NULL, array($this->view->storage->base_table)), '#default_value' => $access['type'], ); @@ -1518,7 +1518,7 @@ public function buildOptionsForm(&$form, &$form_state) { $cache = $this->getOption('cache'); $form['cache']['type'] = array( '#type' => 'radios', - '#options' => views_fetch_plugin_names('cache', NULL, array($this->view->base_table)), + '#options' => views_fetch_plugin_names('cache', NULL, array($this->view->storage->base_table)), '#default_value' => $cache['type'], ); @@ -1584,7 +1584,7 @@ public function buildOptionsForm(&$form, &$form_state) { // Doesn't make sense to show a field setting here if we aren't querying // an entity base table. Also, we make sure that there's at least one // entity type with a translation handler attached. - if (in_array($this->view->base_table, $entity_tables) && $has_translation_handlers) { + if (in_array($this->view->storage->base_table, $entity_tables) && $has_translation_handlers) { $languages = array( '***CURRENT_LANGUAGE***' => t("Current user's language"), '***DEFAULT_LANGUAGE***' => t("Default site language"), @@ -1613,7 +1613,7 @@ public function buildOptionsForm(&$form, &$form_state) { $form['#title'] .= t('How should this view be styled'); $form['style_plugin'] = array( '#type' => 'radios', - '#options' => views_fetch_plugin_names('style', $this->getStyleType(), array($this->view->base_table)), + '#options' => views_fetch_plugin_names('style', $this->getStyleType(), array($this->view->storage->base_table)), '#default_value' => $this->getOption('style_plugin'), '#description' => t('If the style you choose has settings, be sure to click the settings button that will appear next to it in the View summary.'), ); @@ -1654,7 +1654,7 @@ public function buildOptionsForm(&$form, &$form_state) { $form['#title'] .= t('How should each row in this view be styled'); $form['row_plugin'] = array( '#type' => 'radios', - '#options' => views_fetch_plugin_names('row', $this->getStyleType(), array($this->view->base_table)), + '#options' => views_fetch_plugin_names('row', $this->getStyleType(), array($this->view->storage->base_table)), '#default_value' => $this->getOption('row_plugin'), ); @@ -1843,7 +1843,7 @@ public function buildOptionsForm(&$form, &$form_state) { '#markup' => '

' . t('This section lists all possible templates for the display plugin and for the style plugins, ordered roughly from the least specific to the most specific. The active template for each plugin -- which is the most specific template found on the system -- is highlighted in bold.') . '

', ); - if (isset($this->view->display[$this->view->current_display]->new_id)) { + if (isset($this->view->displayHandlers[$this->view->current_display]->new_id)) { $form['important']['new_id'] = array( '#prefix' => '
', '#suffix' => '
', @@ -2012,7 +2012,7 @@ public function buildOptionsForm(&$form, &$form_state) { $exposed_form = $this->getOption('exposed_form'); $form['exposed_form']['type'] = array( '#type' => 'radios', - '#options' => views_fetch_plugin_names('exposed_form', NULL, array($this->view->base_table)), + '#options' => views_fetch_plugin_names('exposed_form', NULL, array($this->view->storage->base_table)), '#default_value' => $exposed_form['type'], ); @@ -2046,7 +2046,7 @@ public function buildOptionsForm(&$form, &$form_state) { $pager = $this->getOption('pager'); $form['pager']['type'] = array( '#type' => 'radios', - '#options' => views_fetch_plugin_names('pager', !$this->usesPager() ? 'basic' : NULL, array($this->view->base_table)), + '#options' => views_fetch_plugin_names('pager', !$this->usesPager() ? 'basic' : NULL, array($this->view->storage->base_table)), '#default_value' => $pager['type'], ); @@ -2465,7 +2465,7 @@ public function renderMoreLink() { if (!empty($this->view->exposed_raw_input)) { $url_options['query'] = $this->view->exposed_raw_input; } - $theme = views_theme_functions('views_more', $this->view, $this->view->display[$this->view->current_display]); + $theme = views_theme_functions('views_more', $this->view, $this->view->displayHandlers[$this->view->current_display]->display); $path = check_url(url($path, $url_options)); return theme($theme, array('more_url' => $path, 'link_text' => check_plain($this->useMoreText()), 'view' => $this->view)); @@ -2690,8 +2690,8 @@ public function getSpecialBlocks() { $blocks = array(); if ($this->usesExposedFormInBlock()) { - $delta = '-exp-' . $this->view->name . '-' . $this->display['id']; - $desc = t('Exposed form: @view-@display_id', array('@view' => $this->view->name, '@display_id' => $this->display['id'])); + $delta = '-exp-' . $this->view->storage->name . '-' . $this->display['id']; + $desc = t('Exposed form: @view-@display_id', array('@view' => $this->view->storage->name, '@display_id' => $this->display['id'])); $blocks[$delta] = array( 'info' => $desc, diff --git a/lib/Drupal/views/Plugin/views/display/Feed.php b/lib/Drupal/views/Plugin/views/display/Feed.php index 0050f94..dbc2ab3 100644 --- a/lib/Drupal/views/Plugin/views/display/Feed.php +++ b/lib/Drupal/views/Plugin/views/display/Feed.php @@ -7,6 +7,7 @@ namespace Drupal\views\Plugin\views\display; +use Drupal\views\ViewExecutable; use Symfony\Component\HttpFoundation\Response; use Symfony\Component\HttpKernel\Exception\NotFoundHttpException; use Drupal\Core\Annotation\Plugin; @@ -43,13 +44,13 @@ class Feed extends Page { */ protected $usesPager = FALSE; - public function init(&$view, &$display, $options = NULL) { + public function init(ViewExecutable $view, &$display, $options = NULL) { parent::init($view, $display, $options); // Set the default row style. Ideally this would be part of the option // definition, but in this case it's dependent on the view's base table, // which we don't know until init(). - $row_plugins = views_fetch_plugin_names('row', $this->getStyleType(), array($view->base_table)); + $row_plugins = views_fetch_plugin_names('row', $this->getStyleType(), array($view->storage->base_table)); $default_row_plugin = key($row_plugins); if ($this->options['row_plugin'] == '') { $this->options['row_plugin'] = $default_row_plugin; diff --git a/lib/Drupal/views/Plugin/views/display/Page.php b/lib/Drupal/views/Plugin/views/display/Page.php index e959f60..d348872 100644 --- a/lib/Drupal/views/Plugin/views/display/Page.php +++ b/lib/Drupal/views/Plugin/views/display/Page.php @@ -80,7 +80,7 @@ public function executeHookMenu($callbacks) { // views_arg_load -- which lives in views.module $bits = explode('/', $this->getOption('path')); - $page_arguments = array($this->view->name, $this->display['id']); + $page_arguments = array($this->view->storage->name, $this->display['id']); $this->view->initHandlers(); $view_arguments = $this->view->argument; @@ -141,7 +141,7 @@ public function executeHookMenu($callbacks) { 'access callback' => 'views_access', 'access arguments' => $access_arguments, // Identify URL embedded arguments and correlate them to a handler - 'load arguments' => array($this->view->name, $this->display['id'], '%index'), + 'load arguments' => array($this->view->storage->name, $this->display['id'], '%index'), ); $menu = $this->getOption('menu'); if (empty($menu)) { @@ -202,7 +202,7 @@ public function executeHookMenu($callbacks) { 'access callback' => 'views_access', 'access arguments' => $access_arguments, // Identify URL embedded arguments and correlate them to a handler - 'load arguments' => array($this->view->name, $this->display['id'], '%index'), + 'load arguments' => array($this->view->storage->name, $this->display['id'], '%index'), 'title' => $tab_options['title'], 'description' => $tab_options['description'], 'menu_name' => $tab_options['name'], diff --git a/lib/Drupal/views/Plugin/views/display_extender/DisplayExtenderPluginBase.php b/lib/Drupal/views/Plugin/views/display_extender/DisplayExtenderPluginBase.php index bc64f12..e7fe061 100644 --- a/lib/Drupal/views/Plugin/views/display_extender/DisplayExtenderPluginBase.php +++ b/lib/Drupal/views/Plugin/views/display_extender/DisplayExtenderPluginBase.php @@ -7,6 +7,7 @@ namespace Drupal\views\Plugin\views\display_extender; +use Drupal\views\ViewExecutable; use Drupal\views\Plugin\views\PluginBase; use Drupal\Core\Annotation\Translation; @@ -17,7 +18,7 @@ */ abstract class DisplayExtenderPluginBase extends PluginBase { - public function init(&$view, &$display) { + public function init(ViewExecutable $view, &$display) { $this->setOptionDefaults($this->options, $this->defineOptions()); $this->view = $view; $this->display = $display; diff --git a/lib/Drupal/views/Plugin/views/exposed_form/ExposedFormPluginBase.php b/lib/Drupal/views/Plugin/views/exposed_form/ExposedFormPluginBase.php index c630167..cb71597 100644 --- a/lib/Drupal/views/Plugin/views/exposed_form/ExposedFormPluginBase.php +++ b/lib/Drupal/views/Plugin/views/exposed_form/ExposedFormPluginBase.php @@ -7,6 +7,7 @@ namespace Drupal\views\Plugin\views\exposed_form; +use Drupal\views\ViewExecutable; use Drupal\views\Plugin\views\PluginBase; /** @@ -37,7 +38,7 @@ * @param $display * The display handler. */ - public function init(&$view, &$display, $options = array()) { + public function init(ViewExecutable $view, &$display, $options = array()) { $this->setOptionDefaults($this->options, $this->defineOptions()); $this->view = &$view; $this->displayHandler = &$display; @@ -314,8 +315,8 @@ function reset_form(&$form, &$form_state) { // remember settings. $display_id = ($this->view->display_handler->isDefaulted('filters')) ? 'default' : $this->view->current_display; - if (isset($_SESSION['views'][$this->view->name][$display_id])) { - unset($_SESSION['views'][$this->view->name][$display_id]); + if (isset($_SESSION['views'][$this->view->storage->name][$display_id])) { + unset($_SESSION['views'][$this->view->storage->name][$display_id]); } // Set the form to allow redirect. diff --git a/lib/Drupal/views/Plugin/views/field/Boolean.php b/lib/Drupal/views/Plugin/views/field/Boolean.php index 4fbab84..5a9ae65 100644 --- a/lib/Drupal/views/Plugin/views/field/Boolean.php +++ b/lib/Drupal/views/Plugin/views/field/Boolean.php @@ -6,6 +6,8 @@ */ namespace Drupal\views\Plugin\views\field; + +use Drupal\views\ViewExecutable; use Drupal\Core\Annotation\Plugin; /** @@ -38,7 +40,7 @@ protected function defineOptions() { return $options; } - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { parent::init($view, $options); $default_formats = array( diff --git a/lib/Drupal/views/Plugin/views/field/FieldPluginBase.php b/lib/Drupal/views/Plugin/views/field/FieldPluginBase.php index 5bc6f95..1ebe1dd 100644 --- a/lib/Drupal/views/Plugin/views/field/FieldPluginBase.php +++ b/lib/Drupal/views/Plugin/views/field/FieldPluginBase.php @@ -1605,22 +1605,22 @@ public function themeFunctions() { $themes = array(); $hook = 'views_view_field'; - $display = $this->view->display[$this->view->current_display]; + $display = $this->view->displayHandlers[$this->view->current_display]->display; if (!empty($display)) { - $themes[] = $hook . '__' . $this->view->name . '__' . $display['id'] . '__' . $this->options['id']; - $themes[] = $hook . '__' . $this->view->name . '__' . $display['id']; + $themes[] = $hook . '__' . $this->view->storage->name . '__' . $display['id'] . '__' . $this->options['id']; + $themes[] = $hook . '__' . $this->view->storage->name . '__' . $display['id']; $themes[] = $hook . '__' . $display['id'] . '__' . $this->options['id']; $themes[] = $hook . '__' . $display['id']; if ($display['id'] != $display['display_plugin']) { - $themes[] = $hook . '__' . $this->view->name . '__' . $display['display_plugin'] . '__' . $this->options['id']; - $themes[] = $hook . '__' . $this->view->name . '__' . $display['display_plugin']; + $themes[] = $hook . '__' . $this->view->storage->name . '__' . $display['display_plugin'] . '__' . $this->options['id']; + $themes[] = $hook . '__' . $this->view->storage->name . '__' . $display['display_plugin']; $themes[] = $hook . '__' . $display['display_plugin'] . '__' . $this->options['id']; $themes[] = $hook . '__' . $display['display_plugin']; } } - $themes[] = $hook . '__' . $this->view->name . '__' . $this->options['id']; - $themes[] = $hook . '__' . $this->view->name; + $themes[] = $hook . '__' . $this->view->storage->name . '__' . $this->options['id']; + $themes[] = $hook . '__' . $this->view->storage->name; $themes[] = $hook . '__' . $this->options['id']; $themes[] = $hook; diff --git a/lib/Drupal/views/Plugin/views/field/Markup.php b/lib/Drupal/views/Plugin/views/field/Markup.php index 8e961fc..d82a529 100644 --- a/lib/Drupal/views/Plugin/views/field/Markup.php +++ b/lib/Drupal/views/Plugin/views/field/Markup.php @@ -8,6 +8,7 @@ namespace Drupal\views\Plugin\views\field; use Drupal\Core\Annotation\Plugin; +use Drupal\views\ViewExecutable; /** * A handler to run a field through check_markup, using a companion @@ -26,7 +27,7 @@ */ class Markup extends FieldPluginBase { - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { parent::init($view, $options); $this->format = $this->definition['format']; diff --git a/lib/Drupal/views/Plugin/views/filter/Broken.php b/lib/Drupal/views/Plugin/views/filter/Broken.php index b708515..02eb9a5 100644 --- a/lib/Drupal/views/Plugin/views/filter/Broken.php +++ b/lib/Drupal/views/Plugin/views/filter/Broken.php @@ -8,6 +8,7 @@ namespace Drupal\views\Plugin\views\filter; use Drupal\Core\Annotation\Plugin; +use Drupal\views\ViewExecutable; /** * A special handler to take the place of missing or broken handlers. @@ -24,7 +25,7 @@ public function adminLabel($short = FALSE) { return t('Broken/missing handler'); } - public function init(&$view, &$options) { } + public function init(ViewExecutable $view, &$options) { } public function defineOptions() { return array(); } public function ensureMyTable() { /* No table to ensure! */ } public function query($group_by = FALSE) { /* No query to run */ } diff --git a/lib/Drupal/views/Plugin/views/filter/FilterPluginBase.php b/lib/Drupal/views/Plugin/views/filter/FilterPluginBase.php index 37a9768..aa3b8d8 100644 --- a/lib/Drupal/views/Plugin/views/filter/FilterPluginBase.php +++ b/lib/Drupal/views/Plugin/views/filter/FilterPluginBase.php @@ -9,6 +9,7 @@ use Drupal\views\Plugin\views\HandlerBase; use Drupal\Core\Annotation\Plugin; +use Drupal\views\ViewExecutable; /** * @defgroup views_filter_handlers Views filter handlers @@ -78,7 +79,7 @@ * This likely has to be overridden by filters which are more complex * than simple operator/value. */ - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { parent::init($view, $options); $this->operator = $this->options['operator']; @@ -1240,8 +1241,8 @@ function store_group_input($input, $status) { // false means that we got a setting that means to recuse ourselves, // so we should erase whatever happened to be there. - if ($status === FALSE && isset($_SESSION['views'][$this->view->name][$display_id])) { - $session = &$_SESSION['views'][$this->view->name][$display_id]; + if ($status === FALSE && isset($_SESSION['views'][$this->view->storage->name][$display_id])) { + $session = &$_SESSION['views'][$this->view->storage->name][$display_id]; if (isset($session[$this->options['group_info']['identifier']])) { unset($session[$this->options['group_info']['identifier']]); @@ -1249,11 +1250,11 @@ function store_group_input($input, $status) { } if ($status !== FALSE) { - if (!isset($_SESSION['views'][$this->view->name][$display_id])) { - $_SESSION['views'][$this->view->name][$display_id] = array(); + if (!isset($_SESSION['views'][$this->view->storage->name][$display_id])) { + $_SESSION['views'][$this->view->storage->name][$display_id] = array(); } - $session = &$_SESSION['views'][$this->view->name][$display_id]; + $session = &$_SESSION['views'][$this->view->storage->name][$display_id]; $session[$this->options['group_info']['identifier']] = $input[$this->options['group_info']['identifier']]; } @@ -1334,8 +1335,8 @@ public function storeExposedInput($input, $status) { // false means that we got a setting that means to recuse ourselves, // so we should erase whatever happened to be there. - if (!$status && isset($_SESSION['views'][$this->view->name][$display_id])) { - $session = &$_SESSION['views'][$this->view->name][$display_id]; + if (!$status && isset($_SESSION['views'][$this->view->storage->name][$display_id])) { + $session = &$_SESSION['views'][$this->view->storage->name][$display_id]; if ($operator && isset($session[$this->options['expose']['operator_id']])) { unset($session[$this->options['expose']['operator_id']]); } @@ -1346,11 +1347,11 @@ public function storeExposedInput($input, $status) { } if ($status) { - if (!isset($_SESSION['views'][$this->view->name][$display_id])) { - $_SESSION['views'][$this->view->name][$display_id] = array(); + if (!isset($_SESSION['views'][$this->view->storage->name][$display_id])) { + $_SESSION['views'][$this->view->storage->name][$display_id] = array(); } - $session = &$_SESSION['views'][$this->view->name][$display_id]; + $session = &$_SESSION['views'][$this->view->storage->name][$display_id]; if ($operator && isset($input[$this->options['expose']['operator_id']])) { $session[$this->options['expose']['operator_id']] = $input[$this->options['expose']['operator_id']]; diff --git a/lib/Drupal/views/Plugin/views/filter/ManyToOne.php b/lib/Drupal/views/Plugin/views/filter/ManyToOne.php index 4e2e3bf..d3ff72c 100644 --- a/lib/Drupal/views/Plugin/views/filter/ManyToOne.php +++ b/lib/Drupal/views/Plugin/views/filter/ManyToOne.php @@ -7,6 +7,7 @@ namespace Drupal\views\Plugin\views\filter; +use Drupal\views\ViewExecutable; use Drupal\views\ManyToOneHelper; use Drupal\Core\Annotation\Plugin; @@ -33,7 +34,7 @@ class ManyToOne extends InOperator { */ var $helper = NULL; - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { parent::init($view, $options); $this->helper = new ManyToOneHelper($this); } diff --git a/lib/Drupal/views/Plugin/views/pager/Full.php b/lib/Drupal/views/Plugin/views/pager/Full.php index 78f6b09..32aae7c 100644 --- a/lib/Drupal/views/Plugin/views/pager/Full.php +++ b/lib/Drupal/views/Plugin/views/pager/Full.php @@ -291,7 +291,7 @@ public function query() { } function render($input) { - $pager_theme = views_theme_functions('pager', $this->view, $this->view->display[$this->view->current_display]); + $pager_theme = views_theme_functions('pager', $this->view, $this->view->displayHandlers[$this->view->current_display]->display); // The 0, 1, 3, 4 index are correct. See theme_pager documentation. $tags = array( 0 => $this->options['tags']['first'], diff --git a/lib/Drupal/views/Plugin/views/pager/Mini.php b/lib/Drupal/views/Plugin/views/pager/Mini.php index 4b454b9..d39eb3c 100644 --- a/lib/Drupal/views/Plugin/views/pager/Mini.php +++ b/lib/Drupal/views/Plugin/views/pager/Mini.php @@ -32,7 +32,7 @@ public function summaryTitle() { } function render($input) { - $pager_theme = views_theme_functions('views_mini_pager', $this->view, $this->view->display[$this->view->current_display]); + $pager_theme = views_theme_functions('views_mini_pager', $this->view, $this->view->displayHandlers[$this->view->current_display]->display); return theme($pager_theme, array( 'parameters' => $input, 'element' => $this->options['id'])); } diff --git a/lib/Drupal/views/Plugin/views/pager/None.php b/lib/Drupal/views/Plugin/views/pager/None.php index 434f0c1..7d6ec89 100644 --- a/lib/Drupal/views/Plugin/views/pager/None.php +++ b/lib/Drupal/views/Plugin/views/pager/None.php @@ -7,6 +7,7 @@ namespace Drupal\views\Plugin\views\pager; +use Drupal\views\ViewExecutable; use Drupal\Core\Annotation\Plugin; use Drupal\Core\Annotation\Translation; @@ -24,7 +25,7 @@ */ class None extends PagerPluginBase { - public function init(&$view, &$display, $options = array()) { + public function init(ViewExecutable $view, &$display, $options = array()) { parent::init($view, $display, $options); // If the pager is set to none, then it should show all items. diff --git a/lib/Drupal/views/Plugin/views/pager/PagerPluginBase.php b/lib/Drupal/views/Plugin/views/pager/PagerPluginBase.php index 1500934..11b8724 100644 --- a/lib/Drupal/views/Plugin/views/pager/PagerPluginBase.php +++ b/lib/Drupal/views/Plugin/views/pager/PagerPluginBase.php @@ -8,6 +8,7 @@ namespace Drupal\views\Plugin\views\pager; use Drupal\views\Plugin\views\PluginBase; +use Drupal\views\ViewExecutable; /** * @defgroup views_pager_plugins Views pager plugins @@ -42,7 +43,7 @@ * @param $display * The display handler. */ - public function init(&$view, &$display, $options = array()) { + public function init(ViewExecutable $view, &$display, $options = array()) { $this->setOptionDefaults($this->options, $this->defineOptions()); $this->view = &$view; $this->displayHandler = &$display; diff --git a/lib/Drupal/views/Plugin/views/query/QueryPluginBase.php b/lib/Drupal/views/Plugin/views/query/QueryPluginBase.php index 4faae11..502ebdc 100644 --- a/lib/Drupal/views/Plugin/views/query/QueryPluginBase.php +++ b/lib/Drupal/views/Plugin/views/query/QueryPluginBase.php @@ -8,6 +8,7 @@ namespace Drupal\views\Plugin\views\query; use Drupal\views\Plugin\views\PluginBase; +use Drupal\views\ViewExecutable; /** * @todo. @@ -46,7 +47,7 @@ public function query($get_count = FALSE) { } * @param view $view * The view which is executed. */ - function alter(&$view) { } + function alter(ViewExecutable $view) { } /** * Builds the necessary info to execute the query. @@ -54,7 +55,7 @@ function alter(&$view) { } * @param view $view * The view which is executed. */ - function build(&$view) { } + function build(ViewExecutable $view) { } /** * Executes the query and fills the associated view object with according @@ -69,7 +70,7 @@ function build(&$view) { } * @param view $view * The view which is executed. */ - function execute(&$view) { } + function execute(ViewExecutable $view) { } /** * Add a signature to the query, if such a thing is feasible. @@ -80,7 +81,7 @@ function execute(&$view) { } * @param view $view * The view which is executed. */ - function add_signature(&$view) { } + function add_signature(ViewExecutable $view) { } /** * Get aggregation info for group by queries. diff --git a/lib/Drupal/views/Plugin/views/query/Sql.php b/lib/Drupal/views/Plugin/views/query/Sql.php index c106be6..a45ca8f 100644 --- a/lib/Drupal/views/Plugin/views/query/Sql.php +++ b/lib/Drupal/views/Plugin/views/query/Sql.php @@ -12,6 +12,7 @@ use Drupal\views\Plugin\views\join\JoinPluginBase; use Drupal\Core\Annotation\Plugin; use Drupal\Core\Annotation\Translation; +use Drupal\views\ViewExecutable; /** * @todo. @@ -1285,7 +1286,7 @@ public function query($get_count = FALSE) { $query = Database::getConnection($target, $key) ->select($this->base_table, $this->base_table, $options) ->addTag('views') - ->addTag('views_' . $this->view->name); + ->addTag('views_' . $this->view->storage->name); // Add the tags added to the view itself. foreach ($this->tags as $tag) { @@ -1402,7 +1403,7 @@ function get_where_args() { /** * Let modules modify the query just prior to finalizing it. */ - function alter(&$view) { + function alter(ViewExecutable $view) { foreach (module_implements('views_query_alter') as $module) { $function = $module . '_views_query_alter'; $function($view, $this); @@ -1412,7 +1413,7 @@ function alter(&$view) { /** * Builds the necessary info to execute the query. */ - function build(&$view) { + function build(ViewExecutable $view) { // Make the query distinct if the option was set. if (!empty($this->options['distinct'])) { $this->set_distinct(TRUE); @@ -1437,7 +1438,7 @@ function build(&$view) { * Values to set: $view->result, $view->total_rows, $view->execute_time, * $view->current_page. */ - function execute(&$view) { + function execute(ViewExecutable $view) { $external = FALSE; // Whether this query will run against an external database. $query = $view->build_info['query']; $count_query = $view->build_info['count_query']; @@ -1516,7 +1517,7 @@ function execute(&$view) { drupal_set_message($e->getMessage(), 'error'); } else { - throw new DatabaseExceptionWrapper(format_string('Exception in @human_name[@view_name]: @message', array('@human_name' => $view->human_name, '@view_name' => $view->name, '@message' => $e->getMessage()))); + throw new DatabaseExceptionWrapper(format_string('Exception in @human_name[@view_name]: @message', array('@human_name' => $view->storage->human_name, '@view_name' => $view->storage->name, '@message' => $e->getMessage()))); } } @@ -1646,8 +1647,8 @@ function load_entities(&$results) { } } - function add_signature(&$view) { - $view->query->add_field(NULL, "'" . $view->name . ':' . $view->current_display . "'", 'view_name'); + function add_signature(ViewExecutable $view) { + $view->query->add_field(NULL, "'" . $view->storage->name . ':' . $view->current_display . "'", 'view_name'); } function get_aggregation_info() { diff --git a/lib/Drupal/views/Plugin/views/relationship/GroupwiseMax.php b/lib/Drupal/views/Plugin/views/relationship/GroupwiseMax.php index 5eb60da..54125ed 100644 --- a/lib/Drupal/views/Plugin/views/relationship/GroupwiseMax.php +++ b/lib/Drupal/views/Plugin/views/relationship/GroupwiseMax.php @@ -8,7 +8,6 @@ namespace Drupal\views\Plugin\views\relationship; use Drupal\Core\Database\Query\AlterableInterface; -use Drupal\views\ViewExecutable; use Drupal\Core\Annotation\Plugin; /** @@ -132,10 +131,10 @@ public function buildOptionsForm(&$form, &$form_state) { // TODO: check the field is the correct sort? // or let users hang themselves at this stage and check later? if ($view->type == 'Default') { - $views[t('Default Views')][$view->name] = $view->name; + $views[t('Default Views')][$view->storage->name] = $view->storage->name; } else { - $views[t('Existing Views')][$view->name] = $view->name; + $views[t('Existing Views')][$view->storage->name] = $view->storage->name; } } } @@ -173,7 +172,7 @@ function get_temporary_view() { * When the form is submitted, take sure to clear the subquery string cache. */ public function submitOptionsForm(&$form, &$form_state) { - $cid = 'views_relationship_groupwise_max:' . $this->view->name . ':' . $this->view->current_display . ':' . $this->options['id']; + $cid = 'views_relationship_groupwise_max:' . $this->view->storage->name . ':' . $this->view->current_display . ':' . $this->options['id']; cache('views_data')->delete($cid); } @@ -360,7 +359,7 @@ public function query() { } else { // Get the stored subquery SQL string. - $cid = 'views_relationship_groupwise_max:' . $this->view->name . ':' . $this->view->current_display . ':' . $this->options['id']; + $cid = 'views_relationship_groupwise_max:' . $this->view->storage->name . ':' . $this->view->current_display . ':' . $this->options['id']; $cache = cache('views_data')->get($cid); if (isset($cache->data)) { $def['left_query'] = $cache->data; diff --git a/lib/Drupal/views/Plugin/views/relationship/RelationshipPluginBase.php b/lib/Drupal/views/Plugin/views/relationship/RelationshipPluginBase.php index 7d9abd5..e879dac 100644 --- a/lib/Drupal/views/Plugin/views/relationship/RelationshipPluginBase.php +++ b/lib/Drupal/views/Plugin/views/relationship/RelationshipPluginBase.php @@ -7,6 +7,7 @@ namespace Drupal\views\Plugin\views\relationship; +use Drupal\views\ViewExecutable; use Drupal\views\Plugin\views\HandlerBase; use Drupal\views\Join; use Drupal\Core\Annotation\Plugin; @@ -47,7 +48,7 @@ * Init handler to let relationships live on tables other than * the table they operate on. */ - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { $this->setOptionDefaults($this->options, $this->defineOptions()); parent::init($view, $options); if (isset($this->definition['relationship table'])) { diff --git a/lib/Drupal/views/Plugin/views/row/RowPluginBase.php b/lib/Drupal/views/Plugin/views/row/RowPluginBase.php index b91079e..2e6348e 100644 --- a/lib/Drupal/views/Plugin/views/row/RowPluginBase.php +++ b/lib/Drupal/views/Plugin/views/row/RowPluginBase.php @@ -8,6 +8,7 @@ namespace Drupal\views\Plugin\views\row; use Drupal\views\Plugin\views\PluginBase; +use Drupal\views\ViewExecutable; /** * @defgroup views_row_plugins Views row plugins @@ -41,7 +42,7 @@ /** * Initialize the row plugin. */ - public function init(&$view, &$display, $options = NULL) { + public function init(ViewExecutable $view, &$display, $options = NULL) { $this->setOptionDefaults($this->options, $this->defineOptions()); $this->view = &$view; $this->displayHandler = &$display; diff --git a/lib/Drupal/views/Plugin/views/sort/GroupByNumeric.php b/lib/Drupal/views/Plugin/views/sort/GroupByNumeric.php index 437d8c4..e52bc06 100644 --- a/lib/Drupal/views/Plugin/views/sort/GroupByNumeric.php +++ b/lib/Drupal/views/Plugin/views/sort/GroupByNumeric.php @@ -8,6 +8,7 @@ namespace Drupal\views\Plugin\views\sort; use Drupal\Core\Annotation\Plugin; +use Drupal\views\ViewExecutable; /** * Handler for GROUP BY on simple numeric fields. @@ -18,7 +19,7 @@ */ class GroupByNumeric extends SortPluginBase { - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { parent::init($view, $options); // Initialize the original handler. diff --git a/lib/Drupal/views/Plugin/views/style/StylePluginBase.php b/lib/Drupal/views/Plugin/views/style/StylePluginBase.php index 4f94b23..9813694 100644 --- a/lib/Drupal/views/Plugin/views/style/StylePluginBase.php +++ b/lib/Drupal/views/Plugin/views/style/StylePluginBase.php @@ -11,6 +11,7 @@ use Drupal\views\Plugin\views\wizard\WizardInterface; use Drupal\Core\Annotation\Plugin; use Drupal\Core\Annotation\Translation; +use Drupal\views\ViewExecutable; /** * @defgroup views_style_plugins Views style plugins @@ -89,7 +90,7 @@ * The style options might come externally as the style can be sourced * from at least two locations. If it's not included, look on the display. */ - public function init(&$view, &$display, $options = NULL) { + public function init(ViewExecutable $view, &$display, $options = NULL) { $this->setOptionDefaults($this->options, $this->defineOptions()); $this->view = &$view; $this->displayHandler = &$display; @@ -438,7 +439,7 @@ function render_grouping_sets($sets, $level = 0) { $row = reset($set['rows']); // Render as a grouping set. if (is_array($row) && isset($row['group'])) { - $output .= theme(views_theme_functions('views_view_grouping', $this->view, $this->view->display[$this->view->current_display]), + $output .= theme(views_theme_functions('views_view_grouping', $this->view, $this->view->displayHandlers[$this->view->current_display]->display), array( 'view' => $this->view, 'grouping' => $this->options['grouping'][$level], diff --git a/lib/Drupal/views/Tests/AnalyzeTest.php b/lib/Drupal/views/Tests/AnalyzeTest.php index a368d39..f5497f2 100644 --- a/lib/Drupal/views/Tests/AnalyzeTest.php +++ b/lib/Drupal/views/Tests/AnalyzeTest.php @@ -41,7 +41,6 @@ function testAnalyzeBasic() { $this->drupalLogin($this->admin); // Enable the frontpage view and click the analyse button. $view = views_get_view('frontpage'); - $view = $view->createDuplicate(); $this->drupalGet('admin/structure/views/view/frontpage/edit'); $this->assertLink(t('analyze view')); diff --git a/lib/Drupal/views/Tests/Field/HandlerFieldFieldTest.php b/lib/Drupal/views/Tests/Field/HandlerFieldFieldTest.php index 2003e52..a738062 100644 --- a/lib/Drupal/views/Tests/Field/HandlerFieldFieldTest.php +++ b/lib/Drupal/views/Tests/Field/HandlerFieldFieldTest.php @@ -116,16 +116,18 @@ public function _testFormatterSimpleFieldRender() { public function _testMultipleFieldRender() { $view = $this->getView(); + $field_name = $this->fields[3]['field_name']; // Test delta limit. - $view->displayHandlers['default']->options['fields'][$this->fields[3]['field_name']]['group_rows'] = TRUE; - $view->displayHandlers['default']->options['fields'][$this->fields[3]['field_name']]['delta_limit'] = 3; + $view->initDisplay(); + $view->displayHandlers['default']->options['fields'][$field_name]['group_rows'] = TRUE; + $view->displayHandlers['default']->options['fields'][$field_name]['delta_limit'] = 3; $this->executeView($view); for ($i = 0; $i < 3; $i++) { - $rendered_field = $view->style_plugin->get_field($i, $this->fields[3]['field_name']); + $rendered_field = $view->style_plugin->get_field($i, $field_name); $items = array(); - $pure_items = $this->nodes[$i]->{$this->fields[3]['field_name']}[LANGUAGE_NOT_SPECIFIED]; + $pure_items = $this->nodes[$i]->{$field_name}[LANGUAGE_NOT_SPECIFIED]; $pure_items = array_splice($pure_items, 0, 3); foreach ($pure_items as $j => $item) { $items[] = $pure_items[$j]['value']; @@ -139,15 +141,16 @@ public function _testMultipleFieldRender() { $view->destroy(); // Test delta limit + offset - $view->display['default']['display_options']['fields'][$this->fields[3]['field_name']]['group_rows'] = TRUE; - $view->display['default']['display_options']['fields'][$this->fields[3]['field_name']]['delta_limit'] = 3; - $view->display['default']['display_options']['fields'][$this->fields[3]['field_name']]['delta_offset'] = 1; + $view->initDisplay(); + $view->displayHandlers['default']->options['fields'][$field_name]['group_rows'] = TRUE; + $view->displayHandlers['default']->options['fields'][$field_name]['delta_limit'] = 3; + $view->displayHandlers['default']->options['fields'][$field_name]['delta_offset'] = 1; $this->executeView($view); for ($i = 0; $i < 3; $i++) { - $rendered_field = $view->style_plugin->get_field($i, $this->fields[3]['field_name']); + $rendered_field = $view->style_plugin->get_field($i, $field_name); $items = array(); - $pure_items = $this->nodes[$i]->{$this->fields[3]['field_name']}[LANGUAGE_NOT_SPECIFIED]; + $pure_items = $this->nodes[$i]->{$field_name}[LANGUAGE_NOT_SPECIFIED]; $pure_items = array_splice($pure_items, 1, 3); foreach ($pure_items as $j => $item) { $items[] = $pure_items[$j]['value']; @@ -157,16 +160,17 @@ public function _testMultipleFieldRender() { $view->destroy(); // Test delta limit + reverse. - $view->display['default']['display_options']['fields'][$this->fields[3]['field_name']]['delta_offset'] = 0; - $view->display['default']['display_options']['fields'][$this->fields[3]['field_name']]['group_rows'] = TRUE; - $view->display['default']['display_options']['fields'][$this->fields[3]['field_name']]['delta_limit'] = 3; - $view->display['default']['display_options']['fields'][$this->fields[3]['field_name']]['delta_reversed'] = TRUE; + $view->initDisplay(); + $view->displayHandlers['default']->options['fields'][$field_name]['delta_offset'] = 0; + $view->displayHandlers['default']->options['fields'][$field_name]['group_rows'] = TRUE; + $view->displayHandlers['default']->options['fields'][$field_name]['delta_limit'] = 3; + $view->displayHandlers['default']->options['fields'][$field_name]['delta_reversed'] = TRUE; $this->executeView($view); for ($i = 0; $i < 3; $i++) { - $rendered_field = $view->style_plugin->get_field($i, $this->fields[3]['field_name']); + $rendered_field = $view->style_plugin->get_field($i, $field_name); $items = array(); - $pure_items = $this->nodes[$i]->{$this->fields[3]['field_name']}[LANGUAGE_NOT_SPECIFIED]; + $pure_items = $this->nodes[$i]->{$field_name}[LANGUAGE_NOT_SPECIFIED]; array_splice($pure_items, 0, -3); $pure_items = array_reverse($pure_items); foreach ($pure_items as $j => $item) { @@ -177,16 +181,17 @@ public function _testMultipleFieldRender() { $view->destroy(); // Test delta first last. - $view->display['default']['display_options']['fields'][$this->fields[3]['field_name']]['group_rows'] = TRUE; - $view->display['default']['display_options']['fields'][$this->fields[3]['field_name']]['delta_limit'] = 0; - $view->display['default']['display_options']['fields'][$this->fields[3]['field_name']]['delta_first_last'] = TRUE; - $view->display['default']['display_options']['fields'][$this->fields[3]['field_name']]['delta_reversed'] = FALSE; + $view->initDisplay(); + $view->displayHandlers['default']->options['fields'][$field_name]['group_rows'] = TRUE; + $view->displayHandlers['default']->options['fields'][$field_name]['delta_limit'] = 0; + $view->displayHandlers['default']->options['fields'][$field_name]['delta_first_last'] = TRUE; + $view->displayHandlers['default']->options['fields'][$field_name]['delta_reversed'] = FALSE; $this->executeView($view); for ($i = 0; $i < 3; $i++) { - $rendered_field = $view->style_plugin->get_field($i, $this->fields[3]['field_name']); + $rendered_field = $view->style_plugin->get_field($i, $field_name); $items = array(); - $pure_items = $this->nodes[$i]->{$this->fields[3]['field_name']}[LANGUAGE_NOT_SPECIFIED]; + $pure_items = $this->nodes[$i]->{$field_name}[LANGUAGE_NOT_SPECIFIED]; $items[] = $pure_items[0]['value']; $items[] = $pure_items[4]['value']; $this->assertEqual($rendered_field, implode(', ', $items), 'Take sure that the amount of items are limited.'); @@ -194,16 +199,17 @@ public function _testMultipleFieldRender() { $view->destroy(); // Test delta limit + custom seperator. - $view->display['default']['display_options']['fields'][$this->fields[3]['field_name']]['delta_first_last'] = FALSE; - $view->display['default']['display_options']['fields'][$this->fields[3]['field_name']]['delta_limit'] = 3; - $view->display['default']['display_options']['fields'][$this->fields[3]['field_name']]['group_rows'] = TRUE; - $view->display['default']['display_options']['fields'][$this->fields[3]['field_name']]['separator'] = ':'; + $view->initDisplay(); + $view->displayHandlers['default']->options['fields'][$field_name]['delta_first_last'] = FALSE; + $view->displayHandlers['default']->options['fields'][$field_name]['delta_limit'] = 3; + $view->displayHandlers['default']->options['fields'][$field_name]['group_rows'] = TRUE; + $view->displayHandlers['default']->options['fields'][$field_name]['separator'] = ':'; $this->executeView($view); for ($i = 0; $i < 3; $i++) { - $rendered_field = $view->style_plugin->get_field($i, $this->fields[3]['field_name']); + $rendered_field = $view->style_plugin->get_field($i, $field_name); $items = array(); - $pure_items = $this->nodes[$i]->{$this->fields[3]['field_name']}[LANGUAGE_NOT_SPECIFIED]; + $pure_items = $this->nodes[$i]->{$field_name}[LANGUAGE_NOT_SPECIFIED]; $pure_items = array_splice($pure_items, 0, 3); foreach ($pure_items as $j => $item) { $items[] = $pure_items[$j]['value']; diff --git a/lib/Drupal/views/Tests/Handler/FilterCombineTest.php b/lib/Drupal/views/Tests/Handler/FilterCombineTest.php index 160fb5c..eb487a2 100644 --- a/lib/Drupal/views/Tests/Handler/FilterCombineTest.php +++ b/lib/Drupal/views/Tests/Handler/FilterCombineTest.php @@ -35,7 +35,7 @@ function setUp() { protected function getBasicView() { $view = parent::getBasicView(); - $view->display['default']['display_options']['fields']['job'] = array( + $view->displayHandlers['default']->display['display_options']['fields']['job'] = array( 'id' => 'job', 'table' => 'views_test_data', 'field' => 'job', diff --git a/lib/Drupal/views/Tests/Plugin/ArgumentDefaultTest.php b/lib/Drupal/views/Tests/Plugin/ArgumentDefaultTest.php index 0c638cc..6c9e120 100644 --- a/lib/Drupal/views/Tests/Plugin/ArgumentDefaultTest.php +++ b/lib/Drupal/views/Tests/Plugin/ArgumentDefaultTest.php @@ -142,7 +142,7 @@ function testArgumentDefaultFixed() { */ protected function getBasicView() { $view = $this->createViewFromConfig('test_argument_default_fixed'); - $view->display['default']['display_options']['arguments']['null']['default_argument_options']['argument'] = $this->random; + $view->displayHandlers['default']->display['display_options']['arguments']['null']['default_argument_options']['argument'] = $this->random; return $view; } diff --git a/lib/Drupal/views/Tests/Plugin/CacheTest.php b/lib/Drupal/views/Tests/Plugin/CacheTest.php index 2c75f44..328fcb6 100644 --- a/lib/Drupal/views/Tests/Plugin/CacheTest.php +++ b/lib/Drupal/views/Tests/Plugin/CacheTest.php @@ -161,7 +161,7 @@ function testHeaderStorage() { // Some hook_views_pre_render in views_test_data.module adds the test css/js file. // so they should be added to the css/js storage. $view = $this->getView(); - $view->name = 'test_cache_header_storage'; + $view->storage->name = 'test_cache_header_storage'; $view->display_handler->overrideOption('cache', array( 'type' => 'time', 'output_lifespan' => '3600', @@ -185,7 +185,7 @@ function testHeaderStorage() { // Now add some css/jss before running the view. // Make sure that this css is not added when running the cached view. - $view->name = 'test_cache_header_storage_2'; + $view->storage->name = 'test_cache_header_storage_2'; $system_css_path = drupal_get_path('module', 'system') . '/system.maintenance.css'; drupal_add_css($system_css_path); diff --git a/lib/Drupal/views/Tests/Plugin/DisplayTest.php b/lib/Drupal/views/Tests/Plugin/DisplayTest.php index 425dabf..82f6250 100644 --- a/lib/Drupal/views/Tests/Plugin/DisplayTest.php +++ b/lib/Drupal/views/Tests/Plugin/DisplayTest.php @@ -54,7 +54,7 @@ function testDisplayPlugin() { // Add a new 'display_test' display and test it's there. $view->addDisplay('display_test'); - $this->assertTrue(isset($view->display['display_test_1']), 'Added display has been assigned to "display_test_1"'); + $this->assertTrue(isset($view->storage->display['display_test_1']), 'Added display has been assigned to "display_test_1"'); // Check the the display options are like expected. $options = array( @@ -64,7 +64,7 @@ function testDisplayPlugin() { 'display_title' => 'Display test', 'position' => NULL, ); - $this->assertEqual($view->display['display_test_1'], $options); + $this->assertEqual($view->storage->display['display_test_1'], $options); $view->setDisplay('display_test_1'); diff --git a/lib/Drupal/views/Tests/Plugin/FilterTest.php b/lib/Drupal/views/Tests/Plugin/FilterTest.php index f200f06..ad6ee1d 100644 --- a/lib/Drupal/views/Tests/Plugin/FilterTest.php +++ b/lib/Drupal/views/Tests/Plugin/FilterTest.php @@ -138,7 +138,7 @@ public function testFilterQuery() { $this->executeView($view); // Check if we have all 5 results. - $this->assertEqual(count($view->result), 5, format_string('All @count results returned', array('@count' => count($view->display)))); + $this->assertEqual(count($view->result), 5, format_string('All @count results returned', array('@count' => count($view->displayHandlers)))); } } diff --git a/lib/Drupal/views/Tests/UpgradeTestCase.php b/lib/Drupal/views/Tests/UpgradeTestCase.php index 6aedec1..4c9da4a 100644 --- a/lib/Drupal/views/Tests/UpgradeTestCase.php +++ b/lib/Drupal/views/Tests/UpgradeTestCase.php @@ -94,7 +94,7 @@ public function testMovedTo() { $view->update(); $view->build(); - $this->assertEqual('views_test_data', $view->base_table, 'Make sure that view->base_table gets automatically converted.'); + $this->assertEqual('views_test_data', $view->storage->base_table, 'Make sure that view->base_table gets automatically converted.'); // $this->assertEqual('id', $view->field['id']->field, 'If we move a whole table fields of this table should work, too.'); $this->assertEqual('id', $view->field['id']->realField, 'To run the query right the realField has to be set right.'); $this->assertEqual('views_test_data', $view->field['id']->table); diff --git a/lib/Drupal/views/Tests/ViewExecutableTest.php b/lib/Drupal/views/Tests/ViewExecutableTest.php index 02eb4d9..d3a80fe 100644 --- a/lib/Drupal/views/Tests/ViewExecutableTest.php +++ b/lib/Drupal/views/Tests/ViewExecutableTest.php @@ -59,18 +59,9 @@ public function testConstructing() { */ public function testProperties() { $view = $this->getView(); - $storage = $view->storage; - foreach ($this->configProperties as $property) { - $this->assertIdentical($view->{$property}, $storage->{$property}, format_string("Property '%property' is set on the stored view.", array('%property' => $property))); - } foreach ($this->executableProperties as $property) { $this->assertTrue(isset($view->{$property})); } - - // Set one storage property manually on the storage and verify that it is - // access on the executable. - $storage->human_name = $this->randomName(); - $this->assertIdentical($view->human_name, $storage->human_name); } /** diff --git a/lib/Drupal/views/Tests/ViewTest.php b/lib/Drupal/views/Tests/ViewTest.php index ed27f4d..52e1d96 100644 --- a/lib/Drupal/views/Tests/ViewTest.php +++ b/lib/Drupal/views/Tests/ViewTest.php @@ -135,11 +135,11 @@ public function testCreateDuplicate() { ); foreach ($config_properties as $property) { - $this->assertIdentical($view->{$property}, $copy->{$property}, format_string('@property property is identical.', array('@property' => $property))); + $this->assertIdentical($view->storage->{$property}, $copy->{$property}, format_string('@property property is identical.', array('@property' => $property))); } // Check the displays are the same. - foreach ($view->display as $id => $display) { + foreach ($view->storage->display as $id => $display) { // assertIdentical will not work here. $this->assertEqual($display, $copy->display[$id], format_string('The @display display has been copied correctly.', array('@display' => $id))); } diff --git a/lib/Drupal/views/ViewExecutable.php b/lib/Drupal/views/ViewExecutable.php index 9914aec..025fdf4 100644 --- a/lib/Drupal/views/ViewExecutable.php +++ b/lib/Drupal/views/ViewExecutable.php @@ -513,33 +513,6 @@ public function __construct(ViewStorage $storage) { $this->storage->setExecutable($this); } - /** - * Implements the magic __get() method. - * - * @todo Remove this once all calls are changed to use storage directly. - */ - public function &__get($name) { - if (property_exists($this->storage, $name)) { - return $this->storage->{$name}; - } - elseif (property_exists($this, $name)) { - return $this->{$name}; - } - } - - /** - * Implements the magic __set() method. - * - * @todo Remove this once all calls are changed to use storage directly. - */ - public function __set($name, $value) { - if (property_exists($this, $name)) { - $this->{$name} = $value; - } - elseif (property_exists($this->storage, $name)) { - $this->storage->{$name} = $value; - } - } /** * Implements the magic __call() method. @@ -726,7 +699,7 @@ public function initDisplay($reset = FALSE) { } // Instantiate all displays - foreach (array_keys($this->display) as $id) { + foreach (array_keys($this->storage->display) as $id) { $this->displayHandlers[$id] = views_get_plugin('display', $this->storage->display[$id]['display_plugin']); if (!empty($this->displayHandlers[$id])) { // Initialize the new display handler with data. @@ -798,9 +771,9 @@ public function setDisplay($display_id = NULL) { } // Ensure the requested display exists. - if (empty($this->display[$display_id])) { + if (empty($this->displayHandlers[$display_id])) { $display_id = 'default'; - if (empty($this->display[$display_id])) { + if (empty($this->displayHandlers[$display_id])) { debug('set_display() called with invalid display ID @display.', array('@display' => $display_id)); return FALSE; } @@ -1087,7 +1060,7 @@ public function initQuery() { // Create and initialize the query object. $views_data = views_fetch_data($this->storage->base_table); - $this->base_field = !empty($views_data['table']['base']['field']) ? $views_data['table']['base']['field'] : ''; + $this->storage->base_field = !empty($views_data['table']['base']['field']) ? $views_data['table']['base']['field'] : ''; if (!empty($views_data['table']['base']['database'])) { $this->base_database = $views_data['table']['base']['database']; } @@ -1103,7 +1076,7 @@ public function initQuery() { return FALSE; } - $this->query->init($this->storage->base_table, $this->base_field, $query_options['options']); + $this->query->init($this->storage->base_table, $this->storage->base_field, $query_options['options']); return TRUE; } @@ -1583,7 +1556,7 @@ public function preExecute($args = array()) { } // Allow hook_views_pre_view() to set the dom_id, then ensure it is set. - $this->dom_id = !empty($this->dom_id) ? $this->dom_id : md5($this->name . REQUEST_TIME . rand()); + $this->dom_id = !empty($this->dom_id) ? $this->dom_id : md5($this->storage->name . REQUEST_TIME . rand()); // Allow the display handler to set up for execution $this->display_handler->preExecute(); @@ -1617,8 +1590,8 @@ public function attachDisplays() { $this->is_attachment = TRUE; // Give other displays an opportunity to attach to the view. - foreach ($this->display as $id => $display) { - if (!empty($this->display[$id])) { + foreach ($this->displayHandlers as $id => $display) { + if (!empty($this->displayHandlers[$id])) { $this->displayHandlers[$id]->attachTo($this->current_display); } } diff --git a/lib/Drupal/views/ViewStorage.php b/lib/Drupal/views/ViewStorage.php index 512174f..a96242d 100644 --- a/lib/Drupal/views/ViewStorage.php +++ b/lib/Drupal/views/ViewStorage.php @@ -126,6 +126,17 @@ class ViewStorage extends ConfigEntityBase implements ViewStorageInterface { public $module = 'views'; /** + * Implements the magic __call() method. + * + * @todo Remove this once all calls are changed to use storage directly. + */ + public function __call($name, $arguments) { + if (method_exists($this->executable, $name)) { + return call_user_func_array(array($this->executable, $name), $arguments); + } + } + + /** * Stores the executable version of this view. * * @param Drupal\views\ViewExecutable $executable @@ -153,45 +164,6 @@ public function initDisplay($reset = FALSE) { } /** - * Implements the magic __call() method. - * - * @todo Remove this once all calls are changed to use executable directly. - */ - public function __call($name, $arguments) { - if (method_exists($this->executable, $name)) { - return call_user_func_array(array($this->executable, $name), $arguments); - } - } - - /** - * Implements the magic __get() method. - * - * @todo Remove this once all calls are changed to use executable directly. - */ - public function &__get($name) { - if (property_exists($this->executable, $name)) { - return $this->executable->{$name}; - } - if (property_exists($this, $name)) { - return $this->{$name}; - } - } - - /** - * Implements the magic __set() method. - * - * @todo Remove this once all calls are changed to use executable directly. - */ - public function __set($name, $value) { - if (property_exists($this, $name)) { - $this->{$name} = $value; - } - elseif (property_exists($this->executable, $name)) { - $this->executable->{$name} = $value; - } - } - - /** * Returns the name of the module implementing this view. */ public function getModule() { @@ -373,6 +345,9 @@ public static function generateItemId($requested_id, $existing_items) { */ public function &newDisplay($plugin_id = 'page', $title = NULL, $id = NULL) { $id = $this->addDisplay($plugin_id, $title, $id); + if (!isset($this->executable)) { + $this->setExecutable(new ViewExecutable($this)); + } // Create a handler. $this->executable->displayHandlers[$id] = views_get_plugin('display', $this->display[$id]['display_plugin']); @@ -384,7 +359,7 @@ public function &newDisplay($plugin_id = 'page', $title = NULL, $id = NULL) { if (!empty($this->executable->displayHandlers[$id])) { // Initialize the new display handler with data. - $this->executable->displayHandlers[$id]->init($this, $this->display[$id]); + $this->executable->displayHandlers[$id]->init($this->executable, $this->display[$id]); // If this is NOT the default display handler, let it know which is if ($id != 'default') { // @todo is the '&' still required in php5? diff --git a/lib/Views/block/Plugin/views/display/Block.php b/lib/Views/block/Plugin/views/display/Block.php index 69f25b6..a1d205a 100644 --- a/lib/Views/block/Plugin/views/display/Block.php +++ b/lib/Views/block/Plugin/views/display/Block.php @@ -52,15 +52,15 @@ protected function defineOptions() { * stuff with it. */ public function executeHookBlockList($delta = 0, $edit = array()) { - $delta = $this->view->name . '-' . $this->display['id']; + $delta = $this->view->storage->name . '-' . $this->display['id']; $desc = $this->getOption('block_description'); if (empty($desc)) { if ($this->display['display_title'] == $this->definition['title']) { - $desc = t('View: !view', array('!view' => $this->view->getHumanName())); + $desc = t('View: !view', array('!view' => $this->view->storage->getHumanName())); } else { - $desc = t('View: !view: !display', array('!view' => $this->view->getHumanName(), '!display' => $this->display['display_title'])); + $desc = t('View: !view: !display', array('!view' => $this->view->storage->getHumanName(), '!display' => $this->display['display_title'])); } } return array( @@ -193,14 +193,14 @@ public function submitOptionsForm(&$form, &$form_state) { parent::submitOptionsForm($form, $form_state); switch ($form_state['section']) { case 'display_id': - $this->updateBlockBid($form_state['view']->name, $this->display['id'], $this->display['new_id']); + $this->updateBlockBid($form_state['view']->storage->name, $this->display['id'], $this->display['new_id']); break; case 'block_description': $this->setOption('block_description', $form_state['values']['block_description']); break; case 'block_caching': $this->setOption('block_caching', $form_state['values']['block_caching']); - $this->saveBlockCache($form_state['view']->name . '-'. $form_state['display_id'], $form_state['values']['block_caching']); + $this->saveBlockCache($form_state['view']->storage->name . '-'. $form_state['display_id'], $form_state['values']['block_caching']); break; } } diff --git a/lib/Views/comment/Plugin/views/field/Comment.php b/lib/Views/comment/Plugin/views/field/Comment.php index dd6afde..fe151ad 100644 --- a/lib/Views/comment/Plugin/views/field/Comment.php +++ b/lib/Views/comment/Plugin/views/field/Comment.php @@ -7,6 +7,7 @@ namespace Views\comment\Plugin\views\field; +use Drupal\views\ViewExecutable; use Drupal\views\Plugin\views\field\FieldPluginBase; use Drupal\Core\Annotation\Plugin; @@ -25,7 +26,7 @@ class Comment extends FieldPluginBase { /** * Override init function to provide generic option to link to comment. */ - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { parent::init($view, $options); if (!empty($this->options['link_to_comment'])) { $this->additional_fields['cid'] = 'cid'; diff --git a/lib/Views/comment/Plugin/views/field/Username.php b/lib/Views/comment/Plugin/views/field/Username.php index 025d4be..52e8851 100644 --- a/lib/Views/comment/Plugin/views/field/Username.php +++ b/lib/Views/comment/Plugin/views/field/Username.php @@ -7,6 +7,7 @@ namespace Views\comment\Plugin\views\field; +use Drupal\views\ViewExecutable; use Drupal\views\Plugin\views\field\FieldPluginBase; use Drupal\Core\Annotation\Plugin; @@ -25,7 +26,7 @@ class Username extends FieldPluginBase { /** * Override init function to add uid and homepage fields. */ - public function init(&$view, &$data) { + public function init(ViewExecutable $view, &$data) { parent::init($view, $data); $this->additional_fields['uid'] = 'uid'; $this->additional_fields['homepage'] = 'homepage'; diff --git a/lib/Views/field/Plugin/views/argument/FieldList.php b/lib/Views/field/Plugin/views/argument/FieldList.php index 8dc99c3..f3b486b 100644 --- a/lib/Views/field/Plugin/views/argument/FieldList.php +++ b/lib/Views/field/Plugin/views/argument/FieldList.php @@ -7,6 +7,7 @@ namespace Views\field\Plugin\views\argument; +use Drupal\views\ViewExecutable; use Drupal\views\Plugin\views\argument\Numeric; use Drupal\Core\Annotation\Plugin; @@ -30,7 +31,7 @@ class FieldList extends Numeric { */ var $allowed_values = NULL; - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { parent::init($view, $options); $field = field_info_field($this->definition['field_name']); $this->allowed_values = options_allowed_values($field); diff --git a/lib/Views/field/Plugin/views/argument/ListString.php b/lib/Views/field/Plugin/views/argument/ListString.php index a50af5a..6ac3f8d 100644 --- a/lib/Views/field/Plugin/views/argument/ListString.php +++ b/lib/Views/field/Plugin/views/argument/ListString.php @@ -7,6 +7,7 @@ namespace Views\field\Plugin\views\argument; +use Drupal\views\ViewExecutable; use Drupal\views\Plugin\views\argument\String; use Drupal\Core\Annotation\Plugin; @@ -30,7 +31,7 @@ class ListString extends String { */ var $allowed_values = NULL; - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { parent::init($view, $options); $field = field_info_field($this->definition['field_name']); $this->allowed_values = options_allowed_values($field); diff --git a/lib/Views/field/Plugin/views/field/Field.php b/lib/Views/field/Plugin/views/field/Field.php index 5239f2f..4a3561d 100644 --- a/lib/Views/field/Plugin/views/field/Field.php +++ b/lib/Views/field/Plugin/views/field/Field.php @@ -7,6 +7,7 @@ namespace Views\field\Plugin\views\field; +use Drupal\views\ViewExecutable; use Drupal\views\Plugin\views\field\FieldPluginBase; use Drupal\Core\Annotation\Plugin; @@ -65,7 +66,7 @@ class Field extends FieldPluginBase { */ public $instance; - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { parent::init($view, $options); $this->field_info = $field = field_info_field($this->definition['field_name']); @@ -114,7 +115,7 @@ public function access() { function get_base_table() { if (!isset($this->base_table)) { // This base_table is coming from the entity not the field. - $this->base_table = $this->view->base_table; + $this->base_table = $this->view->storage->base_table; // If the current field is under a relationship you can't be sure that the // base table of the view is the base table of the current field. diff --git a/lib/Views/field/Plugin/views/relationship/EntityReverse.php b/lib/Views/field/Plugin/views/relationship/EntityReverse.php index 2dc97a7..c15c78f 100644 --- a/lib/Views/field/Plugin/views/relationship/EntityReverse.php +++ b/lib/Views/field/Plugin/views/relationship/EntityReverse.php @@ -7,6 +7,7 @@ namespace Views\field\Plugin\views\relationship; +use Drupal\views\ViewExecutable; use Drupal\views\Plugin\views\relationship\RelationshipPluginBase; use Drupal\Core\Annotation\Plugin; @@ -22,7 +23,7 @@ */ class EntityReverse extends RelationshipPluginBase { - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { parent::init($view, $options); $this->field_info = field_info_field($this->definition['field_name']); diff --git a/lib/Views/file/Plugin/views/field/File.php b/lib/Views/file/Plugin/views/field/File.php index c4b6a87..a10f88d 100644 --- a/lib/Views/file/Plugin/views/field/File.php +++ b/lib/Views/file/Plugin/views/field/File.php @@ -7,6 +7,7 @@ namespace Views\file\Plugin\views\field; +use Drupal\views\ViewExecutable; use Drupal\Core\Annotation\Plugin; use Drupal\views\Plugin\views\field\FieldPluginBase; @@ -25,7 +26,7 @@ class File extends FieldPluginBase { /** * Constructor to provide additional field to add. */ - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { parent::init($view, $options); if (!empty($options['link_to_file'])) { $this->additional_fields['uri'] = 'uri'; diff --git a/lib/Views/node/Plugin/views/argument/CreatedDay.php b/lib/Views/node/Plugin/views/argument/CreatedDay.php index 2d2b7cc..22ea081 100644 --- a/lib/Views/node/Plugin/views/argument/CreatedDay.php +++ b/lib/Views/node/Plugin/views/argument/CreatedDay.php @@ -16,6 +16,8 @@ * * @Plugin( * id = "node_created_day", + * arg_format = "d", + * format = "j", * module = "node" * ) */ @@ -27,9 +29,7 @@ class CreatedDay extends Date { public function __construct(array $configuration, $plugin_id, DiscoveryInterface $discovery) { parent::__construct($configuration, $plugin_id, $discovery); - $this->formula = views_date_sql_extract('DAY', "***table***.$this->realField"); - $this->format = 'j'; - $this->arg_format = 'd'; + $this->formula = views_date_sql_extract('DAY', '***table***.' $this->definition['field']); } /** @@ -38,7 +38,7 @@ public function __construct(array $configuration, $plugin_id, DiscoveryInterface function summary_name($data) { $day = str_pad($data->{$this->name_alias}, 2, '0', STR_PAD_LEFT); // strtotime respects server timezone, so we need to set the time fixed as utc time - return format_date(strtotime("2005" . "05" . $day . " 00:00:00 UTC"), 'custom', $this->format, 'UTC'); + return format_date(strtotime("2005" . "05" . $day . " 00:00:00 UTC"), 'custom', $this->definition['format'], 'UTC'); } /** @@ -46,7 +46,7 @@ function summary_name($data) { */ function title() { $day = str_pad($this->argument, 2, '0', STR_PAD_LEFT); - return format_date(strtotime("2005" . "05" . $day . " 00:00:00 UTC"), 'custom', $this->format, 'UTC'); + return format_date(strtotime("2005" . "05" . $day . " 00:00:00 UTC"), 'custom', $this->definition['format'], 'UTC'); } function summary_argument($data) { diff --git a/lib/Views/node/Plugin/views/argument/CreatedFullDate.php b/lib/Views/node/Plugin/views/argument/CreatedFullDate.php index 934b67e..e92b2ca 100644 --- a/lib/Views/node/Plugin/views/argument/CreatedFullDate.php +++ b/lib/Views/node/Plugin/views/argument/CreatedFullDate.php @@ -16,6 +16,8 @@ * * @Plugin( * id = "node_created_fulldate", + * arg_format = "Ymd", + * format = "F j, Y", * module = "node" * ) */ @@ -27,9 +29,7 @@ class CreatedFullDate extends Date { public function __construct(array $configuration, $plugin_id, DiscoveryInterface $discovery) { parent::__construct($configuration, $plugin_id, $discovery); - $this->format = 'F j, Y'; - $this->arg_format = 'Ymd'; - $this->formula = views_date_sql_format($this->arg_format, "***table***.$this->realField"); + $this->formula = views_date_sql_format($this->definition['arg_format'], '***table***.' . $this->definition['field']); } /** @@ -37,14 +37,14 @@ public function __construct(array $configuration, $plugin_id, DiscoveryInterface */ function summary_name($data) { $created = $data->{$this->name_alias}; - return format_date(strtotime($created . " 00:00:00 UTC"), 'custom', $this->format, 'UTC'); + return format_date(strtotime($created . " 00:00:00 UTC"), 'custom', $this->definition['format'], 'UTC'); } /** * Provide a link to the next level of the view */ function title() { - return format_date(strtotime($this->argument . " 00:00:00 UTC"), 'custom', $this->format, 'UTC'); + return format_date(strtotime($this->argument . " 00:00:00 UTC"), 'custom', $this->definition['format'], 'UTC'); } } diff --git a/lib/Views/node/Plugin/views/argument/CreatedMonth.php b/lib/Views/node/Plugin/views/argument/CreatedMonth.php index ed51d20..80d7671 100644 --- a/lib/Views/node/Plugin/views/argument/CreatedMonth.php +++ b/lib/Views/node/Plugin/views/argument/CreatedMonth.php @@ -16,6 +16,8 @@ * * @Plugin( * id = "node_created_month", + * arg_format = "m", + * format = "F", * module = "node" * ) */ @@ -27,9 +29,7 @@ class CreatedMonth extends Date { public function __construct(array $configuration, $plugin_id, DiscoveryInterface $discovery) { parent::__construct($configuration, $plugin_id, $discovery); - $this->formula = views_date_sql_extract('MONTH', "***table***.$this->realField"); - $this->format = 'F'; - $this->arg_format = 'm'; + $this->formula = views_date_sql_extract('MONTH', '***table***.' $this->definition['field']); } /** @@ -37,7 +37,7 @@ public function __construct(array $configuration, $plugin_id, DiscoveryInterface */ function summary_name($data) { $month = str_pad($data->{$this->name_alias}, 2, '0', STR_PAD_LEFT); - return format_date(strtotime("2005" . $month . "15" . " 00:00:00 UTC" ), 'custom', $this->format, 'UTC'); + return format_date(strtotime("2005" . $month . "15" . " 00:00:00 UTC" ), 'custom', $this->definition['format'], 'UTC'); } /** @@ -45,7 +45,7 @@ function summary_name($data) { */ function title() { $month = str_pad($this->argument, 2, '0', STR_PAD_LEFT); - return format_date(strtotime("2005" . $month . "15" . " 00:00:00 UTC"), 'custom', $this->format, 'UTC'); + return format_date(strtotime("2005" . $month . "15" . " 00:00:00 UTC"), 'custom', $this->definition['format'], 'UTC'); } function summary_argument($data) { diff --git a/lib/Views/node/Plugin/views/argument/CreatedWeek.php b/lib/Views/node/Plugin/views/argument/CreatedWeek.php index 9e54758..8d408f0 100644 --- a/lib/Views/node/Plugin/views/argument/CreatedWeek.php +++ b/lib/Views/node/Plugin/views/argument/CreatedWeek.php @@ -16,6 +16,7 @@ * * @Plugin( * id = "node_created_week", + * arg_format = "w", * module = "node" * ) */ @@ -27,8 +28,7 @@ class CreatedWeek extends Date { public function __construct(array $configuration, $plugin_id, DiscoveryInterface $discovery) { parent::__construct($configuration, $plugin_id, $discovery); - $this->arg_format = 'w'; - $this->formula = views_date_sql_extract('WEEK', "***table***.$this->realField"); + $this->formula = views_date_sql_extract('WEEK', '***table***.' $this->definition['field']); } /** diff --git a/lib/Views/node/Plugin/views/argument/CreatedYear.php b/lib/Views/node/Plugin/views/argument/CreatedYear.php index 55d4cf7..99d0222 100644 --- a/lib/Views/node/Plugin/views/argument/CreatedYear.php +++ b/lib/Views/node/Plugin/views/argument/CreatedYear.php @@ -16,6 +16,7 @@ * * @Plugin( * id = "node_created_year", + * arg_format = "Y", * module = "node" * ) */ @@ -27,8 +28,7 @@ class CreatedYear extends Date { public function __construct(array $configuration, $plugin_id, DiscoveryInterface $discovery) { parent::__construct($configuration, $plugin_id, $discovery); - $this->arg_format = 'Y'; - $this->formula = views_date_sql_extract('YEAR', "***table***.$this->realField"); + $this->formula = views_date_sql_extract('YEAR', '***table***.' $this->definition['field']); } } diff --git a/lib/Views/node/Plugin/views/argument/CreatedYearMonth.php b/lib/Views/node/Plugin/views/argument/CreatedYearMonth.php index 758f3d7..9924a40 100644 --- a/lib/Views/node/Plugin/views/argument/CreatedYearMonth.php +++ b/lib/Views/node/Plugin/views/argument/CreatedYearMonth.php @@ -16,6 +16,8 @@ * * @Plugin( * id = "node_created_year_month", + * format = "F Y", + * arg_format = "Ym", * module = "node" * ) */ @@ -27,9 +29,7 @@ class CreatedYearMonth extends Date { public function __construct(array $configuration, $plugin_id, DiscoveryInterface $discovery) { parent::__construct($configuration, $plugin_id, $discovery); - $this->format = 'F Y'; - $this->arg_format = 'Ym'; - $this->formula = views_date_sql_format($this->arg_format, "***table***.$this->realField"); + $this->formula = views_date_sql_format($this->definition['arg_format'], '***table***.' . $this->definition['field']); } /** @@ -37,14 +37,14 @@ public function __construct(array $configuration, $plugin_id, DiscoveryInterface */ function summary_name($data) { $created = $data->{$this->name_alias}; - return format_date(strtotime($created . "15" . " 00:00:00 UTC"), 'custom', $this->format, 'UTC'); + return format_date(strtotime($created . "15" . " 00:00:00 UTC"), 'custom', $this->definition['format'], 'UTC'); } /** * Provide a link to the next level of the view */ function title() { - return format_date(strtotime($this->argument . "15" . " 00:00:00 UTC"), 'custom', $this->format, 'UTC'); + return format_date(strtotime($this->argument . "15" . " 00:00:00 UTC"), 'custom', $this->definition['format'], 'UTC'); } } diff --git a/lib/Views/node/Plugin/views/field/HistoryUserTimestamp.php b/lib/Views/node/Plugin/views/field/HistoryUserTimestamp.php index 74b63b7..c45d62b 100644 --- a/lib/Views/node/Plugin/views/field/HistoryUserTimestamp.php +++ b/lib/Views/node/Plugin/views/field/HistoryUserTimestamp.php @@ -7,6 +7,7 @@ namespace Views\node\Plugin\views\field; +use Drupal\views\ViewExecutable; use Views\node\Plugin\views\field\Node; use Drupal\Core\Annotation\Plugin; @@ -25,7 +26,7 @@ */ class HistoryUserTimestamp extends Node { - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { parent::init($view, $options); global $user; if ($user->uid) { diff --git a/lib/Views/node/Plugin/views/field/Node.php b/lib/Views/node/Plugin/views/field/Node.php index 2995cb6..974c6be 100644 --- a/lib/Views/node/Plugin/views/field/Node.php +++ b/lib/Views/node/Plugin/views/field/Node.php @@ -7,6 +7,7 @@ namespace Views\node\Plugin\views\field; +use Drupal\views\ViewExecutable; use Drupal\views\Plugin\views\field\FieldPluginBase; use Drupal\Core\Annotation\Plugin; @@ -24,7 +25,7 @@ */ class Node extends FieldPluginBase { - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { parent::init($view, $options); // Don't add the additional fields to groupby if (!empty($this->options['link_to_node'])) { diff --git a/lib/Views/node/Plugin/views/field/Revision.php b/lib/Views/node/Plugin/views/field/Revision.php index aeb000c..2e1b12a 100644 --- a/lib/Views/node/Plugin/views/field/Revision.php +++ b/lib/Views/node/Plugin/views/field/Revision.php @@ -7,6 +7,7 @@ namespace Views\node\Plugin\views\field; +use Drupal\views\ViewExecutable; use Views\node\Plugin\views\field\Node; use Drupal\Core\Annotation\Plugin; @@ -22,7 +23,7 @@ */ class Revision extends Node { - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { parent::init($view, $options); if (!empty($this->options['link_to_node_revision'])) { $this->additional_fields['vid'] = 'vid'; diff --git a/lib/Views/node/Plugin/views/row/View.php b/lib/Views/node/Plugin/views/row/View.php index e49fb85..2a081d4 100644 --- a/lib/Views/node/Plugin/views/row/View.php +++ b/lib/Views/node/Plugin/views/row/View.php @@ -7,6 +7,7 @@ namespace Views\node\Plugin\views\row; +use Drupal\views\ViewExecutable; use Drupal\Core\Annotation\Plugin; use Drupal\Core\Annotation\Translation; use Drupal\views\Plugin\views\row\RowPluginBase; @@ -37,7 +38,7 @@ class View extends RowPluginBase { // Stores the nodes loaded with pre_render. var $nodes = array(); - public function init(&$view, &$display, $options = NULL) { + public function init(ViewExecutable $view, &$display, $options = NULL) { parent::init($view, $display, $options); // Handle existing views with the deprecated 'teaser' option. if (isset($this->options['teaser'])) { diff --git a/lib/Views/statistics/Plugin/views/field/AccesslogPath.php b/lib/Views/statistics/Plugin/views/field/AccesslogPath.php index 6bdb347..4b16e20 100644 --- a/lib/Views/statistics/Plugin/views/field/AccesslogPath.php +++ b/lib/Views/statistics/Plugin/views/field/AccesslogPath.php @@ -7,6 +7,7 @@ namespace Views\statistics\Plugin\views\field; +use Drupal\views\ViewExecutable; use Drupal\views\Plugin\views\field\FieldPluginBase; use Drupal\Core\Annotation\Plugin; @@ -25,7 +26,7 @@ class AccesslogPath extends FieldPluginBase { /** * Override init function to provide generic option to link to node. */ - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { parent::init($view, $options); if (!empty($this->options['display_as_link'])) { $this->additional_fields['path'] = 'path'; diff --git a/lib/Views/taxonomy/Plugin/views/argument_default/Tid.php b/lib/Views/taxonomy/Plugin/views/argument_default/Tid.php index 9086e66..583a030 100644 --- a/lib/Views/taxonomy/Plugin/views/argument_default/Tid.php +++ b/lib/Views/taxonomy/Plugin/views/argument_default/Tid.php @@ -7,6 +7,7 @@ namespace Views\taxonomy\Plugin\views\argument_default; +use Drupal\views\ViewExecutable; use Drupal\Core\Annotation\Plugin; use Drupal\Core\Annotation\Translation; use Drupal\views\Plugin\views\argument_default\ArgumentDefaultPluginBase; @@ -22,7 +23,7 @@ */ class Tid extends ArgumentDefaultPluginBase { - public function init(&$view, &$argument, $options) { + public function init(ViewExecutable $view, &$argument, $options) { parent::init($view, $argument, $options); // Convert legacy vids option to machine name vocabularies. diff --git a/lib/Views/taxonomy/Plugin/views/argument_validator/Term.php b/lib/Views/taxonomy/Plugin/views/argument_validator/Term.php index 16c1b80..73f567f 100644 --- a/lib/Views/taxonomy/Plugin/views/argument_validator/Term.php +++ b/lib/Views/taxonomy/Plugin/views/argument_validator/Term.php @@ -7,6 +7,7 @@ namespace Views\taxonomy\Plugin\views\argument_validator; +use Drupal\views\ViewExecutable; use Drupal\Core\Annotation\Plugin; use Drupal\Core\Annotation\Translation; use Drupal\views\Plugin\views\argument_validator\ArgumentValidatorPluginBase; @@ -22,7 +23,7 @@ */ class Term extends ArgumentValidatorPluginBase { - public function init(&$view, &$argument, $options) { + public function init(ViewExecutable $view, &$argument, $options) { parent::init($view, $argument, $options); // Convert legacy vids option to machine name vocabularies. diff --git a/lib/Views/taxonomy/Plugin/views/field/TaxonomyIndexTid.php b/lib/Views/taxonomy/Plugin/views/field/TaxonomyIndexTid.php index 6deeb34..c4ec914 100644 --- a/lib/Views/taxonomy/Plugin/views/field/TaxonomyIndexTid.php +++ b/lib/Views/taxonomy/Plugin/views/field/TaxonomyIndexTid.php @@ -7,6 +7,7 @@ namespace Views\taxonomy\Plugin\views\field; +use Drupal\views\ViewExecutable; use Drupal\views\Plugin\views\field\PrerenderList; use Drupal\Core\Annotation\Plugin; @@ -22,7 +23,7 @@ */ class TaxonomyIndexTid extends PrerenderList { - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { parent::init($view, $options); // @todo: Wouldn't it be possible to use $this->base_table and no if here? if ($view->base_table == 'node_revision') { diff --git a/lib/Views/taxonomy/Plugin/views/filter/TaxonomyIndexTid.php b/lib/Views/taxonomy/Plugin/views/filter/TaxonomyIndexTid.php index 1e11a3b..f4300b0 100644 --- a/lib/Views/taxonomy/Plugin/views/filter/TaxonomyIndexTid.php +++ b/lib/Views/taxonomy/Plugin/views/filter/TaxonomyIndexTid.php @@ -7,6 +7,7 @@ namespace Views\taxonomy\Plugin\views\filter; +use Drupal\views\ViewExecutable; use Drupal\Core\Annotation\Plugin; use Drupal\views\Plugin\views\filter\ManyToOne; @@ -25,7 +26,7 @@ class TaxonomyIndexTid extends ManyToOne { // Stores the exposed input for this filter. var $validated_exposed_input = NULL; - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { parent::init($view, $options); if (!empty($this->definition['vocabulary'])) { $this->options['vocabulary'] = $this->definition['vocabulary']; diff --git a/lib/Views/taxonomy/Plugin/views/relationship/NodeTermData.php b/lib/Views/taxonomy/Plugin/views/relationship/NodeTermData.php index a00a331..f2bbd9a 100644 --- a/lib/Views/taxonomy/Plugin/views/relationship/NodeTermData.php +++ b/lib/Views/taxonomy/Plugin/views/relationship/NodeTermData.php @@ -7,6 +7,7 @@ namespace Views\taxonomy\Plugin\views\relationship; +use Drupal\views\ViewExecutable; use Drupal\views\Plugin\views\relationship\RelationshipPluginBase; use Drupal\Core\Annotation\Plugin; @@ -22,7 +23,7 @@ */ class NodeTermData extends RelationshipPluginBase { - public function init(&$view, &$options) { + public function init(ViewExecutable $view, &$options) { parent::init($view, $options); // Convert legacy vids option to machine name vocabularies. diff --git a/lib/Views/user/Plugin/views/field/Name.php b/lib/Views/user/Plugin/views/field/Name.php index 3c76de4..8ce583d 100644 --- a/lib/Views/user/Plugin/views/field/Name.php +++ b/lib/Views/user/Plugin/views/field/Name.php @@ -9,6 +9,7 @@ use Views\user\Plugin\views\field\User; use Drupal\Core\Annotation\Plugin; +use Drupal\views\ViewExecutable; /** * Field handler to provide simple renderer that allows using a themed user link. @@ -25,7 +26,7 @@ class Name extends User { /** * Add uid in the query so we can test for anonymous if needed. */ - public function init(&$view, &$data) { + public function init(ViewExecutable $view, &$data) { parent::init($view, $data); if (!empty($this->options['overwrite_anonymous']) || !empty($this->options['format_username'])) { $this->additional_fields['uid'] = 'uid'; diff --git a/lib/Views/user/Plugin/views/field/User.php b/lib/Views/user/Plugin/views/field/User.php index 641d773..1fee9d2 100644 --- a/lib/Views/user/Plugin/views/field/User.php +++ b/lib/Views/user/Plugin/views/field/User.php @@ -9,6 +9,7 @@ use Drupal\views\Plugin\views\field\FieldPluginBase; use Drupal\Core\Annotation\Plugin; +use Drupal\views\ViewExecutable; /** * Field handler to provide simple renderer that allows linking to a user. @@ -25,7 +26,7 @@ class User extends FieldPluginBase { /** * Override init function to provide generic option to link to user. */ - public function init(&$view, &$data) { + public function init(ViewExecutable $view, &$data) { parent::init($view, $data); if (!empty($this->options['link_to_user'])) { $this->additional_fields['uid'] = 'uid'; diff --git a/tests/views_test_data/views_test_data.module b/tests/views_test_data/views_test_data.module index 62a67df..1341f46 100644 --- a/tests/views_test_data/views_test_data.module +++ b/tests/views_test_data/views_test_data.module @@ -47,8 +47,8 @@ function views_test_data_test_dynamic_access_callback($access, $argument1, $argu /** * Implements hook_views_pre_render(). */ -function views_test_data_views_pre_render(&$view) { - if ($view->name == 'test_cache_header_storage') { +function views_test_data_views_pre_render(ViewExecutable $view) { + if ($view->storage->name == 'test_cache_header_storage') { drupal_add_js(drupal_get_path('module', 'views_test_data') . '/views_cache.test.js'); drupal_add_css(drupal_get_path('module', 'views_test_data') . '/views_cache.test.css'); $view->build_info['pre_render_called'] = TRUE; @@ -58,8 +58,8 @@ function views_test_data_views_pre_render(&$view) { /** * Implements hook_views_post_build(). */ -function views_test_data_views_post_build(ViewExecutable &$view) { - if ($view->name == 'test_page_display') { +function views_test_data_views_post_build(ViewExecutable $view) { + if ($view->storage->name == 'test_page_display') { if ($view->current_display == 'page_1') { $view->build_info['denied'] = TRUE; } diff --git a/theme/theme.inc b/theme/theme.inc index 0a42b9e..9a4a227 100644 --- a/theme/theme.inc +++ b/theme/theme.inc @@ -6,6 +6,7 @@ */ use Drupal\Core\Template\Attribute; +use Drupal\views\ViewExecutable; /** * Provide a full array of possible themes to try for a given hook. @@ -17,23 +18,23 @@ * @param $display * The display being rendered, if applicable. */ -function _views_theme_functions($hook, $view, $display = NULL) { +function _views_theme_functions($hook, ViewExecutable $view, $display = NULL) { $themes = array(); if ($display) { - $themes[] = $hook . '__' . $view->name . '__' . $display['id']; + $themes[] = $hook . '__' . $view->storage->name . '__' . $display['id']; $themes[] = $hook . '__' . $display['id']; // Add theme suggestions for each single tag. - foreach (drupal_explode_tags($view->tag) as $tag) { + foreach (drupal_explode_tags($view->storage->tag) as $tag) { $themes[] = $hook . '__' . preg_replace('/[^a-z0-9]/', '_', strtolower($tag)); } if ($display['id'] != $display['display_plugin']) { - $themes[] = $hook . '__' . $view->name . '__' . $display['display_plugin']; + $themes[] = $hook . '__' . $view->storage->name . '__' . $display['display_plugin']; $themes[] = $hook . '__' . $display['display_plugin']; } } - $themes[] = $hook . '__' . $view->name; + $themes[] = $hook . '__' . $view->storage->name; $themes[] = $hook; return $themes; } @@ -48,8 +49,8 @@ function template_preprocess_views_view(&$vars) { $vars['rows'] = (!empty($view->result) || $view->style_plugin->even_empty()) ? $view->style_plugin->render($view->result) : ''; - $vars['css_name'] = drupal_clean_css_identifier($view->name); - $vars['name'] = $view->name; + $vars['css_name'] = drupal_clean_css_identifier($view->storage->name); + $vars['name'] = $view->storage->name; $vars['display_id'] = $view->current_display; // Basic classes @@ -124,7 +125,7 @@ function template_preprocess_views_view(&$vars) { 'ajax_path' => url('views/ajax'), 'ajaxViews' => array( 'views_dom_id:' . $vars['dom_id'] => array( - 'view_name' => $view->name, + 'view_name' => $view->storage->name, 'view_display_id' => $view->current_display, 'view_args' => check_plain(implode('/', $view->args)), 'view_path' => check_plain(current_path()), @@ -866,7 +867,7 @@ function template_preprocess_views_view_rss(&$vars) { // Figure out which display which has a path we're using for this feed. If there isn't // one, use the global $base_url $link_display_id = $view->display_handler->getLinkDisplay(); - if ($link_display_id && !empty($view->display[$link_display_id])) { + if ($link_display_id && !empty($view->displayHandlers[$link_display_id])) { $path = $view->displayHandlers[$link_display_id]->getPath(); } diff --git a/theme/views-ui-edit-view.tpl.php b/theme/views-ui-edit-view.tpl.php index 5c3732c..566b8a4 100644 --- a/theme/views-ui-edit-view.tpl.php +++ b/theme/views-ui-edit-view.tpl.php @@ -21,7 +21,7 @@ @base.', - array('%name' => $view->name, '@base' => $base_table)); ?> + array('%name' => $view->storage->name, '@base' => $base_table)); ?> diff --git a/views.module b/views.module index 7cd7ae6..9c95cd9 100644 --- a/views.module +++ b/views.module @@ -131,7 +131,7 @@ function views_forms($form_id, $args) { function views_form_id($view) { $parts = array( 'views_form', - $view->name, + $view->storage->name, $view->current_display, ); @@ -373,7 +373,7 @@ function views_plugin_list() { } // Add this view to the list for this plugin. - $plugins[$key]['views'][$view->name] = $view->name; + $plugins[$key]['views'][$view->storage->name] = $view->storage->name; } } } @@ -390,11 +390,11 @@ function views_plugin_list() { */ function views_preprocess_node(&$vars) { // The 'view' attribute of the node is added in views_preprocess_node() - if (!empty($vars['node']->view) && !empty($vars['node']->view->name)) { + if (!empty($vars['node']->view) && !empty($vars['node']->view->storage->name)) { $vars['view'] = $vars['node']->view; - $vars['theme_hook_suggestions'][] = 'node__view__' . $vars['node']->view->name; + $vars['theme_hook_suggestions'][] = 'node__view__' . $vars['node']->view->storage->name; if (!empty($vars['node']->view->current_display)) { - $vars['theme_hook_suggestions'][] = 'node__view__' . $vars['node']->view->name . '__' . $vars['node']->view->current_display; + $vars['theme_hook_suggestions'][] = 'node__view__' . $vars['node']->view->storage->name . '__' . $vars['node']->view->current_display; // If a node is being rendered in a view, and the view does not have a path, // prevent drupal from accidentally setting the $page variable: @@ -416,11 +416,11 @@ function views_preprocess_node(&$vars) { */ function views_preprocess_comment(&$vars) { // The 'view' attribute of the node is added in template_preprocess_views_view_row_comment() - if (!empty($vars['node']->view) && !empty($vars['node']->view->name)) { + if (!empty($vars['node']->view) && !empty($vars['node']->view->storage->name)) { $vars['view'] = &$vars['node']->view; - $vars['theme_hook_suggestions'][] = 'comment__view__' . $vars['node']->view->name; + $vars['theme_hook_suggestions'][] = 'comment__view__' . $vars['node']->view->storage->name; if (!empty($vars['node']->view->current_display)) { - $vars['theme_hook_suggestions'][] = 'comment__view__' . $vars['node']->view->name . '__' . $vars['node']->view->current_display; + $vars['theme_hook_suggestions'][] = 'comment__view__' . $vars['node']->view->storage->name . '__' . $vars['node']->view->current_display; } } } @@ -699,7 +699,7 @@ function views_block_info() { } $view->initDisplay(); - foreach ($view->displayHandlers as $display) { + foreach ($view->executable->displayHandlers as $display) { if (isset($display) && !empty($display->definition['uses_hook_block'])) { $result = $display->executeHookBlockList(); @@ -817,7 +817,7 @@ function views_block_view($delta) { * The type of the block. If it's block it's a regular views display, * but 'special_block_-exp' exist as well. */ -function views_add_block_contextual_links(&$block, $view, $display_id, $block_type = 'block') { +function views_add_block_contextual_links(&$block, ViewExecutable $view, $display_id, $block_type = 'block') { // Do not add contextual links to an empty block. if (!empty($block['content'])) { // Contextual links only work on blocks whose content is a renderable @@ -895,14 +895,14 @@ function views_add_block_contextual_links(&$block, $view, $display_id, $block_ty * @see views_page_alter() * @see template_preprocess_views_view() */ -function views_add_contextual_links(&$render_element, $location, $view, $display_id) { +function views_add_contextual_links(&$render_element, $location, ViewExecutable $view, $display_id) { // Do not do anything if the view is configured to hide its administrative // links. if (empty($view->hide_admin_links)) { // Also do not do anything if the display plugin has not defined any // contextual links that are intended to be displayed in the requested // location. - $plugin = views_get_plugin_definition('display', $view->display[$display_id]['display_plugin']); + $plugin = views_get_plugin_definition('display', $view->displayHandlers[$display_id]->display['display_plugin']); // If contextual_links_locations are not set, provide a sane default. (To // avoid displaying any contextual links at all, a display plugin can still // set 'contextual_links_locations' to, e.g., {""}.) @@ -945,7 +945,7 @@ function views_add_contextual_links(&$render_element, $location, $view, $display $render_element['#views_contextual_links_info'][$module] = array( 'location' => $location, 'view' => $view, - 'view_name' => $view->name, + 'view_name' => $view->storage->name, 'view_display_id' => $display_id, ); } @@ -1555,8 +1555,8 @@ function views_get_applicable_views($type) { // Loop on array keys because something seems to muck with $view->display // a bit in PHP4. - foreach (array_keys($view->display) as $id) { - $plugin = views_get_plugin_definition('display', $view->display[$id]['display_plugin']); + foreach (array_keys($view->storage->display) as $id) { + $plugin = views_get_plugin_definition('display', $view->storage->display[$id]['display_plugin']); if (!empty($plugin[$type])) { // This view uses_hook_menu. Clone it so that different handlers // don't trip over each other, and add it to the list. @@ -1615,7 +1615,7 @@ function views_get_disabled_views() { * @param mixed $exclude_view * view or current display to exclude * either a - * - views object (containing $exclude_view->name and $exclude_view->current_display) + * - views object (containing $exclude_view->storage->name and $exclude_view->current_display) * - views name as string: e.g. my_view * - views name and display id (separated by ':'): e.g. my_view:default * @param bool $optgroup @@ -1648,7 +1648,7 @@ function views_get_views_as_options($views_only = FALSE, $filter = 'all', $exclu $exclude_view_display = ''; } elseif (is_object($exclude_view)) { - $exclude_view_name = $exclude_view->name; + $exclude_view_name = $exclude_view->storage->name; $exclude_view_display = $exclude_view->current_display; } else { @@ -1658,18 +1658,18 @@ function views_get_views_as_options($views_only = FALSE, $filter = 'all', $exclu $options = array(); foreach ($views as $view) { // Return only views. - if ($views_only && $view->name != $exclude_view_name) { - $options[$view->name] = $view->getHumanName(); + if ($views_only && $view->storage->name != $exclude_view_name) { + $options[$view->storage->name] = $view->getHumanName(); } // Return views with display ids. else { foreach ($view->display as $display_id => $display) { - if (!($view->name == $exclude_view_name && $display_id == $exclude_view_display)) { + if (!($view->storage->name == $exclude_view_name && $display_id == $exclude_view_display)) { if ($optgroup) { - $options[$view->name][$view->name . ':' . $display['id']] = t('@view : @display', array('@view' => $view->name, '@display' => $display['id'])); + $options[$view->storage->name][$view->storage->name . ':' . $display['id']] = t('@view : @display', array('@view' => $view->storage->name, '@display' => $display['id'])); } else { - $options[$view->name . ':' . $display['id']] = t('View: @view - Display: @display', array('@view' => $view->name, '@display' => $display['id'])); + $options[$view->storage->name . ':' . $display['id']] = t('View: @view - Display: @display', array('@view' => $view->storage->name, '@display' => $display['id'])); } } } @@ -1741,20 +1741,6 @@ function views_move_table($table) { return $table; } -/** - * Export CRUD callback to save a view. - */ -function views_save_view(&$view) { - return $view->save(); -} - -/** - * Export CRUD callback to delete a view. - */ -function views_delete_view(&$view) { - return $view->delete(TRUE); -} - // ------------------------------------------------------------------ // Views form (View with form elements) @@ -1784,7 +1770,7 @@ function views_view_has_form_elements($view) { * step (the default 'views_form_views_form' step). That way it is actually * possible for modules to have a multistep form if they need to. */ -function views_form($form, &$form_state, $view, $output) { +function views_form($form, &$form_state, ViewExecutable $view, $output) { $form_state['step'] = isset($form_state['step']) ? $form_state['step'] : 'views_form_views_form'; // Cache the built form to prevent it from being rebuilt prior to validation // and submission, which could lead to data being processed incorrectly, @@ -1809,7 +1795,7 @@ function views_form($form, &$form_state, $view, $output) { * Callback for the main step of a Views form. * Invoked by views_form(). */ -function views_form_views_form($form, &$form_state, $view, $output) { +function views_form_views_form($form, &$form_state, ViewExecutable $view, $output) { $form['#prefix'] = '
'; $form['#suffix'] = '
'; $form['#theme'] = 'views_form_views_form'; @@ -1972,7 +1958,7 @@ function views_exposed_form($form, &$form_state) { // Let form plugins know this is for exposed widgets. $form_state['exposed'] = TRUE; // Check if the form was already created - if ($cache = views_exposed_form_cache($view->name, $view->current_display)) { + if ($cache = views_exposed_form_cache($view->storage->name, $view->current_display)) { return $cache; } @@ -2006,12 +1992,12 @@ function views_exposed_form($form, &$form_state) { '#name' => '', '#type' => 'submit', '#value' => t('Apply'), - '#id' => drupal_html_id('edit-submit-' . $view->name), + '#id' => drupal_html_id('edit-submit-' . $view->storage->name), ); $form['#action'] = url($view->display_handler->getUrl()); $form['#theme'] = views_theme_functions('views_exposed_form', $view, $display); - $form['#id'] = drupal_clean_css_identifier('views_exposed_form-' . check_plain($view->name) . '-' . check_plain($display['id'])); + $form['#id'] = drupal_clean_css_identifier('views_exposed_form-' . check_plain($view->storage->name) . '-' . check_plain($display['id'])); // $form['#attributes']['class'] = array('views-exposed-form'); // If using AJAX, we need the form plugin. @@ -2023,7 +2009,7 @@ function views_exposed_form($form, &$form_state) { $exposed_form_plugin->exposed_form_alter($form, $form_state); // Save the form - views_exposed_form_cache($view->name, $view->current_display, $form); + views_exposed_form_cache($view->storage->name, $view->current_display, $form); return $form; } @@ -2111,7 +2097,7 @@ function views_exposed_form_cache($views_name, $display_name, $form_output = NUL /** * Build a list of theme function names for use most everywhere. */ -function views_theme_functions($hook, $view, $display = NULL) { +function views_theme_functions($hook, ViewExecutable $view, $display = NULL) { module_load_include('inc', 'views', 'theme/theme'); return _views_theme_functions($hook, $view, $display); } diff --git a/views.tokens.inc b/views.tokens.inc index 4858fba..f65a345 100644 --- a/views.tokens.inc +++ b/views.tokens.inc @@ -58,15 +58,15 @@ function views_tokens($type, $tokens, array $data = array(), array $options = ar foreach ($tokens as $name => $original) { switch ($name) { case 'name': - $replacements[$original] = $sanitize ? check_plain($view->human_name) : $view->human_name; + $replacements[$original] = $sanitize ? check_plain($view->storage->human_name) : $view->storage->human_name; break; case 'description': - $replacements[$original] = $sanitize ? check_plain($view->description) : $view->description; + $replacements[$original] = $sanitize ? check_plain($view->storage->description) : $view->storage->description; break; case 'machine-name': - $replacements[$original] = $view->name; + $replacements[$original] = $view->storage->name; break; case 'title': diff --git a/views_ui.module b/views_ui.module index 3622229..4a39f33 100644 --- a/views_ui.module +++ b/views_ui.module @@ -271,9 +271,9 @@ function views_ui_custom_theme() { function views_ui_edit_page_title($view) { module_load_include('inc', 'views_ui', 'includes/admin'); $bases = views_fetch_base_tables(); - $name = $view->getHumanName(); - if (isset($bases[$view->base_table])) { - $name .= ' (' . $bases[$view->base_table]['title'] . ')'; + $name = $view->storage->getHumanName(); + if (isset($bases[$view->storage->base_table])) { + $name .= ' (' . $bases[$view->storage->base_table]['title'] . ')'; } return $name; @@ -298,7 +298,7 @@ function views_ui_cache_load($name) { $view = $original_view; if (!empty($view)) { // Check to see if someone else is already editing this view. - $view->locked = $views_temp_store->isLocked($view->name); + $view->locked = $views_temp_store->isLocked($view->storage->name); // Set a flag to indicate that this view is being edited. // This flag will be used e.g. to determine whether strings // should be localized. @@ -308,7 +308,7 @@ function views_ui_cache_load($name) { else { // Keep disabled/enabled status real. if ($original_view) { - $view->disabled = $original_view->disabled; + $view->storage->disabled = $original_view->storage->disabled; } } @@ -325,7 +325,7 @@ function views_ui_cache_load($name) { * Specialized cache function to add a flag to our view, include an appropriate * include, and cache more easily. */ -function views_ui_cache_set(&$view) { +function views_ui_cache_set(ViewExecutable $view) { if (!empty($view->locked)) { drupal_set_message(t('Changes cannot be made to a locked view.'), 'error'); return; @@ -343,10 +343,8 @@ function views_ui_cache_set(&$view) { unset($view->display_handler); unset($view->default_display); $view->query = NULL; - foreach (array_keys($view->display) as $id) { - unset($view->displayHandlers[$id]); - } - views_temp_store()->set($view->name, $view); + $view->displayHandlers = array(); + views_temp_store()->set($view->storage->name, $view); } /** @@ -475,13 +473,13 @@ function views_ui_view_preview_section_handler_links($view, $type, $title = FALS $field_name = $handler->adminLabel(TRUE); $links[$type . '-edit-' . $id] = array( 'title' => t('Edit @section', array('@section' => $field_name)), - 'href' => "admin/structure/views/nojs/config-item/$view->name/{$display['id']}/$type/$id", + 'href' => "admin/structure/views/nojs/config-item/{$view->storage->name}/{$display['id']}/$type/$id", 'attributes' => array('class' => array('views-ajax-link')), ); } $links[$type . '-add'] = array( 'title' => t('Add new'), - 'href' => "admin/structure/views/nojs/add-item/$view->name/{$display['id']}/$type", + 'href' => "admin/structure/views/nojs/add-item/{$view->storage->name}/{$display['id']}/$type", 'attributes' => array('class' => array('views-ajax-link')), ); @@ -496,7 +494,7 @@ function views_ui_view_preview_section_display_category_links($view, $type, $tit $links = array( $type . '-edit' => array( 'title' => t('Edit @section', array('@section' => $title)), - 'href' => "admin/structure/views/nojs/display/$view->name/{$display['id']}/$type", + 'href' => "admin/structure/views/nojs/display/{$view->storage->name}/{$display['id']}/$type", 'attributes' => array('class' => array('views-ajax-link')), ), ); @@ -598,7 +596,7 @@ function views_ui_get_form_wizard_instance($wizard) { function views_ui_views_plugins_alter(&$plugins) { // @todo This currently does not work, investigate annotation alters. // Attach contextual links to each display plugin. The links will point to - // paths underneath "admin/structure/views/view/{$view->name}" (i.e., paths + // paths underneath "admin/structure/views/view/{$view->storage->name}" (i.e., paths // for editing and performing other contextual actions on the view). foreach ($plugins['display'] as &$display) { $display['contextual links']['views_ui'] = array( diff --git a/views_ui_listing/views_ui_listing.module b/views_ui_listing/views_ui_listing.module index 34a926f..bb08f93 100644 --- a/views_ui_listing/views_ui_listing.module +++ b/views_ui_listing/views_ui_listing.module @@ -63,7 +63,7 @@ function views_ui_listing_entity_listing_page($entity_id) { */ function views_ui_listing_ajax_callback(EntityListControllerInterface $controller, ViewExecutable $entity, $op) { // Perform the operation. - $entity->$op(); + $entity->storage->$op(); // If the request is via AJAX, return the rendered list as JSON. if (drupal_container()->get('request')->request->get('js')) {