Issue #2897929 by bojanz: Move Stores to admin/commerce/config

Author: bojanz

modules/price/commerce_price.links.menu.yml

@@ -3,3 +3,4 @@ entity.commerce_currency.collection:
   route_name: entity.commerce_currency.collection
   parent: 'commerce.store_configuration'
   description: 'Manage your currencies.'
+  weight: -20

modules/store/commerce_store.links.menu.yml

@@ -1,11 +1,13 @@
 entity.commerce_store.collection:
   title: 'Stores'
   route_name: 'entity.commerce_store.collection'
-  parent: 'commerce.admin_commerce'
+  parent: 'commerce.store_configuration'
   description: 'Manage your stores.'
+  weight: -10
 
 entity.commerce_store_type.collection:
   title: 'Store types'
   route_name: 'entity.commerce_store_type.collection'
   parent: 'commerce.store_configuration'
   description: 'Manage fields, form and display settings for your stores.'
+  weight: -5

modules/store/commerce_store.post_update.php

@@ -36,3 +36,35 @@ function commerce_store_post_update_1() {
 
   return $message;
 }
+
+/**
+ * Revert the Stores view.
+ */
+function commerce_store_post_update_2() {
+  /** @var \Drupal\commerce\Config\ConfigUpdaterInterface $config_updater */
+  $config_updater = \Drupal::service('commerce.config_updater');
+
+  $views = [
+    'views.view.commerce_stores',
+  ];
+  $result = $config_updater->revert($views, FALSE);
+
+  $success_results = $result->getSucceeded();
+  $failure_results = $result->getFailed();
+  $message = '';
+  if ($success_results) {
+    $message = t('Succeeded:') . '<br>';
+    foreach ($success_results as $success_message) {
+      $message .= $success_message . '<br>';
+    }
+    $message .= '<br>';
+  }
+  if ($failure_results) {
+    $message .= t('Failed:') . '<br>';
+    foreach ($failure_results as $failure_message) {
+      $message .= $failure_message . '<br>';
+    }
+  }
+
+  return $message;
+}

modules/store/config/install/views.view.commerce_stores.yml

@@ -422,7 +422,7 @@ display:
     position: 1
     display_options:
       display_extenders: {  }
-      path: admin/commerce/stores
+      path: admin/commerce/config/stores
     cache_metadata:
       contexts:
         - 'languages:language_content'

modules/store/src/Entity/Store.php

@@ -63,8 +63,8 @@
  *     "add-form" = "/store/add/{commerce_store_type}",
  *     "edit-form" = "/store/{commerce_store}/edit",
  *     "delete-form" = "/store/{commerce_store}/delete",
- *     "delete-multiple-form" = "/admin/commerce/stores/delete",
- *     "collection" = "/admin/commerce/stores",
+ *     "delete-multiple-form" = "/admin/commerce/config/stores/delete",
+ *     "collection" = "/admin/commerce/config/stores",
  *   },
  *   bundle_entity_type = "commerce_store_type",
  *   field_ui_base_route = "entity.commerce_store_type.edit_form",

modules/store/tests/src/FunctionalJavascript/StoreTest.php

@@ -35,7 +35,7 @@ protected function getAdministratorPermissions() {
    * Tests creating a store.
    */
   public function testCreateStore() {
-    $this->drupalGet('admin/commerce/stores');
+    $this->drupalGet('admin/commerce/config/stores');
     $this->getSession()->getPage()->clickLink('Add store');
 
     // Check the integrity of the form.