hook_schema_alter() is missing an @ingroup schemaapi [#1963206]

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

Same problem on D7 docs.

Comment	File	Size	Author
#4	0001-Issue-1963206-by-fgm-hook_schema_alter-is-missing-in.patch	775 bytes	fgm
#4
#2	0001-Issue-1963206-hook_schema_alter-D7.patch	753 bytes	fgm
#2
#1	0001-Issue-1963206-by-fgm-hook_schema_alter-is-missing-in.patch	775 bytes	fgm
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

File	Size
0001-Issue-1963206-hook_schema_alter-D7.patch	753 bytes

Comment #3

6 April 2013 at 15:34

Status:

Needs review

» Needs work

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

Comment #4

fgm

French

Paris, France

CreditAttribution: fgm commented 7 April 2013 at 07:43

Status:

Needs work

» Needs review

File	Size
0001-Issue-1963206-by-fgm-hook_schema_alter-is-missing-in.patch	775 bytes

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

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

Comment #5

jhodgdon

she/her

English

CreditAttribution: jhodgdon commented 10 April 2013 at 14:37

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!