Problem/Motivation

Fix the following issues from PHPCS.

FILE: ...ocs/d9/web/modules/contrib/inxmail_xml/src/Form/InxmailXmlSettingsForm.php
--------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
--------------------------------------------------------------------------------
65 | ERROR | [x] Expected 1 blank line after function; 0 found
66 | ERROR | [x] The closing brace for the class must have an empty line
| | before it
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

FILE: ...MP/htdocs/d9/web/modules/contrib/inxmail_xml/src/Entity/TemplateEntity.php
--------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------
44 | ERROR | [x] Doc comment long description must end with a full stop
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

FILE: ...d9/web/modules/contrib/inxmail_xml/src/Controller/InxmailXmlController.php
--------------------------------------------------------------------------------
FOUND 27 ERRORS AFFECTING 27 LINES
--------------------------------------------------------------------------------
17 | ERROR | [x] Expected 1 space(s) before asterisk; 0 found
20 | ERROR | [x] Expected 1 space(s) before asterisk; 0 found
49 | ERROR | [x] Expected 3 space(s) before asterisk; 2 found
50 | ERROR | [x] Expected 3 space(s) before asterisk; 2 found
52 | ERROR | [x] Inline control structures are not allowed
67 | ERROR | [x] Expected newline after closing brace
85 | ERROR | [x] Parameter comment must end with a full stop
87 | ERROR | [ ] Description for the @return value is missing
88 | ERROR | [x] Separate the @return and @throws sections by a blank line.
91 | ERROR | [x] Inline control structures are not allowed
102 | ERROR | [x] Case breaking statements must be followed by a single blank
| | line
116 | ERROR | [ ] Description for the @return value is missing
117 | ERROR | [x] Separate the @return and @throws sections by a blank line.
119 | ERROR | [x] Expected 1 space before opening brace; found 2
120 | ERROR | [x] Inline control structures are not allowed
135 | ERROR | [x] Inline control structures are not allowed
138 | ERROR | [x] Expected 1 space after "="; 2 found
156 | ERROR | [ ] Description for the @return value is missing
157 | ERROR | [x] Separate the @return and @throws sections by a blank line.
160 | ERROR | [x] Inline control structures are not allowed
163 | ERROR | [x] Expected 1 space(s) after cast statement; 0 found
188 | ERROR | [x] Inline control structures are not allowed
191 | ERROR | [x] Expected 1 space after "="; 2 found
229 | ERROR | [x] Expected newline after closing brace
233 | ERROR | [x] Concat operator must be surrounded by a single space
255 | ERROR | [ ] Description for the @return value is missing
260 | ERROR | [x] Expected newline after closing brace
--------------------------------------------------------------------------------

CommentFileSizeAuthor
#2 3340131-2.patch6.65 KBnoorulshameera

Comments

noorulshameera created an issue. See original summary.

noorulshameera’s picture

Status: Active » Needs review
StatusFileSize
new6.65 KB

Adding patch to fix the issue.

rohan-sinha’s picture

Status: Needs review » Reviewed & tested by the community

Patch on #2 worked successfull. Great Work @noorulshameera !!

  • noorulshameera authored af04fa32 on 8.x-1.x
    Issue #3340131 by noorulshameera: Fix Coding Standard Issue | PHPCS
    
antonín slejška’s picture

Status: Reviewed & tested by the community » Fixed

Thank you @noorulshameera! I have commited and pushed your patch:
https://git.drupalcode.org/project/inxmail_xml/-/commit/af04fa3250a29435...

Status: Fixed » Closed (fixed)

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