I applied the latest patch in #651478: Mapping on import, but my Batch is still failing.

I tried dpm($view_details) in the constructor for FeedsViewBatch and it seems like nothing is being passed to the constructor.

Are there any other patches I need to apply to get this working?

CommentFileSizeAuthor
#12 update-for-feeds-beta10-928770-12.patch578 bytesmegachriz

Comments

Anonymous’s picture

Anonymous (not verified) commented
Title: Is this module functioning? » Module only works with Feeds-alpha16

So I downloaded Feeds-alpha16 as was used in the screencast. This solved the problem that I was having with the Batch failing.

I still get the following warning when I import:

warning: Division by zero in /Users/clark/Documents/sites/feeds-iswc/sites/all/modules/feeds_view_parser/FeedsViewParser.inc on line 42.
dawehner’s picture

dawehner
Primary language German
commented
Category: support » bug

Well this is a bug

adityakg’s picture

adityakg commented
Assigned: Unassigned » adityakg
Status: Active » Needs work

Ah, sorry for the broken dependencies. The last time I check, this module is working with feeds beta1, while now it is beta9 I think it is worth updating. I will try to do it on this weekend.

@linclark: thanks for pointing out!
@dereine: indeed this is a bug. will fix it asap :)

Anonymous’s picture

Anonymous (not verified) commented

@adityakg Thank you for the awesome module, I was thrilled when I got it working. I'm sure lots of SemWeb geeks will be too once I show them how to use it.

I can test it when you have the updated version ready if you need.

adityakg’s picture

adityakg commented

Sorry my school life has been unexpected with lots of urgent work suddenly popped up. I will definitely look at it on this weekend.

Anonymous’s picture

Anonymous (not verified) commented

Thanks for the update (I definitely understand). It is not urgent for me, I probably won't demo it for anyone until November.

Mercury500’s picture

Mercury500 commented

So, it's not working now, correct? I.e. we should wait for a fix (using feeds 6.x-1.0-beta9).

kika’s picture

kika commented

Confirmed bug described on #1 on 2010-Aug-12 -dev release, on import. Please fix!

kika’s picture

kika commented

*bump* - any update on this?

areynolds’s picture

areynolds commented

subscribe

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
commented
Priority: Normal » Critical

Subscribing.

megachriz’s picture

megachriz
Primary language Dutch
commented
StatusFileSize
new update-for-feeds-beta10-928770-12.patch578 bytes

The reason why this module doesn't work with the latest Feeds version is because the way of how getSourceElement() should be implemented has been changed. Previously the item was given as a first parameter. Now it is a FeedsImportBatch object.

Patch attached.

megachriz’s picture

megachriz
Primary language Dutch
commented
Status: Needs work » Needs review
R.J. Steinert’s picture

R.J. Steinert commented

subscribing