the hook_requirements does not work. patch attached fixes it. also upgrades the severity to the ERROR level to display a message on the administer page.

Side note: there is a duplicate message in the .module file in getid3_admin_settings_form() which could call hook_requirements() and display the message from there instead.

CommentFileSizeAuthor
#2 getid3_313345.patch1.35 KBdrewish
getid3.demos_.patch1 KBscor

Comments

scor’s picture

scor commented
Status: Active » Needs review

setting status accordingly.
Also, a note concerning the demo scripts could be added in the README.txt and on the project page.

drewish’s picture

drewish commented
StatusFileSize
new getid3_313345.patch1.35 KB

that didn't have the right path in the message. i copied the code from the module that we use on the settings page. give this a try.

scor’s picture

scor commented

yeah that looks even better! will let someone else set to RTBC!

drewish’s picture

drewish commented
Status: Needs review » Fixed

thanks went ahead and committed this to HEAD.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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