Warning on trying to add imagefield node

nirvanajyothi - June 13, 2009 - 05:25
Project:Image filter
Version:6.x-1.0-rc1
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed
Description

warning: Invalid argument supplied for foreach() in C:\wamp\www\domain\sites\all\modules\image_filter\image_filter.module on line 136.

Happens when i try to add image added using http://drupal.org/project/imagefield into other content type pages using [image:nid].

Is this module compatible only with images added using conventional Image module http://drupal.org/project/image ?

-thank you.

#1

raintonr - June 15, 2009 - 00:43
Category:support request» feature request

Q. "Is this module compatible only with images added using conventional Image module?"

A. Yes. For now. I take if your node is not an 'image' node?

I would like to have imagefilter cope with imagefield fields. Something like:

[image:123] - If this isn't an image node, find the first imagefield field and display it's first image.

[image:123.xyz] - Find the imagefield named 'xyz' on node 123 and display it's first image.

[image:123.xyz.99] - Find the imagefield named 'xyz' on node 123 and display the 99th image.

Does that sound like a good idea?

#2

nirvanajyothi - June 15, 2009 - 11:26

Yes. That is exhaustive, all use cases i guess, an imagefilter having features anyone could ever ask for!
It will be very useful in the future as imagefield cck takes over images in drupal websites.

For a node with a single imagefield (as in current image node) the option to customize imagefilter module to call that particular field name may be sufficient. This will be the basic use case.

-thank you

#3

raintonr - August 6, 2009 - 12:02
Status:active» needs review

Have just committed a change to allow the filter to work with imagefield and imagecache. size= attribute is the imagecache preset.

Just handles the first imagefield and delta at the moment but step in the right direction.

There's no development build so you'll have to take it from CSV to test at the moment. Anyone game?

#4

nirvanajyothi - August 6, 2009 - 16:47

Will be happy to try it out. Am not accustomed to CVS stuff. Think i just have to download those files and place them in image_filter folder and run the update.php. Please correct if i am not right.

#5

raintonr - August 7, 2009 - 10:08

RE: #4... yeah, just download the .module and .info from CVS and plonk in your directory.

You can grab them here:

http://cvs.drupal.org/viewvc.py/drupal/contributions/modules/image_filter/

There is no need to run update, no changes there.

#6

nirvanajyothi - August 7, 2009 - 11:52

It works fine! I tested for a content type with a single cck image field. The align,width and size options working alright as well! The image is linked to the node where it belongs.

Think this feature makes Imagefilter module more powerful and we may see more users shift ing to CCK for Images instead of the Image module.

Will a dev version be coming out now?

#7

raintonr - August 19, 2009 - 04:38
Status:needs review» fixed

Sweet - thanks for confirming.

Have made a v1.0 release for D6 with this included.

#8

nirvanajyothi - August 19, 2009 - 16:23

Congrats! Will try the new release.

Thank you.

#9

System Message - September 2, 2009 - 16:30
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.