When you want to import a csv without headers and map the first column from the csv, you get the error 'You must select a mapping source'.

CommentFileSizeAuthor
#6 feeds-allow-zero-1984962-6.patch1.44 KBtwistor
#5 test-1984962-5.patch3.16 KBredsd
#3 feeds_use_0_as_source_1984962_1.patch1.55 KBmvd81NL
#1 feeds_use_0_as_source_1984962_1.patch1.57 KBmvd81NL

Comments

mvd81NL’s picture

mvd81NL commented
Status: Active » Needs review
StatusFileSize
new feeds_use_0_as_source_1984962_1.patch1.57 KB

Created a patch for this issue.

Status: Needs review » Needs work

The last submitted patch, feeds_use_0_as_source_1984962_1.patch, failed testing.

mvd81NL’s picture

mvd81NL commented
StatusFileSize
new feeds_use_0_as_source_1984962_1.patch1.55 KB
mvd81NL’s picture

mvd81NL commented
Status: Needs work » Needs review
redsd’s picture

redsd commented
StatusFileSize
new test-1984962-5.patch3.16 KB

Maybe this patch will work.

twistor’s picture

twistor commented
Version: 7.x-2.0-alpha8 » 7.x-2.x-dev
StatusFileSize
new feeds-allow-zero-1984962-6.patch1.44 KB

Does this solve your issue?

mvd81NL’s picture

mvd81NL commented

Thanks but my (#3) patch already works. It only failed (#1) because of the paths (first lines) in the patch, not the patch itself.

Fugazi-2’s picture

Fugazi-2 commented

#3 worked for me.

twistor’s picture

twistor commented

#3 is technically correct, but too verbose. It's not going in that way.

fureigh’s picture

fureigh commented

Confirming that #3 worked for me as well.

mikran’s picture

mikran commented
Status: Needs review » Reviewed & tested by the community

#6 fixed it.

twistor’s picture

twistor commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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

joelwallis’s picture

joelwallis commented
Issue summary: View changes

I just stumbled upon this bug today and fixed the code. And I was generating the patch to send back to the community I saw a LOT of commits into 7.x-2.x branch, which already had this one, fixing the bug.

Why the hell there's no releases containing these fixes?

marc.groth’s picture

marc.groth commented
Version: 7.x-2.x-dev » 7.x-2.0-alpha9
Category: Bug report » Task
Status: Closed (fixed) » Patch (to be ported)

Agreed with @joelwallis. It's been 2 years and this is still not part of a stable release (even though the latest one was released less than 2 weeks ago). Patch #6 works well and was already committed to dev. Could it please be added to the next planned release?

twistor’s picture

twistor commented
Version: 7.x-2.0-alpha9 » 7.x-2.x-dev
Category: Task » Bug report
Status: Patch (to be ported) » Closed (fixed)
marc.groth’s picture

marc.groth commented

It would be great if we could get an explanation as to why this hasn't made it into a stable release in the past 2 years; rather than just updating the status without a word. Does it require further work?

twistor’s picture

twistor commented

There hasn't been a release since this was fixed. Only a security release that contained security fixes, no other fixes.