A customer found a problem with statistical reports. They found the circulation stats for returns grouped all returns on the NULL itemtype. What? Well, yes.
My first idea was to fill a bug for that [1]. But looking for the bug on the reports code, I found that the problem was in C4::Circulation. So filled a bug for it [2] and provided regression tests and the fix. Cool.
But what about the wrong data on the statistics table? Basically, all returns have the wrong itemtype (NULL). Should we fix that table's data? Should we make the report do the proper JOIN operation on the items table? I'd go with the first option, but probably worth discussing, because there are other bugs related to how the 'statistics' table have been filled. [3]
Regards
--
Tomás Cohen Arazi
Prosecretaría de Informática
Universidad Nacional de Córdoba
✆ +54 351 5353750 ext 13168
GPG: B76C 6E7C 2D80 551A C765 E225 0A27 2EA1 B2F3 C15F