22b4106398
The ind_tag of TransformHtmlToXml is unused. Some calls to this function incorrectly revert indicator and ind_tag (which is not a problem when both are empty..) Patch of Srdjan Jankovic, amended and signed off by Marcel de Rooy. The following calls are fixed: call in acqui/addorder.pl: switched indicator with ind_tag call in acqui/addorderiso2709.pl replaced too acqui/finishreceive.pl replaced too These calls are fine: two calls in cataloguing/additem.pl are fine call in serials/serials-edit.pl is fine call in tools/batchMod.pl is fine The folllow-up patch adds a simple unit test. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> With AcqCreateItem=='placing an order', tested if adding an order still worked (covered both addorder.pl and addorderiso2709.pl). Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> |
||
|---|---|---|
| .. | ||
| pdfformat | ||
| acqui-home.pl | ||
| add_user_search.pl | ||
| addorder.pl | ||
| addorderiso2709.pl | ||
| ajax-getauthvaluedropbox.pl | ||
| basket.pl | ||
| basketgroup.pl | ||
| basketheader.pl | ||
| booksellers.pl | ||
| cancelorder.pl | ||
| check_budget_total.pl | ||
| check_duplicate_barcode_ajax.pl | ||
| check_uniqueness.pl | ||
| currency.pl | ||
| finishreceive.pl | ||
| histsearch.pl | ||
| invoice-files.pl | ||
| invoice.pl | ||
| invoices.pl | ||
| lateorders-export.pl | ||
| lateorders.pl | ||
| modordernotes.pl | ||
| neworderbiblio.pl | ||
| neworderempty.pl | ||
| newordersubscription.pl | ||
| newordersuggestion.pl | ||
| ordered.pl | ||
| orderreceive.pl | ||
| parcel.pl | ||
| parcels.pl | ||
| spent.pl | ||
| supplier.pl | ||
| transferorder.pl | ||
| uncertainprice.pl | ||
| updatesupplier.pl | ||
| z3950_search.pl | ||