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 commit3c0ac4cb9a) Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com> (cherry picked from commit72e78f72a3) Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
This commit is contained in:
parent
3f1fa1918e
commit
cc83d8e19a
|
|
@ -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