Fix the issues reported by phpcs

I have Created a patch to fix this issue.

Hi

The patch#2 applied cleanly but found 1 phpcbf warning that is fixed in the MR.

Thanks.

 /**
  * @file
- *
  * Hook implementations.
  */

Since that comment is edited, the description should be changed too. (Hook implementations for the Secure html module.)

 /**
+ * A filter that Wraps content in an iframe.
+ *

Wraps is misspelled, since it has been capitalized when it should not.

Also, the report shown in the issue summary is for a single file, but the MR changes two files. Either the report is updated or the MR changes what the report says to change.

Hi
I changed the 2 comment which reported above, while running phpcs command before applying patch changes are in 2 files.

Thanks

+ /**
++ * A filter that Wraps content in an iframe.
++ *
+  * @Filter(

+-      list($k, $v) = explode("=", $str);
++      [$k, $v] = explode("=", $str);

+   public function settingsForm(array $form, FormStateInterface $form_state) {
+-    $form['secure_html_settings'] = array(
++    $form['secure_html_settings'] = [

There are extra plus/minus signs that need to be removed.

- * Hook implementations.
+ * Hook implementations for the Secure html module..

There are two periods at the end of the description instead of one.

Patch updated.
Please review.

 /**
  * @file
- *
- * Hook implementations.
+ * Hook implementations for Secure HTML module.
  */

The module name is Secure html, not Secure HTML.
A definite article is missing after for.

The report shows warnings/errors for a single file, but the patch changes two files. Either the patch changes only the file for which the issue summary reports warnings/errors or the report is changed to reflect the changes done.

Patch updated.
Please review.

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