Coding standards

layout_builder_ids.services.yml

 services:
   layout_builder_ids.render_block_component_subscriber:
-    class: Drupal\layout_builder_ids\EventSubscriber\LayoutBuilderIdsRenderSubscriber
+    class: 'Drupal\layout_builder_ids\EventSubscriber\LayoutBuilderIdsRenderSubscriber'
     tags:
       - { name: event_subscriber }
   layout_builder_ids.configure_section_form:

src/EventSubscriber/LayoutBuilderIdsConfigureBlock.php

@@ -25,26 +25,17 @@ class LayoutBuilderIdsConfigureBlock implements EventSubscriberInterface {
     $form = &$event->getForm();
 
     // If we are on a configure section form, alter it.
-    if ($form['#form_id'] == 'layout_builder_add_block' || $form['#form_id'] == 'layout_builder_update_block') {
-
-      // Get the form state.
-      $form_state = &$event->getFormState();
-
-      // Load in the form object.
-      $formObject = $form_state->getFormObject();
-
-      // Load in the component/block.
-      $component = $formObject->getCurrentComponent();
+    if (in_array($form['#form_id'], ['layout_builder_add_block', 'layout_builder_update_block'], TRUE)) {
 
       // Pull out the layout_builder_id from config.
-      $layout_builder_id = $component->get('layout_builder_id');
+      $layout_builder_id = &$event->getFormState()->getFormObject()->getCurrentComponent()->get('layout_builder_id');
 
       // Add the section id to the configure form.
       $form['settings']['layout_builder_id'] = [
         '#type' => 'textfield',
         '#title' => 'Block ID',
         '#weight' => 0,
-        '#default_value' => $layout_builder_id ? $layout_builder_id : NULL,
+        '#default_value' => $layout_builder_id ?: NULL,
         '#description' => t('Enter an ID for the block. IDs can contain letters, numbers, underscore, hyphen and period characters, and should start with a letter.'),
       ];
 
@@ -64,11 +55,8 @@ class LayoutBuilderIdsConfigureBlock implements EventSubscriberInterface {
     // If there is in id, save it in config.
     if ($layout_builder_id !== NULL) {
 
-      // Load in the form object.
-      $formObject = $form_state->getFormObject();
-
       // Load in the component/block.
-      $component = $formObject->getCurrentComponent();
+      $component = $form_state->getFormObject()->getCurrentComponent();
 
       // Set the layout_builder_id.
       $component->set('layout_builder_id', Html::getId($layout_builder_id));

src/EventSubscriber/LayoutBuilderIdsConfigureSection.php

@@ -29,21 +29,15 @@ class LayoutBuilderIdsConfigureSection implements EventSubscriberInterface {
     // If we are on a configure section form, alter it.
     if ($form['#form_id'] == 'layout_builder_configure_section') {
 
-      // Get the form state.
-      $form_state = $event->getFormState();
-
-      // Get the form object.
-      $formObject = $form_state->getFormObject();
-
       // Get the config for the section.
-      $config = $formObject->getLayout()->getConfiguration();
+      $config = $event->getFormState()->getFormObject()->getLayout()->getConfiguration();
 
       // Add the section id to the configure form.
       $form['layout_settings']['layout_builder_id'] = [
         '#type' => 'textfield',
         '#title' => 'Section ID',
         '#weight' => 0,
-        '#default_value' => isset($config['layout_builder_id']) ? $config['layout_builder_id'] : '',
+        '#default_value' => $config['layout_builder_id'] ?: NULL,
         '#description' => t('Enter an ID for the section.  IDs can contain letters, numbers, underscore, hyphen and period characters, and should start with a letter.'),
       ];