Title says it all: hook_schema_alter() is part of the Schema API and should be tagged as such.

Same problem on D7 docs.

CommentFileSizeAuthor
#4 0001-Issue-1963206-by-fgm-hook_schema_alter-is-missing-in.patch775 bytesfgm
#2 0001-Issue-1963206-hook_schema_alter-D7.patch753 bytesfgm
#1 0001-Issue-1963206-by-fgm-hook_schema_alter-is-missing-in.patch775 bytesfgm

Comments

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
StatusFileSize
new 0001-Issue-1963206-by-fgm-hook_schema_alter-is-missing-in.patch775 bytes

Trivial patch attached.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Status: Active » Needs review
StatusFileSize
new 0001-Issue-1963206-hook_schema_alter-D7.patch753 bytes

And equivalent D7 patch.

Status: Needs review » Needs work

The last submitted patch, 0001-Issue-1963206-hook_schema_alter-D7.patch, failed testing.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Status: Needs work » Needs review
StatusFileSize
new 0001-Issue-1963206-by-fgm-hook_schema_alter-is-missing-in.patch775 bytes

Hmm... I thought the bot didn't run tests named *-D(version).patch ?

Resubmitting D8 version just to make "needs review" stick.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Assigned: Unassigned » jhodgdon
Status: Needs review » Reviewed & tested by the community

Regarding the D7 suffix, please read the description on the File Attachments field, which explains the current state of affairs regarding testing. Also, it's not normally necessary to even make a D7 patch for something like this, because when we commit the 8.x patch we can generally use -p2 and commit to 7.x at the same time.

Anyway, looks like a good patch -- I'll get it committed shortly. Thanks!

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Assigned: jhodgdon » Unassigned
Status: Reviewed & tested by the community » Fixed

Thanks! Committed latest patch to 8.x and 7.x.

Status: Fixed » Closed (fixed)

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