drupal-media / entity_browser

src/Tests/FormElementTest.php

Spacing +5 added lines, -5 removed lines

@@ -51,22 +51,22 @@
     $this->assertFieldByXPath("//input[@type='hidden' and @id='edit-fancy-entity-browser-target']", '', "Entity browser's hidden element found.");
 
     $edit = [
-      'fancy_entity_browser[entity_ids]' => $this->nodes[0]->getEntityTypeId() . ':' . $this->nodes[0]->id() . ' ' . $this->nodes[1]->getEntityTypeId() . ':' . $this->nodes[1]->id(),
+      'fancy_entity_browser[entity_ids]' => $this->nodes[0]->getEntityTypeId().':'.$this->nodes[0]->id().' '.$this->nodes[1]->getEntityTypeId().':'.$this->nodes[1]->id(),
     ];
     $this->drupalPostForm(NULL, $edit, 'Submit');
-    $expected = 'Selected entities: ' . $this->nodes[0]->label() . ', ' . $this->nodes[1]->label();
+    $expected = 'Selected entities: '.$this->nodes[0]->label().', '.$this->nodes[1]->label();
     $this->assertText($expected, 'Selected entities detected.');
 
-    $default_entity = $this->nodes[0]->getEntityTypeId() . ':' . $this->nodes[0]->id();
+    $default_entity = $this->nodes[0]->getEntityTypeId().':'.$this->nodes[0]->id();
     $this->drupalGet('/test-element', ['query' => ['default_entity' => $default_entity]]);
     $this->assertLink('Select entities', 0, 'Trigger link found.');
     $this->assertFieldByXPath("//input[@type='hidden' and @id='edit-fancy-entity-browser-target']", $default_entity, "Entity browser's hidden element found.");
 
     $edit = [
-      'fancy_entity_browser[entity_ids]' => $this->nodes[1]->getEntityTypeId() . ':' . $this->nodes[1]->id() . ' ' . $this->nodes[0]->getEntityTypeId() . ':' . $this->nodes[0]->id(),
+      'fancy_entity_browser[entity_ids]' => $this->nodes[1]->getEntityTypeId().':'.$this->nodes[1]->id().' '.$this->nodes[0]->getEntityTypeId().':'.$this->nodes[0]->id(),
     ];
     $this->drupalPostForm(NULL, $edit, 'Submit');
-    $expected = 'Selected entities: ' . $this->nodes[1]->label() . ', ' . $this->nodes[0]->label();
+    $expected = 'Selected entities: '.$this->nodes[1]->label().', '.$this->nodes[0]->label();
     $this->assertText($expected, 'Selected entities detected.');
   }
 

src/WidgetBase.php

Doc Comments +1 added lines, -1 removed lines

@@ -276,7 +276,7 @@
   /**
    * Run widget validators.
    *
-   * @param array $entities
+   * @param \Drupal\Core\Entity\EntityInterface[] $entities
    *   Array of entity ids to validate.
    * @param array $validators
    *   Array of widget validator ids.

src/Tests/ConfigUITest.php

Spacing +10 added lines, -10 removed lines

@@ -139,9 +139,9 @@ 
     $xpath = new \DOMXPath($dom);
     foreach ($commands as $command) {
       if ($command['command'] == 'insert' && $command['method'] == 'replaceWith') {
-        $wrapperNode = $xpath->query('//*[@id="' . ltrim($command['selector'], '#') . '"]')->item(0);
+        $wrapperNode = $xpath->query('//*[@id="'.ltrim($command['selector'], '#').'"]')->item(0);
         $newDom = new \DOMDocument();
-        @$newDom->loadHTML('<div>' . $command['data'] . '</div>');
+        @$newDom->loadHTML('<div>'.$command['data'].'</div>');
         $newNode = @$dom->importNode($newDom->documentElement->firstChild->firstChild, TRUE);
         $wrapperNode->parentNode->replaceChild($newNode, $wrapperNode);
         $content = $dom->saveHTML();
@@ -232,14 +232,14 @@ 
     $this->assertUrl('/admin/config/content/entity_browser/test_entity_browser/selection_display', ['query' => ['js' => 'nojs']]);
 
     $this->drupalPostForm(NULL, [], 'Next');
-    $this->assertFieldById('edit-table-' . $first_uuid . '-label', 'upload', 'Correct value for widget label found.');
-    $this->assertFieldById('edit-table-' . $first_uuid . '-form-upload-location', 'public://', 'Correct value for upload location found.');
-    $this->assertFieldByXPath("//input[@data-drupal-selector='edit-table-" . $first_uuid . "-form-submit-text']", 'Select files', 'Correct value for submit text found.');
-    $this->assertFieldById('edit-table-' . $second_uuid . '-label', 'entity_form', 'Correct value for widget label found.');
-    $this->assertOptionSelectedWithDrupalSelector('edit-table-' . $second_uuid . '-form-entity-type', 'user', 'Correct value for entity type found.');
-    $this->assertOptionSelectedWithDrupalSelector('edit-table-' . $second_uuid . '-form-bundle-select', 'user', 'Correct value for bundle found.');
-    $this->assertOptionSelectedWithDrupalSelector('edit-table-' . $second_uuid . '-form-form-mode-form-select', 'register', 'Correct value for form modes found.');
-    $this->assertFieldByXPath("//input[@data-drupal-selector='edit-table-" . $second_uuid . "-form-submit-text']", 'But some are more equal than others', 'Correct value for submit text found.');
+    $this->assertFieldById('edit-table-'.$first_uuid.'-label', 'upload', 'Correct value for widget label found.');
+    $this->assertFieldById('edit-table-'.$first_uuid.'-form-upload-location', 'public://', 'Correct value for upload location found.');
+    $this->assertFieldByXPath("//input[@data-drupal-selector='edit-table-".$first_uuid."-form-submit-text']", 'Select files', 'Correct value for submit text found.');
+    $this->assertFieldById('edit-table-'.$second_uuid.'-label', 'entity_form', 'Correct value for widget label found.');
+    $this->assertOptionSelectedWithDrupalSelector('edit-table-'.$second_uuid.'-form-entity-type', 'user', 'Correct value for entity type found.');
+    $this->assertOptionSelectedWithDrupalSelector('edit-table-'.$second_uuid.'-form-bundle-select', 'user', 'Correct value for bundle found.');
+    $this->assertOptionSelectedWithDrupalSelector('edit-table-'.$second_uuid.'-form-form-mode-form-select', 'register', 'Correct value for form modes found.');
+    $this->assertFieldByXPath("//input[@data-drupal-selector='edit-table-".$second_uuid."-form-submit-text']", 'But some are more equal than others', 'Correct value for submit text found.');
 
     $this->drupalPostForm(NULL, [], 'Finish');
 

src/Tests/EntityBrowserUpdateHookTest.php

Spacing +3 added lines, -3 removed lines

@@ -16,8 +16,8 @@ 
    */
   protected function setDatabaseDumpFiles() {
     $this->databaseDumpFiles = [
-      DRUPAL_ROOT . '/core/modules/system/tests/fixtures/update/drupal-8.bare.standard.php.gz',
-      __DIR__ . '/../../tests/fixtures/update/entity_browser.update-hook-test.php',
+      DRUPAL_ROOT.'/core/modules/system/tests/fixtures/update/drupal-8.bare.standard.php.gz',
+      __DIR__.'/../../tests/fixtures/update/entity_browser.update-hook-test.php',
     ];
   }
 
@@ -52,7 +52,7 @@ 
       ->get('entity_browser.browser.test_update');
 
     $this->assertNull($browser->get('submit_text'), 'Old submit text is gone');
-    $this->assertEqual($browser->get('widgets.a4ad947c-9669-497c-9988-24351955a02f.settings.submit_text'), 'All animals are created equal','New submit text appears on the widget.');
+    $this->assertEqual($browser->get('widgets.a4ad947c-9669-497c-9988-24351955a02f.settings.submit_text'), 'All animals are created equal', 'New submit text appears on the widget.');
   }
 
 }

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();
         }

modules/entity_form/src/Plugin/EntityBrowser/Widget/EntityForm.php

Spacing +9 added lines, -9 removed lines

@@ -98,7 +98,7 @@ 
    * {@inheritdoc}
    */
   public function getForm(array &$original_form, FormStateInterface $form_state, array $aditional_widget_parameters) {
-    if (empty($this->configuration['entity_type']) || empty($this->configuration['bundle'])  || empty($this->configuration['form_mode'])) {
+    if (empty($this->configuration['entity_type']) || empty($this->configuration['bundle']) || empty($this->configuration['form_mode'])) {
       return ['#markup' => $this->t('The settings for this widget (Entity type, Bundle or Form mode) are not configured correctly.')];
     }
 
@@ -106,7 +106,7 @@ 
 
     // Pretend to be IEFs submit button.
     $form['#submit'] = [['Drupal\inline_entity_form\ElementSubmit', 'trigger']];
-    $form['actions']['submit']['#ief_submit_trigger']  = TRUE;
+    $form['actions']['submit']['#ief_submit_trigger'] = TRUE;
     $form['actions']['submit']['#ief_submit_trigger_all'] = TRUE;
 
     $form['inline_entity_form'] = [
@@ -134,7 +134,7 @@ 
     $entities = $this->prepareEntities($form, $form_state);
     array_walk(
       $entities,
-      function (EntityInterface $entity) {
+      function(EntityInterface $entity) {
         $entity->save();
       }
     );
@@ -155,7 +155,7 @@ 
     $definitions = $this->entityTypeManager->getDefinitions();
     $entity_types = array_combine(
       array_keys($definitions),
-      array_map(function (EntityTypeInterface $item) {
+      array_map(function(EntityTypeInterface $item) {
         return $item->getLabel();
       }, $definitions)
     );
@@ -173,7 +173,7 @@ 
     $bundles = [];
     if ($entity_type) {
       $definitions = $this->entityTypeBundleInfo->getBundleInfo($entity_type);
-      $bundles = array_map(function ($item) {
+      $bundles = array_map(function($item) {
         return $item['label'];
       }, $definitions);
     }
@@ -186,7 +186,7 @@ 
         '#options' => $bundles,
         '#default_value' => $bundle,
       ],
-      '#attributes' => ['id' => 'bundle-wrapper-' . $this->uuid()],
+      '#attributes' => ['id' => 'bundle-wrapper-'.$this->uuid()],
     ];
 
     $form['form_mode'] = [
@@ -197,7 +197,7 @@ 
         '#default_value' => $form_mode,
         '#options' => $this->entityDisplayRepository->getFormModeOptions($entity_type),
       ],
-      '#attributes' => ['id' => 'form-mode-wrapper-' . $this->uuid()],
+      '#attributes' => ['id' => 'form-mode-wrapper-'.$this->uuid()],
     ];
 
     return $form;
@@ -222,8 +222,8 @@ 
    */
   public function updateFormElements($form, FormStateInterface $form_state) {
     $response = new AjaxResponse();
-    $response->addCommand(new ReplaceCommand('#bundle-wrapper-' . $this->uuid(), $this->updateBundle($form, $form_state)));
-    $response->addCommand(new ReplaceCommand('#form-mode-wrapper-' . $this->uuid(), $this->updateFormMode($form, $form_state)));
+    $response->addCommand(new ReplaceCommand('#bundle-wrapper-'.$this->uuid(), $this->updateBundle($form, $form_state)));
+    $response->addCommand(new ReplaceCommand('#form-mode-wrapper-'.$this->uuid(), $this->updateFormMode($form, $form_state)));
     return $response;
   }
 

src/Plugin/EntityBrowser/SelectionDisplay/View.php

Spacing +3 added lines, -3 removed lines

@@ -50,7 +50,7 @@ 
     // can't do much about it.
     $selected_entities = $form_state->get(['entity_browser', 'selected_entities']);
     if (!empty($selected_entities)) {
-      $ids = array_map(function (EntityInterface $item) {
+      $ids = array_map(function(EntityInterface $item) {
         return $item->id();
       }, $selected_entities);
       $storage['selection_display_view']->setArguments([implode(',', $ids)]);
@@ -86,14 +86,14 @@ 
     $views = Views::getAllViews();
     foreach ($views as $view_id => $view) {
       foreach ($view->get('display') as $display_id => $display) {
-        $options[$view_id . '.' . $display_id] = $this->t('@view : @display', array('@view' => $view->label(), '@display' => $display['display_title']));
+        $options[$view_id.'.'.$display_id] = $this->t('@view : @display', array('@view' => $view->label(), '@display' => $display['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,
       '#required' => TRUE,
       '#description' => $this->t('View display to use for displaying currently selected items. Do note that to get something usefull out of this display, its first contextual filter should be a filter on the primary identifier field of your entity type (e.g., Node ID, Media ID).'),

src/Plugin/EntityBrowser/SelectionDisplay/MultiStepDisplay.php

Spacing +4 added lines, -4 removed lines

@@ -102,7 +102,7 @@ 
         $display = ['#markup' => $display];
       }
 
-      $form['selected']['items_' . $entity->id() . '_' . $id] = [
+      $form['selected']['items_'.$entity->id().'_'.$id] = [
         '#theme_wrappers' => ['container'],
         '#attributes' => [
           'class' => ['item-container'],
@@ -113,10 +113,10 @@ 
           '#type' => 'submit',
           '#value' => $this->t('Remove'),
           '#submit' => [[get_class($this), 'removeItemSubmit']],
-          '#name' => 'remove_' . $entity->id() . '_' . $id,
+          '#name' => 'remove_'.$entity->id().'_'.$id,
           '#attributes' => [
             'data-row-id' => $id,
-            'data-remove-entity' => 'items_' . $entity->id(),
+            'data-remove-entity' => 'items_'.$entity->id(),
           ],
         ],
         'weight' => [
@@ -158,7 +158,7 @@ 
     // Remove weight of entity being removed.
     $form_state->unsetValue([
       'selected',
-      $triggering_element['#attributes']['data-remove-entity'] . '_' . $triggering_element['#attributes']['data-row-id'],
+      $triggering_element['#attributes']['data-remove-entity'].'_'.$triggering_element['#attributes']['data-row-id'],
     ]);
 
     // Remove entity itself.

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

@@ -146,7 +146,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',
@@ -189,7 +189,7 @@ 
       '#type' => 'fieldset',
       '#title' => $this->t('Entity display plugin configuration'),
       '#tree' => TRUE,
-      '#prefix' => '<div id="' . $id . '">',
+      '#prefix' => '<div id="'.$id.'">',
       '#suffix' => '</div>',
     ];
 
@@ -272,7 +272,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();
   }
 
   /**
@@ -284,7 +284,7 @@ 
 
     // Get correct ordered list of entity IDs.
     $ids = array_map(
-      function (EntityInterface $entity) {
+      function(EntityInterface $entity) {
         return $entity->id();
       },
       $entities
@@ -298,8 +298,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,
@@ -314,8 +314,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
         )),
@@ -364,7 +364,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;
   }
 
@@ -469,7 +469,7 @@ 
       '#theme_wrappers' => ['container'],
       '#attributes' => ['class' => ['entities-list']],
       '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);
           if (is_string($display)) {
             $display = ['#markup' => $display];
@@ -478,7 +478,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,
@@ -490,10 +490,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'),
@@ -631,7 +631,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']);
       }
 
@@ -688,7 +688,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;