Fix the errors/warnings reported by PHP_CodeSniffer

Found some more phpcs errors and solved them. Here is the updated patch.

Hello,

Reviewed the #4 patch and after applying the patch still below issues are remaining.

FILE: /home/drupal/drupal-9.5/drupal/default_paragraphs/src/Plugin/Field/FieldWidget/DefaultParagraphsWidget.php
------------------------------------------------------------------------------------------------------------------------
FOUND 4 ERRORS AND 2 WARNINGS AFFECTING 6 LINES
------------------------------------------------------------------------------------------------------------------------
 245 | ERROR   | [x] Use null coalesce operator instead of ternary operator.
 253 | ERROR   | [x] Use null coalesce operator instead of ternary operator.
 268 | ERROR   | [x] Use null coalesce operator instead of ternary operator.
 286 | WARNING | [ ] Unused variable $key.
 293 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 361 | ERROR   | [x] Use null coalesce operator instead of ternary operator.
------------------------------------------------------------------------------------------------------------------------

Resolved the issue and create a new patch.
Please review the patch.

Thanks!

 .field--widget-default-paragraphs th .paragraphs-actions {
-    float: right;
-    margin-right: -11px;
+  float: right;
+  margin-right: -11px;
 }
 
 .field--widget-default-paragraphs .paragraphs-dropbutton-wrapper {
-    display: inline-flex;
+  display: inline-flex;
 }
 
 .field--widget-default-paragraphs .dropbutton-wrapper {
-    width: auto;
-    margin-right: 0;
-    padding-right: 0;
+  width: auto;
+  margin-right: 0;
+  padding-right: 0;
 }
 
 .field--widget-default-paragraphs .dropbutton-widget {
-    position: static;
+  position: static;
 }
 
 .field--widget-default-paragraphs .field-multiple-table {
-    margin-bottom: 10px;
+  margin-bottom: 10px;
 }
 
 .field--widget-default-paragraphs td {
-    padding: 10px 0;
+  padding: 10px 0;
 }
 
 .field--widget-default-paragraphs .field-multiple-drag {
-    vertical-align: top;
+  vertical-align: top;
 }
 
 .field--widget-default-paragraphs .draggable .tabledrag-handle {
-    margin-left: 0;
+  margin-left: 0;
 }
 
 .field--widget-default-paragraphs .tabledrag-handle .handle {
-    height: 22px;
+  height: 22px;
 }
 
 .field--widget-default-paragraphs .delta-order {
-    padding-right: 10px;
-    text-align: right;
+  padding-right: 10px;
+  text-align: right;
 }

Those changes aren't related to the report shown in the issue summary, like other changes present in the last patch.

Hi, I'm working on it .

Hi, I have created a patch . Please review it .

diff --git a/css/default_paragraphs.widget.css b/css/default_paragraphs.widget.css
index fc6f5b5..282e15b 100644
--- a/css/default_paragraphs.widget.css
+++ b/css/default_paragraphs.widget.css
@@ -1,43 +1,43 @@
 .field--widget-default-paragraphs th .paragraphs-actions {
-    float: right;
-    margin-right: -11px;
+  float: right;
+  margin-right: -11px;
 }
 
 .field--widget-default-paragraphs .paragraphs-dropbutton-wrapper {
-    display: inline-flex;
+  display: inline-flex;
 }
 

The report shown in the issue summary is for PHP files, not .css files.

Hi, I have excluded the css files from this patch. Please review it.

+// Use \Drupal\Core\StringTranslation\StringTranslationTrait;.

Either that trait is used, and the use statement is necessary, or that trait is not used and the use statement is not necessary.

Use statement has been removed. Please review this new patch.

The patch fixes what reported.

Tested and verified , ready for merge .

The person who provided the patch/MR cannot change the status to Reviewed & tested by the community.

I tested the issue fork locally. It seems like all issues are fixed.