search

tsgrp / HPI

OpenContent Management Suite (OCMS)
http://www.tsgrp.com/products
17 stars 5 forks source link

Export to Excel does not work in View All Documents #1605

Closed gsteimer closed 6 years ago

gsteimer commented 6 years ago

The export to excel action no longer works properly in View All Documents:

image

It just spins here and no longer returns

image

I'm not seeing any errors in the browser console or in OC logs.

joehof commented 6 years ago

So this turned into a way bigger thing than expected due to a couple of things. Original error was due to the fact we called getSearchConfig - which did not return anything if we were not on the search page, so this was an error missed from the blum merge for VAD. Then, the call for currentType is super scary and error prone in Stage, since multiple things overwrite the global currentTypeConfig() in Stage, which led to race conditions. So we need to pass in the types that we searched on for VAD - which now can be more than one type. Also added in some more graceful fail states in case stuff is not configured correctly. Another sidenote is excelStageUrl also needed a slight change since currentType is fetched differently.

Fixed with HPI commit 19881