Bug 36298: In patrons search fix road type with authorized value
In new patrons search via API, the road type code is displayed in patron address instead of authorized value description.
Also road type is missing a space separator in DE and FR format.
Test plan :
1) Create two entries in authorized value category "ROADTYPE" :
R1 "Road"
R2 "Street"
2) Edit a patron (ie Delgado, Floyd) to set road type to "Road"
3) Perform a search on patrons (click on letter D)
4) Check you see in address "Road" and not "R1"
5) Edit authorized value category "ROADTYPE" to delete entry "R1"
6) Perform a search on patrons (click on letter D)
7) Check you see in address "R1"
Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
(cherry picked from commit 3c0ac4cb9a)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
This commit is contained in:
parent
6d1125fa8f
commit
72e78f72a3
|
|
@ -7,7 +7,7 @@
|
|||
|
||||
const roadtypes = [% To.json(AuthorisedValues.Get('ROADTYPE')) | $raw %];
|
||||
let roadtypes_map = roadtypes.reduce((map, r) => {
|
||||
map[r.street_type] = r;
|
||||
map[r.authorised_value] = r.lib;
|
||||
return map;
|
||||
}, {});
|
||||
|
||||
|
|
@ -17,7 +17,7 @@
|
|||
if ( data.address ) {
|
||||
let roadtype_desc = '';
|
||||
if ( data.street_type ) {
|
||||
roadtype_desc = roadtypes_map[data.street_type] ? roadtypes_map[data.street_type].street_type : data.street_type;
|
||||
roadtype_desc = roadtypes_map[data.street_type] ? roadtypes_map[data.street_type] : data.street_type;
|
||||
}
|
||||
address += include_li ? '<li class="patronaddress1">' : '';
|
||||
address += escape_str(data.street_number) + ' ' + escape_str(data.address) + ' ' + escape_str(roadtype_desc);
|
||||
|
|
@ -59,10 +59,10 @@
|
|||
if ( data.address ) {
|
||||
let roadtype_desc = '';
|
||||
if ( data.street_type ) {
|
||||
roadtype_desc = roadtypes_map[data.street_type] ? roadtypes_map[data.street_type].street_type : data.street_type;
|
||||
roadtype_desc = roadtypes_map[data.street_type] ? roadtypes_map[data.street_type] : data.street_type;
|
||||
}
|
||||
address += include_li ? '<li class="patronaddress1">' : ' ';
|
||||
address += escape_str(data.address) + ' ' + escape_str(roadtype_desc) + escape_str(data.street_number);
|
||||
address += escape_str(data.address) + ' ' + escape_str(roadtype_desc) + ' ' + escape_str(data.street_number);
|
||||
address += include_li ? '</li>' : '';
|
||||
}
|
||||
}
|
||||
|
|
@ -88,10 +88,10 @@
|
|||
if ( data.address ) {
|
||||
let roadtype_desc = '';
|
||||
if ( data.street_type ) {
|
||||
roadtype_desc = roadtypes_map[data.street_type] ? roadtypes_map[data.street_type].street_type : data.street_type;
|
||||
roadtype_desc = roadtypes_map[data.street_type] ? roadtypes_map[data.street_type] : data.street_type;
|
||||
}
|
||||
address += include_li ? '<li class="patronaddress1">' : '';
|
||||
address += escape_str(data.street_number) + ' ' + escape_str(roadtype_desc) + escape_str(data.address);
|
||||
address += escape_str(data.street_number) + ' ' + escape_str(roadtype_desc) + ' ' + escape_str(data.address);
|
||||
address += include_li ? '</li>' : '';
|
||||
}
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in New Issue