bug 2265: fix problem when cloning MARC field
Correctly set indicators when cloning a field in the MARC editor. Prior this fix, a cloned field would not be saved. Bug introduced in patch for bug 2207. Signed-off-by: Joshua Ferraro <jmf@liblime.com>
This commit is contained in:
parent
71c7afd90d
commit
7ac682c859
2 changed files with 10 additions and 4 deletions
|
@ -100,8 +100,10 @@ function CloneField(index) {
|
|||
|
||||
<!-- TMPL_UNLESS NAME='hide_marc'--> // No indicator if hide_marc
|
||||
// setting a new name for the new indicator
|
||||
var indicator = clone.getElementsByTagName('input')[0];
|
||||
for(var i=0; i < 2; i++) {
|
||||
var indicator = clone.getElementsByTagName('input')[i];
|
||||
indicator.setAttribute('name',indicator.getAttribute('name')+new_key);
|
||||
}
|
||||
<!-- /TMPL_UNLESS -->
|
||||
|
||||
// settings all subfields
|
||||
|
@ -216,6 +218,7 @@ function CloneField(index) {
|
|||
if(divs[i].getAttribute('name') == 'div_indicator'){
|
||||
var inputs = divs[i].getElementsByTagName('input');
|
||||
inputs[0].setAttribute('id',inputs[0].getAttribute('id')+new_key);
|
||||
inputs[1].setAttribute('id',inputs[1].getAttribute('id')+new_key);
|
||||
|
||||
var CloneButtonPlus;
|
||||
try {
|
||||
|
|
|
@ -159,8 +159,10 @@ function CloneField(index) {
|
|||
|
||||
<!-- TMPL_UNLESS NAME='hide_marc'--> // No indicator if hide_marc
|
||||
// setting a new name for the new indicator
|
||||
var indicator = clone.getElementsByTagName('input')[0];
|
||||
for(var i=0; i < 2; i++) {
|
||||
var indicator = clone.getElementsByTagName('input')[i];
|
||||
indicator.setAttribute('name',indicator.getAttribute('name')+new_key);
|
||||
}
|
||||
<!-- /TMPL_UNLESS -->
|
||||
|
||||
// settings all subfields
|
||||
|
@ -274,6 +276,7 @@ function CloneField(index) {
|
|||
if(divs[i].getAttribute('id').match(/^div_indicator/)){
|
||||
var inputs = divs[i].getElementsByTagName('input');
|
||||
inputs[0].setAttribute('id',inputs[0].getAttribute('id')+new_key);
|
||||
inputs[1].setAttribute('id',inputs[1].getAttribute('id')+new_key);
|
||||
|
||||
var CloneButtonPlus;
|
||||
try {
|
||||
|
|
Loading…
Reference in a new issue