---
serials/reorder_members.pl |5 +
1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/serials/reorder_members.pl b/serials/reorder_members.pl
index 66d4ab2..28175fb 100755
--- a/serials/reorder_members.pl
+++ b/serials/reorder_members.pl
@@ -21,12 +21,17 @@
use strict;
use
We've checked the return for definedness and
assigned an empty string to it ao we don't want
to make it undef again
This is a good argument that cascading if-elses expecially with
long unstructured elses are a bad idea
---
acqui/basket.pl |2 +-
1 files changed, 1 insertions(+), 1
Script was written as though it had an array rather than
a single supplier. Replaced the excess punctuation with a single
supplier variable
replaced the C-style currency loop with a simpler perl-style one
we only need 1 loop not multiples
Changed variable name $GST to $tax_rate it's not magical
Script was written as though it had an array rather than
a single supplier. Replaced the excess punctuation with a single
supplier variable
replaced the C-style currency loop with a simpler perl-style one
Changed variable name $GST to $tax it's not magical and not a constant
and we all know what
basket.pl calculates variables i, odd and count which are
no longer used in the template
removing them allows some simplification of the main loop
and cuts out on an extra db call
Loop through basketgroups also made more readable
Try to dimish the number of undef warnings filling the logs
---
The only serials routine without warnings enabled was a
script with the least code to generate them
removed the numerous Module use declarations that were not
required
aligned the parameters for a more aesthetically pleasing effect
---
serials/reorder_members.pl | 20 +++-
1
Script was written as though it had an array rather than
a single supplier. Replaced the excess punctuation with a single
supplier variable
replaced the C-style currency loop with a simpler perl-style one
Changed variable name $GST to $tax it's not magical and not a constant
and we all know what
The empty vendor created by the select if any subscriptions
have no associated vendor generates a 'wrong-looking' display
and selecting it gives no options
---
C4/Serials.pm |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/C4/Serials.pm b/C4/Serials.pm
index
- is it not safer to use NULL rather than 0 as the
behaviour of auto_increment passed 0 is dependent on the SQL mode that
MySQL is running under? (sorry scarred by an early experience of
restoring a db to find all our auto_incremented row ids had changed!!)
Colin
--
Colin Campbell
Chief
if $index is unnecessary as we have made this true 5 lines above
variables should not be declared in conditionals if used outside of them
set $struct_attr to a sensible default to avoid generating warnings
in this assigment and elsewhere
---
C4/Search.pm |6 +++---
1 files changed, 3
Changed arriveddate format to follow syspref
Fixed a whitespace error
---
serials/serials-edit.pl |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/serials/serials-edit.pl b/serials/serials-edit.pl
index bd7d420..b407ca9 100755
--- a/serials/serials-edit.pl
+++
Script was incorrectly asking user to define an active currency
Now test for active currency in the standard way
Use the currency routines in Budgets where applicable
Move the code paths in the if else to subroutines to clarify flow
Enable warnings
---
admin/currency.pl | 257
Script was incorrectly asking user to define an active currency
Now test for active currency in the standard way
Use the currency routines in Budgets where applicable
Move the code paths in the if else to subroutines to clarify flow
Enable warnings
---
admin/currency.pl | 258
Script was incorrectly asking user to define an active currency
Now test for active currency in the standard way
Use the currency routines in Budgets where applicable
Move the code paths in the if else to subroutines to clarify flow
Enable warnings
---
admin/currency.pl | 258
labeled_marc ioption was omitted (spelling police?)
options available now match those in code
---
.../en/modules/admin/preferences/cataloguing.pref |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git
a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/cataloguing.pref
on current 3.2 HEAD.
These are not major new functionality, but a series of fixes to
eradicate some randomness caused by undefined values. Includes a commit
to fix Licenses in the serials directory.
Credit due to the staff at fao for their testing, feedback and perseverance.
Colin
--
Colin
Identify Patron Expiry so that it is returned in Patron Info Response
---
C4/SIP/ILS/Patron.pm | 25 -
1 files changed, 20 insertions(+), 5 deletions(-)
diff --git a/C4/SIP/ILS/Patron.pm b/C4/SIP/ILS/Patron.pm
index 386287b..2a73307 100644
--- a/C4/SIP/ILS/Patron.pm
+++
Change timestamp routine to do the right thing when passed an iso date
Otherwise was generating an epoch date of 0
---
C4/SIP/Sip.pm |8
1 files changed, 8 insertions(+), 0 deletions(-)
diff --git a/C4/SIP/Sip.pm b/C4/SIP/Sip.pm
index 558a1d4..44d877e 100644
--- a/C4/SIP/Sip.pm
+++
Allow partial payment of outstanding fines
either against individual fine entries or as a lump payment.
Sponsered by East Brunswick Public Library, East Brunswick, NJ, USA
---
C4/Accounts.pm | 101 ++-
.../intranet-tmpl/prog/en/modules/members/pay.tmpl |
Also remove the reverse column if no contents to be displayed
( otherwise looks like a markup error )
Rationalize the main loop now toggle not required
---
.../prog/en/modules/members/boraccount.tmpl|6 ++-
members/boraccount.pl | 60 +++
---
C4/Accounts.pm|3 +--
members/mancredit.pl |5 -
members/maninvoice.pl |5 -
3 files changed, 9 insertions(+), 4 deletions(-)
diff --git a/C4/Accounts.pm b/C4/Accounts.pm
index bce9c19..c7e05bc 100644
--- a/C4/Accounts.pm
+++ b/C4/Accounts.pm
@@ -454,7 +454,6 @@
Script was incorrectly asking user to define an active currency
Now test for active currency in the standard way
Use the currency routines in Budgets where applicable
Move the code paths in the if else to subroutines to clarify flow
Enable warnings
---
admin/currency.pl | 258
Remove unnecessary db calls and processing from acqui-home.pl
Do not duplicate processing of fields
Declare variables closer to use
Remove unnecessary holding vars
Replace some anonymous var names by something more meaningful
Do not generate a toggle variable use Template's LOOP vars
---
Fixes generation of empty index page when total_parcels is a multiple of
pagesize
Remove some calculations whose results were not used
Make variable names meaningful (results and count arnt)
Fix errors in pod syntax
Make what code does clearer to readers who are not computers
Use a subroutine
. [ undef, ] ). Better to just return; which returns
undef in a scalar context and an empty array in an array context. Makes
for dafer code.
Colin
--
Colin Campbell
Chief Software Engineer,
PTFS Europe Limited
Content Management and Library Solutions
+44 (0) 208 366 1295 (phone)
+44 (0) 7759
On 18/03/10 14:30, Colin Campbell wrote:
for dafer code.
Oops that should be safer (though I've generated my fair share of dafter
code over the years)
C.
--
Colin Campbell
Chief Software Engineer,
PTFS Europe Limited
Content Management and Library Solutions
+44 (0) 208 366 1295 (phone)
+44 (0
Garry Collum's patch correctly identified the core flaw in operation
looking closer revealed some other flaws (2 blank options, some vars
and documentation belonging to a superceded version, etc.
Changed GetCities to return a structure rather than the messy two returns
Let the template do the
Remove any carriage returns in text data as they will be
interpreted as end of message by clients and cause communication failure
---
C4/SIP/Sip.pm |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/C4/SIP/Sip.pm b/C4/SIP/Sip.pm
index 558a1d4..f176516 100644
---
return undef or empty array in error cases
hopefully if we're consistent someone may start checking it
removed unnecessary ()s
---
C4/Bookseller.pm |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/C4/Bookseller.pm b/C4/Bookseller.pm
index 655d25e..63df90d 100644
---
mysql comments require a space after the --
---
installer/data/mysql/en/optional/auth_val.sql |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/installer/data/mysql/en/optional/auth_val.sql
b/installer/data/mysql/en/optional/auth_val.sql
index 5a39534..12856d0 100644
---
dom is the recommended authorities indexing mode in external docs
making it default accepts what appears to be the de facto preference
see Indexdata's documentation for more details on the options
---
Makefile.PL |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git
Before formatting and doing arithmetic with them we should
ensure undefined numeric elements are numbers
(This should be enforced in the db)
---
acqui/acqui-home.pl | 10 ++
1 files changed, 10 insertions(+), 0 deletions(-)
diff --git a/acqui/acqui-home.pl b/acqui/acqui-home.pl
index
---
acqui/neworderempty.pl | 30 ---
.../prog/en/modules/acqui/neworderempty.tmpl |8 -
2 files changed, 20 insertions(+), 18 deletions(-)
diff --git a/acqui/neworderempty.pl b/acqui/neworderempty.pl
index 9fb1870..cedec85 100755
---
An earlier patch inadvertantly reverted a name change in the template
---
.../opac-tmpl/prog/en/modules/opac-basket.tmpl |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-basket.tmpl
Script test.pl was added to the top level directory
Due to the name this gets run in a make test and end in error
I suspect it was comitted in error
moved and renamed so it dosent get in the way
---
misc/split_test.pl | 30 ++
test.pl| 30
On 08/02/10 20:15, Chris Nighswonger wrote:
+our @EXPORT = qw(get_all_templates
-our @EXPORT_OK = qw(get_all_templates
I'm not sure replacing EXPORT_OK with EXPORT can be called progress. A
migration in the other direction might be considered 'A Good Thing'
Colin
--
Colin Campbell
quantityreceived drives other parts of acq noticeably budget balances
ensure that a number (usually 0) is set in NewOrder
give field a valid default and set as not null
reinterpret exsting nulls in table as zero
---
C4/Acquisition.pm |3 +++
The return from Mail::Sendmail should always be checked
Output the error message if it fails so we have some
indication of where to start looking if it fails
---
C4/Letters.pm | 21 +
1 files changed, 9 insertions(+), 12 deletions(-)
diff --git a/C4/Letters.pm
changed link to point at labels script
---
.../en/modules/labels/label-edit-template.tmpl |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git
a/koha-tmpl/intranet-tmpl/prog/en/modules/labels/label-edit-template.tmpl
Fixed two calls where new interface was not used
Arguably new syntax allows more options than we require but it was not catching
error cases (when 0 or undef is passed). It also can now be called in ways
which would validly return multiple hits but explicitly does not
(nor does calling code expect
---
C4/Budgets.pm | 11 ++-
1 files changed, 6 insertions(+), 5 deletions(-)
diff --git a/C4/Budgets.pm b/C4/Budgets.pm
index 4a8b3b6..8d0a2f0 100644
--- a/C4/Budgets.pm
+++ b/C4/Budgets.pm
@@ -630,14 +630,14 @@ sub DelBudget {
return $rc;
}
-=back
-
=head2 FUNCTIONS ABOUT
New version of patch to apply against 3.2alpha
Also remove related FIXME message that was displaying to the user
---
acqui/neworderbiblio.pl| 14 +-
.../prog/en/modules/acqui/neworderbiblio.tmpl |2 +-
2 files changed, 6 insertions(+), 10
due date was not being formatted as a timestamp
but being passed as epoch seconds
---
C4/SIP/Sip/MsgType.pm |6 +-
1 files changed, 5 insertions(+), 1 deletions(-)
diff --git a/C4/SIP/Sip/MsgType.pm b/C4/SIP/Sip/MsgType.pm
index 1b210ca..d0550e3 100644
--- a/C4/SIP/Sip/MsgType.pm
+++
Subscription end date is required before receiving against
the subscription. If not set the subscription needs
renewal before use.
---
.../prog/en/modules/serials/subscription-add.tmpl |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git
Calls to the unexported subroutine result in system error
---
C4/Serials.pm |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/C4/Serials.pm b/C4/Serials.pm
index 0b70099..d72e98c 100644
--- a/C4/Serials.pm
+++ b/C4/Serials.pm
@@ -56,6 +56,7 @@ BEGIN {
reorder_members
Add use warnings where not present
Remove unused variables
Avoid redeclarations of variables
trailing spaces trimmed
---
serials/acqui-search-result.pl |4 +-
serials/checkexpiration.pl |2 +-
serials/claims.pl |1 -
serials/lateissues-excel.pl|
Some koha acq terminology is confusing to new users so lets at least
be consistent in our usages we are adding to the basket here not an order
---
.../prog/en/modules/acqui/basket.tmpl |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git
Fix obvious warning generators
use of string comparison on numeric values
use of capture variables without testing comparison
reuse of variable names in same lexical scope
Tidy some layout issues
remove commented out code
remove unused variables
remove tabs from mixed space tab layouts
Let templating system do what its designed for
pass an array ref with the suppliers
---
.../prog/en/modules/serials/claims.tmpl|7 -
serials/claims.pl | 30 ---
2 files changed, 19 insertions(+), 18 deletions(-)
diff --git
Previous patch still used old variable name to get count causing
incorrect value to be returned
---
serials/claims.pl |3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/serials/claims.pl b/serials/claims.pl
index d6a7ce7..8ff672f 100755
--- a/serials/claims.pl
+++
Required CPAN module was missing from deoendency list
---
Makefile.PL |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/Makefile.PL b/Makefile.PL
index aeeba7f..ae93b7f 100644
--- a/Makefile.PL
+++ b/Makefile.PL
@@ -537,6 +537,7 @@ WriteMakefile(
NO_META
Field is varchar(10) allow all characters to be used
---
.../prog/en/modules/admin/aqbookfund.tmpl |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbookfund.tmpl
Script was supplying an extra empty record to the template
Changed loop to remove the extra variable that was tempting
us into the error.
Removed the element highlight as the template didn't use (or need) it
---
acqui/neworderbiblio.pl | 17 -
1 files changed, 8 insertions(+), 9
Removed some old code from basket display
Added use warnings to basket.pl
Suppress some warnings generated in basket.pl
---
C4/Acquisition.pm | 11 +---
acqui/basket.pl| 53 ++--
Allows searching by purchase order number in acq screens
Add PO Number to histsearch results display for consistency
This feature sponsored by UN-FAO
---
C4/Acquisition.pm | 29 +++
acqui/posearch.pl | 56 ++
Dont display unpopulated field
---
.../prog/en/modules/acqui/neworderempty.tmpl |1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tmpl
b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tmpl
index
Remove dead code
Clarify the loop assigning dates
Remove the cause of undef warnings in the log
---
serials/routing.pl | 44
1 files changed, 16 insertions(+), 28 deletions(-)
diff --git a/serials/routing.pl b/serials/routing.pl
index
Moved generation of dropdown to template from script
Removed the unnecessary count variable returned by
C4::Serials::GetLateIssues
C4::Serials::GetLateMissingIssues
---
C4/Serials.pm | 37 +++-
.../prog/en/modules/serials/claims.tmpl
Removed some useless counts from serials-edit / Serials interfaces
Removed old commented out code. Unrequired variables
Reformatted code so that improving logic can be done
more easily
---
C4/Serials.pm | 15 +--
serials/serials-edit.pl | 442
If an invalid item barcode is passed to checkin
the sip2 connection dies. This is because although
no item object is created its mehods are called in Checkin
To maintain the connection properly catch the condition
and return the correct response to the unit
(should also fix Bug #3696 )
---
While the comment correctly notes that the order of variable length
fields is not fixed some units expect mandatory fields to
follow the sequence in the protocol definition and fail
parsing otherwise. Moved institution id to its expected
place in the patron information response.
Indented the
When editing items the user library rather than the
existing value was being set as the default value in
branch fields of the item
---
cataloguing/additem.pl |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/cataloguing/additem.pl b/cataloguing/additem.pl
index
As subsequent operations don't get what they expect
all manner of nastiness ensues
---
serials/serials-edit.pl |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/serials/serials-edit.pl b/serials/serials-edit.pl
index 9994e44..86e594e 100755
--- a/serials/serials-edit.pl
Although the due date of an on loan item was being retrieved
the appropriate field in the Item object was not set
causing the due date not to be returned in the item
information response
---
C4/SIP/ILS/Item.pm |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git
Also cleaned the interface to the claims related functions
in Serials so they do not return and extra count variable
moved generation of dropdown to template instead of inline code
---
C4/Serials.pm | 37 +++-
Fixed obvious warnings generators in scripts
with mismatched comparisons or undefined variables
removed temporary variable selected while ensuring the
comparison it represented was between two defined variables
---
C4/AuthoritiesMarc.pm|2 +-
authorities/auth_finder.pl
Fix code generating warnings with warnings enabled
---
admin/koha2marclinks.pl | 24 +++-
1 files changed, 11 insertions(+), 13 deletions(-)
diff --git a/admin/koha2marclinks.pl b/admin/koha2marclinks.pl
index e65e1e5..8e0e105 100755
--- a/admin/koha2marclinks.pl
+++
Fix a couple of comparisons generating warnings
---
admin/marctagstructure.pl |5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/admin/marctagstructure.pl b/admin/marctagstructure.pl
index acd8cd7..a9ae892 100755
--- a/admin/marctagstructure.pl
+++
Previously could not add or update subfields in MARC Authority
Subfield Structure if no Librarian Text supplied
Requirement removed
---
admin/auth_subfields_structure.pl |3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/admin/auth_subfields_structure.pl
Item is a hash reference. We need to specify the field in
the hash to fill the data field not the hash itself.
Where no barcode field is present but we have an itemnumber
retrieve the item details to provide the barcode in
the hash to fill the data field not the hash itself.
---
holds
and transits as if a normal checkin and one where it checks in the book
but needs a second manual checkin before placing the book on the
hold-shelf (much as if the pickup location was another branch).
Comments and suggestions welcome.
C.
--
Colin Campbell
Software Engineer, PTFS
---
reserve/modrequest.pl | 11 ++-
1 files changed, 6 insertions(+), 5 deletions(-)
diff --git a/reserve/modrequest.pl b/reserve/modrequest.pl
index 8b7d8ee..f47e112 100755
--- a/reserve/modrequest.pl
+++ b/reserve/modrequest.pl
@@ -70,11 +70,12 @@ else {
}
}
my
Pull the code into its own subr so that the dangers of
inaccurate copy paste are avoided in future
---
C4/Biblio.pm | 27 +++
1 files changed, 19 insertions(+), 8 deletions(-)
diff --git a/C4/Biblio.pm b/C4/Biblio.pm
index cdb119f..b851bc0 100644
--- a/C4/Biblio.pm
+++
remove unnecessary calls to finish
let DBI return an array instead of looping to create one
make some SQL oneliners more succinct
---
C4/Bookfund.pm | 43 ---
1 files changed, 8 insertions(+), 35 deletions(-)
diff --git a/C4/Bookfund.pm b/C4/Bookfund.pm
Behave consistently with other patron_ids as many sip clients
use this in a subsequent patron information or patron status request
---
C4/SIP/ILS/Item.pm| 13 +
C4/SIP/Sip/MsgType.pm |2 +-
2 files changed, 14 insertions(+), 1 deletions(-)
diff --git a/C4/SIP/ILS/Item.pm
---
tools/overduerules.pl | 35 ---
1 files changed, 16 insertions(+), 19 deletions(-)
diff --git a/tools/overduerules.pl b/tools/overduerules.pl
index ec50ff8..e14d1d8 100755
--- a/tools/overduerules.pl
+++ b/tools/overduerules.pl
@@ -18,12 +18,13 @@
# Suite
to declare @issue_list outside of the
conditional?
C.
--
Colin Campbell
Software Engineer, PTFS Europe Limited
Content Management and Library Solutions
+44 (0) 208 366 1295 (phone)
+44 (0) 7759 633626 (mobile)
colin.campb...@ptfs-europe.com
skype: colin_campbell2
http://www.ptfs-europe.com
Numbers in perl with leading zeros are interpreted in octal
Ensure that comparisons are done using string operators
or where appropriate use the MARC::Field method
---
C4/Record.pm |2 +-
cataloguing/addbiblio.pl |2 +-
Fixed inconsistent usuage of primary key in delete
Refactored:
Moved operations to separate subroutines to clarify data flow
Removed unnecessary redirects to self
Renamed confusingly named else variable passed to template
Other changes for code clarity
NB Outstanding:
database reading/writing
that causes the
silly subject line)
Cheers
Colin
--
Colin Campbell
Software Engineer, PTFS Europe Limited
Content Management and Library Solutions
+44 (0) 208 366 1295 (phone)
+44 (0) 7759 633626 (mobile)
colin.campb...@ptfs-europe.com
skype: colin_campbell2
http://www.ptfs-europe.com
Public Library, East Brunswick, NJ USA
Signed-off-by: Colin Campbell colin.campb...@ptfs-europe.com
---
admin/systempreferences.pl |1 +
installer/data/mysql/en/mandatory/sysprefs.sql |1 +
.../1-Obligatoire/unimarc_standard_systemprefs.sql |1 +
installer
by
Middletown Township Public Library, Middletown, NJ USA and
East Brunswick Public Library, East Brunswick, NJ USA
Signed-off-by: Colin Campbell colin.campb...@ptfs-europe.com
---
C4/Circulation.pm |3 +
C4/Items.pm| 24
On 07/26/2009 05:26 PM, Colin Campbell wrote:
Apologies - Subject line should have been 'Allow temporary processing
shelving locations
C.
--
Colin Campbell
Software Engineer, PTFS Europe Limited
Content Management and Library Solutions
+44 (0) 208 366 1295 (phone)
+44 (0) 7759 633626 (mobile
number of renewals.
This work was sponsored by:
Middletown Township Public Library, Middletown, NJ USA
Signed-off-by: Colin Campbell colin.campb...@ptfs-europe.com
---
C4/Circulation.pm | 55 +++-
circ/circulation.pl
: PTFS Contractor dbavous...@ptfs.com
This work co-sponsered by Middletown Township Public Library, Middletown, NJ,
USA
and East Brunswick Public Library, East Brunswick, NJ, USA
Signed-off-by: Colin Campbell colin.campb...@ptfs-europe.com
---
C4/Members.pm
it happened.
Sponsored by East Brunswick Public Library, East Brunswick, NJ USA
Signed-off-by: J. David Bavousett dbavous...@ptfs.com
Signed-off-by: Colin Campbell colin.campb...@ptfs-europe.com
---
C4/Members.pm | 45 ++--
.../prog/en/modules
to the
template to find out what they mean.
Colin
--
Colin Campbell
Software Engineer, PTFS Europe Limited
Content Management and Library Solutions
+44 (0) 208 366 1295 (phone)
+44 (0) 7759 633626 (mobile)
colin.campb...@ptfs-europe.com
skype: colin_campbell2
http://www.ptfs-europe.com
---
C4/Serials.pm |3 ++-
.../en/modules/serials/subscription-detail.tmpl|3 +++
2 files changed, 5 insertions(+), 1 deletions(-)
diff --git a/C4/Serials.pm b/C4/Serials.pm
index 478dd87..5f4b3ec 100644
--- a/C4/Serials.pm
+++ b/C4/Serials.pm
@@
---
C4/Serials.pm | 15 +++
1 files changed, 7 insertions(+), 8 deletions(-)
diff --git a/C4/Serials.pm b/C4/Serials.pm
index 26b08db..478dd87 100644
--- a/C4/Serials.pm
+++ b/C4/Serials.pm
@@ -320,15 +320,14 @@ Update Claimdate for issues in @$serialids list with date
$date
---
.../en/modules/serials/subscription-detail.tmpl|3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git
a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-detail.tmpl
b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-detail.tmpl
index
op was not being returned to claims.pl. It seems better logic
to use whether the there are ids the user has flagged for claims
to decide whether to generate the claims
---
.../prog/en/modules/serials/claims.tmpl|3 +--
serials/claims.pl | 14
Avoids generating a slew of warnings in the log when the undefined
value was concatenated or used in regexps
---
C4/Biblio.pm | 12
1 files changed, 8 insertions(+), 4 deletions(-)
diff --git a/C4/Biblio.pm b/C4/Biblio.pm
index 803a921..d20d747 100644
--- a/C4/Biblio.pm
+++
---
.../en/modules/admin/auth_subfields_structure.tmpl |8 +---
1 files changed, 5 insertions(+), 3 deletions(-)
diff --git
a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_subfields_structure.tmpl
b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_subfields_structure.tmpl
index
---
admin/auth_subfields_structure.pl |5 -
.../en/modules/admin/auth_subfields_structure.tmpl |9 +++--
2 files changed, 11 insertions(+), 3 deletions(-)
diff --git a/admin/auth_subfields_structure.pl
b/admin/auth_subfields_structure.pl
index 133559f..b166777
---
C4/AuthoritiesMarc.pm | 63
1 files changed, 37 insertions(+), 26 deletions(-)
diff --git a/C4/AuthoritiesMarc.pm b/C4/AuthoritiesMarc.pm
index 633ced8..882211d 100644
--- a/C4/AuthoritiesMarc.pm
+++ b/C4/AuthoritiesMarc.pm
@@ -526,8 +526,13
---
authorities/authorities.pl |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/authorities/authorities.pl b/authorities/authorities.pl
index a0ce203..2e9f2da 100755
--- a/authorities/authorities.pl
+++ b/authorities/authorities.pl
@@ -431,7 +431,7 @@ sub build_tabs
---
C4/AuthoritiesMarc.pm | 63
1 files changed, 37 insertions(+), 26 deletions(-)
diff --git a/C4/AuthoritiesMarc.pm b/C4/AuthoritiesMarc.pm
index 633ced8..882211d 100644
--- a/C4/AuthoritiesMarc.pm
+++ b/C4/AuthoritiesMarc.pm
@@ -526,8 +526,13
97 matches
Mail list logo
