Support from Acquia helps fund testing for Drupal Acquia logo

Comments

N.kishorekumar created an issue. See original summary.

N.kishorekumar’s picture

Please review this Patch

N.kishorekumar’s picture

Updated code formatting and fixed security issue
please review this patch

N.kishorekumar’s picture

Please review only this updated patch

nico.knaepen’s picture

Status: Needs review » Reviewed & tested by the community

Patch reviewed & tested. No issues found.
Thanks for helping a hand in formatting the code following the coding standards.

N.kishorekumar’s picture

Issue tags: +Quick fix
truls1502’s picture

Version: 7.x-3.0-rc5 » 7.x-3.x-dev
Status: Reviewed & tested by the community » Needs review
FileSize
1.56 KB

Corrected the patch from #4 because of PHP 7.2.

renatog’s picture

Status: Needs review » Reviewed & tested by the community

The patch works good.

Thank you guys for contribution.

  • RenatoG committed a45b44b on 7.x-3.x authored by truls1502
    Issue #2555783 by N.kishorekumar, truls1502: Code formatting
    
renatog’s picture

Status: Reviewed & tested by the community » Fixed

Committed to the dev branch.

Thanks.

Status: Fixed » Closed (fixed)

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