Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Buvaneshwari.V created an issue. See original summary.

Buvaneshwari.V’s picture

Assigned: Buvaneshwari.V » Unassigned
FileSize
3.29 KB

Please review the attached patch for the same!

Thanks.

sun’s picture

Title: Convert module to use short array syntax (new coding standard). » Coding standards: Use short array syntax
Status: Needs review » Postponed

Thanks for contributing this cleanup!

A coding style cleanup patch (like this) potentially breaks many other patches in the queue. As there is no stable release of this module yet, there are more important changes/patches in the issue queue that should be performed first. For that reason, I'm marking this issue as postponed – which doesn't mean that it shouldn't be committed; it just means that we should not risk to break more substantial patches at this point in time. I hope you can understand :-)

Mario Steinitz’s picture

Status: Postponed » Needs review

Breaking other patches in the issue queue is no blocker for complying with coding standards. I set the issue back to "needs review".

@sun: Your enthusiasm for the openid_connect module is very much appreciated. We'd also love to see this module changing from beta to RC or a feature complete first version for Drupal 8. Yet, postponing issues or setting them to RTBC should be something the community (at least two or three active users of the module plus a maintainer) should decide together.

A side-note: Actually, we already applied this patch and added some further fixes regarding coding standards (as a consistent use of the string translation trait, dependency injection, fixes to some comments/inline documentation, as well as last lines within some configuration files) within a development version of the module waiting to be pushed to the drupal.org repository, once the maintainers allow co-maintainership.

sun’s picture

If you have further code changes to contribute, then you should attach a patch here or in a separate issue. Project maintainers usually do not use their commit access to sidestep Drupal's regular issue/patch and peer review process.

Mario Steinitz’s picture

Already created according issues in the issue queue. See https://www.drupal.org/project/openid_connect/issues/2960885

Yet, have to extract the related patches from the development version. I'll do so within the next few days, after our local tests are done.

Mario Steinitz’s picture

FileSize
3.92 KB

This patch includes the above changes and also fixes a missing comma within an array in openid_connect_connect_current_user().

  • Mario Steinitz committed 5eee698 on 8.x-1.x
    Issue #2938027 by Buvaneshwari.V, Mario Steinitz: Coding standards: Use...
Mario Steinitz’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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