Hi,
I followed the tutorial and am certain I have everything set up correctly. I have updated the URL_to Absolute.inc code. However although the nodes are created and other fields correct, the images are not being saved to the FIG image field I created. For each imported item I get the following log entries:

Deprecated function: Function split() is deprecated in feeds_imagegrabber_feeds_set_target() (line 172 of /home/dev/public_html/sites/all/modules/feeds_imagegrabber/feeds_imagegrabber.module).

Notice: Undefined property: stdClass::$redirect_code in feeds_imagegrabber_webpage_scraper() (line 428 of /home/dev/public_html/sites/all/modules/feeds_imagegrabber/feeds_imagegrabber.module)

I am using this feed: https://picasaweb.google.com/data/feed/base/user/113121343407001669583/a...
with this class: scaledimage-onscreenpane

thanks!

CommentFileSizeAuthor
#4 0004-Passing-settings-into-feeds_imagegrabber_scrape_imag.patch1.37 KBmcboof
#3 0001-Removing-deprecated-split.patch895 bytesmcboof
#3 0003-Empty-redirect-code-warning.patch834 bytesmcboof
#2 tt_bizzklein.txt644 bytessummit

Comments

nooysters’s picture

nooysters commented
Priority: Normal » Major
summit’s picture

summit commented
StatusFileSize
new tt_bizzklein.txt644 bytes

Hi,

I am having exactly the same error messages on a completely different feed.
I add an example feed for references.

Looking forward to your reply. And thanks for making this module!
Greetings, Martijn

mcboof’s picture

mcboof commented
StatusFileSize
new 0003-Empty-redirect-code-warning.patch834 bytes
new 0001-Removing-deprecated-split.patch895 bytes

I don't think these warnings cause any issues, but I've attached patches to fix both of them. They were filling up my error logs.

mcboof’s picture

mcboof commented
StatusFileSize
new 0004-Passing-settings-into-feeds_imagegrabber_scrape_imag.patch1.37 KB

Ooops. I forgot another change I made to remove warnings. At the most, the feeds_imagegrabber_scrape_images functions uses the $settings array, but doesn't have it. This patch passes it in.

rooby’s picture

rooby commented
Status: Active » Needs review

Different bugs should have different issues.
It makes it easier to find that way.
I created a patch for the redirect code before I realised this issue had multiple patches that did differnet things. See #1618328: Notice: Undefined property: redirect_code in feeds_imagegrabber_webpage_scraper()

Also, for anyone to know there is a patch in here you need to set to needs review.

Please make different issues so that each patch that fixes an individual problem has it's own issue with its own description.

If multiple patches are required to fix a single issue (note that this issue is not a single issue), they shoul dbe all in one patch file, not multiple or people won't know what to apply.

rooby’s picture

rooby commented
Status: Needs review » Closed (duplicate)

OK I've split this out into individual issues, as they were all separate and the issue from #4 had another issue already.

It just makes it easier for people to find a solution if they have one of the three different problems.

Marking as duplicate of:

From comment:
#3 - #1618360: Deprecated function: Function split() is deprecated in feeds_imagegrabber_feeds_set_target()
#3 - #1618328: Notice: Undefined property: redirect_code in feeds_imagegrabber_webpage_scraper()
#4 - #1511010: Feeling lucky option doesn't work

I have credited mcboof in those tickets for the patches here.