Coding standards [#3337018]

This task is devoted to the LutskGCW23, please, don't work with it until it starts.


FILE: /var/www/html/web/modules/contrib/field_properties/src/Plugin/FieldPropertyTypeInterface.php
----------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------
 3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
----------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------


FILE: /var/www/html/web/modules/contrib/field_properties/src/Plugin/FieldProperties/Type/JSONPropertyType.php
-------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------
 3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
-------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------


FILE: /var/www/html/web/modules/contrib/field_properties/src/Plugin/FieldProperties/Type/NumberPropertyType.php
---------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------
 3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
---------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------


FILE: /var/www/html/web/modules/contrib/field_properties/src/Plugin/FieldProperties/Type/StringPropertyType.php
---------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------
 3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
---------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------


FILE: /var/www/html/web/modules/contrib/field_properties/src/Plugin/Field/FieldFormatter/FieldPropertiesFormatter.php
---------------------------------------------------------------------------------------------------------------------
FOUND 5 ERRORS AFFECTING 5 LINES
---------------------------------------------------------------------------------------------------------------------
  3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
 31 | ERROR | [x] Short array syntax must be used to define arrays
 37 | ERROR | [x] Short array syntax must be used to define arrays
 40 | ERROR | [x] Short array syntax must be used to define arrays
 48 | ERROR | [x] Short array syntax must be used to define arrays
---------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 5 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------


FILE: /var/www/html/web/modules/contrib/field_properties/src/Plugin/Field/FieldType/FieldPropertiesItem.php
-----------------------------------------------------------------------------------------------------------
FOUND 12 ERRORS AFFECTING 12 LINES
-----------------------------------------------------------------------------------------------------------
   3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
  48 | ERROR | [x] Short array syntax must be used to define arrays
  49 | ERROR | [x] Short array syntax must be used to define arrays
  50 | ERROR | [x] Short array syntax must be used to define arrays
  55 | ERROR | [x] Short array syntax must be used to define arrays
  60 | ERROR | [x] Short array syntax must be used to define arrays
  66 | ERROR | [x] Short array syntax must be used to define arrays
  67 | ERROR | [x] Short array syntax must be used to define arrays
  68 | ERROR | [x] Short array syntax must be used to define arrays
  70 | ERROR | [x] Short array syntax must be used to define arrays
  90 | ERROR | [x] Short array syntax must be used to define arrays
 100 | ERROR | [x] Short array syntax must be used to define arrays
-----------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 12 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------


FILE: /var/www/html/web/modules/contrib/field_properties/src/Plugin/Field/FieldWidget/FieldPropertiesWidget.php
---------------------------------------------------------------------------------------------------------------
FOUND 8 ERRORS AFFECTING 8 LINES
---------------------------------------------------------------------------------------------------------------
  3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
 35 | ERROR | [x] Short array syntax must be used to define arrays
 39 | ERROR | [x] Use null coalesce operator instead of ternary operator.
 44 | ERROR | [x] Short array syntax must be used to define arrays
 49 | ERROR | [x] Short array syntax must be used to define arrays
 54 | ERROR | [x] Use null coalesce operator instead of ternary operator.
 56 | ERROR | [x] Short array syntax must be used to define arrays
 59 | ERROR | [x] Use null coalesce operator instead of ternary operator.
---------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 8 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------


FILE: /var/www/html/web/modules/contrib/field_properties/src/Plugin/FieldPropertyTypeManager.php
-----------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
-----------------------------------------------------------------------------------------------------
  3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
 23 | ERROR | [x] Short array syntax must be used to define arrays
-----------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------

Time: 252ms; Memory: 10MB

Issue fork field_properties-3337018

Show commands

Start within a Git clone of the project using the version control instructions.

Add & fetch this issue fork’s repository

Or, if you do not have SSH keys set up on git.drupalcode.org:

Add & fetch this issue fork’s repository

3337018-coding-standards changes, plain diff MR !2
Check out this branch
3337021-coding-standards compare
Check out this branch

About issue forks

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

27 January 2023 at 16:36

_shY created an issue. See original summary.

Comment #2

snap_x

🇺🇦 Ukraine, Lutsk

CreditAttribution: snap_x at DevBranch, Drupal Ukraine Community commented 28 January 2023 at 14:17

Comment #3

28 January 2023 at 14:29

snap_x opened merge request !2

Comment #4

apaderno

he/him

Italian

Brescia, 🇮🇹 🇪🇺

CreditAttribution: apaderno as a volunteer commented 29 January 2023 at 10:27

Status:	Active	» Needs work
Issue tags:		+Needs issue summary update

The issue summary needs to describe exactly what the task is. In the case of coding standards, it should also show how the command used to verify the code follows the coding standards is called (for example, phpcs --standard=DrupalPractice), and possibly the output of that command.