Hi, I still experience issues with the "'this field cannot hold more than 1 values" error.
Found that the array_merge didn't override indexed (non-keyed) elements. So, I replaced it with `array_replace_recursive` which did the job in my setup.- 🇳🇱Netherlands megachriz
@akrrev
If you can provide your feed type configuration (yml file) and a sample source file, we could perhaps add test coverage for your use case. :) @MegaChriz
I just found that there were multiple places that required the change.
I've posted an updated patch.- 🇳🇱Netherlands megachriz
@akrrev
I see also that your patch is breaking a feature in Feeds, so I think it's not the correct solution. More specifically, it breaks a feature where you map multiple times to the same target. Mapping multiple times to the same target is useful in a situation when you want to import data into a field that accepts multiple values and your data is structured as follows: - Status changed to Needs review
over 1 year ago 2:40am 6 April 2023 - Status changed to RTBC
about 1 year ago 4:18pm 2 September 2023 - last update
about 1 year ago 701 pass, 1 fail The last submitted patch, 12: 3172709-enforce-recursive-merge-2.patch, failed testing. View results →
- 🇳🇱Netherlands megachriz
@Qusai Taha
@Venkatesh Rajan.J
I would like to add test coverage for this issue. Can you provide an export of your feed type and the file that you were trying to import? - 🇳🇱Netherlands megachriz
Qusai Taha responded on Slack:
Hi @MegaChriz Actually, I don't have a specific scenario for this issue, but usually, it appears when we are working on multilingual imports.
So therefore, I think it's better to give this issue attention again when I'll be focussing on multilingual issues.