drupal-media / entity_browser

src/RouteSubscriber.php

Spacing +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
       /** @var $browser \Drupal\entity_browser\EntityBrowserInterface */
       $browser = $this->browserStorage->load($id);
       if ($route = $browser->route()) {
-        $collection->add('entity_browser.' . $browser->id(), $route);
+        $collection->add('entity_browser.'.$browser->id(), $route);
       }
     }
 

src/WidgetSelectorInterface.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
    * @return array
    *   Form structure.
    */
-  public function getForm(array &$form, FormStateInterface &$form_state);
+  public function getForm(array &$form, FormStateInterface&$form_state);
 
   /**
    * Sets the default widget.

src/Wizard/EntityBrowserWizard.php

Doc Comments +1 added lines, -3 removed lines

@@ -34,9 +34,7 @@
    *   The entity manager.
    * @param $tempstore_id
    *   The shared temp store factory collection name.
-   * @param null $machine_name
-   *   The SharedTempStore key for our current wizard values.
-   * @param null $step
+   * @param string|null $step
    *   The current active step of the wizard.
    */
   public function __construct(SharedTempStoreFactory $tempstore, FormBuilderInterface $builder, ClassResolverInterface $class_resolver, EventDispatcherInterface $event_dispatcher, EntityManagerInterface $entity_manager, RouteMatchInterface $route_match, $tempstore_id, $entity_browser = NULL, $step = 'general') {

src/Plugin/EntityBrowser/Widget/Upload.php

Spacing +1 added lines, -1 removed lines

@@ -125,7 +125,7 @@
       $files = $this->prepareEntities($form, $form_state);
       array_walk(
         $files,
-        function (FileInterface $file) {
+        function(FileInterface $file) {
           $file->setPermanent();
           $file->save();
         }

Braces +1 added lines, -2 removed lines

@@ -208,8 +208,7 @@
       $extensions = implode(' ', array_unique($extensions));
       if (!preg_match('/^([a-z0-9]+([.][a-z0-9])* ?)+$/', $extensions)) {
         $form_state->setError($element, t('The list of allowed extensions is not valid, be sure to exclude leading dots and to separate extensions with a comma or space.'));
-      }
-      else {
+      } else {
         $form_state->setValueForElement($element, $extensions);
       }
     }

src/Controllers/EntityBrowserController.php

Spacing +2 added lines, -2 removed lines

@@ -43,12 +43,12 @@
       // Return the form as a modal dialog.
       $form['#attached']['library'][] = 'core/drupal.dialog.ajax';
       $title = $this->t('Edit entity @entity', ['@entity' => $entity->label()]);
-      $response = AjaxResponse::create()->addCommand(new OpenDialogCommand('#' . $entity->getEntityTypeId() . '-' . $entity->id() . '-edit-dialog', $title, $form, ['modal' => TRUE, 'width' => 800]));
+      $response = AjaxResponse::create()->addCommand(new OpenDialogCommand('#'.$entity->getEntityTypeId().'-'.$entity->id().'-edit-dialog', $title, $form, ['modal' => TRUE, 'width' => 800]));
       return $response;
     }
     else {
       // Return command for closing the modal.
-      $response = AjaxResponse::create()->addCommand(new CloseDialogCommand('#' . $entity->getEntityTypeId() . '-' . $entity->id() . '-edit-dialog'));
+      $response = AjaxResponse::create()->addCommand(new CloseDialogCommand('#'.$entity->getEntityTypeId().'-'.$entity->id().'-edit-dialog'));
       // Also refresh the widget if "details_id" is provided.
       $details_id = $request->query->get('details_id');
       if (!empty($details_id)) {

Braces +1 added lines, -2 removed lines

@@ -45,8 +45,7 @@
       $title = $this->t('Edit entity @entity', ['@entity' => $entity->label()]);
       $response = AjaxResponse::create()->addCommand(new OpenDialogCommand('#' . $entity->getEntityTypeId() . '-' . $entity->id() . '-edit-dialog', $title, $form, ['modal' => TRUE, 'width' => 800]));
       return $response;
-    }
-    else {
+    } else {
       // Return command for closing the modal.
       $response = AjaxResponse::create()->addCommand(new CloseDialogCommand('#' . $entity->getEntityTypeId() . '-' . $entity->id() . '-edit-dialog'));
       // Also refresh the widget if "details_id" is provided.

src/Plugin/Field/FieldWidget/EntityReferenceBrowserWidget.php

Braces +4 added lines, -8 removed lines

@@ -453,8 +453,7 @@ 
     // and use this information to return correct part of the form.
     if (!empty($trigger['#ajax']['event']) && $trigger['#ajax']['event'] == 'entity_browser_value_updated') {
       $parents = array_slice($trigger['#array_parents'], 0, -1);
-    }
-    elseif ($trigger['#type'] == 'submit' && strpos($trigger['#name'], '_remove_')) {
+    } elseif ($trigger['#type'] == 'submit' && strpos($trigger['#name'], '_remove_')) {
       $parents = array_slice($trigger['#array_parents'], 0, -static::$deleteDepth);
     }
 
@@ -621,12 +620,10 @@ 
     $entity_browser_id = $this->getSetting('entity_browser');
     if (empty($entity_browser_id)) {
       return [$this->t('No entity browser selected.')];
-    }
-    else {
+    } else {
       if ($browser = $this->entityTypeManager->getStorage('entity_browser')->load($entity_browser_id)) {
         $summary[] = $this->t('Entity browser: @browser', ['@browser' => $browser->label()]);
-      }
-      else {
+      } else {
         drupal_set_message($this->t('Missing entity browser!'), 'error');
         return [$this->t('Missing entity browser!')];
       }
@@ -636,8 +633,7 @@ 
     $selection_mode_options = EntityBrowserElement::getSelectionModeOptions();
     if (isset($selection_mode_options[$selection_mode])) {
       $summary[] = $this->t('Selection mode: @selection_mode', ['@selection_mode' => $selection_mode_options[$selection_mode]]);
-    }
-    else {
+    } else {
       $summary[] = $this->t('Undefined selection mode.');
     }
 

Spacing +15 added lines, -15 removed lines

@@ -170,7 +170,7 @@ 
       }
     }
 
-    $id = Html::getUniqueId('field-' . $this->fieldDefinition->getName() . '-display-settings-wrapper');
+    $id = Html::getUniqueId('field-'.$this->fieldDefinition->getName().'-display-settings-wrapper');
     $element['field_widget_display'] = [
       '#title' => $this->t('Entity display plugin'),
       '#type' => 'select',
@@ -220,7 +220,7 @@ 
       '#type' => 'fieldset',
       '#title' => $this->t('Entity display plugin configuration'),
       '#tree' => TRUE,
-      '#prefix' => '<div id="' . $id . '">',
+      '#prefix' => '<div id="'.$id.'">',
       '#suffix' => '</div>',
     ];
 
@@ -303,7 +303,7 @@ 
    *   A key for form state storage.
    */
   protected function getFormStateKey(FieldItemListInterface $items) {
-    return $items->getEntity()->uuid() . ':' . $items->getFieldDefinition()->getName();
+    return $items->getEntity()->uuid().':'.$items->getFieldDefinition()->getName();
   }
 
   /**
@@ -315,7 +315,7 @@ 
 
     // Get correct ordered list of entity IDs.
     $ids = array_map(
-      function (EntityInterface $entity) {
+      function(EntityInterface $entity) {
         return $entity->id();
       },
       $entities
@@ -329,8 +329,8 @@ 
     // might result in unpredictable results.
     $form_state->set(['entity_browser_widget', $this->getFormStateKey($items)], $ids);
 
-    $hidden_id = Html::getUniqueId('edit-' . $this->fieldDefinition->getName() . '-target-id');
-    $details_id = Html::getUniqueId('edit-' . $this->fieldDefinition->getName());
+    $hidden_id = Html::getUniqueId('edit-'.$this->fieldDefinition->getName().'-target-id');
+    $details_id = Html::getUniqueId('edit-'.$this->fieldDefinition->getName());
 
     $element += [
       '#id' => $details_id,
@@ -345,8 +345,8 @@ 
         // We need to repeat ID here as it is otherwise skipped when rendering.
         '#attributes' => ['id' => $hidden_id],
         '#default_value' => implode(' ', array_map(
-            function (EntityInterface $item) {
-              return $item->getEntityTypeId() . ':' . $item->id();
+            function(EntityInterface $item) {
+              return $item->getEntityTypeId().':'.$item->id();
             },
             $entities
         )),
@@ -398,7 +398,7 @@ 
    */
   public static function processEntityBrowser(&$element, FormStateInterface $form_state, &$complete_form) {
     $uuid = key($element['#attached']['drupalSettings']['entity_browser']);
-    $element['#attached']['drupalSettings']['entity_browser'][$uuid]['selector'] = '#' . $element['#custom_hidden_id'];
+    $element['#attached']['drupalSettings']['entity_browser'][$uuid]['selector'] = '#'.$element['#custom_hidden_id'];
     return $element;
   }
 
@@ -508,7 +508,7 @@ 
       '#theme_wrappers' => ['container'],
       '#attributes' => ['class' => $classes],
       'items' => array_map(
-        function (ContentEntityInterface $entity, $row_id) use ($field_widget_display, $details_id, $field_parents) {
+        function(ContentEntityInterface $entity, $row_id) use ($field_widget_display, $details_id, $field_parents) {
           $display = $field_widget_display->view($entity);
           $edit_button_access = $this->getSetting('field_widget_edit') && $entity->access('update', $this->currentUser);
           if ($entity->getEntityTypeId() == 'file') {
@@ -524,7 +524,7 @@ 
             '#theme_wrappers' => ['container'],
             '#attributes' => [
               'class' => ['item-container', Html::getClass($field_widget_display->getPluginId())],
-              'data-entity-id' => $entity->getEntityTypeId() . ':' . $entity->id(),
+              'data-entity-id' => $entity->getEntityTypeId().':'.$entity->id(),
               'data-row-id' => $row_id,
             ],
             'display' => $display,
@@ -536,10 +536,10 @@ 
                 'wrapper' => $details_id,
               ],
               '#submit' => [[get_class($this), 'removeItemSubmit']],
-              '#name' => $this->fieldDefinition->getName() . '_remove_' . $entity->id() . '_' . $row_id . '_' . md5(json_encode($field_parents)),
+              '#name' => $this->fieldDefinition->getName().'_remove_'.$entity->id().'_'.$row_id.'_'.md5(json_encode($field_parents)),
               '#limit_validation_errors' => [array_merge($field_parents, [$this->fieldDefinition->getName()])],
               '#attributes' => [
-                'data-entity-id' => $entity->getEntityTypeId() . ':' . $entity->id(),
+                'data-entity-id' => $entity->getEntityTypeId().':'.$entity->id(),
                 'data-row-id' => $row_id,
               ],
               '#access' => (bool) $this->getSetting('field_widget_remove'),
@@ -678,7 +678,7 @@ 
       }
       // Submit was triggered by one of the "Remove" buttons. We need to walk
       // few levels up to read value of "target_id" element.
-      elseif ($trigger['#type'] == 'submit' && strpos($trigger['#name'], $this->fieldDefinition->getName() . '_remove_') === 0) {
+      elseif ($trigger['#type'] == 'submit' && strpos($trigger['#name'], $this->fieldDefinition->getName().'_remove_') === 0) {
         $parents = array_merge(array_slice($trigger['#parents'], 0, -static::$deleteDepth), ['target_id']);
       }
 
@@ -735,7 +735,7 @@ 
     // If an entity browser is being used in this widget, add it as a config
     // dependency.
     if ($browser_name = $this->getSetting('entity_browser')) {
-      $dependencies['config'][] = 'entity_browser.browser.' . $browser_name;
+      $dependencies['config'][] = 'entity_browser.browser.'.$browser_name;
     }
 
     return $dependencies;

modules/entity_form/entity_browser_entity_form.module

Spacing +4 added lines, -4 removed lines

@@ -18,7 +18,7 @@ 
 /**
  * Implements hook_inline_entity_form_reference_form_alter().
  */
-function entity_browser_entity_form_inline_entity_form_reference_form_alter(&$reference_form, FormStateInterface &$form_state) {
+function entity_browser_entity_form_inline_entity_form_reference_form_alter(&$reference_form, FormStateInterface&$form_state) {
   /** @var \Drupal\field\FieldConfigInterface $instance */
   $instance = $form_state->get([
     'inline_entity_form',
@@ -27,7 +27,7 @@ 
   ]);
 
   /** @var \Drupal\Core\Entity\Display\EntityFormDisplayInterface $form_display */
-  $entity_form_id = $instance->getTargetEntityTypeId() . '.' . $instance->getTargetBundle() . '.default';
+  $entity_form_id = $instance->getTargetEntityTypeId().'.'.$instance->getTargetBundle().'.default';
   // TODO - 'default' might become configurable or something else in the future.
   // See https://www.drupal.org/node/2510274
   $form_display = \Drupal::entityTypeManager()->getStorage('entity_form_display')->load($entity_form_id);
@@ -47,7 +47,7 @@ 
   ]));
 
   $cardinality = FieldStorageDefinitionInterface::CARDINALITY_UNLIMITED;
-  if($instance->getFieldStorageDefinition()->get('cardinality') !== $cardinality) {
+  if ($instance->getFieldStorageDefinition()->get('cardinality') !== $cardinality) {
     $cardinality = $instance->getFieldStorageDefinition()->get('cardinality') - $count_existing_selection;
   }
 
@@ -119,7 +119,7 @@ 
   $ief_id = $reference_form['#ief_id'];
   $form_values = NestedArray::getValue($form_state->getValues(), $reference_form['#parents']);
   $attach_entities = EntityBrowserElement::processEntityIds($form_values['entity_browser']['entity_ids']);
-  $entities =& $form_state->get(['inline_entity_form', $ief_id, 'entities']);
+  $entities = & $form_state->get(['inline_entity_form', $ief_id, 'entities']);
 
   // Determine the correct weight of the new element.
   $weight = 0;

Braces +1 added lines, -2 removed lines

@@ -165,8 +165,7 @@
         '#empty_value' => '_none',
         '#default_value' => $plugin->getThirdPartySetting('entity_browser_entity_form', 'entity_browser_id'),
       ];
-    }
-    else {
+    } else {
       $element['message'] = [
         '#markup' => t('There are no entity browsers available. You can create one <a href="@url">here</a>.', ['@url' => Url::fromRoute('entity.entity_browser.collection')->toString()]),
       ];

src/Form/GeneralInfoConfig.php

Spacing +9 added lines, -9 removed lines

@@ -90,9 +90,9 @@ 
     $entity_browser = $cached_values['entity_browser'];
 
     if (empty($entity_browser->id())) {
-      $help_text = '<div class="clearfix eb-help-text"><h2>' . $this->t('Entity Browser creation instructions') . '</h2>';
-      $help_text .= '<p>' . $this->t('This is a multi-step form. In this first step you need to define the main characteristics of the Entity Browser (in other words, which plugins will be used for each functionality). In the following steps of this wizard, each individual plugin can be configured, when necessary.') . '</p>';
-      $help_text .= '<p>' . $this->t('You can find more detailed information about creating and configuring Entity Browsers at the <a href="@guide_href" target="_blank">official documentation</a>.', ['@guide_href' => 'https://drupal-media.gitbooks.io/drupal8-guide/content/modules/entity_browser/intro.html']) . '</p>';
+      $help_text = '<div class="clearfix eb-help-text"><h2>'.$this->t('Entity Browser creation instructions').'</h2>';
+      $help_text .= '<p>'.$this->t('This is a multi-step form. In this first step you need to define the main characteristics of the Entity Browser (in other words, which plugins will be used for each functionality). In the following steps of this wizard, each individual plugin can be configured, when necessary.').'</p>';
+      $help_text .= '<p>'.$this->t('You can find more detailed information about creating and configuring Entity Browsers at the <a href="@guide_href" target="_blank">official documentation</a>.', ['@guide_href' => 'https://drupal-media.gitbooks.io/drupal8-guide/content/modules/entity_browser/intro.html']).'</p>';
       $help_text .= '</div>';
       $form['help_text'] = [
         '#markup' => $help_text,
@@ -100,10 +100,10 @@ 
     }
 
     $displays = [];
-    $display_description = $this->t('Choose here how the browser(s) should be presented to the end user. The available plugins are:') . '<ul>';
+    $display_description = $this->t('Choose here how the browser(s) should be presented to the end user. The available plugins are:').'<ul>';
     foreach ($this->displayManager->getDefinitions() as $plugin_id => $plugin_definition) {
       $displays[$plugin_id] = $plugin_definition['label'];
-      $display_description .= '<li><b>' . $plugin_definition['label'] . ':</b> ' . $plugin_definition['description'] . '</li>';
+      $display_description .= '<li><b>'.$plugin_definition['label'].':</b> '.$plugin_definition['description'].'</li>';
     }
     $display_description .= '</ul>';
     $form['display'] = [
@@ -116,10 +116,10 @@ 
     ];
 
     $widget_selectors = [];
-    $widget_description = $this->t('In the last step of the entity browser configuration you can decide how the widgets will be available to the editor. The available plugins are:') . '<ul>';
+    $widget_description = $this->t('In the last step of the entity browser configuration you can decide how the widgets will be available to the editor. The available plugins are:').'<ul>';
     foreach ($this->widgetSelectorManager->getDefinitions() as $plugin_id => $plugin_definition) {
       $widget_selectors[$plugin_id] = $plugin_definition['label'];
-      $widget_description .= '<li><b>' . $plugin_definition['label'] . ':</b> ' . $plugin_definition['description'] . '</li>';
+      $widget_description .= '<li><b>'.$plugin_definition['label'].':</b> '.$plugin_definition['description'].'</li>';
     }
     $widget_description .= '</ul>';
     $form['widget_selector'] = [
@@ -132,10 +132,10 @@ 
     ];
 
     $selection_display = [];
-    $selection_description = $this->t('You can optionally allow a "work-in-progress selection zone" to be available to the editor, while still navigating, browsing and selecting the entities. The available plugins are:') . '<ul>';
+    $selection_description = $this->t('You can optionally allow a "work-in-progress selection zone" to be available to the editor, while still navigating, browsing and selecting the entities. The available plugins are:').'<ul>';
     foreach ($this->selectionDisplayManager->getDefinitions() as $plugin_id => $plugin_definition) {
       $selection_display[$plugin_id] = $plugin_definition['label'];
-      $selection_description .= '<li><b>' . $plugin_definition['label'] . ':</b> ' . $plugin_definition['description'] . '</li>';
+      $selection_description .= '<li><b>'.$plugin_definition['label'].':</b> '.$plugin_definition['description'].'</li>';
     }
     $selection_description .= '</ul>';
     $form['selection_display'] = [

src/Form/WidgetsConfig.php

Spacing +4 added lines, -4 removed lines

@@ -53,10 +53,10 @@ 
     $entity_browser = $form_state->getTemporaryValue('wizard')['entity_browser'];
 
     $widgets = [];
-    $description = $this->t('The available plugins are:') . '<ul>';
+    $description = $this->t('The available plugins are:').'<ul>';
     foreach ($this->widgetManager->getDefinitions() as $plugin_id => $plugin_definition) {
       $widgets[$plugin_id] = $plugin_definition['label'];
-      $description .= '<li><b>' . $plugin_definition['label'] . ':</b> ' . $plugin_definition['description'] . '</li>';
+      $description .= '<li><b>'.$plugin_definition['label'].':</b> '.$plugin_definition['description'].'</li>';
     }
     $description .= '</ul>';
     $default_widgets = [];
@@ -67,7 +67,7 @@ 
     $form['widget'] = [
       '#type' => 'select',
       '#title' => $this->t('Add widget plugin'),
-      '#options' => ['_none_' => '- ' . $this->t('Select a widget to add it') . ' -'] + $widgets,
+      '#options' => ['_none_' => '- '.$this->t('Select a widget to add it').' -'] + $widgets,
       '#description' => $description,
       '#ajax' => [
         'callback' => [get_class($this), 'tableUpdatedAjaxCallback'],
@@ -121,7 +121,7 @@ 
       $row['remove'] = [
         '#type' => 'submit',
         '#value' => $this->t('Delete'),
-        '#name' => 'remove' . $uuid,
+        '#name' => 'remove'.$uuid,
         '#ajax' => [
           'callback' => [get_class($this), 'tableUpdatedAjaxCallback'],
           'wrapper' => 'widgets',