- ๐ท๐ดRomania reszli Tรขrgu Mureศ
changes to improve functionality and code:
- I needed some more elements inside the root node, so I introduced an additional config field called "metadata" as a textarea
- also, I replaced the custom replacement of {{ lang }} by allowing the use of token in the attributes and metadata fields
- additionally I switched to dep. inj. where possible
still left to do:
- instead of strpos and str_replace, switch to a better way of manipulating the XML (i.e. using DOMDocument)
- Status changed to Needs work
almost 2 years ago 8:38pm 22 February 2023 - ๐บ๐ฆUkraine chizh273
The #47 has one issue.
If you open the settings of the data export format with enabled warning logging ($config['system.logging']['error_level'] = 'verbose';
) you will get a warning "Warning: Undefined array key "metadata" in /var/www/html/web/modules/contrib/views_data_export/src/Plugin/views/style/DataExport.php on line 241".I have updated the #47 patch to fix this warning.
- ๐จ๐ญSwitzerland sir_squall
Thank you very much the path #50 work well!
- ๐บ๐ฆUkraine proweb.ua
how to add attributes to Item node name?
can this be done via the API? - Status changed to Needs review
6 months ago 3:26pm 24 July 2024 - ๐ซ๐ทFrance jibus
#50 works also.
I would suggest to move the "Item wrapper node name" before "Item node name" configuration field.
- ๐บ๐ธUnited States newme154
Hello,
I am not seeing the drop down info for the xml settings. however, I do see it for the CSV settings.
- ๐ฏ๐ตJapan bassline
#50 works
The XML configuration worked in my environment
- Status changed to RTBC
6 days ago 7:28am 16 January 2025 - ๐ฌ๐งUnited Kingdom steven jones
This looks great, and thanks for the hard work everyone, but this sort of change really should have some tests to make sure we're not breaking things for existing sites, and that the new settings work correctly etc. so setting back to needs work.