[Koha-bugs] [Bug 15498] Replace ExportWithCsvProfile with ExportCircHistory

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15498

Marcel de Rooy <m.de.rooy at rijksmuseum.nl> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |m.de.rooy at rijksmuseum.nl
             Status|Signed Off                  |Failed QA

--- Comment #18 from Marcel de Rooy <m.de.rooy at rijksmuseum.nl> ---
QA Comment:
It would have been helpful if I had a more clear description of what this patch
set wants to accomplish.

[1] Going through the code changes, you remove ExportWithCsvProfile and you add
ExportCircHistory. You also touch ExportRemoveFields. ExportRemoveFields is
still found in
koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref. But
it is not in sysprefs.sql. Still in use in tools/export.pl. But I would also
expect these fields to pop up at the circulation screen in the Dont export
fields box.

[2] I do not see the combo (select name="csv_profile_id"). I guess it is hidden
by js in checkouts.js. But also if I select csv, it does not show up.
Refreshed, restarted, flushed, etc. No results. (I expected a combo with two
csv profiles.)

Failed QA

-- 
You are receiving this mail because:
You are watching all bug changes.