Bugzilla – Attachment 4914 Details for
Bug 6690
If multiple issues are received in serials-edit.pl, changing the status of any issue changes the date of the first issue.
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Corrects a problem with the wrong date being updated in serials-edit.pl
0001-Corrects-a-problem-with-the-wrong-date-being-updated.patch (text/plain), 3.56 KB, created by
Frédérick Capovilla
on 2011-08-09 20:42:56 UTC
(
hide
)
Description:
Corrects a problem with the wrong date being updated in serials-edit.pl
Filename:
MIME Type:
Creator:
Frédérick Capovilla
Created:
2011-08-09 20:42:56 UTC
Size:
3.56 KB
patch
obsolete
>From 1bd23cd4a99aebec0b61350f1b7dba55adfda006 Mon Sep 17 00:00:00 2001 >From: =?utf-8?q?Fr=C3=A9d=C3=A9rick=20Capovilla?= <frederick.capovilla@sys-tech.net> >Date: Tue, 9 Aug 2011 16:22:24 -0400 >Subject: [PATCH] Corrects a problem with the wrong date being updated in serials-edit.pl > >If multiple issues are in the list, changing the status of any issue in >the table would always change the "Expected on" date of the first issue. >Modified the javascript so the date is modified on the correct row. >--- > .../prog/en/modules/serials/serials-edit.tmpl | 9 ++++----- > 1 files changed, 4 insertions(+), 5 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-edit.tmpl b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-edit.tmpl >index c2b724b..f8a996f 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-edit.tmpl >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-edit.tmpl >@@ -84,9 +84,8 @@ function HideItems(index,labelindex) { > function setStatus(serialid){ > $("#status"+serialid).val("2").attr("selected","selected"); > } >-function changeDate(adate) { >- var elem = document.getElementById("expecteddate"); >- elem.value = adate; >+function changeDate(elem,adate) { >+ $(elem).closest('tr').find('#expecteddate').val(adate); > } > function changeDate2(adate) { > var elem = document.getElementById("supexpecteddate"); >@@ -231,9 +230,9 @@ function CloneSubfield(index){ > <select name="status" size="1" disabled="disabled"> > <!--TMPL_ELSE--> > <!--TMPL_IF Name="serialsadditems"--> >- <select name="status" size="1" id="status<!-- TMPL_VAR NAME="serialid" -->" onchange="if (this.value==2){unHideItems('items'+<!-- TMPL_VAR NAME="subscriptionid" --><!-- TMPL_VAR NAME="serialid" -->,'label<!-- TMPL_VAR NAME="subscriptionid" --><!--TMPL_VAR Name="serialid"-->','<!-- TMPL_VAR NAME="serialid" -->'); changeDate('<!-- TMPL_VAR NAME="arriveddate" -->')} else if (this.value==7){changeDate('<!-- TMPL_VAR NAME="arriveddate" -->')} else { HideItems('items'+<!-- TMPL_VAR NAME="subscriptionid" -->+<!-- TMPL_VAR NAME="serialid" -->,'label<!-- TMPL_VAR NAME="subscriptionid" --><!--TMPL_VAR Name="serialid"-->'); changeDate('<!-- TMPL_VAR NAME="planneddate" -->')}" > >+ <select name="status" size="1" id="status<!-- TMPL_VAR NAME="serialid" -->" onchange="if (this.value==2){unHideItems('items'+<!-- TMPL_VAR NAME="subscriptionid" --><!-- TMPL_VAR NAME="serialid" -->,'label<!-- TMPL_VAR NAME="subscriptionid" --><!--TMPL_VAR Name="serialid"-->','<!-- TMPL_VAR NAME="serialid" -->'); changeDate(this,'<!-- TMPL_VAR NAME="arriveddate" -->')} else if (this.value==7){changeDate(this,'<!-- TMPL_VAR NAME="arriveddate" -->')} else { HideItems('items'+<!-- TMPL_VAR NAME="subscriptionid" -->+<!-- TMPL_VAR NAME="serialid" -->,'label<!-- TMPL_VAR NAME="subscriptionid" --><!--TMPL_VAR Name="serialid"-->'); changeDate(this,'<!-- TMPL_VAR NAME="planneddate" -->')}" > > <!--TMPL_ELSE --> >- <select name="status" size="1" id="status<!-- TMPL_VAR NAME="serialid" -->" onchange="if (this.value==2 || this.value==7){changeDate('<!-- TMPL_VAR NAME="arriveddate" -->')} else {changeDate('<!-- TMPL_VAR NAME="planneddate" -->')}" > >+ <select name="status" size="1" id="status<!-- TMPL_VAR NAME="serialid" -->" onchange="if (this.value==2 || this.value==7){changeDate(this,'<!-- TMPL_VAR NAME="arriveddate" -->')} else {changeDate(this,'<!-- TMPL_VAR NAME="planneddate" -->')}" > > <!--/TMPL_IF--> > <!--/TMPL_IF--> > <!--TMPL_IF name="status1" --> >-- >1.5.6.5 >
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 6690
: 4914 |
5002
|
5010