Hello,
Some time ago I made an extra_filters_logo_resize module to work with imagecache actions. With updates made about a month ago, I started getting some really odd url "temporary redirect" issues with new logo images I uploaded. I updated to the latest dev version yesterday, and the issues still remain.

The log mentions a ton of 'invalid argument supplied' errors for the $image variable I'm passing in. Has the extra_filters handling changed recently? here's a sample of my module:

<?php
function extra_filters_logo_resize_image(&$image, $data = array()) {
  $maxWidth = $data['maxWidth'] ? $data['maxWidth'] : 240;
  $maxHeight = $data['maxHeight'] ? $data['maxHeight'] : 240;
  $maxArea = $data['maxArea'] ? $data['maxArea'] : 18000;
  $canvasWidth = $data['canvasWidth'] ? $data['canvasWidth'] : 0;
  $canvasHeight = $data['canvasHeight'] ? $data['canvasHeight'] : 0;
  $image = ResizeLogo($image, $maxWidth, $maxHeight, $maxArea, $canvasWidth, $canvasHeight);
  return true;
}

function ResizeLogo(&$image, $maxWidth, $maxHeight, $maxArea, $canvasWidth, $canvasHeight) {
...
}
?>

Let me know your thoughts. I have people breathing down my neck while I troubleshoot this. Your feedback is greatly appreciated. :D

Best regards,

Chris

Comments

dman’s picture

I can't really tell from anything you've described so far.
The 2.x-dev is mostly about moving some of the library files around, and a revamp of textactions. No change in argument handling, which is defined by imagecache and imageAPI further upstream.

If you can't quote the exact error message and line number, I'm in the dark.
Your $image should contain an $image->resource and $image->data, but beyond that, I can't say what's missing or where.

fietserwin’s picture

Version: » 6.x-1.x-dev
Status: Active » Postponed (maintainer needs more info)

Is this issue still relevant? If so, please assign a version to it (and be more specific about the messages as states in #1).

Note: since reporting this issue, the version field has become required. So I have to fill in a version now: guessing 6.x-1.x.

owntheweb’s picture

Status: Postponed (maintainer needs more info) » Closed (fixed)

Hello,
I don't think this issue is still relevant. I have totally forgotten about this post (meaning I probably experienced an "ID10T" error, fixed it and moved on), and am happily creating custom ImageCache Actions in 7.x now.

Thanks for an excellent module!!

Chris