the following error is returned when I go to "http://www.mydomain.com/audio/download" without any argument:

* warning: Invalid argument supplied for foreach() in /home/mydomain/modules/node.module on line 359.
* warning: implode() [function.implode]: Bad arguments. in /home/mydomain/modules/node.module on line 363.
* user warning: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1 query: SELECT n.nid, n.vid, n.type, n.status, n.created, n.changed, n.comment, n.promote, n.moderate, n.sticky, r.timestamp AS revision_timestamp, r.title, r.body, r.teaser, r.log, r.format, u.uid, u.name, u.picture, u.data FROM node n INNER JOIN users u ON u.uid = n.uid INNER JOIN node_revisions r ON r.vid = n.vid WHERE in /home/mydomain/includes/database.mysql.inc on line 121.

the expected behaviour would be returning "404 page not found". the included patch fixes this bug.

CommentFileSizeAuthor
#2 audio.module.patch_1.txt1.05 KBluperry
audio.module.patch_0.txt687 bytesluperry

Comments

drewish’s picture

drewish commented

that looks like a good patch, i want to check that it and play work the same way.

one side note, for example URLs you should use example.com. see RFC2606 for more.

luperry’s picture

luperry commented
StatusFileSize
new audio.module.patch_1.txt1.05 KB

it seems "http://www.example.com/audio/play" has the same issue. here is another patch.

and thanks for the tip. :)

drewish’s picture

drewish commented
Status: Needs review » Fixed

thanks, i've committed this to the 4.7 branch.

Anonymous’s picture

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