Bug 19974: (QA follow-up) Move lost logic to else branch
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Fri, 23 Mar 2018 08:34:46 +0000 (09:34 +0100)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Fri, 20 Apr 2018 14:57:31 +0000 (11:57 -0300)
commit8d8dc552418dc0a95c8baadcfc524382e76d971c
tree70753ef6b52966df85298960f757912b7c0dfb15
parent6fc6715350084bbba2bef92b52aabf58afd70a9f
Bug 19974: (QA follow-up) Move lost logic to else branch

If an error is raised for the barcode, we should not try to perform the
lost logic succeeding it.
Futhermore there is no need to go to GetMarcFromKohaField etc. if we just
use the output of ModItemFromMarc.

Note: It seems unnecessary to clear $itemnumber, but I can understand
the anxiety about passing it to the template with op=additem. So just
leaving it here.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
cataloguing/additem.pl