Closed (fixed)
Project:
Feeds Image Grabber
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
11 Mar 2011 at 12:03 UTC
Updated:
14 Aug 2011 at 08:51 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
nimi commentedLooking for this one too... Any news?
Thanks,
Nimi
Comment #2
johnvsame for me..
Comment #3
drupalthemernet commentedI'm waiting for this release too.
Comment #4
brad.mrumlinski commentedMe too... Are there large requirements needed to upgrade this to work with D7? I'm willing to put in some work.
Comment #5
drupalicious commentedSubscribe
Comment #6
Krummrey commentedI am not a coder, but I've taken the latest DEV and ran it through the coder module at http://upgrade.boombatower.com/
The resulting patch is attached, but there are 11 bugs to be fixed as it seems, three of them critical.
+++++++++++++++++++++++++++++
feeds_imagegrabber/feeds_imagegrabber.module
Summary
minor: 4, normal: 1, critical: 3, ignored: 0
Details
+11: [critical] Module .info files must now specify all loadable code files explicitly.
+40: [critical] Parameters to hook_form_alter have changed.
+81: [normal] Use "elseif" in place of "else if"
+173: [critical] hook_nodeapi_xxx() has been renamed to hook_node_xxx().
+647: [minor] in most cases, replace the string function with the drupal_ equivalent string functions
+648: [minor] in most cases, replace the string function with the drupal_ equivalent string functions
+844: [minor] @see should always be in a comment block rather than as an inline comment.
+870: [minor] in most cases, replace the string function with the drupal_ equivalent string functions
+++++++++++++++++++++++++++++
feeds_imagegrabber/feeds_imagegrabber.install
Summary
minor: 0, normal: 3, critical: 0, ignored: 0
Details
+69: [normal] Use st() instead of t() in hook_install(), hook_uninstall() and hook_update_N()
+69: [normal] The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.
+70: [normal] The $text argument to l() should be enclosed within st() so that it is translatable from within the install.
Maybe someone can take it from here.
Comment #7
rfayThere is already a D7 branch, and it looks like some work went into it in December (on the 7.x-1.x branch)
Comment #8
publicmind commentedYes, I already started working on the 7.x branch last year and was only able to upgrade some utility functions as Feeds was not ready for integration then. I have been out of touch with Feeds development recently and will have to spend some time which is scarce these days :(
Regards,
Comment #9
gberm commentedInterested in this too.
Comment #10
Slim Pickens commentedSubscribe
Comment #11
gberm commentedHas anyone used the D7 branch. Is it at all stable?
Comment #12
TallDavid commentedSubscribe
Comment #13
paulgemini commentedSubscribe
Comment #14
jensimmons commentedsubscribe
Comment #15
sachbearbeiter commentedsubscribe
Comment #16
Anonymous (not verified) commentedThis is something I'm very much looking for at the moment as well. Need to download images from a remote XML feed to my Drupal 7 install. When can we have a stable release for D7?
Comment #17
RonP commentedVery interested as well. Just figured out feeds and was about to (most likely) break everything by trying to add an image to the articles.
You are a very smart and intuitive person C-=
Subscribe me
Comment #18
endend commentedSubscribe
Comment #19
rodych commentedSubscribe
Comment #20
inversed commentedsubscribe
Comment #21
Christopher Riley commentedToday I got a notice
Feeds Image Grabber 7.x-1.x-dev (2011-Feb-25)
Project not supported: This project is no longer supported, and is no longer available for download. Disabling everything included by this project is strongly recommended!
Includes: Feeds Image Grabber
Is there no plans to have a D7 version of this?
Comment #22
Jason Dean commentedsubscribe me
Comment #23
mrsinguyen commented+1
Comment #24
gberm commentedIs this dead?
Comment #25
paulgemini commentedNothing's dead if someone else wants to work on it.
Is there another module that accomplishes the same thing (or similar)?
I went ahead and downloaded Fig for D7 and am going to test it and try to create a patch that gets it working. Don't get your hopes up, as i'm a novice, but I need this!
Comment #26
rfayJust to reiterate so that everybody is aware of it: You don't need a module like FIG in D7 if you have a URL to your image. Feeds D7 is perfectly capable of handling an image URL like http://... or ftp://... or whatever. Out of the box. You only would need FIG (as I understand it) if you have to scrape a remote site for an image.
I did a few screencasts on importing at http://www.drupalcommerce.org/node/467. While these are Commerce-oriented, the part about the image import just uses regular D7 feeds capabilities, and you'll notice that all the image imports are URLs (remote).
Comment #27
paulgemini commentedWorked for me! Then again, I'm not really sure if I'm using this module or just regular old Feeds.
I used an xpath parser to import images into nodes' image field using only their url (supplied by an xml file).
Comment #28
publicmind commentedI finally got some time this weekend, so hurry and grab the D7 version from the repository. :)
And as rfay already mentioned, FIG is not for those who have their images in the Feed XML. For this, Feeds with SimplePie enclosures does a good job. FIG is needed when you need to download images from the webpage of the feed item.
Thanks for the patience.
Regards,