[Koha-bugs] [Bug 26145] Add the ability to attach a cover image at item level

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

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

--- Comment #78 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
(In reply to Katrin Fischer from comment #61)
> 1) Fixed some missing filters - posting follow-up.

Thanks!

> 2) Hm, the 2 separate updatedatabase statements are a bit confusing. Please
> merge them.

Ok, I will merge them before pushing them.
They are 2 different things, so I wanted to make it clear in the output.

> 3) New column is not added to the column configuration for the items table
> in the OPAC. I've tried to fix it, but just adding it to the .yml didn't
> have the intended effect. Can you please have a look?

Done.

> Note: I think it would have been nice to have this split into several bugs,
> starting with the rewrite of C4::Images into the Koha namespace for the
> existing functionality.

I took advantage of being in this part of the code to move legacy C4 code to
the Koha namespace. I did not plan to do that before writing the first patches
(hence it is not done at the beginning of the patch stack).
The only thing I could have separated is the "allow multiple covers", but I
thought it was good to have it from scratch (as it is the current behaviour for
biblio records).

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