The YouTube module has a function called

function youtube_api_search($term, $start_at = 1, $num_items = 10, $constraints = array())

however this collides with hook_search (which takes completely different parameters), meaning that the search admin page (which happens to call this hook) bombs out with a PHP/datatype error. See http://api.drupal.org/api/function/hook_search/5 for details.

Suggest renaming function to youtube_api_search_videos() or similar. This would be an API change, and should be called out in release notes. See attachment.

CommentFileSizeAuthor
youtube_api.rename_search.patch672 byteslyricnz

Comments

lyricnz’s picture

lyricnz commented
Status: Active » Needs review

Needs review, I suppose :)

beeradb’s picture

beeradb commented
Status: Needs review » Reviewed & tested by the community

Thanks for the bug report, as well as the patch.

I'll commit with the next round of work.

beeradb’s picture

beeradb commented

Fixed in http://drupal.org/cvs?commit=124629

thanks again.

beeradb’s picture

beeradb commented
Status: Reviewed & tested by the community » Closed (fixed)