Coding Standard issues

Created on 16 June 2017, over 7 years ago
Updated 22 August 2024, 3 months ago

There are some minor coding standard issues in module.

šŸ› Bug report
Status

RTBC

Version

1.0

Component

Code

Created by

šŸ‡®šŸ‡³India jayesh_makwana

Live updates comments and jobs are added and updated live.
  • Coding standards

    It involves compliance with, or the content of coding standards. Requires broad community agreement.

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.

  • Merge request !1Issue #2886679: Coding Standard issues ā†’ (Merged) created by urvashi_vora
  • Hi @urvashi_vora,

    Applied the patch you provided, it was applied successfully, however, there are 2 errors remaining. Please see below:

    āžœ  edit_and_save git:(8.x-1.x) curl https://www.drupal.org/files/issues/2022-07-16/coding-standards-fixed-2886679-7.patch | patch -p1
      % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                     Dload  Upload   Total   Spent    Left  Speed
    100  1157  100  1157    0     0   4630      0 --:--:-- --:--:-- --:--:--  4861
    patching file README.txt
    patching file edit_and_save.info.yml
    patching file src/Repository/EditAndSaveRepository.php
    āžœ  edit_and_save git:(8.x-1.x) āœ— ..
    āžœ  contrib git:(master) āœ— phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig edit_and_save
    
    FILE: ...v9/web/modules/contrib/edit_and_save/src/Form/EditAndSaveConfigureForm.php
    --------------------------------------------------------------------------------
    FOUND 2 ERRORS AFFECTING 2 LINES
    --------------------------------------------------------------------------------
       8 | ERROR | [x] Use statements should be sorted alphabetically. The first
         |       |     wrong one is Drupal\Core\Form\ConfigFormBase.
     153 | ERROR | [x] Use null coalesce operator instead of ternary operator.
    --------------------------------------------------------------------------------
    PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
    --------------------------------------------------------------------------------
    
    Time: 187ms; Memory: 10MB

    And as expected the MR!1 also have errors remaining, so moving this to Needs Work.

    Kindly check

    Thanks,
    Jake

  • Assigned to akshaydalvi212
  • šŸ‡®šŸ‡³India akshaydalvi212

    I will remove the remaining issues.

  • Issue was unassigned.
  • Status changed to Needs review 3 months ago
  • šŸ‡®šŸ‡³India akshaydalvi212

    hi @a.aaronjake,

    Removed the remaining coding standard errors and now you can review.

    Thanks and regards
    Akshay Dalvi

  • Status changed to RTBC 3 months ago
  • Hi @akshaydalvi212,

    Applied the latest commits on MR!1, it was applied successfully and confirmed no remaining issue reported.

    āžœ  edit_and_save git:(8.x-1.x) curl https://git.drupalcode.org/project/edit_and_save/-/merge_requests/1.diff | patch -p1
      % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                     Dload  Upload   Total   Spent    Left  Speed
    100  2624    0  2624    0     0   7964      0 --:--:-- --:--:-- --:--:--  8251
    patching file README.txt
    patching file edit_and_save.info.yml
    patching file src/Form/EditAndSaveConfigureForm.php
    patching file src/Repository/EditAndSaveRepository.php
    āžœ  edit_and_save git:(8.x-1.x) āœ— ..
    āžœ  contrib git:(master) āœ— phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig edit_and_save
    āžœ  contrib git:(master) āœ—

    Will now move this to RTBC

    Thanks,
    Jake

  • šŸ‡§šŸ‡¬Bulgaria ovidenov

    Thanks, merged

Production build 0.71.5 2024