Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rodrigoac created an issue. See original summary.

rodrigoac’s picture

Assigned: rodrigoac » Unassigned
Status: Needs work » Needs review
FileSize
225 bytes

Patch attached

mgoncalves’s picture

Assigned: Unassigned » mgoncalves
dhruveshdtripathi’s picture

FileSize
4.72 KB

Got an error after applying the patch.

dhruveshdtripathi’s picture

Status: Needs review » Needs work
mgoncalves’s picture

Hello @dhruveshdtripathi,

It's import to follow the process to fix a bug.
When was assigned to someone, means that person is working on.
Please, be attention with that.

Another point is that, the fix of this bug not means isn't working, but there is another one that is interfering to test it.

I created another bug indicate this problem. See #2865333 Database Exception during install module.

After fix the error reported above, we can back and test it again with the same patch submitted.

Thank you

mgoncalves’s picture

Added related issue.

dhruveshdtripathi’s picture

I saw that it was assigned to you but the status was needs review, that's why I pointed the error.

Thank you!

hgunicamp’s picture

I applied the patch attached to 2793285 issue and it fixed the database problem.
After that, I tested the patch attached here and it worked for.

and

hgunicamp’s picture

Assigned: mgoncalves » Unassigned
Status: Needs work » Reviewed & tested by the community
hgunicamp’s picture