Attachment 31902 Details for Bug 5342 – Bug 5342: (qa-followup) FIX some issues

[patch] Bug 5342: (qa-followup) FIX some issues

Bug-5342-qa-followup-FIX-some-issues.patch (text/plain), 2.37 KB, created by Jonathan Druart on 2014-09-26 15:15:24 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2014-09-26 15:15:24 UTC

Size: 2.37 KB

patch

obsolete

>From 0c656fb7da049abb276f7368ec6177879b6dbbdb Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@biblibre.com>
>Date: Fri, 26 Sep 2014 17:14:00 +0200
>Subject: [PATCH] Bug 5342: (qa-followup) FIX some issues
>
>this patch:
>- reintroduces the ISSN column
>- fix a wording (already there before the main patch)
>- fix the tests if a serial was already in late
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/serials/claims.tt | 4 +++-
> t/db_dependent/Serials/Claims.t                           | 2 ++
> 2 files changed, 5 insertions(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/claims.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/claims.tt
>index 49b8448..433f51a 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/claims.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/claims.tt
>@@ -64,7 +64,7 @@
> 	// Checks if the form can be sent (at least one checkbox must be checked)
> 	function checkForm() {
> 	    if ($("input:checked").length == 0) {
>-		alert(_("Please select at least one item."));
>+		alert(_("Please select at least one issue."));
> 		return false;
> 	    }
> 	}
>@@ -276,6 +276,7 @@
>                         <th>Library</th>
>                         <th class="anti-the">Title</th>
>                         <th>Issue number</th>
>+                        <th>ISSN</th>
>                         <th>Status</th>
>                         <th class="title-string">Since</th>
>                         <th>Claims count</th>
>@@ -294,6 +295,7 @@
>                         <td>
>                         <a href="/cgi-bin/koha/serials/subscription-detail.pl?subscriptionid=[% missingissue.subscriptionid %]">[% missingissue.title |html %]</a>
>                         </td>
>+                        <td>[% missingissue.issn %]</td>
>                         <td>[% missingissue.serialseq %]</td>
>                         <td>
>                             [% IF ( missingissue.status1 ) %]<span class="status-expected">Expected</span>[% END %]
>diff --git a/t/db_dependent/Serials/Claims.t b/t/db_dependent/Serials/Claims.t
>index a90563d..60f851a 100644
>--- a/t/db_dependent/Serials/Claims.t
>+++ b/t/db_dependent/Serials/Claims.t
>@@ -11,6 +11,8 @@ my $dbh = C4::Context->dbh;
> $dbh->{AutoCommit} = 0;
> $dbh->{RaiseError} = 1;
> 
>+$dbh->do(q|DELETE FROM issues|);
>+
> my $branchcode = 'CPL';
> my $bpid = AddBudgetPeriod({
>     budget_period_startdate   => '2015-01-01',
>-- 
>2.1.0

Actions: View | Diff | Splinter Review

Attachments on bug 5342: 8115 | 8116 | 8309 | 8334 | 8335 | 8611 | 8682 | 8683 | 8900 | 8910 | 9168 | 9169 | 9976 | 9977 | 10211 | 10212 | 10245 | 10258 | 14444 | 14445 | 14446 | 16217 | 16218 | 16219 | 19512 | 31277 | 31375 | 31454 | 31694 | 31902 | 32171 | 32172