The foreign key definition for advanced_help_index in the advanced_help.install file is defined as follows:
'foreign keys' = array('module' => array('system' => 'name')
This does not follow Drupal 7 guidelines.
It should be defined using the foreign keys 'table'/'columns' syntax as found at
http://api.drupal.org/api/drupal/includes%21database%21schema.inc/group/....

CommentFileSizeAuthor
#1 foreign-key-issues-1524936-1.patch497 bytesvaibhavjain
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vaibhavjain’s picture

vaibhavjain
he/him
Primary language Hindi
Location Delhi
CreditAttribution: vaibhavjain commented
Status: Active » Needs review
FileSize
foreign-key-issues-1524936-1.patch497 bytes

Here is a patch for the same.

euk’s picture

euk CreditAttribution: euk commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Why such a small thing it not pushed yet?

No matter if Drupal does not support FK in the core, it is still may be used by other modules.

Patch #1 worked for me.

euk’s picture

euk CreditAttribution: euk commented
Version: 7.x-1.0 » 7.x-1.2

  • gisle committed 088a67a on 7.x-1.x authored by vaibhavjain 
    Issue #1524936 by vaibhavjain: Foreign Key Definition In Install File...
gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
CreditAttribution: gisle as a volunteer commented
Status: Reviewed & tested by the community » Fixed

Fix now in version 7.x-1.3.

gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
CreditAttribution: gisle as a volunteer commented
Version: 7.x-1.2 » 7.x-1.3

Setting correct version.

Status: Fixed » Closed (fixed)

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