bdragon posted a patch at http://drupal.org/node/105334 for banner HEAD that gets the size of remote banners, and drupal_http_request() instead of getimagesize(). On my local Windows development machine, the old use of getimagesize() caused Apache/PHP to die horribly but the new version works nicely.

The current patch is a backport of that fix to banner 4.7

Differences from http://drupal.org/node/105334:

1) No change to $node->content field name, since this is not a problem in 4.7, so no need to run update.php after applying the patch. Also patch from http://drupal.org/node/105310 not included.

2) Additional specific check for 404 (Not Found) error return with a specific error message.

Best wishes, Mark.

CommentFileSizeAuthor
banner.module.4-7.remote.patch3.19 KBplumbley

Comments

wulff’s picture

Will be fixed in next commit. Thanks.

wulff’s picture

Status: Needs review » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)