Fix the issues reported by phpcs

FILE: /Applications/MAMP/htdocs/d9/web/modules/contrib/commerce_webform_order/README.md --------------------------------------------------------------------------------------- FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES --------------------------------------------------------------------------------------- 4 | WARNING | Line exceeds 80 characters; contains 142 characters 8 | WARNING | Line exceeds 80 characters; contains 172 characters 10 | WARNING | Line exceeds 80 characters; contains 137 characters 15 | WARNING | Line exceeds 80 characters; contains 89 characters --------------------------------------------------------------------------------------- FILE: /Applications/MAMP/htdocs/d9/web/modules/contrib/commerce_webform_order/commerce_webform_order.install ------------------------------------------------------------------------------------------------------------ FOUND 3 ERRORS AFFECTING 1 LINE ------------------------------------------------------------------------------------------------------------ 54 | ERROR | [x] Expected 1 space after CATCH keyword; 0 found 54 | ERROR | [x] Newline required after opening brace 54 | ERROR | [x] Closing brace must be on a line by itself ------------------------------------------------------------------------------------------------------------ PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY ------------------------------------------------------------------------------------------------------------ FILE: /Applications/MAMP/htdocs/d9/web/modules/contrib/commerce_webform_order/src/Plugin/WebformHandler/CommerceWebformOrderHandler.php ----------------------------------------------------------------------------------------------------------------------------------------------------------- FOUND 2 ERRORS AND 2 WARNINGS AFFECTING 4 LINES ----------------------------------------------------------------------------------------------------------------------------------------------------------- 412 | WARNING | [x] '@TODO: Use AJAX to reload order item bundle fields on product variation change.' should match the format '@todo Fix problem X here.' 436 | ERROR | [x] Use null coalesce operator instead of ternary operator. 441 | WARNING | [x] '@TODO: Use the same type of this order item field.' should match the format '@todo Fix problem X here.' 786 | ERROR | [x] Functions must not contain multiple empty lines in a row; found 2 empty lines ----------------------------------------------------------------------------------------------------------------------------------------------------------- PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY ----------------------------------------------------------------------------------------------------------------------------------------------------------- FILE: /Applications/MAMP/htdocs/d9/web/modules/contrib/commerce_webform_order/src/Element/EntityAutocompleteOrToken.php ----------------------------------------------------------------------------------------------------------------------- FOUND 1 ERROR AFFECTING 1 LINE ----------------------------------------------------------------------------------------------------------------------- 185 | ERROR | [x] Use null coalesce operator instead of ternary operator. ----------------------------------------------------------------------------------------------------------------------- PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY -----------------------------------------------------------------------------------------------------------------------

Comments & Activities

Not all content is available!

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

Comment about 2 years ago →
🇮🇳India Charchil Khandelwal
Charchil Khandelwal → made their first commit to this issue’s fork.
@charchil-khandelwal opened merge request.
Comment about 2 years ago →
🇮🇳India Charchil Khandelwal
Created MR !2 for this issue.
Please review.

Thank you.
Status changed to Needs work almost 2 years ago6:36pm 6 May 2023
Comment almost 2 years ago →
🇮🇹Italy apaderno Brescia, 🇮🇹
The issue summary should always describe what the issue is trying to fix and, in the case, of coding standards issues, show which command has been used, which arguments have been used, and which report that command shown.
Status changed to Needs review almost 2 years ago7:21am 8 May 2023
Comment almost 2 years ago →
🇨🇦Canada imustakim Canada
Issue summary updated.
Status changed to Needs work almost 2 years ago7:29am 8 May 2023

Comment almost 2 years ago →

🇮🇹Italy apaderno Brescia, 🇮🇹

-    * Product variation (allowing to override certain properties such price or currency).
+    * Product variation (allowing to override certain properties
+    such price or currency).

The such price or currency). part must be indented respect the previous line.

Comment over 1 year ago →
System Message

facine → committed 723feeec on 3.x authored by noorulshameera →
Issue #3333154 by Charchil Khandelwal, noorulshameera, apaderno: Fix the...
Status changed to Fixed over 1 year ago5:39pm 23 August 2023
Comment over 1 year ago →
🇪🇸Spain facine
Fixed, thanks!
Comment over 1 year ago →
System Message
facine → closed merge request !2
Comment over 1 year ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

Fix the issues reported by phpcs

Steps to reproduce

Problem/Motivation

Comments & Activities