[Koha-bugs] [Bug 27272] Move C4::Items::GetItemsInfo to Koha namespace

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Fri Aug 5 03:58:10 CEST 2022


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

Victor Grousset/tuxayo <victor at tuxayo.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |Signed Off

--- Comment #113 from Victor Grousset/tuxayo <victor at tuxayo.net> ---
The test suite passes.

---

Here is the only complain from the QA script:
 FAIL   C4/Items.pm
   FAIL   pod coverage
                POD coverage was greater before, try perl
-MPod::Coverage=PackageName -e666
Is it legit?

---

Tell me (Jonathan or a QAer that looks into this) if there is particular stuff
to test manually.

For lack of clues I just reread all the patches in case there where something
obviously suspicious.
I haven't done development with those functions and in all the places they are
used so I can't tell if the replacement of GetItemsInfo & co is correct.

Let's move on and see what QA has to say. These kind of patches with nothing
clear to test can easily get stuck forever waiting for SO because testers don't
know what to do.
I would happily come back if there is precise stuff to test manually.

Thanks Jonathan for the big refactoring!

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


More information about the Koha-bugs mailing list