The call by ad_manager_block_view() to theme('ad_manager_ad') was passing a string instead of an array. Likewise, theme_ad_manager_ad() was accepting a string instead of an array.

CommentFileSizeAuthor
#2 ad_manager-change_hook_theme_arguments_variables-1859598-2.patch404 bytescodycraven
theme_ad_manager_parameter_fix.patch828 bytesmatt poole

Comments

flacoman91’s picture

flacoman91 commented

thank you! i couldn't get this module to put out the blocks on my page and I didn't understand the php errors on a clean install.
Your patch fixes it.

codycraven’s picture

codycraven commented
Status: Needs review » Fixed
StatusFileSize
new ad_manager-change_hook_theme_arguments_variables-1859598-2.patch404 bytes

Thanks Matt, committed and pushed.

Also as a note this needed the arguments key converted to variables (another D6 to D7 change), attached is a patch that I've committed and pushed as well.

Status: Fixed » Closed (fixed)

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