As part of starting to use this module I realised that the module wasn't showing when listing the the modules (via UI) when typing "Search API", and that's due to the module key used as name in the info file.

Also note that the module requires at least Search API >= 8.x-1.14.

As part of fixing this I also thought I could fix the coding standards reported by drupalcs and drupalcsp, so a patch will follow for that.

CommentFileSizeAuthor
#5 3153778-4-module_name_and_coding_standards.patch16.62 KBfjgarlin
#2 3153778-2-module_name_and_coding_standards.patch16.6 KBfjgarlin

Comments

fjgarlin created an issue. See original summary.

fjgarlin’s picture

fjgarlin commented
StatusFileSize
new 3153778-2-module_name_and_coding_standards.patch16.6 KB

Patch attached.

fjgarlin’s picture

fjgarlin commented
Status: Active » Needs review
fjgarlin’s picture

fjgarlin commented
Issue summary: View changes
fjgarlin’s picture

fjgarlin commented
StatusFileSize
new 3153778-4-module_name_and_coding_standards.patch16.62 KB

Included the version constraint as Search API needs to be >= 8.x-1.14

nbaosullivan’s picture

nbaosullivan commented
Status: Needs review » Fixed

Thanks @fjgarlin! Committed!

Status: Fixed » Closed (fixed)

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