Previous works on this issue, fixed in D8.

CommentFileSizeAuthor
#7 interdiff-2819787-3-7.txt758 byteschiranjeeb2410
#7 2819787-7.patch825 byteschiranjeeb2410
#3 2819787-3.patch821 bytesshashikant_chauhan

Comments

joyceg created an issue. See original summary.

Lal_’s picture

Lal_
He\Him
commented
shashikant_chauhan’s picture

shashikant_chauhan commented
Assigned: Unassigned » shashikant_chauhan
Status: Active » Needs review
StatusFileSize
new 2819787-3.patch821 bytes

adding patch.

joyceg’s picture

joyceg commented

@shashikant_chauhan, Thanks for the patch. It applies well.

chiranjeeb2410’s picture

chiranjeeb2410 commented
Status: Needs review » Reviewed & tested by the community

@joyceg,

The change is good to go. Patch applies clearly. Updating to RTBC

stefan.r’s picture

stefan.r commented
Status: Reviewed & tested by the community » Needs review

Can we link to https://www.drupal.org/requirements/database just to be consistent?

chiranjeeb2410’s picture

chiranjeeb2410 commented
StatusFileSize
new 2819787-7.patch825 bytes
new interdiff-2819787-3-7.txt758 bytes

@stefan.r

Uploading fresh patch with suggested fix in #6.
Please review.

stefan.r’s picture

stefan.r commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +Pending Drupal 7 commit
David_Rothstein’s picture

David_Rothstein commented
Status: Reviewed & tested by the community » Fixed
Issue tags: -Pending Drupal 7 commit

Either page seems OK to me to be honest, but I guess the one that is database-specific and more detailed is a bit better.

Committed to 7.x - thanks!

  • David_Rothstein committed c55fe93 on 7.x 
    Issue #2819787 by chiranjeeb2410, shashikant_chauhan, stefan.r, Kartagis...

Status: Fixed » Closed (fixed)

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