Bugzilla – Attachment 11490 Details for
Bug 7688
Change subscriptions numbering pattern and frequencies
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[SIGNED-OFF] Bug 7688 follow-up: Small fixes for QA
Bug-7688-follow-up-Small-fixes-for-QA.patch (text/plain), 4.52 KB, created by
Kyle M Hall (khall)
on 2012-08-09 12:36:23 UTC
(
hide
)
Description:
[SIGNED-OFF] Bug 7688 follow-up: Small fixes for QA
Filename:
MIME Type:
Creator:
Kyle M Hall (khall)
Created:
2012-08-09 12:36:23 UTC
Size:
4.52 KB
patch
obsolete
>From d4d84655c6066771eccead5b842b6202dcff1fe0 Mon Sep 17 00:00:00 2001 >From: Julian Maurice <julian.maurice@biblibre.com> >Date: Thu, 2 Aug 2012 16:55:24 +0200 >Subject: [PATCH] Bug 7688 follow-up: Small fixes for QA > >- # Subroutines::ProhibitExplicitReturnUndef: Got 1 violation(s) in > C4::Serials::GetSubscriptionIrregularities >- Bad template constructions fixed in serials/subscription-add.tt > >Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> >--- > C4/Serials.pm | 2 +- > .../prog/en/modules/serials/subscription-add.tt | 20 +++++++++++++++++--- > 2 files changed, 18 insertions(+), 4 deletions(-) > >diff --git a/C4/Serials.pm b/C4/Serials.pm >index 7b7de86..95af5f9 100644 >--- a/C4/Serials.pm >+++ b/C4/Serials.pm >@@ -1247,7 +1247,7 @@ get the list of irregularities for a subscription > sub GetSubscriptionIrregularities { > my $subscriptionid = shift; > >- return undef unless $subscriptionid; >+ return unless $subscriptionid; > > my $dbh = C4::Context->dbh; > my $query = qq{ >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt >index bfebddd..7323afb 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt >@@ -653,7 +653,11 @@ $(document).ready(function() { > <select name="frequency" size="1" id="frequency"> > <option value="">-- please choose --</option> > [% FOREACH frequency IN frequencies %] >- <option value="[% frequency.id %]" [% IF (frequency.selected) %] selected="selected" [% END %]> >+ [% IF (frequency.selected) %] >+ <option value="[% frequency.id %]" selected="selected"> >+ [% ELSE %] >+ <option value="[% frequency.id %]"> >+ [% END %] > [% frequency.label %] > </option> > [% END %] >@@ -663,7 +667,11 @@ $(document).ready(function() { > <label for="subtype">Subscription length:</label> > <select name="subtype" id="subtype"> > [% FOREACH subt IN subtype %] >- <option value="[% subt.name %]" [% IF (subt.selected) %] selected="selected" [% END %] > >+ [% IF (subt.selected) %] >+ <option value="[% subt.name %]" selected="selected"> >+ [% ELSE %] >+ <option value="[% subt.name %]"> >+ [% END %] > [% subt.name %] > </option> > [% END %] >@@ -684,7 +692,13 @@ $(document).ready(function() { > <select name="numbering_pattern" size="1" id="numberpattern"> > <option value="">-- please choose --</option> > [% FOREACH numberpattern IN numberpatterns %] >- <option value="[% numberpattern.id %]" [% IF (numberpattern.selected) %] selected="selected" [% END %]>[% numberpattern.label %]</option> >+ [% IF (numberpattern.selected) %] >+ <option value="[% numberpattern.id %]"> >+ [% ELSE %] >+ <option value="[% numberpattern.id %]"> >+ [% END %] >+ [% numberpattern.label %] >+ </option> > [% END %] > </select> > </li> >-- >1.7.2.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 7688
:
8110
|
8941
|
8942
|
8943
|
10145
|
10146
|
10468
|
10469
|
10470
|
10739
|
10740
|
10742
|
10743
|
10903
|
10920
|
10985
|
11299
|
11300
|
11489
|
11490
|
12268
|
12269
|
13193
|
13194
|
13195
|
13196
|
14122
|
14123
|
14124
|
14125
|
14996
|
15492
|
15493
|
15494
|
15495
|
15496
|
15534
|
15536
|
15537
|
15538
|
15539
|
15540
|
15541
|
16679
|
16680
|
16681
|
16682
|
16683
|
16684
|
16744
|
17283
|
17284
|
17285
|
17855
|
17856
|
17857
|
18298
|
18299
|
18300
|
18793
|
18794
|
18795
|
19368
|
19369
|
19370
|
19393
|
19395