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

--- Comment #28 from Martin Renvoize <[email protected]> ---
I've now uploaded a correctly rebased patchset so the tests should all now be
passing.

Having discussed this with Josef in IRC we are both in agreement that
statistucs.proccode should be removed as is. I've looked through the SQL
Reports library on the wiki and can't find a single mention of it's use and
neither of us could work out a consistent and correct way to report against it
which remained accurate.  We also couldn't work out any form of migration path
from accountno to accountline_id.

So.. I believe this patchset is now sound, although I would like one minor
followup which actually removed the field from the statistics table too.

I believe if we do decide we want a link to accountlines in this table we
should treat that as a separate issue.

I'll add the followup to remove the field from the database.

-- 
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