Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.We have Feeds set up with Image Grabber to create nodes and pull images from some RSS feeds. We are getting many of these errors:
Undefined index: cardinality in feeds_imagegrabber_scrape_images() (line 555 of feeds_imagegrabber.module)
Nodes are being created, but I'm wondering if anyone has any ideas about this error?
Thanks in advance for anyone's help and guidance.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | feeds_image_grabber-undefined_index_cardinality-1990784-1.patch | 1.05 KB | brianbarker |
Comments
Comment #1
brianbarker CreditAttribution: brianbarker commentedI was getting these same messages. It turns out there are two lines of code where the cardinality value is being accessed from a $settings variable while it should be the $options variable. Simple typo. I made my first drupal patch, hope it works!
Comment #2
brianbarker CreditAttribution: brianbarker commentedI probably should change the status on this I guess?
Comment #3
rooby CreditAttribution: rooby commentedThanks for the patch.
It has been committed to the dev version and will be available there within the next 12 hours.
A new release will be made once more of the issues in the queue have been reviewed.
http://drupalcode.org/project/feeds_imagegrabber.git/commit/3bc3d64
As for it being your first patch, it was properly made, named, etc.
The issue status should have been "Needs review", patch to be ported would be if another version (like the D6 version) now needed this same fix, the patch would have to be ported to work for the drupal 6 version (not necessary in this case).
For more information on status see https://drupal.org/node/156119
Keep up the contributions :)