Thanks for a GREAT module!
Unless I'm not using it correctly, "grabbed" images don't seem to be paying attention to any of the File Field Paths module settings. All of my grabbed images are being placed directly into the /files directory.

Any plans to support this or is this a case where FileField Paths module needs to support Image Grabber?

Thank you,
Scott

CommentFileSizeAuthor
#4 webserver-directory-structure.jpg207.41 KBSc0tt
#4 settings-for-feed-display-node-type.jpg67.78 KBSc0tt
#4 image-grabber-settings-within-feed.jpg94.16 KBSc0tt
#2 Untitled-1.jpg117.36 KBSc0tt
#1 config.jpg85.06 KBpublicmind

Comments

publicmind’s picture

publicmind commented
Status: Active » Fixed
StatusFileSize
new config.jpg85.06 KB

The module correctly saves the images into the directory mentioned in the path settings of the field. You will have to set the path of each field from the "manage fields" page. Refer to the screenshot attached. Let me know if it doesn't work out for you.

Sc0tt’s picture

Sc0tt commented
StatusFileSize
new Untitled-1.jpg117.36 KB

Wow! Thanks for the great response!

I looked at the manage fields page for my CCK image field and I do have a path; but I did notice a difference. Your image shows the path settings under "path settings" and mine is under "FileField Path settings". I tested the CCK image filed in my story node and images are being saved to the correct path.

I've attached a screenshot.

Thanks again for your help,
Scott

publicmind’s picture

publicmind commented

Scott,

If your story images are being uploaded to the right folder, then ImageGrabber must as well store it in that directory as ImageGrabber inquires from filefield about the field directory. I am not sure why this is happening, can you lend me some details about your supporting module versions (Filefield, Imagefield, FeedAPI especially)? Also, if possible include the page URL in the screenshot. I am currently using v3.2 of Filefield and it works as expected on this version.

Regards,

Sc0tt’s picture

Sc0tt commented
StatusFileSize
new image-grabber-settings-within-feed.jpg94.16 KB
new settings-for-feed-display-node-type.jpg67.78 KB
new webserver-directory-structure.jpg207.41 KB

Thank you!
Please find below the information you requested along with a few more screen captures of my settings and directory structure. If you need more information or access to my site, please let me know.
Thank you again,
Scott

FileField 6.x-3.2
FileField Paths 6.x-1.3
ImageField 6.x-3.2
FeedAPI 6.x-1.9-beta2
Image Grabber 6.x-1.9

Drupal 6.14
MySQL database 5.0.77mm0.1
PHP 5.2.9
PHP memory limit 96M
PHP register globals Disabled
Unicode library PHP Mbstring Extension
Update notifications Enabled
Web server Apache/1.3.41 (Unix) mod_auth_passthrough/1.8 mod_log_bytes/1.2 mod_bwlimited/1.4 FrontPage/5.0.2.2635.SR1.2 mod_ssl/2.8.31 OpenSSL/0.9.8b mod_status/0.2mm
cURL Library 7.15.5
jQuery UI 1.6

publicmind’s picture

publicmind commented

Hi Scott,

First, I apologize. I thought you were talking about the default paths settings of the filefield, when you were actually discussing Filefield Paths module. I just got confused, as I never used Filefield Paths module or heard of it. I am extremely sorry for wasting your as well as my time.

Before I write, what my view is, can you do this:
1) go to a feed-item node created by imagegrabber.
2) click on edit and make some changes in the title or body
3) save it.

Then check if the image associated with this node is moved to the desired directory or not. Please report what happens, in the meantime I will do some checking as well.

Thanks and Regards.

publicmind’s picture

publicmind commented

Here is the conclusion after day's work. I was able to integrate ImageGrabber with Filefield Paths but unfortunately it required some changes in Filefield Paths. At the moment, Filefield only updates filepaths when a node form is submitted. 

Line 47, filefield_paths/modules/filfield.inc

function filefield_filefield_paths_get_fields(&$node, &$ffp) {
  if (is_object($node) && $node->form_id == $node->type .'_node_form') {

The form id of the node is not set when imagegrabber updates a node, as it uses node_load and node_save functions for the purpose. I think we must push this issue into the Filefield Paths queue, before anything can be done from imagegrabber side.

Regards.

Sc0tt’s picture

Sc0tt commented

No need to apologize. Thank you for taking the time to investigate this and submit the support request to the FileFile Paths project.
Scott

publicmind’s picture

publicmind commented
Status: Fixed » Postponed
publicmind’s picture

publicmind commented
Assigned: Unassigned » publicmind
Status: Postponed » Fixed

The issue about form id has now been fixed in the dev version of Filefield Paths. Though I have still written the patch code to make it work with 1.3 release. It will be available in the dev version (will be available after the packaging scripts run in next 12 hours) and its stable for use on production site.

Regards,
publicmind

Status: Fixed » Closed (fixed)

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

paganwinter’s picture

paganwinter commented

Subscribing...