Coding standards issues found in php files.

CommentFileSizeAuthor
#2 leaflet-drupal_coding_standard_issues-2582861-2.patch6.04 KBshridhars
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

shridhars created an issue. See original summary.

shridhars’s picture

shridhars CreditAttribution: shridhars as a volunteer and at Faichi Solutions Pvt Ltd commented
FileSize
leaflet-drupal_coding_standard_issues-2582861-2.patch6.04 KB

I found some coding standard issues in below files,
leaflet.module
leaflet.api.php
leaflet.formatters.inc

Please find the attached "leaflet-drupal_coding_standard_issues-2582861-2.patch" file to fix these coding standard issues.

shridhars’s picture

shridhars CreditAttribution: shridhars as a volunteer and at Faichi Solutions Pvt Ltd commented
Status: Active » Needs review
RdeBoer’s picture

RdeBoer
Location Melbourne
CreditAttribution: RdeBoer commented

What... spaces and superfluous comma's?

Nikolay Shapovalov’s picture

Nikolay Shapovalov
Primary language Russian
CreditAttribution: Nikolay Shapovalov as a volunteer commented
Assigned: shridhars » Unassigned
apaderno’s picture

apaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
CreditAttribution: apaderno as a volunteer commented
Title: Coding standard issues found » Fix the errors/warnings reported by PHP_CodeSniffer
Category: Bug report » Task
Status: Needs review » Needs work
Issue tags: +Needs issue summary update

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.

itamair’s picture

itamair CreditAttribution: itamair commented
Status: Needs work » Closed (outdated)

Super old ... nothing applies anymore from this. Closing as outdated ...