drupal-media / entity_browser

src/Plugin/EntityBrowser/Display/Modal.php

Doc Comments +1 added lines, -1 removed lines

@@ -279,7 +279,7 @@
   /**
    * Helper function to return commands to return in AjaxResponse.
    *
-   * @return array
+   * @return SelectEntitiesCommand[]
    *   An array of ajax commands.
    */
   public function getAjaxCommands(FormStateInterface $form_state) {

Spacing +10 added lines, -10 removed lines

@@ -119,7 +119,7 @@ 
     $uuid = $this->getUuid();
     $js_event_object = new RegisterJSCallbacks($this->configuration['entity_browser_id'], $uuid);
     $js_event_object->registerCallback('Drupal.entityBrowser.selectionCompleted');
-    $js_event = $this->eventDispatcher->dispatch(Events::REGISTER_JS_CALLBACKS, $js_event_object );
+    $js_event = $this->eventDispatcher->dispatch(Events::REGISTER_JS_CALLBACKS, $js_event_object);
     $original_path = $this->currentPath->getPath();
     $data = [
       'query_parameters' => [
@@ -139,21 +139,21 @@ 
       '#theme_wrappers' => ['container'],
       'path' => [
         '#type' => 'hidden',
-        '#value' => Url::fromRoute('entity_browser.' . $this->configuration['entity_browser_id'], [], $data['query_parameters'])->toString(),
+        '#value' => Url::fromRoute('entity_browser.'.$this->configuration['entity_browser_id'], [], $data['query_parameters'])->toString(),
       ],
       'open_modal' => [
         '#type' => 'submit',
         '#value' => $this->configuration['link_text'],
         '#limit_validation_errors' => [],
         '#submit' => [],
-        '#name' => Html::getId('op_' . $this->configuration['entity_browser_id'] . '_' . $uuid),
+        '#name' => Html::getId('op_'.$this->configuration['entity_browser_id'].'_'.$uuid),
         '#ajax' => [
           'callback' => [$this, 'openModal'],
           'event' => 'click',
         ],
         '#attributes' => $data['attributes'],
         '#attached' => [
-          'library' => ['core/drupal.dialog.ajax',  'entity_browser/modal'],
+          'library' => ['core/drupal.dialog.ajax', 'entity_browser/modal'],
           'drupalSettings' => [
             'entity_browser' => [
               'modal' => [
@@ -201,13 +201,13 @@ 
         'height' => $this->configuration['height'] - 90,
         'frameborder' => 0,
         'style' => 'padding:0',
-        'name' => 'entity_browser_iframe_' . Html::cleanCssIdentifier($element_name)
+        'name' => 'entity_browser_iframe_'.Html::cleanCssIdentifier($element_name)
       ],
     ];
     $html = drupal_render($content);
 
     $response = new AjaxResponse();
-    $response->addCommand(new OpenDialogCommand('#' . Html::getUniqueId($field_name . '-' . $element_name . '-dialog'), $this->configuration['link_text'], $html, [
+    $response->addCommand(new OpenDialogCommand('#'.Html::getUniqueId($field_name.'-'.$element_name.'-dialog'), $this->configuration['link_text'], $html, [
       'width' => 'auto',
       'height' => 'auto',
       'modal' => TRUE,
@@ -283,7 +283,7 @@ 
    *   An array of ajax commands.
    */
   public function getAjaxCommands(FormStateInterface $form_state) {
-    $entities = array_map(function(EntityInterface $item) {return [$item->id(), $item->uuid(), $item->getEntityTypeId()];}, $form_state->get(['entity_browser', 'selected_entities']));
+    $entities = array_map(function(EntityInterface $item) {return [$item->id(), $item->uuid(), $item->getEntityTypeId()]; }, $form_state->get(['entity_browser', 'selected_entities']));
 
     $commands = [];
     $commands[] = new SelectEntitiesCommand($this->uuid, $entities);
@@ -303,13 +303,13 @@ 
   public function propagateSelection(FilterResponseEvent $event) {
     $render = [
       'labels' => [
-        '#markup' => 'Labels: ' . implode(', ', array_map(function (EntityInterface $item) {return $item->label();}, $this->entities)),
+        '#markup' => 'Labels: '.implode(', ', array_map(function(EntityInterface $item) {return $item->label(); }, $this->entities)),
         '#attached' => [
           'library' => ['entity_browser/modal_selection'],
           'drupalSettings' => [
             'entity_browser' => [
               'modal' => [
-                'entities' => array_map(function (EntityInterface $item) {return [$item->id(), $item->uuid(), $item->getEntityTypeId()];}, $this->entities),
+                'entities' => array_map(function(EntityInterface $item) {return [$item->id(), $item->uuid(), $item->getEntityTypeId()]; }, $this->entities),
                 'uuid' => $this->request->query->get('uuid'),
               ],
             ],
@@ -325,7 +325,7 @@ 
    * {@inheritdoc}
    */
   public function path() {
-    return '/entity-browser/modal/' . $this->configuration['entity_browser_id'];
+    return '/entity-browser/modal/'.$this->configuration['entity_browser_id'];
   }
 
   /**

src/Entity/EntityBrowser.php

Spacing +1 added lines, -1 removed lines

@@ -336,7 +336,7 @@
           '_title_callback' => 'Drupal\entity_browser\Controllers\StandalonePage::title',
           'entity_browser_id' => $this->id(),
         ],
-        ['_permission' => 'access ' . $this->id() . ' entity browser pages'],
+        ['_permission' => 'access '.$this->id().' entity browser pages'],
         ['_admin_route' => \Drupal::config('node.settings')->get('use_admin_theme')]
       );
     }

src/Permissions.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
 
     foreach ($browsers as $browser) {
       if ($browser->route()) {
-        $permissions['access ' . $browser->id() . ' entity browser pages'] = array(
+        $permissions['access '.$browser->id().' entity browser pages'] = array(
           'title' => $this->translationManager->translate('Access @name pages', array('@name' => $browser->label())),
           'description' => $this->translationManager->translate('Access pages that %browser uses to operate.', array('%browser' => $browser->label())),
         );

src/Plugin/EntityBrowser/Widget/Upload.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
     $uploaded_files = $form_state->getValue(['upload'], []);
     $trigger = $form_state->getTriggeringElement();
     // Only validate if we are uploading a file.
-    if (empty($uploaded_files)  && $trigger['#value'] == 'Upload') {
+    if (empty($uploaded_files) && $trigger['#value'] == 'Upload') {
       $form_state->setError($form['widget']['upload'], t('At least one file should be uploaded.'));
     }
   }

src/Plugin/EntityBrowser/Widget/View.php

Indentation +3 added lines, -3 removed lines

@@ -72,9 +72,9 @@
    * @param \Drupal\Core\Session\AccountInterface $current_user
    *   The current user.
    */
-   public function __construct(array $configuration, $plugin_id, $plugin_definition, EventDispatcherInterface $event_dispatcher, EntityManagerInterface $entity_manager, AccountInterface $current_user) {
-     parent::__construct($configuration, $plugin_id, $plugin_definition, $event_dispatcher, $entity_manager);
-     $this->currentUser = $current_user;
+    public function __construct(array $configuration, $plugin_id, $plugin_definition, EventDispatcherInterface $event_dispatcher, EntityManagerInterface $entity_manager, AccountInterface $current_user) {
+      parent::__construct($configuration, $plugin_id, $plugin_definition, $event_dispatcher, $entity_manager);
+      $this->currentUser = $current_user;
   }
 
   /**

Braces +2 added lines, -4 removed lines

@@ -124,8 +124,7 @@ 
         return [
           '#markup' => t('Entity browser select form field not found on a view. <a href=":link">Go fix it</a>!', [':link' => $url]),
         ];
-      }
-      else {
+      } else {
         return [
           '#markup' => t('Entity browser select form field not found on a view. Go fix it!'),
         ];
@@ -183,8 +182,7 @@ 
                 ]);
                 $form_state->setError($form['widget']['view']['entity_browser_select'], $message);
               }
-            }
-            catch (PluginNotFoundException $e) {
+            } catch (PluginNotFoundException $e) {
               $message = t('The Entity Type @type does not exist.', [
                 '@type' => $parts[0],
               ]);

Spacing +3 added lines, -3 removed lines

@@ -116,7 +116,7 @@ 
     $form['view'] = $view->executeDisplay($this->configuration['view_display']);
 
     if (empty($view->field['entity_browser_select'])) {
-      $url = Url::fromRoute('entity.view.edit_form',['view'=>$this->configuration['view']])->toString();
+      $url = Url::fromRoute('entity.view.edit_form', ['view'=>$this->configuration['view']])->toString();
       if ($this->currentUser->hasPermission('administer views')) {
         return [
           '#markup' => t('Entity browser select form field not found on a view. <a href=":link">Go fix it</a>!', [':link' => $url]),
@@ -220,13 +220,13 @@ 
     foreach ($displays as $display) {
       list($view_id, $display_id) = $display;
       $view = $this->entityManager->getStorage('view')->load($view_id);
-      $options[$view_id . '.' . $display_id] = $this->t('@view : @display', array('@view' => $view->label(), '@display' => $view->get('display')[$display_id]['display_title']));
+      $options[$view_id.'.'.$display_id] = $this->t('@view : @display', array('@view' => $view->label(), '@display' => $view->get('display')[$display_id]['display_title']));
     }
 
     $form['view'] = [
       '#type' => 'select',
       '#title' => $this->t('View : View display'),
-      '#default_value' => $this->configuration['view'] . '.' . $this->configuration['view_display'],
+      '#default_value' => $this->configuration['view'].'.'.$this->configuration['view_display'],
       '#options' => $options,
     ];
 

src/Plugin/EntityBrowser/WidgetSelector/DropDown.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
   /**
    * {@inheritdoc}
    */
-  public function getForm(array &$form = array(), FormStateInterface &$form_state = NULL) {
+  public function getForm(array &$form = array(), FormStateInterface&$form_state = NULL) {
     // Set a wrapper container for us to replace the form on ajax call.
     $form['#prefix'] = '<div id="entity-browser-form">';
     $form['#suffix'] = '</div>';

src/Plugin/EntityBrowser/WidgetSelector/Single.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
   /**
    * {@inheritdoc}
    */
-  public function getForm(array &$form, FormStateInterface &$form_state) {
+  public function getForm(array &$form, FormStateInterface&$form_state) {
     return array();
   }
 

src/Plugin/EntityBrowser/WidgetSelector/Tabs.php

Spacing +2 added lines, -2 removed lines

@@ -24,10 +24,10 @@
   /**
    * {@inheritdoc}
    */
-  public function getForm(array &$form = array(), FormStateInterface &$form_state = NULL) {
+  public function getForm(array &$form = array(), FormStateInterface&$form_state = NULL) {
     $element = [];
     foreach ($this->widget_ids as $id => $label) {
-      $name = 'tab_selector_' . $id;
+      $name = 'tab_selector_'.$id;
       $element[$name] = array(
         '#type' => 'button',
         '#attributes' => ['class' => ['tab']],

src/Plugin/Field/FieldWidget/EntityReference.php

Braces +2 added lines, -4 removed lines

@@ -183,8 +183,7 @@ 
 
     if (empty($entity_browser_id)) {
       return [t('No entity browser selected.')];
-    }
-    else {
+    } else {
       $browser = $this->entityManager->getStorage('entity_browser')
         ->load($entity_browser_id);
       $summary[] = t('Entity browser: @browser', ['@browser' => $browser->label()]);
@@ -355,8 +354,7 @@ 
     // 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, -2);
-    }
-    elseif ($trigger['#type'] == 'submit' && strpos($trigger['#name'], '_remove_')) {
+    } elseif ($trigger['#type'] == 'submit' && strpos($trigger['#name'], '_remove_')) {
       $parents = array_slice($trigger['#array_parents'], 0, -4);
     }
 

Unused Use Statements -2 removed lines

@@ -16,8 +16,6 @@
 use Drupal\Core\Field\WidgetBase;
 use Drupal\Core\Form\FormStateInterface;
 use Drupal\Core\Plugin\ContainerFactoryPluginInterface;
-use Drupal\entity_browser\Events\Events;
-use Drupal\entity_browser\Events\RegisterJSCallbacks;
 use Drupal\entity_browser\FieldWidgetDisplayManager;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 use Symfony\Component\EventDispatcher\EventDispatcherInterface;

Indentation -1 removed lines

@@ -25,7 +25,6 @@
 
 /**
  * Plugin implementation of the 'entity_reference' widget for entity browser.
-
  * @FieldWidget(
  *   id = "entity_browser_entity_reference",
  *   label = @Translation("Entity browser"),

Spacing +8 added lines, -8 removed lines

@@ -150,7 +150,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' => t('Entity display plugin'),
       '#type' => 'select',
@@ -184,7 +184,7 @@ 
       '#type' => 'fieldset',
       '#title' => t('Entity display plugin configuration'),
       '#tree' => TRUE,
-      '#prefix' => '<div id="' . $id . '">',
+      '#prefix' => '<div id="'.$id.'">',
       '#suffix' => '</div>',
     ];
 
@@ -281,7 +281,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();
   }
 
   /**
@@ -317,7 +317,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']);
       }
 
@@ -354,8 +354,8 @@ 
     // 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());
     /** @var \Drupal\entity_browser\EntityBrowserInterface $entity_browser */
     $entity_browser = $this->entityManager->getStorage('entity_browser')->load($this->getSetting('entity_browser'));
 
@@ -485,7 +485,7 @@ 
       '#theme_wrappers' => ['container'],
       '#attributes' => ['class' => ['entities-list']],
       'items' => array_map(
-        function (ContentEntityInterface $entity) use ($field_widget_display, $details_id, $field_parents) {
+        function(ContentEntityInterface $entity) use ($field_widget_display, $details_id, $field_parents) {
           $display = $field_widget_display->view($entity);
           if (is_string($display)) {
             $display = ['#markup' => $display];
@@ -505,7 +505,7 @@ 
                 'wrapper' => $details_id,
               ],
               '#submit' => [[get_class($this), 'removeItemSubmit']],
-              '#name' => $this->fieldDefinition->getName() . '_remove_' . $entity->id(),
+              '#name' => $this->fieldDefinition->getName().'_remove_'.$entity->id(),
               '#limit_validation_errors' => [array_merge($field_parents, [$this->fieldDefinition->getName()])],
               '#attributes' => ['data-entity-id' => $entity->id()],
               '#access' => (bool) $this->getSetting('field_widget_remove')