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

Marcel de Rooy <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]
             Status|Signed Off                  |Failed QA

--- Comment #18 from Marcel de Rooy <[email protected]> ---
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.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to