Coding Standards Issues (many)

Created on 17 January 2022, almost 3 years ago
Updated 6 September 2024, 3 months ago

Problem/Motivation

PHP files in this module don't follow Drupal Coding Standards β†’ . Issues include basics like proper indentation and whitespace, docs formatting, variable naming, and more.

Steps to reproduce

Run phpcbf --standard=Drupal --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml ./ on the module files.

Proposed resolution

Review and fix the issues.

πŸ“Œ Task
Status

RTBC

Version

2.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States froboy Chicago, IL

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡¬πŸ‡§United Kingdom ipwa

    ipwa β†’ changed the visibility of the branch 3259014-coding-standards-issues to hidden.

  • πŸ‡¬πŸ‡§United Kingdom ipwa

    ipwa β†’ changed the visibility of the branch 3259014-coding-standards-issues to hidden.

  • First commit to issue fork.
  • πŸ‡ΊπŸ‡ΈUnited States froboy Chicago, IL

    @danielzigo same here... it'd be great to assign credit β†’ to the folks who worked on this.

  • πŸ‡¬πŸ‡§United Kingdom danielzigo

    Hi @froboy. Thanks for your comment. It wasn't long ago that I was made a maintainer of this module after it stopped being maintained for a good while. So, I've been reviewing changes, and after the merge of the changes for this issue, there was one other addition I made that I've asked my colleague to review. Credits would be assigned to you all after that.

    Thank you.

  • πŸ‡¬πŸ‡§United Kingdom welly

    I think something has gone awry here since some of these changes have been merged in. I wonder if there was a merge conflict when merging in the recent changes?

    When enabling the dev version of the module, we get several errors including:

    ParseError: Unmatched '}' in Drupal\Core\Extension\Extension->load() (line 633 of modules/custom/coc_forms_auto_export/coc_forms_auto_export.module).

    And when you remove that addition curly brace, it causes the next following functions to break the code. I think this needs reworking. I'm going to create a new issue and a merge request.

Production build 0.71.5 2024