Attached patch fixes coding standard issues.

CommentFileSizeAuthor
#5 After.PNG34.26 KBriddhi.addweb
#5 Before.PNG52.42 KBriddhi.addweb
#2 fix_coding_standards-2858390-2.patch4.39 KBMunavijayalakshmi
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Munavijayalakshmi created an issue. See original summary.

Munavijayalakshmi’s picture

Munavijayalakshmi CreditAttribution: Munavijayalakshmi at Valuebound commented
Assigned: Munavijayalakshmi » Unassigned
Status: Active » Needs review
FileSize
fix_coding_standards-2858390-2.patch4.39 KB
riddhi.addweb’s picture

riddhi.addweb CreditAttribution: riddhi.addweb at AddWeb Solution Pvt. Ltd. commented

Hi @Munavijayalakshmi,thanks for the patch it works well. I checked it over Simplytest.Me with the help of coder module and with patch it removed the coding standard errors.PFA

riddhi.addweb’s picture

riddhi.addweb CreditAttribution: riddhi.addweb at AddWeb Solution Pvt. Ltd. commented
Status: Needs review » Reviewed & tested by the community
riddhi.addweb’s picture

riddhi.addweb CreditAttribution: riddhi.addweb at AddWeb Solution Pvt. Ltd. commented
FileSize
Before.PNG52.42 KB
After.PNG34.26 KB

sgabe’s picture

sgabe CreditAttribution: sgabe commented
Status: Reviewed & tested by the community » Fixed

Committed, thanks!

Status: Fixed » Closed (fixed)

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