The lines 41 and 42 should have 10 indentation spaces but have 12.

CommentFileSizeAuthor
#5 wrong_indentation_into-2864430-5.patch511 byteshgunicamp
CodeStandard.png36.55 KBhgunicamp
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hgunicamp created an issue. See original summary.

hgunicamp’s picture

hgunicamp CreditAttribution: hgunicamp at CI&T commented
Title: Wrong identation into 'ldadmin.module' file » Wrong indentation into 'ldadmin.module' file
hgunicamp’s picture

hgunicamp CreditAttribution: hgunicamp at CI&T commented
Status: Active » Needs work
hgunicamp’s picture

hgunicamp CreditAttribution: hgunicamp at CI&T commented
Assigned: Unassigned » hgunicamp
Status: Needs work » Active

Working.

hgunicamp’s picture

hgunicamp CreditAttribution: hgunicamp at CI&T commented
Status: Active » Needs review
FileSize
wrong_indentation_into-2864430-5.patch511 bytes

Sending a fixing patch.

renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
Assigned: hgunicamp » renatog
Status: Needs review » Reviewed & tested by the community

Works Good.

Thanks.

  • renatog committed 7e3ab50 on 7.x-1.x authored by hgunicamp 
    Issue #2864430 by hgunicamp, renatog: Wrong indentation into 'ldadmin....
renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
Assigned: renatog » Unassigned
Status: Reviewed & tested by the community » Fixed

Commited in dev branch

Thank you very much @hgunicamp.

Good Work and Good Week.

Regards.

Status: Fixed » Closed (fixed)

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