correct coding standard

Author: bisw

accordion_menus.module

@@ -6,7 +6,7 @@
  */
 
 /**
- * implement hook_theme().
+ * Implements hook_theme().
  */
 function accordion_menus_theme($existing, $type, $theme, $path) {
   return [

css/accordion_menus.css

@@ -1,3 +1,3 @@
 .accordion_menus-block-content h2 {
-  display:none;
-}
+  display: none;
+}

js/accordion_menus.js

@@ -1,5 +1,5 @@
-(function($) {  
-  $(document).ready(function(){
-    $('.accordion_menus_block_container').accordion({header: "h2.accordion-title", autoHeight: false}); 
+( function($) {
+  $(document).ready( function() {
+    $('.accordion_menus_block_container').accordion({header: 'h2.accordion-title', autoHeight: false});
   });
-})(jQuery);
+})(jQuery);

src/Form/AccordionMenusConfigForm.php

@@ -10,17 +10,19 @@
  */
 class AccordionMenusConfigForm extends ConfigFormBase {
 
-  /** @var string Config settings */
+  /**
+   * @var string Config settings
+   */
   const SETTINGS = 'accordion_menus.settings';
 
-  /** 
+  /**
    * {@inheritdoc}
    */
   public function getFormId() {
     return 'accordion_menus_config_form';
   }
 
-  /** 
+  /**
    * {@inheritdoc}
    */
   protected function getEditableConfigNames() {
@@ -29,7 +31,7 @@ protected function getEditableConfigNames() {
     ];
   }
 
-  /** 
+  /**
    * {@inheritdoc}
    */
   public function buildForm(array $form, FormStateInterface $form_state) {
@@ -47,19 +49,20 @@ public function buildForm(array $form, FormStateInterface $form_state) {
     return parent::buildForm($form, $form_state);
   }
 
-  /** 
+  /**
    * {@inheritdoc}
    */
   public function submitForm(array &$form, FormStateInterface $form_state) {
-    // Retrieve the configuration and Set the submitted configuration setting
+    // Retrieve the configuration and Set the submitted configuration setting.
     $this->configFactory->getEditable(static::SETTINGS)
       ->set('accordion_menus', $form_state->getValue('accordion_menus'))
       ->save();
 
     parent::submitForm($form, $form_state);
 
-    // Clear cache is needed to effect this value on block derivetive plugin system.
-    // See @src/Plugin/Derivative/AccordionMenusBlock
+    // Clear cache is needed to effect this value on block derivetive plugin
+    // system. See @src/Plugin/Derivative/AccordionMenusBlock.
     drupal_flush_all_caches();
   }
+
 }

src/Plugin/Block/AccordionMenusBlock.php

@@ -30,20 +30,20 @@ public function build() {
     $parameters->setMinDepth(0)->onlyEnabledLinks();
 
     $tree = $menu_tree->load($menu_name, $parameters);
-    $manipulators = array(
-      array('callable' => 'menu.default_tree_manipulators:checkAccess'),
-      array('callable' => 'menu.default_tree_manipulators:generateIndexAndSort'),
-    );
+    $manipulators = [
+      ['callable' => 'menu.default_tree_manipulators:checkAccess'],
+      ['callable' => 'menu.default_tree_manipulators:generateIndexAndSort'],
+    ];
     $tree = $menu_tree->transform($tree, $manipulators);
 
     $output['#theme'] = 'accordian_menus_block';
     $output['#attached']['library'][] = 'accordion_menus/accordion_menus_widget';
-   
+  
     foreach ($tree as $key => $menu_item) {
       if ($menu_item->hasChildren) {
         $elements[$key] = [
           'content' => $this->generateSubMenuTree($menu_item->subtree),
-          'title' => $menu_item->link->getTitle()
+          'title' => $menu_item->link->getTitle(),
         ];
       }
     }
@@ -52,11 +52,14 @@ public function build() {
     return $output;
   }
 
+  /**
+   * Generate submenu output.
+   */
   private function generateSubMenuTree($menu) {
     $output = [];
     $item_lists = [];
-    foreach($menu as $item) {
-      //If menu element disabled skip this branch
+    foreach ($menu as $item) {
+      // If menu element disabled skip this branch.
       if ($item->link->isEnabled()) {
         $item_lists[] = Link::fromTextAndUrl($item->link->getTitle(), $item->link->getUrlObject());
       }

src/Plugin/Derivative/AccordionMenusBlock.php

@@ -1,10 +1,5 @@
 <?php
 
-/**
- * @file
- * Contains \Drupal\accordion_menus\Plugin\Derivative\AccordionMenusBlock.
- */
-
 namespace Drupal\accordion_menus\Plugin\Derivative;
 
 use Drupal\Component\Plugin\Derivative\DeriverBase;
@@ -52,9 +47,9 @@ public function getDerivativeDefinitions($base_plugin_definition) {
     // Get menus from accordion configuration.
     $accordion_menus = \Drupal::config('accordion_menus.settings')->get('accordion_menus');
     foreach ($this->menuStorage->loadMultiple() as $menu => $entity) {
-      if (in_array($menu, $accordion_menus, true)) {
+      if (in_array($menu, $accordion_menus, TRUE)) {
         $this->derivatives[$menu] = $base_plugin_definition;
-        $this->derivatives[$menu]['admin_label'] = t('Accordion ') . $entity->label();
+        $this->derivatives[$menu]['admin_label'] = t('Accordion') . ' ' . $entity->label();
         $this->derivatives[$menu]['config_dependencies']['config'] = [$entity->getConfigDependencyName()];
       }
     }