Reviewed & tested by the community
Project:
Node Form Panes
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
24 May 2018 at 13:55 UTC
Updated:
4 Dec 2018 at 10:17 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
jigish.addweb commented@msankhala, Kindly review the patch attached below where I corrected all Coding standard errors as per the requirement.
thanks
Comment #3
msankhala commented@jigish.addweb Thanks for your work but there are a couple of errors are thrown while applying this patch.
Comment #4
jigish.addweb commented@msankhala, Thanks for reviewing the patch, I worked for it & attaching a new patch with some more necessary changes I hope this works for you now. Please share your valuable feedback on the same.
Comment #5
msankhala commented@jigish.addweb Thanks for your work on this issue but there are still many issue reported by PHPCS. My phpcs version is 3.3.2 and Drupal coder module version 8.3.1. May be you are using some other version of PHPCS or coder modue thats why it is not showing these warnings for you.
Comment #6
deepanker_bhalla commentedI am uploading the patch with all the fixed issues.
To which I checked the patch #6 is applying cleanly. Here is the output:
❯ node_form_panes 7.x-1.x* git apply -v 2975055-Fix-coding-standard-6.patch
Checking patch README.txt...
Checking patch css/node_form_panes.css...
Checking patch node_form_panes.module...
Checking patch plugins/content_types/node_form/node_form_attachments.inc...
Checking patch plugins/content_types/node_form/node_form_author.inc...
Checking patch plugins/content_types/node_form/node_form_book.inc...
Checking patch plugins/content_types/node_form/node_form_comment.inc...
Checking patch plugins/content_types/node_form/node_form_flag.inc...
Checking patch plugins/content_types/node_form/node_form_log.inc...
Checking patch plugins/content_types/node_form/node_form_menu.inc...
Checking patch plugins/content_types/node_form/node_form_metatag.inc...
Checking patch plugins/content_types/node_form/node_form_panelizer.inc...
Checking patch plugins/content_types/node_form/node_form_path.inc...
Checking patch plugins/content_types/node_form/node_form_publishing.inc...
Checking patch plugins/content_types/node_form/node_form_rabbit_hole.inc...
Checking patch plugins/content_types/node_form/node_form_redirect.inc...
Checking patch plugins/content_types/node_form/node_form_scheduler.inc...
Checking patch plugins/content_types/node_form/node_form_spamicide.inc...
Checking patch plugins/content_types/node_form/node_form_spamicide_field.inc...
Checking patch plugins/content_types/node_form/node_form_summary.inc...
Checking patch plugins/content_types/node_form/node_form_workbench_moderation.inc...
Checking patch plugins/content_types/node_form/node_form_xml_sitemap.inc...
Applied patch README.txt cleanly.
Applied patch css/node_form_panes.css cleanly.
Applied patch node_form_panes.module cleanly.
Applied patch plugins/content_types/node_form/node_form_attachments.inc cleanly.
Applied patch plugins/content_types/node_form/node_form_author.inc cleanly.
Applied patch plugins/content_types/node_form/node_form_book.inc cleanly.
Applied patch plugins/content_types/node_form/node_form_comment.inc cleanly.
Applied patch plugins/content_types/node_form/node_form_flag.inc cleanly.
Applied patch plugins/content_types/node_form/node_form_log.inc cleanly.
Applied patch plugins/content_types/node_form/node_form_menu.inc cleanly.
Applied patch plugins/content_types/node_form/node_form_metatag.inc cleanly.
Applied patch plugins/content_types/node_form/node_form_panelizer.inc cleanly.
Applied patch plugins/content_types/node_form/node_form_path.inc cleanly.
Applied patch plugins/content_types/node_form/node_form_publishing.inc cleanly.
Applied patch plugins/content_types/node_form/node_form_rabbit_hole.inc cleanly.
Applied patch plugins/content_types/node_form/node_form_redirect.inc cleanly.
Applied patch plugins/content_types/node_form/node_form_scheduler.inc cleanly.
Applied patch plugins/content_types/node_form/node_form_spamicide.inc cleanly.
Applied patch plugins/content_types/node_form/node_form_spamicide_field.inc cleanly.
Applied patch plugins/content_types/node_form/node_form_summary.inc cleanly.
Applied patch plugins/content_types/node_form/node_form_workbench_moderation.inc cleanly.
Applied patch plugins/content_types/node_form/node_form_xml_sitemap.inc cleanly.
❯ node_form_panes 7.x-1.x* phpcs --standard=Drupal .
❯ node_form_panes 7.x-1.x*
PHPCS is reporting no coding standard warning after applying the patch #6.
Comment #7
msankhala commentedGreat work @deepanker_bhalla. I can confirm the path #6 is fixing all the coding standard issue reported by PHPCS.