Regarding standard coding for curly bracket in latest version

  • I have found like bug for wordpress standard coding in all theme for below path:
    twentyseventeen/template-parts/post/content-gallery.php file.
    Please review below link where I have attached screenshot.

    Link: https://ufile.io/b5joe

    So what I can do? I have found that in all theme twentyfifteen,twentysexteen,twentyseventeen of latest version. Please suggest asap.

    Thank You.

    The page I need help with: [log in to see the link]

Viewing 3 replies - 1 through 3 (of 3 total)
  • Andrew Nevins

    (@anevins)

    WCLDN 2018 Contributor | Volunteer support

    You haven’t found a bug, you have a disagreement with the coding style. Relax, this isn’t urgent.

    Thread Starter kinjaldalwadi

    (@kinjaldalwadi)

    So is it valid }; in php as per standard coding? Because our technical client believe standard wordpress code so we have to remove one by one in all files where we are facing same syntax like it.

    Andrew Nevins

    (@anevins)

    WCLDN 2018 Contributor | Volunteer support

    It is valid PHP, that’s why it causes no notices, warnings or errors. Try it outside of WordPress and see the outcome.

    It is normal for organisations like yours to have coding standards, but you can’t control the coding standards of things you don’t own. That’s why you shouldn’t edit themes, plugins or WordPress core to suit your coding standards. Instead you should make an exception to files you cannot control.

    If your tests are failing then exclude those files from your tests.

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘Regarding standard coding for curly bracket in latest version’ is closed to new replies.