exportDX2excel can't cope with multiple filters
Export DX to CSV fails if multiple filters are found.
Task remapp.exports.dx_export.exportDX2excel[] raised unexpected: MultipleObjectsReturned('get() returned more than one XrayFilters -- it returned 2!',)
Comments (21)
-
reporter -
reporter - changed milestone to 0.7.4
This needs to go out in a fix release
-
reporter MIssed one. Now works for xlsx. Refs
#430→ <<cset d5fc639fdfb3>>
-
reporter - changed status to resolved
Both xlsx exports and csv exports tested with multiple filters. Fixes
#430. Propose releasing as a bug fix release 0.7.4→ <<cset 907a286121e6>>
-
reporter Bringing ref
#430fix into develop→ <<cset 46dc3102e6c3>>
-
reporter Rearranged the protocol sheet data gathering from that proposed by @LuukO. Reference to https://docs.djangoproject.com/en/1.10/ref/models/querysets/#all added by @LuukO would indicate fix for ref
#430is not suitable. Possibly need to check for .first() or stick with get() and check for MultipleObjectsReturned. Refs#442.→ <<cset 344fa4a34f43>>
-
reporter - changed status to open
Fix wasn't appropriate.
-
reporter Removed concept of pre-testing to check for tables for DX csv export. Dropped use of export_safe too, using replace_comma instead. Might not have been a good idea as now have unicode errors so currently broken. Refs
#430,#442→ <<cset c76adeb4cf22>>
-
reporter All unicode errors resolved. DX export to csv should now work more robustly! Need to see what is happening with filters on the skeletal survey example I have. Refs
#430,#442→ <<cset e3789d896d69>>
-
reporter Commit fe5ba59 referenced this issue
-
reporter Removed use of replace_comma on each field in dx csv in favour of checking for commas just before writing the line. Refs
#430,#442→ <<cset 3380dda7226e>>
-
reporter Adding test files for ref
#430→ <<cset 250b87a235df>>
-
reporter Adding start of test for ref
#430→ <<cset a96802db64fa>>
-
reporter Added test of thickness values for ref
#430→ <<cset f0ec0528d808>>
-
reporter - changed status to resolved
Factored out filter and filter_thickness value population to replace the three occurancers with one. Refs
#454for xlsx, already fixed for csv. Also fixes#430for all three occurances in csv and xlsx. Test needs updating.→ <<cset 28c86f8aad16>>
-
reporter Changed name of get_filters to _get_xray_filterinfo to match rf version. Refs
#430,#442→ <<cset 6697aba758ef>>
-
reporter Added else to material list in case there is something odd recorded or there is a filter sequence but no material element. Refs
#430→ <<cset ba4730356a16>>
-
reporter Adding ref
#430and#442to changes→ <<cset acb2069515ae>>
-
reporter Adding ref
#430and#442to changes→ <<cset ef9f77515678>>
-
reporter Throwing in a placeholder comment to release notes refs
#430,#442→ <<cset f65d7c2b52be>>
-
reporter - Log in to comment
Hopefully this fixes ref
#430. Currently testing.→ <<cset 9f1093d39811>>