From f98362c46264171ef491db242b20c0ef0be095de Mon Sep 17 00:00:00 2001 From: romainj Date: Sun, 22 May 2016 22:04:58 +0200 Subject: [PATCH] Minor code/typo cleaning --- .../admin_toolbar_tools.libraries.yml | 1 - .../admin_toolbar_tools.links.menu.yml | 14 +++++----- .../admin_toolbar_tools.module | 15 +++++----- .../src/Controller/ToolbarController.php | 28 +++++++++---------- 4 files changed, 29 insertions(+), 29 deletions(-) diff --git a/admin_toolbar_tools/admin_toolbar_tools.libraries.yml b/admin_toolbar_tools/admin_toolbar_tools.libraries.yml index ddb22fd..1936ed5 100644 --- a/admin_toolbar_tools/admin_toolbar_tools.libraries.yml +++ b/admin_toolbar_tools/admin_toolbar_tools.libraries.yml @@ -1,5 +1,4 @@ toolbar.icon: - version: VERSION css: theme: css/tools.css: {} \ No newline at end of file diff --git a/admin_toolbar_tools/admin_toolbar_tools.links.menu.yml b/admin_toolbar_tools/admin_toolbar_tools.links.menu.yml index 44360ef..a5dde51 100644 --- a/admin_toolbar_tools/admin_toolbar_tools.links.menu.yml +++ b/admin_toolbar_tools/admin_toolbar_tools.links.menu.yml @@ -1,18 +1,18 @@ admin_toolbar_tools.flush: - title: Flush all caches + title: 'Flush all caches' route_name: admin_toolbar_tools.flush weight: -9 parent: admin_toolbar_tools.help admin_toolbar_tools.cssjs: - title: CSS and Javascript + title: 'CSS and Javascript' route_name: admin_toolbar_tools.cssjs parent: admin_toolbar_tools.flush admin_toolbar_tools.plugin: - title: Plugins + title: 'Plugins' route_name: admin_toolbar_tools.plugin parent: admin_toolbar_tools.flush admin_toolbar_tools.flush_static: - title: Static caches + title: 'Static caches' route_name: admin_toolbar_tools.flush_static parent: admin_toolbar_tools.flush admin_toolbar_tools.flush_menu: @@ -20,17 +20,17 @@ admin_toolbar_tools.flush_menu: route_name: admin_toolbar_tools.flush_menu parent: admin_toolbar_tools.flush admin_toolbar_tools.drupalorg: - title: Drupal.org + title: 'Drupal.org' weight: -5 route_name: admin_toolbar_tools.drupalorg parent: admin_toolbar_tools.help admin_toolbar_tools.listchanges: - title: List changes D8 + title: 'Change records for Drupal core' weight: -6 route_name: admin_toolbar_tools.listchanges parent: admin_toolbar_tools.drupalorg admin_toolbar_tools.doc: - title: Documentation D8 + title: 'D8 API documentation' weight: -5 route_name: admin_toolbar_tools.doc parent: admin_toolbar_tools.drupalorg diff --git a/admin_toolbar_tools/admin_toolbar_tools.module b/admin_toolbar_tools/admin_toolbar_tools.module index caea74a..c8392b8 100644 --- a/admin_toolbar_tools/admin_toolbar_tools.module +++ b/admin_toolbar_tools/admin_toolbar_tools.module @@ -36,7 +36,7 @@ function admin_toolbar_tools_help($route_name, RouteMatchInterface $route_match) case 'help.page.admin_toolbar_tools': $output = ''; $output .= '

' . t('About') . '

'; - $output .= '

' . t('The Admin Toolbar Extra Tools module allows you to to add more fonctionalities to the default toolbar like flush caches, runnig cron.') . '

'; + $output .= '

' . t('The Admin Toolbar Extra Tools module allows you to to add more functionalities to the default toolbar like flushing caches, running cron.') . '

'; return $output; } @@ -47,6 +47,7 @@ function admin_toolbar_tools_help($route_name, RouteMatchInterface $route_match) */ function admin_toolbar_tools_menu_links_discovered_alter(&$links) { $moduleHandler = \Drupal::moduleHandler(); + $links['admin_toolbar_tools.help'] = array( 'title' => t('Tools'), 'route_name' => '', @@ -73,7 +74,7 @@ function admin_toolbar_tools_menu_links_discovered_alter(&$links) { 'weight' => -6 ); $links['system.modules_uninstall'] = array( - 'title' => t('Uninstall'), + 'title' => t('Uninstall module'), 'route_name' => 'system.modules_uninstall', 'parent' => 'system.modules_list', ); @@ -102,10 +103,10 @@ function admin_toolbar_tools_menu_links_discovered_alter(&$links) { ); /** * Add common links to entities if parent module exists. - * The common links for entities are : - * - Editing entity, Deleting entity,manage fields for entity + * The common links for entities are: + * - Editing entity, Deleting entity, manage fields for entity * manage form display for an entity, entity view display - * and others links relating to entity + * and others links related to entity. */ foreach ($content_entities as $module_name => $entities) { if ($moduleHandler->moduleExists($module_name)) { @@ -173,7 +174,7 @@ function admin_toolbar_tools_menu_links_discovered_alter(&$links) { 'route_name' => 'entity.user_role.collection', 'parent' => 'entity.user.collection', ); - $links['user.logoutt'] = array( + $links['user.logout'] = array( 'title' => t('Logout'), 'route_name' => 'user.logout', 'parent' => 'admin_toolbar_tools.help', @@ -332,7 +333,7 @@ function admin_toolbar_tools_menu_links_discovered_alter(&$links) { 'parent' => 'block.admin_display', ); } - // If module contact is enabled. + // If module Contact is enabled. if ($moduleHandler->moduleExists('contact')) { $links['contact.form_add'] = array( 'title' => t('Add contact form'), diff --git a/admin_toolbar_tools/src/Controller/ToolbarController.php b/admin_toolbar_tools/src/Controller/ToolbarController.php index 9e894e7..03517a9 100644 --- a/admin_toolbar_tools/src/Controller/ToolbarController.php +++ b/admin_toolbar_tools/src/Controller/ToolbarController.php @@ -1,4 +1,5 @@ get('plugin.manager.menu.local_action') ); } - //Reload the previous page. + + // Reload the previous page. public function reload_page() { $request = \Drupal::request(); return $request->server->get('HTTP_REFERER'); } - //Flush all caches. + // Flushes all caches. public function flushAll() { drupal_flush_all_caches(); drupal_set_message($this->t('All cache cleared.')); return new RedirectResponse($this->reload_page()); } - //This function flush css and javascript caches. + // Flushes css and javascript caches. public function flush_js_css() { \Drupal::state() ->set('system.css_js_query_string', base_convert(REQUEST_TIME, 10, 36)); @@ -85,22 +87,21 @@ class ToolbarController extends ControllerBase { return new RedirectResponse($this->reload_page()); } - //This function flush plugins caches. + // Flushes plugins caches. public function flush_plugins() { - // Clear all plugin caches. \Drupal::service('plugin.cache_clearer')->clearCachedDefinitions(); drupal_set_message($this->t('Plugin cache cleared.')); return new RedirectResponse($this->reload_page()); } - // Reset all static caches. + // Resets all static caches. public function flush_static() { drupal_static_reset(); drupal_set_message($this->t('All static caches cleared.')); return new RedirectResponse($this->reload_page()); } -// Clears all cached menu data. + // Clears all cached menu data. public function flush_menu() { menu_cache_clear_all(); $this->menuLinkManager->rebuild(); @@ -111,26 +112,26 @@ class ToolbarController extends ControllerBase { return new RedirectResponse($this->reload_page()); } -// this function allow to access in documentation via admin_toolbar module + // Links to drupal.org home page. public function drupal_org() { $response = new RedirectResponse("https://www.drupal.org"); $response->send(); return $response; } - //This function display the administration link Development + // Displays the administration link Development. public function development() { return new RedirectResponse('/admin/structure/menu/'); } - // this function allow to access in documentation(list changes of the different versions of drupal core) via admin_toolbar module. + // Access to Drupal 8 changes (list changes of the different versions of drupal core). public function list_changes() { $response = new RedirectResponse("https://www.drupal.org/list-changes"); $response->send(); return $response; } - //this function allow to add + // Adds a link to the Drupal 8 documentation. public function documentation() { $response = new RedirectResponse("https://api.drupal.org/api/drupal/8"); $response->send(); @@ -143,5 +144,4 @@ class ToolbarController extends ControllerBase { return new RedirectResponse($this->reload_page()); } - } \ No newline at end of file -- GitLab