diff --git a/core/modules/taxonomy/lib/Drupal/taxonomy/Form/OverviewTerms.php b/core/modules/taxonomy/lib/Drupal/taxonomy/Form/OverviewTerms.php
index 2f737ff..8666334 100644
--- a/core/modules/taxonomy/lib/Drupal/taxonomy/Form/OverviewTerms.php
+++ b/core/modules/taxonomy/lib/Drupal/taxonomy/Form/OverviewTerms.php
@@ -12,7 +12,7 @@
use Drupal\Core\Entity\EntityManager;
use Drupal\Core\Config\ConfigFactory;
use Drupal\Core\Database\Connection;
-use Drupal\Core\Extension\ModuleHandler;
+use Drupal\Core\Extension\ModuleHandlerInterface;
use Drupal\taxonomy\Plugin\Core\Entity\Vocabulary;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\DependencyInjection\ContainerInterface;
@@ -58,10 +58,10 @@ class OverviewTerms implements ControllerInterface, FormInterface {
* The current database connection.
* @param \Symfony\Component\HttpFoundation\Request $request
* The current request.
- * @param \Drupal\Core\Extension\ModuleHandler $module_handler
+ * @param \Drupal\Core\Extension\ModuleHandlerInterface $module_handler
* The module handler service.
*/
- public function __construct(EntityManager $entity_manager, ConfigFactory $config_factory, Connection $connection, Request $request, ModuleHandler $module_handler) {
+ public function __construct(EntityManager $entity_manager, ConfigFactory $config_factory, Connection $connection, Request $request, ModuleHandlerInterface $module_handler) {
$this->entityManager = $entity_manager;
$this->config = $config_factory->get('taxonomy.settings');
$this->database = $connection;
@@ -70,7 +70,7 @@ public function __construct(EntityManager $entity_manager, ConfigFactory $config
}
/**
- * Injects plugin manager.
+ * {@inheritdoc}
*/
public static function create(ContainerInterface $container) {
return new static(
@@ -108,22 +108,22 @@ public function buildForm(array $form, array &$form_state, Vocabulary $taxonomy_
$form_state['taxonomy']['vocabulary'] = $taxonomy_vocabulary;
$parent_fields = FALSE;
- $page = isset($_GET['page']) ? $_GET['page'] : 0;
+ ($page = $this->request->query->get('page')) ?: 0;
// Number of terms per page.
- $page_increment = $this->config->get('terms_per_page_admin');
+ $page_increment = $this->config->get('terms_per_page_admin');
// Elements shown on this page.
- $page_entries = 0;
+ $page_entries = 0;
// Elements at the root level before this page.
- $before_entries = 0;
+ $before_entries = 0;
// Elements at the root level after this page.
- $after_entries = 0;
+ $after_entries = 0;
// Elements at the root level on this page.
- $root_entries = 0;
+ $root_entries = 0;
// Terms from previous and next pages are shown if the term tree would have
- // been cut in the middle. Keep track of how many extra terms we show on each
- // page of terms.
- $back_step = NULL;
+ // been cut in the middle. Keep track of how many extra terms we show on
+ // each page of terms.
+ $back_step = NULL;
$forward_step = 0;
// An array of the terms to be displayed on this page.
@@ -178,14 +178,15 @@ public function buildForm(array $form, array &$form_state, Vocabulary $taxonomy_
$forward_step++;
}
- // Finally, if we've gotten down this far, we're rendering a term on this page.
+ // Finally, if we've gotten down this far, we're rendering a term on this
+ // page.
$page_entries++;
- $term_deltas[$term->tid] = isset($term_deltas[$term->tid]) ? $term_deltas[$term->tid] + 1 : 0;
- $key = 'tid:' . $term->tid . ':' . $term_deltas[$term->tid];
+ $term_deltas[$term->id()] = isset($term_deltas[$term->id()]) ? $term_deltas[$term->id()] + 1 : 0;
+ $key = 'tid:' . $term->id() . ':' . $term_deltas[$term->id()];
// Keep track of the first term displayed on this page.
if ($page_entries == 1) {
- $form['#first_tid'] = $term->tid;
+ $form['#first_tid'] = $term->id();
}
// Keep a variable to make sure at least 2 root elements are displayed.
if ($term->parents[0] == 0) {
@@ -201,7 +202,8 @@ public function buildForm(array $form, array &$form_state, Vocabulary $taxonomy_
$pager_total[0] = ceil($total_entries / $page_increment);
// If this form was already submitted once, it's probably hit a validation
- // error. Ensure the form is rebuilt in the same order as the user submitted.
+ // error. Ensure the form is rebuilt in the same order as the user
+ // submitted.
if (!empty($form_state['input'])) {
// Get the $_POST order.
$order = array_flip(array_keys($form_state['input']['terms']));
@@ -226,31 +228,24 @@ public function buildForm(array $form, array &$form_state, Vocabulary $taxonomy_
$form['terms'] = array(
'#type' => 'table',
'#header' => array(t('Name'), t('Weight'), t('Operations')),
- '#empty' => t('No terms available. Add term.', array('@link' => url('admin/structure/taxonomy/' . $taxonomy_vocabulary->id() . '/add'))),
+ '#empty' => t('No terms available. Add term.', array('@link' => url('admin/structure/taxonomy/manage/' . $taxonomy_vocabulary->id() . '/add'))),
'#attributes' => array(
'id' => 'taxonomy',
),
);
foreach ($current_page as $key => $term) {
- // Save the term for the current page so we don't have to load it a second
- // time.
- $form['terms'][$key]['#term'] = (array) $term;
- if (isset($term->parents)) {
- $form['terms'][$key]['#term']['parent'] = $term->parent = $term->parents[0];
- unset($form['terms'][$key]['#term']['parents'], $term->parents);
- }
-
+ $form['terms'][$key]['#term'] = $term;
$form['terms'][$key]['term'] = array(
- '#prefix' => isset($term->depth) && $term->depth > 0 ? theme('indentation', array('size' => $term->depth)) : '',
+ '#prefix' => isset($term->depth->value) && $term->depth->value > 0 ? theme('indentation', array('size' => $term->depth->value)) : '',
'#type' => 'link',
'#title' => $term->label(),
- '#href' => "taxonomy/term/$term->tid",
+ '#href' => "taxonomy/term/$term->id()",
);
if ($taxonomy_vocabulary->hierarchy != TAXONOMY_HIERARCHY_MULTIPLE && count($tree) > 1) {
$parent_fields = TRUE;
$form['terms'][$key]['term']['tid'] = array(
'#type' => 'hidden',
- '#value' => $term->tid,
+ '#value' => $term->id(),
'#attributes' => array(
'class' => array('term-id'),
),
@@ -259,7 +254,7 @@ public function buildForm(array $form, array &$form_state, Vocabulary $taxonomy_
'#type' => 'hidden',
// Yes, default_value on a hidden. It needs to be changeable by the
// javascript.
- '#default_value' => $term->parent,
+ '#default_value' => $term->parent->value,
'#attributes' => array(
'class' => array('term-parent'),
),
@@ -279,7 +274,7 @@ public function buildForm(array $form, array &$form_state, Vocabulary $taxonomy_
'#delta' => $delta,
'#title_display' => 'invisible',
'#title' => t('Weight for added term'),
- '#default_value' => $term->weight,
+ '#default_value' => $term->weight->value,
'#attributes' => array(
'class' => array('term-weight'),
),
@@ -287,19 +282,19 @@ public function buildForm(array $form, array &$form_state, Vocabulary $taxonomy_
$operations = array(
'edit' => array(
'title' => t('edit'),
- 'href' => 'taxonomy/term/' . $term->tid . '/edit',
+ 'href' => 'taxonomy/term/' . $term->id() . '/edit',
'query' => $destination,
),
'delete' => array(
'title' => t('delete'),
- 'href' => 'taxonomy/term/' . $term->tid . '/delete',
+ 'href' => 'taxonomy/term/' . $term->id() . '/delete',
'query' => $destination,
),
);
- if (module_invoke('translation_entity', 'translate_access', $term)) {
+ if ($this->moduleHandler->invoke('translation_entity', 'translate_access', array($term))) {
$operations['translate'] = array(
'title' => t('translate'),
- 'href' => 'taxonomy/term/' . $term->tid . '/translations',
+ 'href' => 'taxonomy/term/' . $term->id() . '/translations',
'query' => $destination,
);
}
@@ -343,7 +338,7 @@ public function buildForm(array $form, array &$form_state, Vocabulary $taxonomy_
'term-parent',
'term-parent',
'term-id',
- FALSE
+ FALSE,
);
$form['terms']['#tabledrag'][] = array(
'depth',
@@ -372,7 +367,7 @@ public function buildForm(array $form, array &$form_state, Vocabulary $taxonomy_
'#type' => 'submit',
'#value' => t('Reset to alphabetical')
);
- $form_state['redirect'] = array($this->request->attributes->get('system_path'), (isset($_GET['page']) ? array('query' => array('page' => $_GET['page'])) : array()));
+ $form_state['redirect'] = array(current_path(), ($this->request->query->get('page') ? array('query' => array('page' => $this->request->query->get('page'))) : array()));
}
return $form;
@@ -387,12 +382,11 @@ public function validateForm(array &$form, array &$form_state) {}
* {@inheritdoc}
*/
public function submitForm(array &$form, array &$form_state) {
- // @todo this needs to be in its own method.
if ($form_state['triggering_element']['#value'] == t('Reset to alphabetical')) {
// Execute the reset action.
+ // @todo this needs to be a separate method.
+ // Will be fixed in http://drupal.org/node/1976296.
if ($form_state['values']['reset_alphabetical'] === TRUE) {
- // @todo this needs to be a method on the vocabulary storage controller.
- // Will be fixed in http://drupal.org/node/1976296.
$this->moduleHandler->loadInclude('taxonomy', 'admin.inc');
return taxonomy_vocabulary_confirm_reset_alphabetical_submit($form, $form_state);
}
@@ -410,9 +404,9 @@ public function submitForm(array &$form, array &$form_state) {
$hierarchy = TAXONOMY_HIERARCHY_DISABLED;
$changed_terms = array();
- // @todo convert taxonomy_get_tree() to a service and inject it.
+ // @todo taxonomy_get_tree needs to be converted to a service and injected.
// Will be fixed in http://drupal.org/node/1976298.
- $tree = taxonomy_get_tree($vocabulary->id());
+ $tree = taxonomy_get_tree($vocabulary->id(), 0, NULL, TRUE);
if (empty($tree)) {
return;
@@ -420,16 +414,15 @@ public function submitForm(array &$form, array &$form_state) {
// Build a list of all terms that need to be updated on previous pages.
$weight = 0;
- $term = (array) $tree[0];
- while ($term['tid'] != $form['#first_tid']) {
- if ($term['parents'][0] == 0 && $term['weight'] != $weight) {
- $term['parent'] = $term['parents'][0];
- $term['weight'] = $weight;
- $changed_terms[$term['tid']] = $term;
+ $term = $tree[0];
+ while ($term->id() != $form['#first_tid']) {
+ if ($term->parent->value == 0 && $term->weight->value != $weight) {
+ $term->weight->value = $weight;
+ $changed_terms[$term->id()] = $term;
}
$weight++;
- $hierarchy = $term['parents'][0] != 0 ? TAXONOMY_HIERARCHY_SINGLE : $hierarchy;
- $term = (array) $tree[$weight];
+ $hierarchy = $term->parent->value != 0 ? TAXONOMY_HIERARCHY_SINGLE : $hierarchy;
+ $term = $tree[$weight];
}
// Renumber the current page weights and assign any new parents.
@@ -438,53 +431,42 @@ public function submitForm(array &$form, array &$form_state) {
if (isset($form['terms'][$tid]['#term'])) {
$term = $form['terms'][$tid]['#term'];
// Give terms at the root level a weight in sequence with terms on previous pages.
- if ($values['term']['parent'] == 0 && $term['weight'] != $weight) {
- $term['weight'] = $weight;
- $changed_terms[$term['tid']] = $term;
+ if ($values['term']['parent'] == 0 && $term->weight->value != $weight) {
+ $term->weight->value = $weight;
+ $changed_terms[$term->id()] = $term;
}
// Terms not at the root level can safely start from 0 because they're all on this page.
elseif ($values['term']['parent'] > 0) {
- $level_weights[$values['term']['parent']] = isset($level_weights[$values['term']['parent']]) ? $level_weights[$values['term']['parent']] + 1 : 0;
- if ($level_weights[$values['term']['parent']] != $term['weight']) {
- $term['weight'] = $level_weights[$values['term']['parent']];
- $changed_terms[$term['tid']] = $term;
+ $level_weights[$values['term']['parent']] = isset($level_weights[$values['term']['parent']]) ? $level_weights[$values['term']->parent->value] + 1 : 0;
+ if ($level_weights[$values['term']['parent']] != $term->weight->value) {
+ $term->weight->value = $level_weights[$values['term']['parent']];
+ $changed_terms[$term->id()] = $term;
}
}
// Update any changed parents.
- if ($values['term']['parent'] != $term['parent']) {
- $term['parent'] = $values['term']['parent'];
- $changed_terms[$term['tid']] = $term;
+ if ($values['term']['parent'] != $term->parent->value) {
+ $term->parent->value = $values['term']['parent'];
+ $changed_terms[$term->id()] = $term;
}
- $hierarchy = $term['parent'] != 0 ? TAXONOMY_HIERARCHY_SINGLE : $hierarchy;
+ $hierarchy = $term->parent->value != 0 ? TAXONOMY_HIERARCHY_SINGLE : $hierarchy;
$weight++;
}
}
// Build a list of all terms that need to be updated on following pages.
for ($weight; $weight < count($tree); $weight++) {
- $term = (array) $tree[$weight];
- if ($term['parents'][0] == 0 && $term['weight'] != $weight) {
- $term['parent'] = $term['parents'][0];
- $term['weight'] = $weight;
- $changed_terms[$term['tid']] = $term;
+ $term = $tree[$weight];
+ if ($term->parent->value == 0 && $term->weight->value != $weight) {
+ $term->parent->value = $term->parent->value;
+ $term->weight->value = $weight;
+ $changed_terms[$term->id()] = $term;
}
- $hierarchy = $term['parents'][0] != 0 ? TAXONOMY_HIERARCHY_SINGLE : $hierarchy;
+ $hierarchy = $term->parent->value != 0 ? TAXONOMY_HIERARCHY_SINGLE : $hierarchy;
}
// Save all updated terms.
- foreach ($changed_terms as $changed) {
- $term = (object) $changed;
- // Update term_hierachy and term_data directly since we don't have a
- // fully populated term object to save.
- $this->database->update('taxonomy_term_hierarchy')
- ->fields(array('parent' => $term->parent))
- ->condition('tid', $term->tid, '=')
- ->execute();
-
- $this->database->update('taxonomy_term_data')
- ->fields(array('weight' => $term->weight))
- ->condition('tid', $term->tid, '=')
- ->execute();
+ foreach ($changed_terms as $term) {
+ $term->save();
}
// Update the vocabulary hierarchy to flat or single hierarchy.
diff --git a/core/modules/taxonomy/taxonomy.admin.inc b/core/modules/taxonomy/taxonomy.admin.inc
index 0703071..ecbbd5d 100644
--- a/core/modules/taxonomy/taxonomy.admin.inc
+++ b/core/modules/taxonomy/taxonomy.admin.inc
@@ -103,376 +103,6 @@ function taxonomy_vocabulary_add() {
}
/**
- * Form builder for the taxonomy terms overview.
- *
- * Display a tree of all the terms in a vocabulary, with options to edit
- * each one. The form is made drag and drop by the theme function.
- *
- * @param Drupal\taxonomy\Plugin\Core\Entity\Vocabulary $vocabulary
- * The taxonomy vocabulary entity to list terms for.
- *
- * @ingroup forms
- * @see taxonomy_overview_terms_submit()
- * @see theme_taxonomy_overview_terms()
- */
-function taxonomy_overview_terms($form, &$form_state, Vocabulary $vocabulary) {
- global $pager_page_array, $pager_total, $pager_total_items;
-
- // Check for confirmation forms.
- if (isset($form_state['confirm_reset_alphabetical'])) {
- return taxonomy_vocabulary_confirm_reset_alphabetical($form, $form_state, $vocabulary->id());
- }
-
- $form_state['taxonomy']['vocabulary'] = $vocabulary;
- $parent_fields = FALSE;
-
- $page = isset($_GET['page']) ? $_GET['page'] : 0;
- $page_increment = config('taxonomy.settings')->get('terms_per_page_admin'); // Number of terms per page.
- $page_entries = 0; // Elements shown on this page.
- $before_entries = 0; // Elements at the root level before this page.
- $after_entries = 0; // Elements at the root level after this page.
- $root_entries = 0; // Elements at the root level on this page.
-
- // Terms from previous and next pages are shown if the term tree would have
- // been cut in the middle. Keep track of how many extra terms we show on each
- // page of terms.
- $back_step = NULL;
- $forward_step = 0;
-
- // An array of the terms to be displayed on this page.
- $current_page = array();
-
- $delta = 0;
- $term_deltas = array();
- $tree = taxonomy_get_tree($vocabulary->id(), 0, NULL, TRUE);
- $term = current($tree);
- do {
- // In case this tree is completely empty.
- if (empty($term)) {
- break;
- }
- $delta++;
- // Count entries before the current page.
- if ($page && ($page * $page_increment) > $before_entries && !isset($back_step)) {
- $before_entries++;
- continue;
- }
- // Count entries after the current page.
- elseif ($page_entries > $page_increment && isset($complete_tree)) {
- $after_entries++;
- continue;
- }
-
- // Do not let a term start the page that is not at the root.
- if (isset($term->depth) && ($term->depth > 0) && !isset($back_step)) {
- $back_step = 0;
- while ($pterm = prev($tree)) {
- $before_entries--;
- $back_step++;
- if ($pterm->depth == 0) {
- prev($tree);
- continue 2; // Jump back to the start of the root level parent.
- }
- }
- }
- $back_step = isset($back_step) ? $back_step : 0;
-
- // Continue rendering the tree until we reach the a new root item.
- if ($page_entries >= $page_increment + $back_step + 1 && $term->depth == 0 && $root_entries > 1) {
- $complete_tree = TRUE;
- // This new item at the root level is the first item on the next page.
- $after_entries++;
- continue;
- }
- if ($page_entries >= $page_increment + $back_step) {
- $forward_step++;
- }
-
- // Finally, if we've gotten down this far, we're rendering a term on this page.
- $page_entries++;
- $term_deltas[$term->id()] = isset($term_deltas[$term->id()]) ? $term_deltas[$term->id()] + 1 : 0;
- $key = 'tid:' . $term->id() . ':' . $term_deltas[$term->id()];
-
- // Keep track of the first term displayed on this page.
- if ($page_entries == 1) {
- $form['#first_tid'] = $term->id();
- }
- // Keep a variable to make sure at least 2 root elements are displayed.
- if ($term->parents[0] == 0) {
- $root_entries++;
- }
- $current_page[$key] = $term;
- } while ($term = next($tree));
-
- // Because we didn't use a pager query, set the necessary pager variables.
- $total_entries = $before_entries + $page_entries + $after_entries;
- $pager_total_items[0] = $total_entries;
- $pager_page_array[0] = $page;
- $pager_total[0] = ceil($total_entries / $page_increment);
-
- // If this form was already submitted once, it's probably hit a validation
- // error. Ensure the form is rebuilt in the same order as the user submitted.
- if (!empty($form_state['input'])) {
- $order = array_flip(array_keys($form_state['input']['terms'])); // Get the $_POST order.
- $current_page = array_merge($order, $current_page); // Update our form with the new order.
- foreach ($current_page as $key => $term) {
- // Verify this is a term for the current page and set at the current
- // depth.
- if (is_array($form_state['input']['terms'][$key]) && is_numeric($form_state['input']['terms'][$key]['term']['tid'])) {
- $current_page[$key]->depth = $form_state['input']['terms'][$key]['term']['depth'];
- }
- else {
- unset($current_page[$key]);
- }
- }
- }
-
- $errors = form_get_errors() != FALSE ? form_get_errors() : array();
- $destination = drupal_get_destination();
- $row_position = 0;
- // Build the actual form.
- $form['terms'] = array(
- '#type' => 'table',
- '#header' => array(t('Name'), t('Weight'), t('Operations')),
- '#empty' => t('No terms available. Add term.', array('@link' => url('admin/structure/taxonomy/manage/' . $vocabulary->id() . '/add'))),
- '#attributes' => array(
- 'id' => 'taxonomy',
- ),
- );
- foreach ($current_page as $key => $term) {
- $form['terms'][$key]['#term'] = $term;
- $form['terms'][$key]['term'] = array(
- '#prefix' => isset($term->depth->value) && $term->depth->value > 0 ? theme('indentation', array('size' => $term->depth->value)) : '',
- '#type' => 'link',
- '#title' => $term->label(),
- '#href' => "taxonomy/term/$term->id()",
- );
- if ($vocabulary->hierarchy != TAXONOMY_HIERARCHY_MULTIPLE && count($tree) > 1) {
- $parent_fields = TRUE;
- $form['terms'][$key]['term']['tid'] = array(
- '#type' => 'hidden',
- '#value' => $term->id(),
- '#attributes' => array(
- 'class' => array('term-id'),
- ),
- );
- $form['terms'][$key]['term']['parent'] = array(
- '#type' => 'hidden',
- // Yes, default_value on a hidden. It needs to be changeable by the
- // javascript.
- '#default_value' => $term->parent->value,
- '#attributes' => array(
- 'class' => array('term-parent'),
- ),
- );
- $form['terms'][$key]['term']['depth'] = array(
- '#type' => 'hidden',
- // Same as above, the depth is modified by javascript, so it's a
- // default_value.
- '#default_value' => $term->depth,
- '#attributes' => array(
- 'class' => array('term-depth'),
- ),
- );
- }
- $form['terms'][$key]['weight'] = array(
- '#type' => 'weight',
- '#delta' => $delta,
- '#title_display' => 'invisible',
- '#title' => t('Weight for added term'),
- '#default_value' => $term->weight->value,
- '#attributes' => array(
- 'class' => array('term-weight'),
- ),
- );
- $operations = array(
- 'edit' => array(
- 'title' => t('edit'),
- 'href' => 'taxonomy/term/' . $term->id() . '/edit',
- 'query' => $destination,
- ),
- 'delete' => array(
- 'title' => t('delete'),
- 'href' => 'taxonomy/term/' . $term->id() . '/delete',
- 'query' => $destination,
- ),
- );
- if (module_invoke('translation_entity', 'translate_access', $term)) {
- $operations['translate'] = array(
- 'title' => t('translate'),
- 'href' => 'taxonomy/term/' . $term->id() . '/translations',
- 'query' => $destination,
- );
- }
- $form['terms'][$key]['operations'] = array(
- '#type' => 'operations',
- '#links' => $operations,
- );
-
- $form['terms'][$key]['#attributes']['class'] = array();
- if ($parent_fields) {
- $form['terms'][$key]['#attributes']['class'][] = 'draggable';
- }
-
- // Add classes that mark which terms belong to previous and next pages.
- if ($row_position < $back_step || $row_position >= $page_entries - $forward_step) {
- $form['terms'][$key]['#attributes']['class'][] = 'taxonomy-term-preview';
- }
-
- if ($row_position !== 0 && $row_position !== count($tree) - 1) {
- if ($row_position == $back_step - 1 || $row_position == $page_entries - $forward_step - 1) {
- $form['terms'][$key]['#attributes']['class'][] = 'taxonomy-term-divider-top';
- }
- elseif ($row_position == $back_step || $row_position == $page_entries - $forward_step) {
- $form['terms'][$key]['#attributes']['class'][] = 'taxonomy-term-divider-bottom';
- }
- }
-
- // Add an error class if this row contains a form error.
- foreach ($errors as $error_key => $error) {
- if (strpos($error_key, $key) === 0) {
- $form['terms'][$key]['#attributes']['class'][] = 'error';
- }
- }
- $row_position++;
- }
-
-
- if ($parent_fields) {
- $form['terms']['#tabledrag'][] = array('match', 'parent', 'term-parent', 'term-parent', 'term-id', FALSE);
- $form['terms']['#tabledrag'][] = array('depth', 'group', 'term-depth', NULL, NULL, FALSE);
- $form['terms']['#attached']['library'][] = array('taxonomy', 'drupal.taxonomy');
- $form['terms']['#attached']['js'][] = array(
- 'data' => array('taxonomy' => array('backStep' => $back_step, 'forwardStep' => $forward_step)),
- 'type' => 'setting',
- );
- }
- $form['terms']['#tabledrag'][] = array('order', 'sibling', 'term-weight');
-
- if ($vocabulary->hierarchy != TAXONOMY_HIERARCHY_MULTIPLE && count($tree) > 1) {
- $form['actions'] = array('#type' => 'actions', '#tree' => FALSE);
- $form['actions']['submit'] = array(
- '#type' => 'submit',
- '#value' => t('Save'),
- '#button_type' => 'primary',
- );
- $form['actions']['reset_alphabetical'] = array(
- '#type' => 'submit',
- '#value' => t('Reset to alphabetical')
- );
- $form_state['redirect'] = array(current_path(), (isset($_GET['page']) ? array('query' => array('page' => $_GET['page'])) : array()));
- }
-
- return $form;
-}
-
-/**
- * Submit handler for terms overview form.
- *
- * Rather than using a textfield or weight field, this form depends entirely
- * upon the order of form elements on the page to determine new weights.
- *
- * Because there might be hundreds or thousands of taxonomy terms that need to
- * be ordered, terms are weighted from 0 to the number of terms in the
- * vocabulary, rather than the standard -10 to 10 scale. Numbers are sorted
- * lowest to highest, but are not necessarily sequential. Numbers may be skipped
- * when a term has children so that reordering is minimal when a child is
- * added or removed from a term.
- *
- * @see taxonomy_overview_terms()
- */
-function taxonomy_overview_terms_submit($form, &$form_state) {
- if ($form_state['triggering_element']['#value'] == t('Reset to alphabetical')) {
- // Execute the reset action.
- if ($form_state['values']['reset_alphabetical'] === TRUE) {
- return taxonomy_vocabulary_confirm_reset_alphabetical_submit($form, $form_state);
- }
- // Rebuild the form to confirm the reset action.
- $form_state['rebuild'] = TRUE;
- $form_state['confirm_reset_alphabetical'] = TRUE;
- return;
- }
-
- // Sort term order based on weight.
- uasort($form_state['values']['terms'], 'drupal_sort_weight');
-
- $vocabulary = $form_state['taxonomy']['vocabulary'];
- // Update the current hierarchy type as we go.
- $hierarchy = TAXONOMY_HIERARCHY_DISABLED;
-
- $changed_terms = array();
- $tree = taxonomy_get_tree($vocabulary->id(), 0, NULL, TRUE);
-
- if (empty($tree)) {
- return;
- }
-
- // Build a list of all terms that need to be updated on previous pages.
- $weight = 0;
- $term = $tree[0];
- while ($term->id() != $form['#first_tid']) {
- if ($term->parent->value == 0 && $term->weight->value != $weight) {
- $term->weight->value = $weight;
- $changed_terms[$term->id()] = $term;
- }
- $weight++;
- $hierarchy = $term->parent->value != 0 ? TAXONOMY_HIERARCHY_SINGLE : $hierarchy;
- $term = $tree[$weight];
- }
-
- // Renumber the current page weights and assign any new parents.
- $level_weights = array();
- foreach ($form_state['values']['terms'] as $tid => $values) {
- if (isset($form['terms'][$tid]['#term'])) {
- $term = $form['terms'][$tid]['#term'];
- // Give terms at the root level a weight in sequence with terms on previous pages.
- if ($values['term']['parent'] == 0 && $term->weight->value != $weight) {
- $term->weight->value = $weight;
- $changed_terms[$term->id()] = $term;
- }
- // Terms not at the root level can safely start from 0 because they're all on this page.
- elseif ($values['term']['parent'] > 0) {
- $level_weights[$values['term']['parent']] = isset($level_weights[$values['term']['parent']]) ? $level_weights[$values['term']->parent->value] + 1 : 0;
- if ($level_weights[$values['term']['parent']] != $term->weight->value) {
- $term->weight->value = $level_weights[$values['term']['parent']];
- $changed_terms[$term->id()] = $term;
- }
- }
- // Update any changed parents.
- if ($values['term']['parent'] != $term->parent->value) {
- $term->parent->value = $values['term']['parent'];
- $changed_terms[$term->id()] = $term;
- }
- $hierarchy = $term->parent->value != 0 ? TAXONOMY_HIERARCHY_SINGLE : $hierarchy;
- $weight++;
- }
- }
-
- // Build a list of all terms that need to be updated on following pages.
- for ($weight; $weight < count($tree); $weight++) {
- $term = $tree[$weight];
- if ($term->parent->value == 0 && $term->weight->value != $weight) {
- $term->parent->value = $term->parent->value;
- $term->weight->value = $weight;
- $changed_terms[$term->id()] = $term;
- }
- $hierarchy = $term->parent->value != 0 ? TAXONOMY_HIERARCHY_SINGLE : $hierarchy;
- }
-
- // Save all updated terms.
- foreach ($changed_terms as $term) {
- $term->save();
- }
-
- // Update the vocabulary hierarchy to flat or single hierarchy.
- if ($vocabulary->hierarchy != $hierarchy) {
- $vocabulary->hierarchy = $hierarchy;
- $vocabulary->save();
- }
- drupal_set_message(t('The configuration options have been saved.'));
-}
-
-/**
* Returns a rendered edit form to create a new term associated to the given vocabulary.
*/
function taxonomy_term_add($vocabulary) {
diff --git a/core/modules/taxonomy/taxonomy.routing.yml b/core/modules/taxonomy/taxonomy.routing.yml
index f886eb3..454de05 100644
--- a/core/modules/taxonomy/taxonomy.routing.yml
+++ b/core/modules/taxonomy/taxonomy.routing.yml
@@ -1,8 +1,6 @@
taxonomy_overview_terms:
- pattern: admin/structure/taxonomy/{taxonomy_vocabulary}
+ pattern: 'admin/structure/taxonomy/manage/{taxonomy_vocabulary}'
defaults:
_form: 'Drupal\taxonomy\Form\OverviewTerms'
requirements:
_entity_access: 'taxonomy_vocabulary.view'
- # Don't allow add or list.
- taxonomy_vocabulary: "^(?:(?!add|list).)*"