Fix the issues reported by phpcs

+- **Key:** The key of the property element to which the attributes
+should be added.

The indentation of the second line is wrong. There are other changes that are wrong for the same reason.

added updated patch and address #4 and it fixed all PHPCS issues

-    {{ content.field_image|add_attr('image_attributes', {class: ['custom-class']}) }}
+    {{ content.field_image|add_attr('image_attributes',
+       {class: ['custom-class']}) }}

It is better to leave that as one-line code.

hi @apaderno but if i change it to one-line code. it'll give phpcs warning

-------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------
34 | WARNING | Line exceeds 80 characters; contains 85 characters
-------------------------------------------------------------------------------------------------------

@Akram Just ignore it. Coding standards do not say that lines must be shorter than 81 characters. There are quite few cases where splitting the code over two or more lines is wrong.

This is what Line length and wrapping → , part of the Drupal coding standards says. (Emphasis is mine.)

• Lines containing longer function names, function/class definitions, variable declarations, etc are allowed to exceed 80 characters.
• Control structure conditions may exceed 80 characters, if they are simple to read and understand.

A tool is not able to understand what is better, and it can be ignored in such cases.

address #8

I made a slight adjustment to the one example line to bring it below 80 characters. I also made some minor changes to the top two paragraphs. I will cherry pick these changes to the 8.x-1.x branch.

Automatically closed - issue fixed for 2 weeks with no activity.