Bug 21347

Summary: bad code for input field in item information tab of addorderiso2709 page
Product: Koha Reporter: Koha Team University Lyon 3 <koha>
Component: AcquisitionsAssignee: Chris Cormack <chris>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: major    
Priority: P5 - low CC: jonathan.druart, katrin.fischer, koha, nick, sandboxes
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 13618    
Bug Blocks:    
Attachments: Bug 21347 : Escaped fields that should be in addorderiso2709
Bug 21347 : Escaped fields that should be in addorderiso2709
Bug 21347: Escaped fields that should be in addorderiso2709

Description Koha Team University Lyon 3 2018-09-14 21:09:13 UTC
In cgi-bin/koha/acqui/addorderiso2709.pl, there is a problem with some fields in item information problem. Instead to have a field, you have the code :
 Cote <input type="text" name="field_value" value="" size="50" maxlength="255" /> 
Note <input type="text" name="field_value" value="" size="50" maxlength="255" />
Comment 1 Chris Cormack 2018-09-14 22:50:41 UTC
Created attachment 78824 [details] [review]
Bug 21347 : Escaped fields that should be in addorderiso2709

Test plan
1/ Stage a MARC file (if you have none staged)
2/ Create a basket in aquisitions
3/ Add an order via staged file
4/ Look at the item information tab, notice there is escaped html
5/ Apply patch
6/ Refresh page, notice it is fixed
Comment 2 Koha Team University Lyon 3 2018-09-14 23:24:12 UTC
I have followed the plan test and now the fields in item information page are correcty displayed.

Thank you Chris!
Comment 3 Biblibre Sandboxes 2018-09-15 16:48:13 UTC
Patch tested with a sandbox, by Sonia <sonia.bouis@univ-lyon3.fr>
Comment 4 Biblibre Sandboxes 2018-09-15 16:48:38 UTC
Created attachment 78858 [details] [review]
Bug 21347 : Escaped fields that should be in addorderiso2709

Test plan
1/ Stage a MARC file (if you have none staged)
2/ Create a basket in aquisitions
3/ Add an order via staged file
4/ Look at the item information tab, notice there is escaped html
5/ Apply patch
6/ Refresh page, notice it is fixed

Signed-off-by: Sonia <sonia.bouis@univ-lyon3.fr>
Comment 5 Jonathan Druart 2018-09-15 20:52:36 UTC
Created attachment 78892 [details] [review]
Bug 21347: Escaped fields that should be in addorderiso2709

Test plan
1/ Stage a MARC file (if you have none staged)
2/ Create a basket in aquisitions
3/ Add an order via staged file
4/ Look at the item information tab, notice there is escaped html
5/ Apply patch
6/ Refresh page, notice it is fixed

Signed-off-by: Sonia <sonia.bouis@univ-lyon3.fr>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 6 Nick Clemens (kidclamp) 2018-09-16 18:26:40 UTC
Awesome work all!

Pushed to master for 18.11