CommentFileSizeAuthor
#9 invite-2871636-8.patch2.39 KBSpartyDan
#6 invite-2871636-6.patch3.32 KBSpartyDan
#3 Captura de Tela 2017-04-20 às 22.11.45.png26.25 KBrenatog
#2 invite_tip_for_token-2.patch3.83 KBrenatog
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

RenatoG created an issue. See original summary.

renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
FileSize
invite_tip_for_token-2.patch3.83 KB

Patch with fix it's in attachment.

renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
Assigned: renatog » Unassigned
Status: Active » Fixed
FileSize
Captura de Tela 2017-04-20 às 22.11.45.png26.25 KB

Fixed.

Commited in dev branch.

Regards.

  • RenatoG committed ad4a188 on 7.x-4.x 
    Issue #2871636 by RenatoG: On install check if token is disable show and...

Status: Fixed » Closed (fixed)

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

SpartyDan’s picture

SpartyDan CreditAttribution: SpartyDan as a volunteer commented
Status: Closed (fixed) » Active
FileSize
invite-2871636-6.patch3.32 KB

The patch in #2 incorrectly changed the format of existing code. This patch reverts that change.

Thank you for the code that you added to address this issue.

In the future, please review your patches to ensure that they don't introduce unwanted changes.

  • SpartyDan committed d4c75b6 on 7.x-4.x 
    Issue #2871636 - Fix coding style issues introduced by commit ad4a188.

  • SpartyDan committed 4945e3e on 7.x-4.x 
    Issue #2871636 - Fix coding style issues - improper 4 space indent...
SpartyDan’s picture

SpartyDan CreditAttribution: SpartyDan as a volunteer commented
FileSize
invite-2871636-8.patch2.39 KB

Patch #6 incorrectly indented some code with 4 spaces. Now correctly indented with 2 spaces.