@@ -, +, @@ patron's data from borrower_data input --- .../prog/en/modules/members/memberentrygen.tt | 6 ++--- koha-tmpl/intranet-tmpl/prog/js/members.js | 26 +++++-------------- members/memberentry.pl | 2 +- 3 files changed, 11 insertions(+), 23 deletions(-) --- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt @@ -1968,10 +1968,10 @@ legend:hover { }); [% END %] $("#guarantor_template").hide(); - var guarantor_attributes; + var prefill_fields; var to_api_mapping; - [% IF guarantor_attributes %] - guarantor_attributes = [% To.json(guarantor_attributes) %]; + [% IF prefill_fields %] + prefill_fields = [% To.json(prefill_fields)%]; [% END %] [% IF to_api_mapping %] to_api_mapping = [% To.json(to_api_mapping) %]; --- a/koha-tmpl/intranet-tmpl/prog/js/members.js +++ a/koha-tmpl/intranet-tmpl/prog/js/members.js @@ -122,26 +122,14 @@ function select_user(borrowernumber, borrower, relationship) { fieldset.find('.new_guarantor_relationship').val(relationship); } - if(guarantor_attributes && to_api_mapping) { - $('#saverecord').prop('disabled', true); - $.ajax({ - url: '/api/v1/patrons/'+borrowernumber, - method: "GET", - dataType: 'json', - success: function(data){ - for (let i = 0; i < parseInt(guarantor_attributes.length, 10); i++) { - let attribute = guarantor_attributes[i]; - let key_data = to_api_mapping[attribute] || attribute; - if(data[key_data] != null && attribute in document.forms.entryform && document.forms.entryform[attribute].value == ""){ - document.forms.entryform[attribute].value = data[key_data]; - } - } - $('#saverecord').prop('disabled', false); - }, - error:function(){ - $('#saverecord').prop('disabled', false); + if ( prefill_fields && to_api_mapping) { + for (let i = 0; i < parseInt(prefill_fields.length, 10); i++) { + let field_name = prefill_fields[i]; + let attribute = to_api_mapping[field_name] || field_name; + if ( borrower[attribute] != null && document.forms.entryform[field_name] && document.forms.entryform[field_name].value == "" ) { + document.forms.entryform[field_name].value = borrower[attribute]; } - }); + } } } --- a/members/memberentry.pl +++ a/members/memberentry.pl @@ -626,7 +626,7 @@ if (C4::Context->preference("IndependentBranches")) { my $prefillguarantorfields=C4::Context->preference("PrefillGuaranteeField"); my @prefill_fields=split(/\,/,$prefillguarantorfields); $template->param( - guarantor_attributes => \@prefill_fields, + prefill_fields => \@prefill_fields, to_api_mapping => Koha::Patron->new->to_api_mapping ); --