Problem/Motivation

Currently taxonomy use custom admin/structure/taxonomy/{vocabulary}/edit path that does not allow use of add/edit names and have a test for that.
Other core parts uses admin/.../{entity_type}/manage/{id} path

Proposed resolution

Change path for all vocabularies to admin/structure/taxonomy/manage/%taxonomy_vocabulary/edit

#843162: Creating vocabularies with machine-names "List" or "Add" breaks links in taxonomy overview admin area.

Files: 
CommentFileSizeAuthor
#11 1978112-taxonomy-path-11.patch32.48 KBandypost
PASSED: [[SimpleTest]]: [MySQL] 55,408 pass(es).
[ View ]
#6 1978112-taxonomy-path-6.patch32.02 KBandypost
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 1978112-taxonomy-path-6.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]
#4 interdiff.txt7.09 KBandypost
#4 1978112-taxonomy-path-4.patch32.53 KBandypost
PASSED: [[SimpleTest]]: [MySQL] 55,328 pass(es).
[ View ]
#2 interdiff.txt635 bytesandypost
#2 1978112-taxonomy-path-2.patch25.14 KBandypost
FAILED: [[SimpleTest]]: [MySQL] 55,832 pass(es), 28 fail(s), and 0 exception(s).
[ View ]
#1 1978112-taxonomy-path-1.patch25.08 KBandypost
FAILED: [[SimpleTest]]: [MySQL] Setup environment: Test cancelled by admin prior to completion.
[ View ]

Comments

Status:Active» Needs review
StatusFileSize
new25.08 KB
FAILED: [[SimpleTest]]: [MySQL] Setup environment: Test cancelled by admin prior to completion.
[ View ]

Trying to keep it simple

StatusFileSize
new25.14 KB
FAILED: [[SimpleTest]]: [MySQL] 55,832 pass(es), 28 fail(s), and 0 exception(s).
[ View ]
new635 bytes

missed hunk

Status:Needs review» Needs work

The last submitted patch, 1978112-taxonomy-path-2.patch, failed testing.

Status:Needs work» Needs review
StatusFileSize
new32.53 KB
PASSED: [[SimpleTest]]: [MySQL] 55,328 pass(es).
[ View ]
new7.09 KB

Fix the rest places

Status:Needs review» Needs work

+++ b/core/modules/shortcut/shortcut.api.phpundefined
@@ -33,7 +33,7 @@
-    return 'admin-shortcuts';

unrelated?

Apart from that looks good.

Status:Needs work» Needs review
StatusFileSize
new32.02 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 1978112-taxonomy-path-6.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

#6: 1978112-taxonomy-path-6.patch queued for re-testing.

Status:Needs review» Needs work

consistency++
but needs a reroll..otherwise looks good

Status:Needs review» Needs work

The last submitted patch, 1978112-taxonomy-path-6.patch, failed testing.

Status:Needs work» Needs review
StatusFileSize
new32.48 KB
PASSED: [[SimpleTest]]: [MySQL] 55,408 pass(es).
[ View ]

Once other one needs work let's normalize path

Status:Needs review» Reviewed & tested by the community

+++ b/core/modules/taxonomy/lib/Drupal/taxonomy/VocabularyFormController.phpundefined
@@ -108,26 +108,6 @@ protected function actions(array $form, array &$form_state) {
   /**
-   * Overrides Drupal\Core\Entity\EntityFormController::validate().
-   */
-  public function validate(array $form, array &$form_state) {
-    parent::validate($form, $form_state);
-
-    // Make sure that the machine name of the vocabulary is not in the
-    // disallowed list (names that conflict with menu items, such as 'list'
-    // and 'add').
-    // During the deletion there is no 'vid' key.
-    if (isset($form_state['values']['vid'])) {
-      // Do not allow machine names to conflict with taxonomy path arguments.
-      $vid = $form_state['values']['vid'];
-      $disallowed = array('add', 'list');
-      if (in_array($vid, $disallowed)) {
-        form_set_error('vid', t('The machine-readable name cannot be "add" or "list".'));
-      }
-    }
-  }

Awesome! I went through the changes and every single line looked perfect!

Title:Convert taxonomy admin path to follow other coreChange notice: Convert taxonomy admin path to follow other core entity patterns
Priority:Normal» Critical
Status:Reviewed & tested by the community» Active
Issue tags:+Needs change record

I was a bit alarmed to see a big hunk of validation logic removed with no replacement until I realized how totally effing janky said validation is and I'm really glad we don't need it anymore. ;)

Committed and pushed to 8.x. Thanks!

Wouldn't hurt to have a change notice for this, for modules whose documentation or whatnot might be pointing to the old URLs.

Assigned:Unassigned» LittleCoding

Write a API change notification

Status:Active» Needs review

Summary

The Drupal taxonomy admin path was converted to follow other core entity admin patterns with the addition of "manage" to the admin path.

Drupal 7

admin/structure/taxonomy/%taxonomy_vocabulary/edit

Drupal 8

admin/structure/taxonomy/manage/%taxonomy_vocabulary/edit

A change record node has been created

http://drupal.org/node/1986784

Assigned:LittleCoding» Unassigned
Priority:Critical» Normal

Changes record looks good

Title:Change notice: Convert taxonomy admin path to follow other core entity patternsConvert taxonomy admin path to follow other core entity patterns
Status:Needs review» Fixed
Issue tags:-Needs change record

This is fixed now, reverting tags and title. Thanks @LittleCoding!

Status:Fixed» Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.