Contrib.social

Fix the issues reported by phpcs

Open on Drupal.org ā†’
Created on 28 February 2019, over 5 years ago
Updated 7 June 2024, 6 months ago

Running phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig shows the following warnings/errors that needs to be fixed.

FILE: ./entity_browser_8.x-2.x/tests/fixtures/update/entity_browser.update-hook-test.php
  4 | ERROR | [x] Doc comment short description must be on the first line
 85 | ERROR | [ ] unserialize() is insecure unless allowed classes are limited. Use a safe format like JSON or use the
    |       |     allowed_classes option.

FILE: ./entity_browser_8.x-2.x/tests/modules/entity_browser_test/entity_browser_test.routing.yml
 7 | WARNING | Open page callback found, please add a comment before the line why there is no access restriction

FILE: ./entity_browser_8.x-2.x/tests/modules/entity_browser_test/src/Form/FormElementTest.php
 33 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
 37 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead

FILE: ./entity_browser_8.x-2.x/tests/modules/entity_browser_test/src/Plugin/EntityBrowser/Widget/DummyWidget.php
 63 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead

FILE: ./entity_browser_8.x-2.x/tests/src/Kernel/Extension/EntityBrowserTest.php
 266 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
     |         | instead
 461 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
     |         | instead
 468 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
     |         | instead

FILE: ./entity_browser_8.x-2.x/tests/src/Kernel/Plugin/FieldWidgetDisplayTest.php
 70 | ERROR | The array declaration extends to column 97 (the limit is 80). The array content should be split up over multiple
    |       | lines

FILE: ./entity_browser_8.x-2.x/tests/src/FunctionalJavascript/ConfigurationTest.php
 202 | WARNING | Unused variable $entity_type.

FILE: ./entity_browser_8.x-2.x/tests/src/FunctionalJavascript/InlineEntityFormTest.php
 424 | ERROR | [x] list(...) is forbidden, use [...] instead.

FILE: ./entity_browser_8.x-2.x/tests/src/FunctionalJavascript/FieldWidgetConfigTest.php
 153 | WARNING | Line exceeds 80 characters; contains 104 characters

FILE: ./entity_browser_8.x-2.x/tests/src/FunctionalJavascript/EntityReferenceWidgetTest.php
 590 | ERROR | [x] list(...) is forbidden, use [...] instead.

FILE: ./entity_browser_8.x-2.x/tests/src/FunctionalJavascript/ParagraphsTest.php
 50 | ERROR | The array declaration extends to column 84 (the limit is 80). The array content should be split up over multiple
    |       | lines
 51 | ERROR | The array declaration extends to column 98 (the limit is 80). The array content should be split up over multiple
    |       | lines

FILE: ./entity_browser_8.x-2.x/tests/src/FunctionalJavascript/EntityBrowserTest.php
  44 | WARNING | [x] '@TODO Test the edit button.' should match the format '@todo Fix problem X here.'
 142 | WARNING | [ ] Unused variable $image2.
 186 | WARNING | [ ] Line exceeds 80 characters; contains 96 characters

FILE: ./entity_browser_8.x-2.x/tests/src/FunctionalJavascript/CardinalityTest.php
 230 | ERROR   | The array declaration extends to column 84 (the limit is 80). The array content should be split up over multiple
     |         | lines
 238 | WARNING | Line exceeds 80 characters; contains 82 characters
 306 | ERROR   | The array declaration extends to column 84 (the limit is 80). The array content should be split up over multiple
     |         | lines
 315 | WARNING | Line exceeds 80 characters; contains 82 characters

FILE: ./entity_browser_8.x-2.x/tests/src/FunctionalJavascript/EntityBrowserWebDriverTestBase.php
 254 | WARNING | Line exceeds 80 characters; contains 83 characters
 288 | WARNING | Line exceeds 80 characters; contains 86 characters

FILE: ./entity_browser_8.x-2.x/tests/src/FunctionalJavascript/PluginsTest.php
 228 | WARNING | [x] 'TODO test if entities were selected. Will most likely need a custom event' should match the format '@todo Fix
     |         |     problem X here.'

FILE: ./entity_browser_8.x-2.x/tests/src/Functional/EntityBrowserUITest.php
 37 | ERROR | Public method name "EntityBrowserUITest::testEntityBrowserUI" is not in lowerCamel format

FILE: ./entity_browser_8.x-2.x/tests/src/Functional/FormElementTest.php
 73 | ERROR | The array declaration extends to column 151 (the limit is 80). The array content should be split up over multiple
    |       | lines
 73 | ERROR | The array declaration extends to column 150 (the limit is 80). The array content should be split up over multiple
    |       | lines

FILE: ./entity_browser_8.x-2.x/entity_browser.install
 70 | WARNING | Unused variable $entity_type_name.
 83 | WARNING | Unused variable $display_name.
 85 | WARNING | Unused variable $field_name.

FILE: ./entity_browser_8.x-2.x/entity_browser.views.inc
 14 | WARNING | Unused variable $entity_type_name.

FILE: ./entity_browser_8.x-2.x/modules/entity_form/entity_browser_entity_form.module
  32 | WARNING | [x] 'TODO - 'default' might become configurable or something else in the future.' should match the format '@todo
     |         |     Fix problem X here.'
  83 | WARNING | [x] 'TODO see if we can get away without overriding entire IEF function.' should match the format '@todo Fix
     |         |     problem X here.'
 106 | WARNING | [ ] Unused variable $key.
 121 | WARNING | [x] 'TODO see if we can get away without overriding entire IEF function.' should match the format '@todo Fix
     |         |     problem X here.'
 158 | ERROR   | [ ] The array declaration extends to column 107 (the limit is 80). The array content should be split up over
     |         |     multiple lines

FILE: ./entity_browser_8.x-2.x/modules/entity_form/src/Plugin/EntityBrowser/Widget/EntityForm.php
 155 | ERROR | The array declaration extends to column 147 (the limit is 80). The array content should be split up over multiple
     |       | lines
 156 | ERROR | The array declaration extends to column 89 (the limit is 80). The array content should be split up over multiple
     |       | lines
 156 | ERROR | The array declaration extends to column 166 (the limit is 80). The array content should be split up over multiple
     |       | lines

FILE: ./entity_browser_8.x-2.x/src/Ajax/ValueUpdatedCommand.php
 17 | ERROR | Class property $details_id should use lowerCamel naming without underscores

FILE: ./entity_browser_8.x-2.x/src/Ajax/SelectEntitiesCommand.php
 9 | ERROR | Doc comment short description must be on a single line, further text should be a separate paragraph

FILE: ./entity_browser_8.x-2.x/src/WidgetInterface.php
 78 | WARNING | Line exceeds 80 characters; contains 83 characters

FILE: ./entity_browser_8.x-2.x/src/WidgetBase.php
 117 | ERROR | The array declaration extends to column 86 (the limit is 80). The array content should be split up over multiple
     |       | lines
 332 | ERROR | Parameter $form_state is not described in comment
 339 | ERROR | The array declaration extends to column 82 (the limit is 80). The array content should be split up over multiple
     |       | lines
 359 | ERROR | Doc comment short description must be on a single line, further text should be a separate paragraph

FILE: ./entity_browser_8.x-2.x/src/DisplayBase.php
 179 | ERROR | The array declaration extends to column 93 (the limit is 80). The array content should be split up over multiple
     |       | lines

FILE: ./entity_browser_8.x-2.x/src/SelectionDisplayBase.php
 133 | ERROR | The trigger_error message 'checkPreselectionSupport method is deprecated. Use supportsPreselection instead.' does
     |       | not match the relaxed standard format: %thing% is deprecated in %deprecation-version% any free text
     |       | %removal-version%. %extra-info%. See %cr-link%

FILE: ./entity_browser_8.x-2.x/src/Element/EntityBrowserElement.php
 257 | ERROR | [x] list(...) is forbidden, use [...] instead.

FILE: ./entity_browser_8.x-2.x/src/EntityBrowserInterface.php
 53 | ERROR | Parameter $widget_selector is not described in comment
 56 | ERROR | Doc comment for parameter $display does not match actual variable name $widget_selector
 64 | ERROR | Parameter $selection_display is not described in comment
 67 | ERROR | Doc comment for parameter $display does not match actual variable name $selection_display

FILE: ./entity_browser_8.x-2.x/src/WidgetSelectorBase.php
 28 | ERROR | [ ] Class property $widgets_ids should use lowerCamel naming without underscores
 43 | ERROR | [x] Use null coalesce operator instead of ternary operator.

FILE: ./entity_browser_8.x-2.x/src/Form/EntityBrowserForm.php
 131 | ERROR | The array declaration extends to column 100 (the limit is 80). The array content should be split up over multiple
     |       | lines
 152 | ERROR | The array declaration extends to column 109 (the limit is 80). The array content should be split up over multiple
     |       | lines

FILE: ./entity_browser_8.x-2.x/src/Form/WidgetsConfig.php
 164 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
     |         | instead

FILE: ./entity_browser_8.x-2.x/src/Form/EntityBrowserEditForm.php
  17 | WARNING | The class short comment should describe what the class does and not simply repeat the class name
 340 | WARNING | Unused variable $plugin_id.

FILE: ./entity_browser_8.x-2.x/src/RouteSubscriber.php
 50 | WARNING | [x] The variable name should be defined after the type
 66 | ERROR   | [ ] Protected method name "RouteSubscriber::getBrowserIDsWithRoute" is not in lowerCamel format

FILE: ./entity_browser_8.x-2.x/src/Events/EventBase.php
 45 | ERROR | Public method name "EventBase::getBrowserID" is not in lowerCamel format
 55 | ERROR | Public method name "EventBase::getBrowserInstanceUUID" is not in lowerCamel format

FILE: ./entity_browser_8.x-2.x/src/Events/Events.php
 12 | ERROR   | Doc comment short description must be on a single line, further text should be a separate paragraph
 19 | WARNING | Line exceeds 80 characters; contains 82 characters
 20 | WARNING | Line exceeds 80 characters; contains 85 characters
 20 | ERROR   | Doc comment short description must be on a single line, further text should be a separate paragraph
 27 | WARNING | Line exceeds 80 characters; contains 82 characters
 29 | ERROR   | Doc comment short description must be on a single line, further text should be a separate paragraph
 36 | WARNING | Line exceeds 80 characters; contains 81 characters
 37 | ERROR   | Doc comment short description must be on a single line, further text should be a separate paragraph

FILE: ./entity_browser_8.x-2.x/src/Events/EntitySelectionEvent.php
 35 | ERROR | Description for the @return value is missing

FILE: ./entity_browser_8.x-2.x/src/SelectionDisplayInterface.php
 69 | ERROR | The text '@deprecated Use ::supportsPreselection instead.' does not match the standard format: @deprecated in
    |       | %deprecation-version% and is removed from %removal-version%. %extra-info%.
 69 | ERROR | Each @deprecated tag must have a @see tag immediately following it

FILE: ./entity_browser_8.x-2.x/src/Plugin/Field/FieldWidget/FileBrowserWidget.php
 101 | ERROR | [ ] Doc comment for parameter $mime_type_guesser does not match actual variable name 
 370 | ERROR | [ ] The array declaration extends to column 133 (the limit is 80). The array content should be split up over
     |       |     multiple lines
 389 | ERROR | [ ] The array declaration extends to column 103 (the limit is 80). The array content should be split up over
     |       |     multiple lines
 406 | ERROR | [ ] The array declaration extends to column 103 (the limit is 80). The array content should be split up over
     |       |     multiple lines
 505 | ERROR | [x] Use null coalesce operator instead of ternary operator.
 511 | ERROR | [ ] The array declaration extends to column 129 (the limit is 80). The array content should be split up over
     |       |     multiple lines

FILE: ./entity_browser_8.x-2.x/src/Plugin/Field/FieldWidget/EntityReferenceBrowserWidget.php
 443 | ERROR   | [ ] The array declaration extends to column 89 (the limit is 80). The array content should be split up over
     |         |     multiple lines
 530 | WARNING | [ ] Unused variable $row_id.
 610 | ERROR   | [ ] The array declaration extends to column 97 (the limit is 80). The array content should be split up over
     |         |     multiple lines
 810 | ERROR   | [ ] The array declaration extends to column 81 (the limit is 80). The array content should be split up over
     |         |     multiple lines
 919 | WARNING | [x] 'TODO Figure out how to avoid using raw user input.' should match the format '@todo Fix problem X here.'

FILE: ./entity_browser_8.x-2.x/src/Plugin/views/display/EntityBrowser.php
 133 | ERROR | [x] Use null coalesce operator instead of ternary operator.

FILE: ./entity_browser_8.x-2.x/src/Plugin/views/filter/ContextualBundle.php
 53 | ERROR | Parameter $selection_storage is not described in comment

FILE: ./entity_browser_8.x-2.x/src/Plugin/EntityBrowser/Widget/View.php
  93 | WARNING | [x] 'TODO - do we need better error handling for view and view_display (in' should match the format '@todo Fix
     |         |     problem X here.'
 108 | ERROR   | [x] Use null coalesce operator instead of ternary operator.
 134 | ERROR   | [ ] The array declaration extends to column 149 (the limit is 80). The array content should be split up over
     |         |     multiple lines
 135 | ERROR   | [ ] The array declaration extends to column 130 (the limit is 80). The array content should be split up over
     |         |     multiple lines
 136 | ERROR   | [ ] The array declaration extends to column 127 (the limit is 80). The array content should be split up over
     |         |     multiple lines
 141 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
 232 | ERROR   | [x] list(...) is forbidden, use [...] instead.
 260 | ERROR   | [x] list(...) is forbidden, use [...] instead.
 262 | ERROR   | [ ] The array declaration extends to column 170 (the limit is 80). The array content should be split up over
     |         |     multiple lines
 285 | ERROR   | [x] list(...) is forbidden, use [...] instead.
 297 | WARNING | [ ] ViewEntity::load calls should be avoided in classes, use dependency injection instead

FILE: ./entity_browser_8.x-2.x/src/Plugin/EntityBrowser/Widget/Upload.php
 101 | ERROR   | The array declaration extends to column 104 (the limit is 80). The array content should be split up over multiple
     |         | lines
 102 | ERROR   | The array declaration extends to column 99 (the limit is 80). The array content should be split up over multiple
     |         | lines
 102 | ERROR   | The array declaration extends to column 177 (the limit is 80). The array content should be split up over multiple
     |         | lines
 157 | WARNING | Line exceeds 80 characters; contains 81 characters

FILE: ./entity_browser_8.x-2.x/src/Plugin/EntityBrowser/SelectionDisplay/View.php
  41 | WARNING | [x] 'TODO - do we need better error handling for view and view_display' should match the format '@todo Fix problem
     |         |     X here.'
  52 | WARNING | [x] 'TODO - if there are entities that are selected multiple times this' should match the format '@todo Fix
     |         |     problem X here.'
  55 | ERROR   | [ ] The array declaration extends to column 81 (the limit is 80). The array content should be split up over
     |         |     multiple lines
  93 | ERROR   | [ ] The array declaration extends to column 146 (the limit is 80). The array content should be split up over
     |         |     multiple lines
 116 | ERROR   | [x] list(...) is forbidden, use [...] instead.

FILE: ./entity_browser_8.x-2.x/src/Plugin/EntityBrowser/SelectionDisplay/MultiStepDisplay.php
  94 | ERROR | The array declaration extends to column 81 (the limit is 80). The array content should be split up over multiple
     |       | lines
 205 | ERROR | The array declaration extends to column 86 (the limit is 80). The array content should be split up over multiple
     |       | lines
 374 | ERROR | The array declaration extends to column 82 (the limit is 80). The array content should be split up over multiple
     |       | lines
 401 | ERROR | The array declaration extends to column 83 (the limit is 80). The array content should be split up over multiple
     |       | lines

FILE: ./entity_browser_8.x-2.x/src/Plugin/EntityBrowser/FieldWidgetDisplay/RenderedEntity.php
 83 | ERROR | [x] Expected 1 space after "?"; 2 found
 84 | ERROR | [x] Expected 1 space after ":"; 2 found

FILE: ./entity_browser_8.x-2.x/src/Plugin/EntityBrowser/FieldWidgetDisplay/ImageThumbnail.php
 74 | ERROR | [x] Expected 1 space after "?"; 2 found
 75 | ERROR | [x] Expected 1 space after ":"; 2 found

FILE: ./entity_browser_8.x-2.x/src/Plugin/EntityBrowser/Display/Standalone.php
 48 | WARNING | [ ] Possible useless method overriding detected
 50 | WARNING | [x] '@TODO Implement it.' should match the format '@todo Fix problem X here.'
 57 | WARNING | [x] '@TODO Implement it.' should match the format '@todo Fix problem X here.'

FILE: ./entity_browser_8.x-2.x/src/Entity/EntityBrowser.php
 323 | WARNING | [x] 'TODO - this doesn't make much sense. Refactor.' should match the format '@todo Fix problem X here.'
 332 | WARNING | [x] 'TODO - this doesn't make much sense. Refactor.' should match the format '@todo Fix problem X here.'
 386 | WARNING | [x] 'TODO: Allow displays to define more than just path.' should match the format '@todo Fix problem X here.'

FILE: ./entity_browser_8.x-2.x/src/Controllers/EntityBrowserController.php
 77 | ERROR | The array declaration extends to column 194 (the limit is 80). The array content should be split up over multiple
    |       | lines

FILE: ./entity_browser_8.x-2.x/src/Controllers/EntityBrowserListBuilder.php
 33 | WARNING | [x] Inline @var declarations should use the /** */ delimiters

FILE: ./entity_browser_8.x-2.x/src/Controllers/EntityBrowserFormController.php
 107 | WARNING | [x] Inline @var declarations should use the /** */ delimiters
 109 | WARNING | [x] Inline @var declarations should use the /** */ delimiters

FILE: ./entity_browser_8.x-2.x/entity_browser.module
  81 | ERROR | [ ] The array declaration extends to column 198 (the limit is 80). The array content should be split up over
     |       |     multiple lines
 148 | ERROR | [x] list(...) is forbidden, use [...] instead.
 166 | ERROR | [x] list(...) is forbidden, use [...] instead.

FILE: ./entity_browser_8.x-2.x/entity_browser.api.php
 14 | WARNING | Line exceeds 80 characters; contains 92 characters
 24 | WARNING | Line exceeds 80 characters; contains 91 characters
 34 | WARNING | Line exceeds 80 characters; contains 88 characters
 44 | WARNING | Line exceeds 80 characters; contains 99 characters
 54 | WARNING | Line exceeds 80 characters; contains 103 characters
 65 | WARNING | Line exceeds 80 characters; contains 101 characters
šŸ“Œ Task
Status

Needs work

Version

2.0

Component

Miscellaneous

Created by

šŸ‡®šŸ‡³India jigish.addweb

Live updates comments and jobs are added and updated live.
  • Coding standards

    It involves compliance with, or the content of coding standards. Requires broad community agreement.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • Comment over 1 year ago ā†’
    šŸ‡®šŸ‡¹Italy apaderno Brescia, šŸ‡®šŸ‡¹

    I am not sure why only the Drupal ruleset is used, when there is also the DrupalPractice ruleset.

  • Status changed to Needs work over 1 year ago
  • Comment over 1 year ago ā†’
    šŸ‡®šŸ‡¹Italy apaderno Brescia, šŸ‡®šŸ‡¹
  • Assigned to Kaustab_Roy
  • Comment over 1 year ago ā†’
    šŸ‡®šŸ‡³India Kaustab_Roy
  • Issue was unassigned.
  • Comment over 1 year ago ā†’
    šŸ‡®šŸ‡¹Italy apaderno Brescia, šŸ‡®šŸ‡¹

    Issues should be assigned to somebody who creates a patch or a MR in the next hours, not after more than 12 hours. This means blocking an issue for which somebody else could have worked on.

  • Comment over 1 year ago ā†’
    šŸ‡®šŸ‡³India sakthi_dev

    Created a patch after verifying with DrupalPractice ruleset with . Please review.

  • Comment over 1 year ago ā†’
    šŸ‡®šŸ‡¹Italy apaderno Brescia, šŸ‡®šŸ‡¹
      * @addtogroup hooks
- * @{

    The last line is necessary and it should not be removed.

     /**
- * Alter the information provided in \Drupal\entity_browser\Annotation\EntityBrowserDisplay.
+ * Alter the information provided in.
+ *
+ * \Drupal\entity_browser\Annotation\EntityBrowserDisplay.

    Short descriptions cannot be split like that.
    Alter must use the third person singular.

    - * TODO see if we can get away without overriding entire IEF function.
+ * @todo see if we can get away without overriding entire IEF function.

    What follows @todo is a sentence: It starts with a capitalized word and ends with a period (preferable), a question mark, or an exclamation mark.

     /**
- * AJAX command to rerender a formatted text field without any transformation
- * filters.
+ * AJAX command to rerender a formatted text without transformation filters.
  */

    I am not sure that removing field is correct. The existing comment is about a field, while the changed comment is about a formatted text (which is not a field).

       /**
-   * The ID for the details element.
+   * Details element ID.

    That description is missing a definite article.

     /**
- * Class EntityBrowserEditForm.
+ * Entity browser edit form class.
  */
 class EntityBrowserEditForm extends EntityForm {

    That description does not say what the class does.

    -    if ($storage = $this->selectionStorage->get($form_state->get(['entity_browser', 'instance_uuid']))) {
+    if ($storage = $this->selectionStorage->get($form_state->get([
+      'entity_browser',
+      'instance_uuid',
+    ]))) {

    As per Drupal coding standards, control structure conditions can be written on a single line, if they are more readable. In this case, the existing code is more readable.

    -  /**
-   * {@inheritdoc}
-   */
-  public function displayEntityBrowser(array $element, FormStateInterface $form_state, array &$complete_form, array $persistent_data = []) {
-    parent::displayEntityBrowser($element, $form_state, $complete_form, $persistent_data);
-    // @TODO Implement it.
-  }
-
   /**
    * {@inheritdoc}
    */
   public function selectionCompleted(array $entities) {
-    // @TODO Implement it.
+    // @todo Implement it.
   }

    The first method cannot be removed, exactly for the same reason the other one has not been removed: It contains a @todo tag.

    +  /**
+   * Turns a render array into an HTML string.
+   *
+   * @var Drupal\Core\Render\RendererInterface
+   */
+  public $renderer;

    That description says for what that property is used; it needs to say what that property contains.

    +  /**
+   * Constructs a new View object.
+   *

    The description for a constructor must start with Constructs a new followed by the class name (including its namespace), and end with object.

    +   * @param string $plugin_id
+   *   The plugin_id for the plugin instance.

    plugin_id is not an English word.
    It is not necessary to say for the plugin instance, since that is a plugin ID.

    +   * @param \Symfony\Component\EventDispatcher\EventDispatcherInterface $event_dispatcher
+   *   Event dispatcher service.

    The description is missing a definite article.

    +   * @param \Drupal\entity_browser\WidgetValidationManager $validation_manager
+   *   The Widget Validation Manager service.

    Only the first word in the description must be capitalized.
    There is no need to say service.

    +   * @param Drupal\Core\Render\RendererInterface $renderer
+   *   Turns a render array into an HTML string.

    That is not the description of what $render contains.

    +    /*
+     * @deprecated and removed in entity_browser:8.x-2.2.
+     *   Use ::supportsPreselection instead.
+     *   @see
+     */

    @deprecated must be added to the method documentation comment, not inside the method.

    +   * @param \Drupal\Core\Form\FormStateInterface $form_state
+   *   Drupal form_state.

    There is no need to say Drupal.
    form_state is not an English word.

    -  protected $widgets_ids;
+  protected $widgetsIds;
 
   /**
    * ID of the default widget.
@@ -40,7 +40,7 @@ abstract class WidgetSelectorBase extends PluginBase implements WidgetSelectorIn
   public function __construct($configuration, $plugin_id, $plugin_definition) {
     parent::__construct($configuration, $plugin_id, $plugin_definition);
     $this->setConfiguration($configuration);
-    $this->widget_ids = isset($this->configuration['widget_ids']) ? $this->configuration['widget_ids'] : [];
+    $this->widget_ids = isset($this->configuration['widget_ids']) ?? [];

    Since the property name has been changed, $this->widget_ids is setting the wrong property.

  • Comment over 1 year ago ā†’
    šŸ‡®šŸ‡³India roshni27
  • Assigned to imustakim
  • Comment over 1 year ago ā†’
    šŸ‡®šŸ‡³India imustakim Ahmedabad
  • Issue was unassigned.
  • Status changed to Needs review over 1 year ago
  • Open in Jenkins ā†’ Open on Drupal.org ā†’
    Core: 9.5.x + Environment: PHP 7.3 & MariaDB 10.3.22
    last update over 1 year ago
    4 pass, 42 fail
  • Comment over 1 year ago ā†’
    šŸ‡®šŸ‡³India imustakim Ahmedabad

    As per suggestions by @apaderno, Changes have been made, and to address some of the concern mentioned in #31 šŸ“Œ Fix the issues reported by phpcs Needs review , only these errors are remaining.
    Assuming these can be ignored. Rest of the errors are fixed.
    Patch is updated, please review.

    FILE: /Users/specbee/Sites/Projects/entity_browser/src/Form/EntityBrowserForm.php
---------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------------------
 131 | ERROR | The array declaration extends to column 100 (the limit is 80). The array content should be split up over multiple lines
---------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/specbee/Sites/Projects/entity_browser/src/Plugin/EntityBrowser/Widget/View.php
-------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-------------------------------------------------------------------------------------------
 41 | WARNING | Line exceeds 80 characters; contains 85 characters
-------------------------------------------------------------------------------------------


FILE: /Users/specbee/Sites/Projects/entity_browser/src/Plugin/EntityBrowser/Display/Standalone.php
--------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------
 48 | WARNING | Possible useless method overriding detected
--------------------------------------------------------------------------------------------------

Time: 3.16 secs; Memory: 18MB
  • Comment over 1 year ago ā†’
    System Message

    The last submitted patch, 34: phpcs-3036556-34.patch, failed testing. View results ā†’

  • First commit to issue fork.
  • Status changed to Needs work 10 months ago
  • Comment 10 months ago ā†’
    šŸ‡®šŸ‡³India Shreyas gowda
  • First commit to issue fork.
  • Comment 9 months ago ā†’
    shruu_rao

    Hi,
    I tried applying #37 patch. The patch throws

    error: corrupt patch at line 16

    Please check.

  • Status changed to Needs review 6 months ago
  • Comment 6 months ago ā†’
    šŸ‡®šŸ‡³India dev16.addweb

    Fixed the issues reported by phpcs.

  • Comment 6 months ago ā†’
    šŸ‡µšŸ‡­Philippines cleavinjosh

    Hi,

    I applied the patch #40 and verified that the issues reported by phpcs are fixed.

     % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 84340  100 84340    0     0   168k      0 --:--:-- --:--:-- --:--:--  168k
Checking patch entity_browser.api.php...
Checking patch entity_browser.module...
Checking patch entity_browser.views.inc...
Checking patch modules/entity_form/entity_browser_entity_form.module...
Checking patch modules/entity_form/src/Plugin/EntityBrowser/Widget/EntityForm.php...
Checking patch src/Ajax/SelectEntitiesCommand.php...
Checking patch src/Ajax/ValueUpdatedCommand.php...
Checking patch src/Controllers/EntityBrowserController.php...
Checking patch src/Controllers/EntityBrowserFormController.php...
Checking patch src/Controllers/EntityBrowserListBuilder.php...
Checking patch src/DisplayBase.php...
Checking patch src/DisplayInterface.php...
Checking patch src/Element/EntityBrowserElement.php...
Checking patch src/Entity/EntityBrowser.php...
Checking patch src/EntityBrowserInterface.php...
Checking patch src/Events/AlterEntityBrowserDisplayData.php...
Checking patch src/Events/EntitySelectionEvent.php...
Checking patch src/Events/EventBase.php...
Checking patch src/Events/Events.php...
Checking patch src/Events/RegisterJSCallbacks.php...
Checking patch src/Form/EntityBrowserEditForm.php...
Checking patch src/Form/EntityBrowserForm.php...
Checking patch src/Plugin/EntityBrowser/Display/Modal.php...
Checking patch src/Plugin/EntityBrowser/Display/Standalone.php...
Checking patch src/Plugin/EntityBrowser/FieldWidgetDisplay/ImageThumbnail.php...
Checking patch src/Plugin/EntityBrowser/FieldWidgetDisplay/RenderedEntity.php...
Checking patch src/Plugin/EntityBrowser/SelectionDisplay/MultiStepDisplay.php...
Checking patch src/Plugin/EntityBrowser/SelectionDisplay/View.php...
Checking patch src/Plugin/EntityBrowser/Widget/Upload.php...
Checking patch src/Plugin/EntityBrowser/Widget/View.php...
Checking patch src/Plugin/EntityBrowser/WidgetSelector/DropDown.php...
Checking patch src/Plugin/EntityBrowser/WidgetSelector/Single.php...
Checking patch src/Plugin/EntityBrowser/WidgetSelector/Tabs.php...
Checking patch src/Plugin/Field/FieldWidget/EntityReferenceBrowserWidget.php...
Checking patch src/Plugin/Field/FieldWidget/FileBrowserWidget.php...
Checking patch src/Plugin/views/display/EntityBrowser.php...
Checking patch src/Plugin/views/field/SelectForm.php...
Checking patch src/Plugin/views/filter/ContextualBundle.php...
Checking patch src/RouteSubscriber.php...
Checking patch src/SelectionDisplayBase.php...
Checking patch src/SelectionDisplayInterface.php...
Checking patch src/WidgetBase.php...
Checking patch src/WidgetInterface.php...
Checking patch src/WidgetSelectorBase.php...
Checking patch tests/modules/entity_browser_test/entity_browser_test.routing.yml...
Checking patch tests/modules/entity_browser_test/src/Form/FormElementTest.php...
Checking patch tests/modules/entity_browser_test/src/Plugin/EntityBrowser/Widget/DummyWidget.php...
Checking patch tests/src/Functional/EntityBrowserUITest.php...
Checking patch tests/src/Functional/FormElementTest.php...
Checking patch tests/src/FunctionalJavascript/CardinalityTest.php...
Checking patch tests/src/FunctionalJavascript/ConfigurationTest.php...
Checking patch tests/src/FunctionalJavascript/EntityBrowserTest.php...
Checking patch tests/src/FunctionalJavascript/EntityBrowserViewsWidgetTest.php...
Checking patch tests/src/FunctionalJavascript/EntityBrowserWebDriverTestBase.php...
Checking patch tests/src/FunctionalJavascript/EntityReferenceWidgetTest.php...
Checking patch tests/src/FunctionalJavascript/FieldWidgetConfigTest.php...
Checking patch tests/src/FunctionalJavascript/InlineEntityFormTest.php...
Checking patch tests/src/FunctionalJavascript/ParagraphsTest.php...
Checking patch tests/src/FunctionalJavascript/PluginsTest.php...
Checking patch tests/src/Kernel/Plugin/FieldWidgetDisplayTest.php...
Applied patch entity_browser.api.php cleanly.
Applied patch entity_browser.module cleanly.
Applied patch entity_browser.views.inc cleanly.
Applied patch modules/entity_form/entity_browser_entity_form.module cleanly.
Applied patch modules/entity_form/src/Plugin/EntityBrowser/Widget/EntityForm.php cleanly.
Applied patch src/Ajax/SelectEntitiesCommand.php cleanly.
Applied patch src/Ajax/ValueUpdatedCommand.php cleanly.
Applied patch src/Controllers/EntityBrowserController.php cleanly.
Applied patch src/Controllers/EntityBrowserFormController.php cleanly.
Applied patch src/Controllers/EntityBrowserListBuilder.php cleanly.
Applied patch src/DisplayBase.php cleanly.
Applied patch src/DisplayInterface.php cleanly.
Applied patch src/Element/EntityBrowserElement.php cleanly.
Applied patch src/Entity/EntityBrowser.php cleanly.
Applied patch src/EntityBrowserInterface.php cleanly.
Applied patch src/Events/AlterEntityBrowserDisplayData.php cleanly.
Applied patch src/Events/EntitySelectionEvent.php cleanly.
Applied patch src/Events/EventBase.php cleanly.
Applied patch src/Events/Events.php cleanly.
Applied patch src/Events/RegisterJSCallbacks.php cleanly.
Applied patch src/Form/EntityBrowserEditForm.php cleanly.
Applied patch src/Form/EntityBrowserForm.php cleanly.
Applied patch src/Plugin/EntityBrowser/Display/Modal.php cleanly.
Applied patch src/Plugin/EntityBrowser/Display/Standalone.php cleanly.
Applied patch src/Plugin/EntityBrowser/FieldWidgetDisplay/ImageThumbnail.php cleanly.
Applied patch src/Plugin/EntityBrowser/FieldWidgetDisplay/RenderedEntity.php cleanly.
Applied patch src/Plugin/EntityBrowser/SelectionDisplay/MultiStepDisplay.php cleanly.
Applied patch src/Plugin/EntityBrowser/SelectionDisplay/View.php cleanly.
Applied patch src/Plugin/EntityBrowser/Widget/Upload.php cleanly.
Applied patch src/Plugin/EntityBrowser/Widget/View.php cleanly.
Applied patch src/Plugin/EntityBrowser/WidgetSelector/DropDown.php cleanly.
Applied patch src/Plugin/EntityBrowser/WidgetSelector/Single.php cleanly.
Applied patch src/Plugin/EntityBrowser/WidgetSelector/Tabs.php cleanly.
Applied patch src/Plugin/Field/FieldWidget/EntityReferenceBrowserWidget.php cleanly.
Applied patch src/Plugin/Field/FieldWidget/FileBrowserWidget.php cleanly.
Applied patch src/Plugin/views/display/EntityBrowser.php cleanly.
Applied patch src/Plugin/views/field/SelectForm.php cleanly.
Applied patch src/Plugin/views/filter/ContextualBundle.php cleanly.
Applied patch src/RouteSubscriber.php cleanly.
Applied patch src/SelectionDisplayBase.php cleanly.
Applied patch src/SelectionDisplayInterface.php cleanly.
Applied patch src/WidgetBase.php cleanly.
Applied patch src/WidgetInterface.php cleanly.
Applied patch src/WidgetSelectorBase.php cleanly.
Applied patch tests/modules/entity_browser_test/entity_browser_test.routing.yml cleanly.
Applied patch tests/modules/entity_browser_test/src/Form/FormElementTest.php cleanly.
Applied patch tests/modules/entity_browser_test/src/Plugin/EntityBrowser/Widget/DummyWidget.php cleanly.
Applied patch tests/src/Functional/EntityBrowserUITest.php cleanly.
Applied patch tests/src/Functional/FormElementTest.php cleanly.
Applied patch tests/src/FunctionalJavascript/CardinalityTest.php cleanly.
Applied patch tests/src/FunctionalJavascript/ConfigurationTest.php cleanly.
Applied patch tests/src/FunctionalJavascript/EntityBrowserTest.php cleanly.
Applied patch tests/src/FunctionalJavascript/EntityBrowserViewsWidgetTest.php cleanly.
Applied patch tests/src/FunctionalJavascript/EntityBrowserWebDriverTestBase.php cleanly.
Applied patch tests/src/FunctionalJavascript/EntityReferenceWidgetTest.php cleanly.
Applied patch tests/src/FunctionalJavascript/FieldWidgetConfigTest.php cleanly.
Applied patch tests/src/FunctionalJavascript/InlineEntityFormTest.php cleanly.
Applied patch tests/src/FunctionalJavascript/ParagraphsTest.php cleanly.
Applied patch tests/src/FunctionalJavascript/PluginsTest.php cleanly.
Applied patch tests/src/Kernel/Plugin/FieldWidgetDisplayTest.php cleanly.
āžœ  entity_browser git:(85a79c7) āœ— cd ..
āžœ  contrib git:(main) āœ— phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml entity_browser
āžœ  contrib git:(main) āœ—

    Retaining the status to needs review so that others can give their feedback.
    Thank you.

  • Status changed to RTBC 6 months ago
  • Comment 6 months ago ā†’
    šŸ‡µšŸ‡­Philippines cleavinjosh

    Hi,

    I applied the patch #40 and verified that the issues reported by phpcs are fixed.

     % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 84340  100 84340    0     0   168k      0 --:--:-- --:--:-- --:--:--  168k
Checking patch entity_browser.api.php...
Checking patch entity_browser.module...
Checking patch entity_browser.views.inc...
Checking patch modules/entity_form/entity_browser_entity_form.module...
Checking patch modules/entity_form/src/Plugin/EntityBrowser/Widget/EntityForm.php...
Checking patch src/Ajax/SelectEntitiesCommand.php...
Checking patch src/Ajax/ValueUpdatedCommand.php...
Checking patch src/Controllers/EntityBrowserController.php...
Checking patch src/Controllers/EntityBrowserFormController.php...
Checking patch src/Controllers/EntityBrowserListBuilder.php...
Checking patch src/DisplayBase.php...
Checking patch src/DisplayInterface.php...
Checking patch src/Element/EntityBrowserElement.php...
Checking patch src/Entity/EntityBrowser.php...
Checking patch src/EntityBrowserInterface.php...
Checking patch src/Events/AlterEntityBrowserDisplayData.php...
Checking patch src/Events/EntitySelectionEvent.php...
Checking patch src/Events/EventBase.php...
Checking patch src/Events/Events.php...
Checking patch src/Events/RegisterJSCallbacks.php...
Checking patch src/Form/EntityBrowserEditForm.php...
Checking patch src/Form/EntityBrowserForm.php...
Checking patch src/Plugin/EntityBrowser/Display/Modal.php...
Checking patch src/Plugin/EntityBrowser/Display/Standalone.php...
Checking patch src/Plugin/EntityBrowser/FieldWidgetDisplay/ImageThumbnail.php...
Checking patch src/Plugin/EntityBrowser/FieldWidgetDisplay/RenderedEntity.php...
Checking patch src/Plugin/EntityBrowser/SelectionDisplay/MultiStepDisplay.php...
Checking patch src/Plugin/EntityBrowser/SelectionDisplay/View.php...
Checking patch src/Plugin/EntityBrowser/Widget/Upload.php...
Checking patch src/Plugin/EntityBrowser/Widget/View.php...
Checking patch src/Plugin/EntityBrowser/WidgetSelector/DropDown.php...
Checking patch src/Plugin/EntityBrowser/WidgetSelector/Single.php...
Checking patch src/Plugin/EntityBrowser/WidgetSelector/Tabs.php...
Checking patch src/Plugin/Field/FieldWidget/EntityReferenceBrowserWidget.php...
Checking patch src/Plugin/Field/FieldWidget/FileBrowserWidget.php...
Checking patch src/Plugin/views/display/EntityBrowser.php...
Checking patch src/Plugin/views/field/SelectForm.php...
Checking patch src/Plugin/views/filter/ContextualBundle.php...
Checking patch src/RouteSubscriber.php...
Checking patch src/SelectionDisplayBase.php...
Checking patch src/SelectionDisplayInterface.php...
Checking patch src/WidgetBase.php...
Checking patch src/WidgetInterface.php...
Checking patch src/WidgetSelectorBase.php...
Checking patch tests/modules/entity_browser_test/entity_browser_test.routing.yml...
Checking patch tests/modules/entity_browser_test/src/Form/FormElementTest.php...
Checking patch tests/modules/entity_browser_test/src/Plugin/EntityBrowser/Widget/DummyWidget.php...
Checking patch tests/src/Functional/EntityBrowserUITest.php...
Checking patch tests/src/Functional/FormElementTest.php...
Checking patch tests/src/FunctionalJavascript/CardinalityTest.php...
Checking patch tests/src/FunctionalJavascript/ConfigurationTest.php...
Checking patch tests/src/FunctionalJavascript/EntityBrowserTest.php...
Checking patch tests/src/FunctionalJavascript/EntityBrowserViewsWidgetTest.php...
Checking patch tests/src/FunctionalJavascript/EntityBrowserWebDriverTestBase.php...
Checking patch tests/src/FunctionalJavascript/EntityReferenceWidgetTest.php...
Checking patch tests/src/FunctionalJavascript/FieldWidgetConfigTest.php...
Checking patch tests/src/FunctionalJavascript/InlineEntityFormTest.php...
Checking patch tests/src/FunctionalJavascript/ParagraphsTest.php...
Checking patch tests/src/FunctionalJavascript/PluginsTest.php...
Checking patch tests/src/Kernel/Plugin/FieldWidgetDisplayTest.php...
Applied patch entity_browser.api.php cleanly.
Applied patch entity_browser.module cleanly.
Applied patch entity_browser.views.inc cleanly.
Applied patch modules/entity_form/entity_browser_entity_form.module cleanly.
Applied patch modules/entity_form/src/Plugin/EntityBrowser/Widget/EntityForm.php cleanly.
Applied patch src/Ajax/SelectEntitiesCommand.php cleanly.
Applied patch src/Ajax/ValueUpdatedCommand.php cleanly.
Applied patch src/Controllers/EntityBrowserController.php cleanly.
Applied patch src/Controllers/EntityBrowserFormController.php cleanly.
Applied patch src/Controllers/EntityBrowserListBuilder.php cleanly.
Applied patch src/DisplayBase.php cleanly.
Applied patch src/DisplayInterface.php cleanly.
Applied patch src/Element/EntityBrowserElement.php cleanly.
Applied patch src/Entity/EntityBrowser.php cleanly.
Applied patch src/EntityBrowserInterface.php cleanly.
Applied patch src/Events/AlterEntityBrowserDisplayData.php cleanly.
Applied patch src/Events/EntitySelectionEvent.php cleanly.
Applied patch src/Events/EventBase.php cleanly.
Applied patch src/Events/Events.php cleanly.
Applied patch src/Events/RegisterJSCallbacks.php cleanly.
Applied patch src/Form/EntityBrowserEditForm.php cleanly.
Applied patch src/Form/EntityBrowserForm.php cleanly.
Applied patch src/Plugin/EntityBrowser/Display/Modal.php cleanly.
Applied patch src/Plugin/EntityBrowser/Display/Standalone.php cleanly.
Applied patch src/Plugin/EntityBrowser/FieldWidgetDisplay/ImageThumbnail.php cleanly.
Applied patch src/Plugin/EntityBrowser/FieldWidgetDisplay/RenderedEntity.php cleanly.
Applied patch src/Plugin/EntityBrowser/SelectionDisplay/MultiStepDisplay.php cleanly.
Applied patch src/Plugin/EntityBrowser/SelectionDisplay/View.php cleanly.
Applied patch src/Plugin/EntityBrowser/Widget/Upload.php cleanly.
Applied patch src/Plugin/EntityBrowser/Widget/View.php cleanly.
Applied patch src/Plugin/EntityBrowser/WidgetSelector/DropDown.php cleanly.
Applied patch src/Plugin/EntityBrowser/WidgetSelector/Single.php cleanly.
Applied patch src/Plugin/EntityBrowser/WidgetSelector/Tabs.php cleanly.
Applied patch src/Plugin/Field/FieldWidget/EntityReferenceBrowserWidget.php cleanly.
Applied patch src/Plugin/Field/FieldWidget/FileBrowserWidget.php cleanly.
Applied patch src/Plugin/views/display/EntityBrowser.php cleanly.
Applied patch src/Plugin/views/field/SelectForm.php cleanly.
Applied patch src/Plugin/views/filter/ContextualBundle.php cleanly.
Applied patch src/RouteSubscriber.php cleanly.
Applied patch src/SelectionDisplayBase.php cleanly.
Applied patch src/SelectionDisplayInterface.php cleanly.
Applied patch src/WidgetBase.php cleanly.
Applied patch src/WidgetInterface.php cleanly.
Applied patch src/WidgetSelectorBase.php cleanly.
Applied patch tests/modules/entity_browser_test/entity_browser_test.routing.yml cleanly.
Applied patch tests/modules/entity_browser_test/src/Form/FormElementTest.php cleanly.
Applied patch tests/modules/entity_browser_test/src/Plugin/EntityBrowser/Widget/DummyWidget.php cleanly.
Applied patch tests/src/Functional/EntityBrowserUITest.php cleanly.
Applied patch tests/src/Functional/FormElementTest.php cleanly.
Applied patch tests/src/FunctionalJavascript/CardinalityTest.php cleanly.
Applied patch tests/src/FunctionalJavascript/ConfigurationTest.php cleanly.
Applied patch tests/src/FunctionalJavascript/EntityBrowserTest.php cleanly.
Applied patch tests/src/FunctionalJavascript/EntityBrowserViewsWidgetTest.php cleanly.
Applied patch tests/src/FunctionalJavascript/EntityBrowserWebDriverTestBase.php cleanly.
Applied patch tests/src/FunctionalJavascript/EntityReferenceWidgetTest.php cleanly.
Applied patch tests/src/FunctionalJavascript/FieldWidgetConfigTest.php cleanly.
Applied patch tests/src/FunctionalJavascript/InlineEntityFormTest.php cleanly.
Applied patch tests/src/FunctionalJavascript/ParagraphsTest.php cleanly.
Applied patch tests/src/FunctionalJavascript/PluginsTest.php cleanly.
Applied patch tests/src/Kernel/Plugin/FieldWidgetDisplayTest.php cleanly.
āžœ  entity_browser git:(85a79c7) āœ— cd ..
āžœ  contrib git:(main) āœ— phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml entity_browser
āžœ  contrib git:(main) āœ—

    Thank you.

  • Status changed to Needs work 6 months ago
  • Comment 6 months ago ā†’
    šŸ‡ØšŸ‡­Switzerland berdir Switzerland

    This is overwhelmingly large, has some bad changes and needs to be done as a merge request.

    Many of these changes probably already have existing dedicated issues, or multiple, it likely makes more sense to focus on these, this will likely never get committed in this form.

    1. +++ b/entity_browser.api.php
@@ -3,15 +3,14 @@
 /**
- * Alter the information provided in \Drupal\entity_browser\Annotation\EntityBrowserDisplay.
+ * Alter the information provided in.
+ *
+ * \Drupal\entity_browser\Annotation\EntityBrowserDisplay.
  *
  * @param array $displays
  *   The array of display plugins, keyed on the machine-readable name.
@@ -21,7 +20,9 @@ function hook_entity_browser_display_info_alter(array &$displays) {

      this doesn't make sense, coder might be happy, but this isn't valid documentation. 

    2. +++ b/modules/entity_form/entity_browser_entity_form.module
@@ -155,7 +155,8 @@ function entity_browser_entity_form_reference_form_submit(array $reference_form,
   $form_state->setValueForElement($reference_form['entity_browser']['entity_ids'], '');
   $input = &$form_state->getUserInput();
-  NestedArray::unsetValue($input, array_merge($reference_form['#parents'], ['entity_browser', 'entity_ids']));
+  NestedArray::unsetValue($input, array_merge($reference_form['#parents'],
+  ['entity_browser', 'entity_ids']));
 }

      this is not more readable and this coder rule is controversial and not enabled by core. We could copy the core phpcs rules, see šŸ“Œ Fix the issues reported by phpcs Needs review 

    3. +++ b/modules/entity_form/src/Plugin/EntityBrowser/Widget/EntityForm.php
@@ -152,8 +152,20 @@ class EntityForm extends WidgetBase {
     $parents = ['table', $this->uuid(), 'form'];
     $entity_type = $form_state->hasValue(array_merge($parents, ['entity_type'])) ? $form_state->getValue(array_merge($parents, ['entity_type'])) : $this->configuration['entity_type'];
-    $bundle = $form_state->hasValue(array_merge($parents, ['bundle', 'select'])) ? $form_state->getValue(array_merge($parents, ['bundle', 'select'])) : $this->configuration['bundle'];
-    $form_mode = $form_state->hasValue(array_merge($parents, ['form_mode', 'form_select'])) ? $form_state->hasValue(array_merge($parents, ['form_mode', 'form_select'])) : $this->configuration['form_mode'];
+    $bundle = $form_state->hasValue(array_merge($parents, [
+      'bundle',
+      'select',
+    ])) ? $form_state->getValue(array_merge($parents, [
+      'bundle',
+      'select',
+    ])) : $this->configuration['bundle'];
+    $form_mode = $form_state->hasValue(array_merge($parents, [
+      'form_mode',
+      'form_select',
+    ])) ? $form_state->hasValue(array_merge($parents, [
+      'form_mode',
+      'form_select',
+    ])) : $this->configuration['form_mode'];
 
     $definitions = $this->entityTypeManager->getDefinitions();

      same, this doesn't make it more readable. getValue() has a default argument, not sure why this code isn't using that. 

    4. +++ b/src/Ajax/ValueUpdatedCommand.php
@@ -10,11 +10,11 @@ use Drupal\Core\Ajax\CommandInterface;
 class ValueUpdatedCommand implements CommandInterface {
 
   /**
-   * The ID for the details element.
+   * Details element ID.
    *
    * @var string
    */
-  protected $details_id;
+  protected $detailsIds;

      This can be considered an API change. 

    5. +++ b/src/Events/Events.php
@@ -8,33 +8,36 @@ namespace Drupal\entity_browser\Events;
   const SELECTED = 'entity_browser.selected';
 
   /**
-   * The DONE event occurs when selection process is done. While it can be emitted
-   * by any part of the system that will usually be done by selection display plugin.
+   * The DONE event occurs when selection process is done.
+   *
+   * While it can be emitted by any part of the system.
+   *
+   * That will usually be done by selection display plugin.
    *

      same here, this blindly makes phpstan happy, but the result is no longer English.

  • Comment 6 months ago ā†’
    šŸ‡®šŸ‡¹Italy apaderno Brescia, šŸ‡®šŸ‡¹

    While it can be emitted by any part of the system that will usually be done by selection display plugin. cannot be split in While it can be emitted by any part of the system. and That will usually be done by selection display plugin. because While it can be emitted by any part of the system. is not a sentence; it is a clause that requires another clause to be a sentence (and to be understandable).
    You cannot randomly split sentences and put a period in random places just because PHP_CodeSniffer wants a shorter sentence.

  • Comment 6 months ago ā†’
    šŸ‡ØšŸ‡­Switzerland berdir Switzerland

    @apaderno: Maybe it's meant to be a Haiku? ;) ( don't actually have any clue about how Haikus work)

  • Comment 6 months ago ā†’
    šŸ‡®šŸ‡¹Italy apaderno Brescia, šŸ‡®šŸ‡¹

    I wish I could write a Haiku about using the correct punctuation. šŸ˜…šŸ˜‰šŸ–Šļø

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024