]> git.koha-community.org Git - koha.git/log
koha.git
10 years agoBug 12763: Add more information for users to OPAC Error 404 page
Marc Véron [Fri, 1 Aug 2014 18:30:04 +0000 (20:30 +0200)]
Bug 12763: Add more information for users to OPAC Error 404 page

This patch adds more information to the OPAC 404 error page to make it more user friendly.

To test:

- Apply patch
- Is the information clearly understandable and does it contain all necessary information?
- Native speakers: Is the text correct?
- Non native speakers: Is the text easy to translate?
- Is the HTML correct?

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 4927 - OPAC advanced search has two call number entries; remove one?
Marc Véron [Mon, 21 Jul 2014 18:52:41 +0000 (20:52 +0200)]
Bug 4927 - OPAC advanced search has two call number entries; remove one?

This patch removes the second Call number entry in OPAC Advanced search with Advanced option (for bootstrap).

To test:
- Go to OPAC, Advanced search, enable Advaced options
- In dropdowns you see two entries "Call number"
- Apply patch
- Verify that the second "Call number" has disappeared (below ISSN)

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Template change only - no problems found.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12609: Add some unit tests for C4::Ratings
Jonathan Druart [Fri, 29 Aug 2014 14:41:29 +0000 (16:41 +0200)]
Bug 12609: Add some unit tests for C4::Ratings

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12609 - Replace use of DBI in C4::Ratings with DBIx::Class
Kyle M Hall [Fri, 18 Jul 2014 14:42:03 +0000 (10:42 -0400)]
Bug 12609 - Replace use of DBI in C4::Ratings with DBIx::Class

Test Plan:
1) Apply this patch
2) prove t/db_dependent/Ratings.t
3) Note all unit tests pass

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Also tested:
- Adding a rating
- Rating displayed on detail
- Rating displayed on results
- Modifying a rating
- Change displayed correctly on detail and results

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12890 - Improve title and breadcrumbs when editing saved SQL report
Owen Leonard [Mon, 8 Sep 2014 18:16:14 +0000 (14:16 -0400)]
Bug 12890 - Improve title and breadcrumbs when editing saved SQL report

When editing an existing SQL report the page title and breadcrumbs are
incomplete:

Home > Reports > Guided reports wizard >

This patch corrects this.

To test, apply the patch and edit an existing SQL report. The page title
and breadcrumbs should be more complete:

Home > Reports > Guided reports wizard > Saved reports > Edit SQL report

Patch behves as expected
Signed-off-by: Marc Véron <veron@veron.ch>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12697 - Surplus closing </div> in OPAC advanced search (bootstrap)
Jacek Ablewicz [Sat, 2 Aug 2014 08:13:16 +0000 (10:13 +0200)]
Bug 12697 - Surplus closing </div> in OPAC advanced search (bootstrap)

In koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-advsearch.tt template,
there is a surplus closing </div> in line 336 (= line 332, in case
Bug 12521 is applied first):

   336: </div> <!-- / .span4 -->

which causes some minor layout problems for this page - i.e.: page
footer ("Powered by Koha" + custom footer set up in system prefrences -
if any) is "glued" to the bottom (to the language selection bar);
unless Bug 12521 is applied first, this affects the placement of the
2nd search bar as well.

To reproduce:

1/ go to the OPAC advanced search page
2/ zoom out (e.g., use Ctrl-<minus> several times) and obeserve the
result
3/ no need for step 2/ (zooming out) when using a display with
(relatively) high vertical resolution; also, this particullar problem
gets more apparent/more noticeable with MARC21/USMASRC "subtype"
options being hidden in the OPAC advanced search (Bug 9043).

To test:

1/ ensure that Bug 12521 is applied first/already applied
2/ apply patch
3/ confirm that this issue is no longer reroductible

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
On top of 12521, don't know exactly what to reproduce, but
surplus '</div>' is real (33 open, 34 close).
Everything looks correct, no koha-qa errors.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12521 - Change layout a bit of the opac advanced search
Liz Rea [Wed, 2 Jul 2014 22:09:22 +0000 (10:09 +1200)]
Bug 12521 - Change layout a bit of the opac advanced search

To test:

Load up the advanced search and make sure it displays properly, with all of the elements within the main div, and only one container-fluid.

Patch behaves as expected. Advanced search looks much better now.
Signed-off-by: Marc Véron <veron@veron.ch>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Checked visually and with HTML validator.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12882 - Translations: Resolve ambiguity for word "checked" in Z39.50 server admin...
Marc Véron [Wed, 10 Sep 2014 14:10:21 +0000 (16:10 +0200)]
Bug 12882 - Translations: Resolve ambiguity for word "checked" in Z39.50 server administration

This patch resolves the ambiguity of the word "Checked" for Z39.50
server administration.

To test:
Apply patch
In staff client, go to Home:Administration:Z39.50/SRU servers
Result: Column heading of 6th column should show "Preselected" (instead
of "Checked")

Edit or create new server for Z39.50 and SRU
Result: Checkbox underneath "Password" should have label "Preselected
(searched by default):"

Click "Help" at top right of screen
Result: Wording changed for selecting targets as default.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12863 [CSS follow-up] News in OPAC: HTML broken with nested p tags and multiple...
Owen Leonard [Fri, 5 Sep 2014 13:04:41 +0000 (09:04 -0400)]
Bug 12863 [CSS follow-up] News in OPAC: HTML broken with nested p tags and multiple thead in same table

This patch contains the revised and compiled CSS which will style the
OPAC's new news markup to match the previous version.

To test, apply the patch and clear your browser cache if necessary. View
news in the OPAC and confirm that it is styled like it was before the
patch making markup changes.

CSS patch behaves as expected. Thanks to Owen for adding this.
Signed-off-by: Marc Véron <veron@veron.ch>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12863 [LESS follow-up] News in OPAC: HTML broken with nested p tags and multiple...
Owen Leonard [Fri, 5 Sep 2014 13:02:28 +0000 (09:02 -0400)]
Bug 12863 [LESS follow-up] News in OPAC: HTML broken with nested p tags and multiple thead in same table

This follow-up modifies the LESS file to add style to the new markup to
match the previous markup. A subsequent patch will include the revised
and compiled CSS.

Signed-off-by: Marc Véron <veron@veron.ch>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12863 - News in OPAC: HTML broken with nested p tags and multiple thead in same...
Marc Véron [Wed, 3 Sep 2014 10:10:26 +0000 (12:10 +0200)]
Bug 12863 - News in OPAC: HTML broken with nested p tags and multiple thead in same table

This patch fixes broken HTML in OPAC news by replacing table display
with divs.

To test:

Create multiple news for OPAC
Validate with HTML validator or check source code
Result:
Error with nested p tags
Table header thead is repeated for every news (not allowed)
Bonus testing: Let a screen reader like NVDA read the news and try to
understand it with closed eyes.

Apply patch
Result:
No more HTML errors in news
Screen reader reads news properly

To style you can use classes newscontainer, newsitem, newsheader,
newsbody and newsfooter.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, passes tests and QA script.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12728: DBRev 3.17.00.025
Tomas Cohen Arazi [Thu, 18 Sep 2014 00:33:49 +0000 (21:33 -0300)]
Bug 12728: DBRev 3.17.00.025

10 years agoBug 12728: [QA Follow-up] Add StatisticsFields to sysprefs.sql
Marcel de Rooy [Thu, 21 Aug 2014 11:06:20 +0000 (13:06 +0200)]
Bug 12728: [QA Follow-up] Add StatisticsFields to sysprefs.sql

This patch adds a dbrev.
If people installed Koha after the initial dbrev (3.09.00.015), an install
with sysprefs.sql did not add the pref. So they still could not have it.

Test plan:
If you have the pref, running an database update should not do anything.
If you do not have the pref (delete it via mysql), it should be added.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12728: Add StatisticsFields to sysprefs.sql
Marcel de Rooy [Thu, 7 Aug 2014 12:16:24 +0000 (14:16 +0200)]
Bug 12728: Add StatisticsFields to sysprefs.sql

Bug 7955: [Statistics tab for Patron checkouts] introduced the pref
StatisticsFields but did not add it to the sql install file.

Test plan:
Run webinstaller or mysql ... < sysprefs.sql
Verify that the new pref is in your systempreferences table.

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
Run webinstall, new syspref loaded.
No koha-qa errors.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12827: NewOrder needs more unit tests
Jonathan Druart [Tue, 26 Aug 2014 14:43:49 +0000 (16:43 +0200)]
Bug 12827: NewOrder needs more unit tests

NewOrder should be more tested!

This patch moves the existing unit tests into a new file and adds some
unit tests.

Note that there is no DB field aqorders.subscription, so the test in
NewOrder can be removed.

Test plan:
  prove t/db_dependent/Acquisition/NewOrder.t
and
  prove t/db_dependent/Acquisition.t
should return green.

Signed-off-by: Zeno Tajoli <z.tajoli@cineca.it>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12827: NewOrder should not return basketno
Jonathan Druart [Tue, 26 Aug 2014 14:06:10 +0000 (16:06 +0200)]
Bug 12827: NewOrder should not return basketno

Since the basketno parameter is needed to insert an order, it is useless
to return it.

This patch changes the prototype for the C4::Acquisition::NewOrder
subroutine. The return value is now a scalar containing the ordernumber
created.

Test plan:
Verify there is no regression on an acquisition workflow:
1/ Create an order with several items
2/ Modify the order
3/ Receive some items
4/ Cancel the receipt
4/ Receive some items
5/ Receive all remaining items
6/ Cancel the receipt

Signed-off-by: Zeno Tajoli <z.tajoli@cineca.it>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12891: NewOrder does not return ordernumber if ordernumber is defined
Jonathan Druart [Mon, 8 Sep 2014 18:25:27 +0000 (20:25 +0200)]
Bug 12891: NewOrder does not return ordernumber if ordernumber is defined

The behavior is quite weird, but
  $schema->resultset('Table')->create($data)->id
does not return the id inserted if $data contains the key.

To be more clear, in this case
  $schema->resultset('Aqorder')->create($new_order)->id
returns an empty string because $new_order->{ordernumber} is an empty
string!

This was not caught by the unit tests, I added one.

Test plan:
- AcqCreateItem set to ordering
- Create an order with items and verify items are correctly linked to the
  order.

Signed-off-by: Dobrica Pavlinusic <dpavlin@rot13.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Confirmed that without the patch the created item is not linked to the
order (entry in aqorders_items). With the patch, it works as expected.
Passes tests and Koha QA script.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 11577: Update DBIx
Tomas Cohen Arazi [Thu, 18 Sep 2014 00:10:20 +0000 (21:10 -0300)]
Bug 11577: Update DBIx

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 10883 - AllowHoldDateInFuture can let a patron jump to the top of the holds list
Kyle M Hall [Wed, 27 Aug 2014 13:05:12 +0000 (09:05 -0400)]
Bug 10883 - AllowHoldDateInFuture can let a patron jump to the top of the holds list

If a patron is allowed to add a future date for the hold using the
"Hold starts on date" field, that patron can put a date in that is
in the *past* which will cause them to move to the top of the holds
queue ( assuming they make the past date far enough back that it is
earlier than all the other hold dates ).

Test Plan:
1) Enable future holds
2) Note you can use dates in the past for "hold starts on date"
3) Note you can use dates in the past for "hold expires on date"
4) Apply this patch
5) Refresh the page
6) Note you can no longer use dates in the past for those fields

Signed-off-by: Christopher Brannon <cbrannon@cdalibrary.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Changed subject: OpacAllowHoldDateInFuture to AllowHoldDateInFuture
as this fix applies to staff.
Works as described, passes tests and QA script.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 11577: DBRev 3.17.00.024
Tomas Cohen Arazi [Wed, 17 Sep 2014 23:08:14 +0000 (20:08 -0300)]
Bug 11577: DBRev 3.17.00.024

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 11577 : Fixing a tiny copy/paste issue
Martin Renvoize [Wed, 27 Aug 2014 17:32:38 +0000 (17:32 +0000)]
Bug 11577 : Fixing a tiny copy/paste issue

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 11577 : Fixing a tiny typo
Chris Cormack [Tue, 29 Jul 2014 05:58:57 +0000 (17:58 +1200)]
Bug 11577 : Fixing a tiny typo

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 11577: Add Hochschule für Gesundheit to contributing institutions
Holger Meißner [Thu, 22 May 2014 08:21:09 +0000 (10:21 +0200)]
Bug 11577: Add Hochschule für Gesundheit to contributing institutions

Automatic renewal is the second feature sponsored by the hsg.

Sponsored-by: Hochschule für Gesundheit (hsg), Germany
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 11577: Unit tests
Holger Meißner [Mon, 19 May 2014 12:00:48 +0000 (14:00 +0200)]
Bug 11577: Unit tests

This patch fixes two unit tests that broke because of the new feature.
Also adds some new test cases.

To test:

1) prove t/db_dependent/Circulation.t
2) prove t/db_dependent/Circulation_Issuingrule.t

Sponsored-by: Hochschule für Gesundheit (hsg), Germany
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 11577: Add an automatic renewal cronjob
Holger Meißner [Fri, 16 May 2014 11:14:32 +0000 (13:14 +0200)]
Bug 11577: Add an automatic renewal cronjob

This patch adds a new cron script automatic_renewals.pl and a new
entry in crontab.example.

To test:

1) You need a few issues, some with automatic renewal and some without.
2) Confirm that each time you run misc/cronjobs/automatic_renewals.pl
   those issues are renewed that meet all of the following criteria:
   - automatic renewal has been scheduled either by issuing rule or by
     checkbox on the checkout page
   - the number of allowed renewals isn't exceeded
   - renewal isn't premature (No renewal before)
3) Confirm that all other issues are not affected.

Sponsored-by: Hochschule für Gesundheit (hsg), Germany
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 11577: OPAC bootstrap theme changes
Holger Meißner [Thu, 15 May 2014 14:48:02 +0000 (16:48 +0200)]
Bug 11577: OPAC bootstrap theme changes

This patch makes opac-user.pl and the bootstrap version of opac-user.tt
handle the new renewal errors "auto_renew" and "auto_too_soon".

To test:

1) Set global syspref "opacthemes" to bootstrap.
2) Set global syspref "OpacRenewalAllowed" to Allow.
3) Test the same things as in previous patch, this time for the OPAC
   summary page.

Sponsored-by: Hochschule für Gesundheit (hsg), Germany
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 11577: Expand description of AllowRenewalLimitOverride
Holger Meißner [Wed, 21 May 2014 13:45:35 +0000 (15:45 +0200)]
Bug 11577: Expand description of AllowRenewalLimitOverride

With the introduction of the new automatic renewal feature global syspref
AllowRenewalLimitOverride is expanded in meaning. It now gives the option
to override blocks if:

- the number of allowed renewals is exceeded
- the renewal is premature (No renewal before)
- the item is scheduled for automatic renewal
- the renewal is premature and the item is scheduled for automatic renewal

Sponsored-by: Hochschule für Gesundheit (hsg), Germany
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 11577: Add 'auto_renew' and 'auto_too_soon' to renewal page
Holger Meißner [Wed, 21 May 2014 13:15:11 +0000 (15:15 +0200)]
Bug 11577: Add 'auto_renew' and 'auto_too_soon' to renewal page

This patch adds the new renewal errors 'auto_renew' and 'auto_too_soon'
to the renewal interface.

To test:

1) Issue two items:
   - one with automatic renewal and no value for "No renewal before"
   - another with automatic renewal and a value for "No renewal before"
2) Try to renew:
   Home > Circulation > Renew
3) Confirm there are error messages explaining that the items have been
   scheduled for automatic renewal and that one of the renewals is also
   premature.
4) If global syspref AllowRenewalLimitOverride is set to "Allow" you
   should be given the option to override.

Sponsored-by: Hochschule für Gesundheit (hsg), Germany
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 11577: Code and intranet template changes
Holger Meißner [Thu, 15 May 2014 14:43:00 +0000 (16:43 +0200)]
Bug 11577: Code and intranet template changes

This patch adds a checkbox for "Automatic renewal" to the checkout page.
CanBookBeRenewed is modified to include two new errors:
- auto_renew (renewal shouldn't be done manually)
- auto_too_soon (renewal is premature and shouldn't be done manually)

To test:

1) Add or edit an issuing rule with "Automatic renewal" and another
   one without it.
2) Issue at least three items:
   - automatic renewal by issuing rule
   - automatic renewal by Checkbox on the checkout page
   - no automatic renewal
3) Test the following steps for both:
   Home > Circulation > Checkouts
   Home > Patrons > Patron details
4) Confirm that issues with automatic renewal cannot be renewed manually,
   even if there are still renewals left and it's not too soon to renew.
5) Confirm that "Scheduled for automatic renewal" and the remaining
   renewals are displayed. If no renewals are left "Not renewable" should
   be displayed.
6) Confirm that issues without automatic renewal behave as usual.

Sponsored-by: Hochschule für Gesundheit (hsg), Germany
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 11577: Add "Automatic renewal" to the circulation and fine rules
Holger Meißner [Thu, 15 May 2014 14:30:39 +0000 (16:30 +0200)]
Bug 11577: Add "Automatic renewal" to the circulation and fine rules

This patch adds a column "Automatic renewal" to the circulation and
fine rules table.

To test:

1) Add or edit some issuing rules.
2) Confirm that "Automatic Renewal" is set to "No" by default and
   can be switched to "Yes".

Sponsored-by: Hochschule für Gesundheit (hsg), Germany
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 11577: Add auto_renew flags to the database
Holger Meißner [Thu, 15 May 2014 14:18:36 +0000 (16:18 +0200)]
Bug 11577: Add auto_renew flags to the database

- issues.auto_renew
- old_issues.auto_renew
- issuingrules.auto_renew

Default value is zero.

To test:

1) Run installer/data/mysql/updatedatabase.pl
2) Create SQL reports like:
   SELECT * FROM issues LIMIT 0,1
3) Confirm that a column auto_renew was added to each of the three tables.

Sponsored-by: Hochschule für Gesundheit (hsg), Germany
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12879: Remove unnecesary diags from Holds.t
Tomas Cohen Arazi [Fri, 5 Sep 2014 15:41:20 +0000 (12:41 -0300)]
Bug 12879: Remove unnecesary diags from Holds.t

There are two unnecessary diag statements:
- Creating biblio instance for testing.
- Creating item instance for testing.

TEST PLAN
---------
1) prove t/db_dependent/Holds.t

t/db_dependent/Holds.t .. 1/38 # Creating biblio instance for testing.
Use of uninitialized value in subroutine entry at /home/tcohen/git/koha-community-src/C4/Charset.pm line 181.
 # Creating item instance for testing.
Use of uninitialized value in subroutine entry at /home/tcohen/git/koha-community-src/C4/Charset.pm line 181.
Use of uninitialized value in subroutine entry at /home/tcohen/git/koha-community-src/C4/Charset.pm line 181.
t/db_dependent/Holds.t .. ok
All tests successful.
Files=1, Tests=38,  1 wallclock secs ( 0.03 usr  0.01 sys +  1.13 cusr  0.11 csys =  1.28 CPU)
Result: PASS

-- They are in the first and fourth lines of this sample output

2) apply patch
3) prove t/db_dependent/Holds.t

t/db_dependent/Holds.t .. 1/38 Use of uninitialized value in subroutine entry at /home/mtompset/kohaclone/C4/Charset.pm line 186.
Use of uninitialized value in subroutine entry at /home/mtompset/kohaclone/C4/Charset.pm line 186.
Use of uninitialized value in subroutine entry at /home/mtompset/kohaclone/C4/Charset.pm line 186.
t/db_dependent/Holds.t .. ok
All tests successful.
Files=1, Tests=38,  1 wallclock secs ( 0.01 usr  0.00 sys +  0.78 cusr  0.09 csys =  0.88 CPU)
Result: PASS

-- They are no longer in the first and fourth lines of this sample output

4) run koha QA test tool

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12899: Row grouping in checkouts table is alphabetical and depends on translation
Owen Leonard [Wed, 10 Sep 2014 12:56:24 +0000 (08:56 -0400)]
Bug 12899: Row grouping in checkouts table is alphabetical and depends on translation

The sort order of the "today's checkouts" and "previous checkouts" row
groupings depends on the label, so in English "today's checkouts" comes
first. However, in other languages the reverse alphabetical order is
incorrect resulting in "previous checkouts" coming first.

This patch adds a dummy column with numeric data on which the sorting
can be done. This should make it translation-agnostic.

To test, apply the patch and install or update a translation which will
demostrate the problem (sv-SE for instance).

- Clear your browser cache and switch to the English templates.
- Check out some items to a patron who has checkouts from a previous
  day.
- Confirm that the sorting of the "today's checkouts" and "previous
  checkouts" row groups is correct.
- Switch to the new/updated translation and reload the circulation page
  for that patron. Confirm that the sort remains correct.
- Confirm that the checkouts table looks correct and that other features
  (sorting, checkboxes) still work correctly.

Revision: Corrected the table footer include to correct the colspan
error causing column misalignment.

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12811 - Patron 'Details' and 'Check out' pages not working for staff users withou...
Jacek Ablewicz [Sat, 23 Aug 2014 14:36:21 +0000 (16:36 +0200)]
Bug 12811 - Patron 'Details' and 'Check out' pages not working for staff users without renewal override permissions

In case when the staff user doesn't have (circulate) ->
(override_renewals) permission granted, this code part

   var AllowRenewalLimitOverride = [%
CAN_user_circulate_override_renewals && AllowRenewalLimitOverride %];

in circ/circulation.tt and members/moremember.tt leads to javascript
error, because TT statement evaluates to empty string.

To reproduce:

- set AllowRenewalLimitOverride syspref to "Don't allow" (this step is
  possibly redundant / not quite relevant),
- for testing purposes, log in as some (sample) staff user whitch does
  not have permission granted for "(override_renewals) Override blocked
  renewals",
- have a look at some patron accounts (preferably, ones with 1+
  check-out), observe that page layout is incorrect in "Check Out" and
  "Details" tab.

To test:

- apply patch,
- retest and ensure that this issue is no longer reproductible,
- make sure that there are no regressions of any kinds regarding renewal
  override permission for staff users (i.e, that it still works like
  intended).

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, passes QA script and tests.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12729 - Overdue items won't show as overdue in red in circulation
Dobrica Pavlinusic [Thu, 4 Sep 2014 13:16:57 +0000 (15:16 +0200)]
Bug 12729 - Overdue items won't show as overdue in red in circulation

It seems that Firefox date parser doesn't like our dates which are
formatted in ISO format like "2014-08-06 00:00:00". This results in
missing red color in overdue dates.

So intead of munching different date formats and JavaScript (and having
to support different browers) this patch moves check for overdue dates
back to mysql and just transfers boolean value to JavaScript so it can
show correct class for date_due.

Test scenario:
1. find borrower with overdue checkouts
2. verify that all dates are black (and are in ISO format)
3. apply this patch
4. reload page and verify that overdue dates turned red

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Tested with different due dates (hourly and not) and different date formats.
Passes tests and QA script.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12859 - Patron import tool doesn't import extened attributes values any longer
Jacek Ablewicz [Tue, 2 Sep 2014 09:46:06 +0000 (11:46 +0200)]
Bug 12859 - Patron import tool doesn't import extened attributes values any longer

To reproduce:

1/ define/set up extended pattron attribute in administration module,
2/ using "Tools -> Import patrons", try to import some sample patron
record[s], having some arbitrary (non-empty) value assigned in
'pattron_attributes' CSV column for attribute defined in step 1/,
3/ observe that extended attribute defined in step 1/ was not imported
(not added to the database), without any warnings.

This patch restores a small code chunk which went AWOL somewhere around
commit b4158f2d2485c9e57bd7f14362b210732bef9d35.

To test:

1/ apply patch & retest,
3/ ensure that this issue is no longer reproductible, and there are no
apparent regressions of any kind,
4/ make sure that the ability to specify default value for extended
attributes added in bug Bug 12716 is still working as intended
(= follow up the test plan for bug 12716).

Signed-off-by: Petter Goksoyr Asen <boutrosboutrosboutros@gmail.com>
Patch fixes import of extended patron attributes.
Step 1-3 in test plan confirmed.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12918: fix failing test case
Robin Sheat [Mon, 15 Sep 2014 02:08:03 +0000 (14:08 +1200)]
Bug 12918: fix failing test case

A bug in a test case causes test case failure. It also required a
database, so I'm moving it out of the way.

Test plan:
1) prove -v t/db_dependent/Templates.t
   -- bombs!
2) apply patch
3) prove -v t/db_dependent/Templates.t
   -- works!

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Good catch Robin, it didn't fail in Perl 5.18 so didn't notice the mistake.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12870 - require a higher version of Test::Simple
Robin Sheat [Thu, 4 Sep 2014 05:02:21 +0000 (17:02 +1200)]
Bug 12870 - require a higher version of Test::Simple

New test cases won't build with the version of this in squeeze.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12898 - Z39.50 title search doesn't work with multiple words
Dobrica Pavlinusic [Wed, 10 Sep 2014 11:27:20 +0000 (13:27 +0200)]
Bug 12898 - Z39.50 title search doesn't work with multiple words

This fixes regression introduced by Bug 6536 so that multiple
words in title search will work.

Test scenario:

1. try z39.50 search with more than one word
2. verify that no results apper
3. apply patch and re-run search
4. verify that there are results

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12898: (regression test) test terms inside double quotes
Tomas Cohen Arazi [Sun, 14 Sep 2014 04:55:33 +0000 (01:55 -0300)]
Bug 12898: (regression test) test terms inside double quotes

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 10226: Add unit tests for GetReservesFromItemnumber
Jonathan Druart [Wed, 10 Sep 2014 14:21:09 +0000 (16:21 +0200)]
Bug 10226: Add unit tests for GetReservesFromItemnumber

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 10226 - suspended holds still show not available
Kyle M Hall [Wed, 27 Aug 2014 12:29:05 +0000 (08:29 -0400)]
Bug 10226 - suspended holds still show not available

If you suspend a hold, the item does not show Available.  It still shows
the person next in line, who isn't eligible for the hold yet because of
the suspension.  This is not the case for a delayed hold, where you
originally place the hold and tell it not to start until a future date.
If you do that, it shows as Available.  This is confusing and
inconsistent.

Test Plan:
1) Create an item level suspended hold for a record with no other holds
2) Note in the record details that the hold shows an item level hold
3) Apply this patch
4) Refresh the record details page, note the item is "Available"
5) Optional: prove t/db_dependent/Holds.t t/db_dependent/Reserves.t

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, passes all tests and QA script.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12574 - BiblioDefaultView doesn't affect XSLT results view
Owen Leonard [Mon, 14 Jul 2014 16:49:27 +0000 (12:49 -0400)]
Bug 12574 - BiblioDefaultView doesn't affect XSLT results view

The OPAC's XSLT results view does not respect the choice set in the
BiblioDefaultView system preference. This patch adds a check of the
preference to the OPAC's results view for MARC21.

To test you must have the OPAC configured to use the XSLT view for
search results. Perform any search and test the link to a title's detail
page. Switching the values of the BiblioDefaultView should be correctly
reflected in the search results links.

Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described for XSLT view in OPAC.

Notes:
- maybe we should change the pref text from 'simple form' to 'normal view'.
- the patch doesn't include changes to the title link generated from an 880
  field - so those links will still lead to the wrong view.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 10571 - IntranetBiblioDefaultView doesn't do anything with MARC21 XSLT
Owen Leonard [Mon, 14 Jul 2014 17:07:42 +0000 (13:07 -0400)]
Bug 10571 - IntranetBiblioDefaultView doesn't do anything with MARC21 XSLT

The staff client's XSLT results view does not respect the choice set in
the IntranetBiblioDefaultView system preference. This patch adds a check
of the preference to the staff client's results view for MARC21.

To test you must have the staff client configured to use the XSLT view
for search results. Perform any search and test the link to a title's
detail page. Switching the value of the IntranetBiblioDefaultView
preference should be correctly reflected in the search results links.

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
Work as described, no koha-qa errors.

Only for MARC21 :(

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, no problems found.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12587: (qa followup) report name consistency
Tomas Cohen Arazi [Tue, 9 Sep 2014 13:47:02 +0000 (10:47 -0300)]
Bug 12587: (qa followup) report name consistency

Reports home should show the same report name (i.e. "Patrons with the most checkouts")
the report itself uses. It is chosen as it is more idiomatic.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12587 - Improve output of filter information on patrons with the most checkouts...
Owen Leonard [Wed, 16 Jul 2014 15:23:00 +0000 (11:23 -0400)]
Bug 12587 - Improve output of filter information on patrons with the most checkouts report

This patch makes some changes to the way filter information is displayed
after running the patrons with the most checkouts report. Before this
patch several untranslatable English strings would be displayed, and
library and item type codes would be shown instead of descriptions.

To test, go to Reports -> Patrons checking out the most. Enter values
for all the filter options:

- Checkout date from
- Checkout date to
- Checkin date from
- Checkin date to
- Library
- Item type
- Patron category

When you submit the form and view the results you should see your date
submissions formatted according to your selected dateformat system
preference. The library name and item type names should be shown instead
of codes.

Signed-off-by: Aleisha <aleishaamohia@hotmail.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Improves translatability of the filters output after running
the report and the readability of them (removing codes).

Note: it would be nice if the read-only would be removed from the date
fields, as forcing people to use the date widget is a known accessibility
issue.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12849 - fix URLs in sent lists
Robin Sheat [Fri, 29 Aug 2014 04:19:22 +0000 (16:19 +1200)]
Bug 12849 - fix URLs in sent lists

This brings back the http(s) to the URLs in sent lists.

Test plan:
* make a list
* send it to yourself
* see that the URLs aren't clickable
* apply the patch
* repeat, except now the URLs are better

Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
By removing this bit of code, the code in Auth.pm is used
instead. The code there is not perfect, but the solution
works and both list and cart use the same code.

Passes tests and QA script.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12557: DBRev 3.17.00.023
Tomas Cohen Arazi [Tue, 9 Sep 2014 13:11:44 +0000 (10:11 -0300)]
Bug 12557: DBRev 3.17.00.023

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12557: Partially received - the change should affect the new order
Jonathan Druart [Thu, 21 Aug 2014 09:02:48 +0000 (11:02 +0200)]
Bug 12557: Partially received - the change should affect the new order

If the receipt in not on the whole order but only on a part of it, the
change should be done on the itemnumber linked to the "new order", the
one we are reverting.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12557: UT: The item is not created
Jonathan Druart [Wed, 20 Aug 2014 10:15:06 +0000 (12:15 +0200)]
Bug 12557: UT: The item is not created

The UT was buggy. I have just fix that in another patch on bug 12555.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12557: Add a way to revert the changes made on items on receving
Jonathan Druart [Fri, 11 Jul 2014 10:09:32 +0000 (12:09 +0200)]
Bug 12557: Add a way to revert the changes made on items on receving

Bug 8307 introduces the AcqItemSetSubfieldsWhenReceived pref.
You can now update an item field on receiving (if you create items on
ordering).
But if the receipt is cancel, there is no way to revert these changes.

This patch adds a new pref AcqItemSetSubfieldsWhenReceiptIsCancelled to
allow to revert changes previously done on receiving

Test plan:
0/ Set the AcqCreateItems to 'ordering'
1/ Fill AcqItemSetSubfieldsWhenReceived with o=1 (UNIMARC) or 7=1
(MARC21).
2/ Fill AcqItemSetSubfieldsWhenReceiptIsCancelled with o=2 (UNIMARC) or
7=2 (MARC21)
3/ Create an order with some items
4/ Receive the order and verify the notforloan value is set to 1
5/ Cancel the receipt and verify the notforloan value is set to 2

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12557: DB changes - Add the AcqItemSetSubfieldsWhenReceiptIsCancelled pref
Jonathan Druart [Fri, 11 Jul 2014 10:08:07 +0000 (12:08 +0200)]
Bug 12557: DB changes - Add the AcqItemSetSubfieldsWhenReceiptIsCancelled pref

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, passes tests and QA script.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12833: Patron search should search on extended attributes
Jonathan Druart [Thu, 28 Aug 2014 08:51:03 +0000 (10:51 +0200)]
Bug 12833: Patron search should search on extended attributes

Before Bug 9811, the patron search searched on extended attributes.

This patch restore this behavior.

Test plan:
0/ Create a patron attribute PA
1/ Create a patron A (cardnumber CNA) with PA="foo"
2/ Create a patron B (cardnumber CNB) with PA="foo bar"
3/ Search for CNA should redirect on the patron detail page.
4/ Search for "foo" should display the search result with 2 results.

Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
'Searchable' patron attributes can now be searched for again.
Works as described, passes stests and QA script.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12698 - If user changes home library in Opac change is not visible in staff client
Marc Véron [Mon, 21 Jul 2014 18:03:01 +0000 (20:03 +0200)]
Bug 12698 - If user changes home library in Opac change is not visible in staff client

To test:
- Set opacuserlogin and OPACPatronDetails to allow
- Log in to OPAC, go to 'your personal details'
- Change Home library
- Go to staff client: Patrons requesting modifications (/cgi-bin/koha/members/members-update.pl)
- Without patch, you do not see the change request, but can Approve, Deny or Ignore it
- With patch, you see the change request, indicating the Home library (branchcode)

Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
I "slightly" rewrote the patch so it applies on current master.

10 years agoBug 7817: (followup) add test on MARC bibliographic framework test
Tomas Cohen Arazi [Mon, 8 Sep 2014 15:28:41 +0000 (12:28 -0300)]
Bug 7817: (followup) add test on MARC bibliographic framework test

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 7817: DBRev 3.17.00.022
Tomas Cohen Arazi [Mon, 8 Sep 2014 15:04:40 +0000 (12:04 -0300)]
Bug 7817: DBRev 3.17.00.022

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 7817: QA Follow-up: Print a warning when permanent location is mapped
Marcel de Rooy [Mon, 25 Aug 2014 11:24:03 +0000 (13:24 +0200)]
Bug 7817: QA Follow-up: Print a warning when permanent location is mapped

Item field permanent location should not be mapped to the MARC structure.
If a library did so, this dbrev will print a warning.
If refers to report 12818 for help in moving the data to more_subfields_xml.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Tested by putting permanent_location in and out of the MARC structure.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 7817: Follow-up for original patch
Marcel de Rooy [Mon, 18 Aug 2014 11:25:48 +0000 (13:25 +0200)]
Bug 7817: Follow-up for original patch

This patch removes the commented line for permanent_location.
It adds a more general comment.
Adjusts the exists check on permanent_location.
Adds a reference to bug 12817 that will deal with paidfor similarly.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 7817 - Items Permanent location (shelving location) is set to NULL when item...
Olli-Antti Kivilahti [Wed, 18 Jun 2014 15:31:13 +0000 (18:31 +0300)]
Bug 7817 - Items Permanent location (shelving location) is set to NULL when item is edited

A quick fix to prevent more damages.
No perceived side-effects so far.

Signed-off-by: David Cook <dcook@prosentient.com.au>
This is actually a perfectly good fix for this issue. I've changed
the explanatory comment to explain why.

Another option would be to remove the 'exists' check in the sub
_do_column_fixes_for_mod(), but this is just as functional.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Detailed comment on Bugzilla.
Adding a small follow-up.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12467 [QA Followup] - Unit Tests
Kyle M Hall [Thu, 28 Aug 2014 17:47:18 +0000 (13:47 -0400)]
Bug 12467 [QA Followup] - Unit Tests

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12467 - Lost items marked as not on loan even if they are!
Kyle M Hall [Mon, 23 Jun 2014 16:21:16 +0000 (12:21 -0400)]
Bug 12467 - Lost items marked as not on loan even if they are!

The cronjob longoverdue.pl does not require that an item marked as lost
be returned automatically, but there is a line in ModItem that
automatically marks the item's onloan as false if itemlost is set!

Test Plan:
1) Mark an item as lost with longoverdue.pl, without --mark-returned
2) Inspect the db, note that items.onloan is now 0
3) Apply this patch
4) Mark repeat step 1
5) Inspect the db, noe that items.onloan is still 1
6) Test marking an item as lost from staff interface,
   ensure there are no regressions.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Patch works according to test plan and fixes a data loss bug.

Some notes:
- This patch would be nicer with a regression test.
- Also checked that returning the item removes lost status and onloan still.
- Tried to test with --mark-returned, but couldn't get it to
  return my item neither with nor without the patch. (see comment on
  bug report)

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12835 - Acquisitions: Show search terms in z39.50 results
Marc Véron [Fri, 15 Aug 2014 04:28:07 +0000 (06:28 +0200)]
Bug 12835 - Acquisitions: Show search terms in z39.50 results

To test:

- Go to Acquisitions, select a bookshop and a basket, click 'Add to basket':'From an external source'
- Do a search
- Your search terms should display at the top of the results page.
- Repeat search with several combinations of search terms (with and without results).

- Check the output for correct html.

Fields Raw (any) and Standard ID removed / MV

Signed-off-by: Paola Rossi <paola.rossi@cineca.it>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Passes QA script, works as described, no problems found.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12828: Update Z39.50/SRU online help
Marcel de Rooy [Tue, 26 Aug 2014 15:06:57 +0000 (17:06 +0200)]
Bug 12828: Update Z39.50/SRU online help

Test plan:
Go to Z39.50/SRU servers, click Help and read the text.

Signed-off-by: Marc Véron <veron@veron.ch>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Additions to help file, no problems found.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12746 - Remove empty onclick attribute from memberentrygen.tt
Marc Véron [Tue, 29 Jul 2014 14:56:44 +0000 (16:56 +0200)]
Bug 12746 - Remove empty onclick attribute from memberentrygen.tt

This patch removes empty onclick attribute from memberentrygen.tt

To test:
Apply patch.
Use git grep 'onclick=""' to make sure that there are no more empty onlick attributes
Go to Home > patrons  and click 'New patron'
Check HTML of 'Cancel' link
Make sure that 'Cancel' link still works

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Changes the link to cancel adding a manual restriction.
No regressions found.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12821: (qa followup) fix History -> Timeline on the help page
Tomas Cohen Arazi [Mon, 8 Sep 2014 13:52:05 +0000 (10:52 -0300)]
Bug 12821: (qa followup) fix History -> Timeline on the help page

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12821: Capitalization: about page fixes
Tomas Cohen Arazi [Mon, 25 Aug 2014 15:29:27 +0000 (12:29 -0300)]
Bug 12821: Capitalization: about page fixes

This patch fixes several capitalization issues in the about page.

To test:
- Go to the about page
=> FAIL: you should see
   * Server Information (tab)
   * Koha Team (tab)
   * Koha Timeline (tab)
   * Additional Thanks To... (at the bottom of the Koha Team tab
- Apply the patch and reload
=> SUCCESS: you should see
  * Server information
  * Koha team
  * Timeline
  * Additional thanks to...
- Sign off :-D

Sponsored-by: Universidad Nacional de Cordoba
Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
String changes, no problems found.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12765: Fix capitalization - "Try another search"
Katrin Fischer [Mon, 8 Sep 2014 09:28:49 +0000 (11:28 +0200)]
Bug 12765: Fix capitalization - "Try another search"

To test:
- Go to cataloguing
- Perform a z39.50 search with no results
- Check "Try another search" is displayed
- Perform a z39.50 search with results
- Verify display is correct as well.

Signed-off-by: George Williams <georgew@latahlibrary.org>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12788: (followup) minor optimization with proper tests
Tomas Cohen Arazi [Wed, 20 Aug 2014 14:39:27 +0000 (11:39 -0300)]
Bug 12788: (followup) minor optimization with proper tests

This patch removes the $facets_info calculation from the _get_facets_data_from_record
sub so it is not done for each record. It introduces a new sub, _get_facets_info
that is called from the getRecords loop, that does the job only once.

To test:
- Apply on top of the previous patches
- Run
  $ prove -v t/db_dependent/Search.t
=> SUCCESS: _get_facets_info gets tested and it passes for both MARC21 and UNIMARC.
  Facets rendering should remain unchaged on the UI.
- Sign off :-D

Sponsored-by: Universidad Nacional de Cordoba
Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Signed-off-by: David Cook <dcook@prosentient.com.au>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12788: facets calculation should skip 100 if ind1=z
Tomas Cohen Arazi [Wed, 20 Aug 2014 03:48:07 +0000 (00:48 -0300)]
Bug 12788: facets calculation should skip 100 if ind1=z

This patch adds a test for field 100, to skip it on facet calculation
if ind1=z.

To test:
- Have IncludeSeeFromInSearches set.
- Create a biblio record, when adding an author, create a new authority record
  that contains a 400$a field (see from).
- Rebuild zebra db.
- Search for the record making sure the search returns more than one record.
=> FAIL: the facets contain the 'see from' field.
- Run
  $ prove -v t/db_dependent/Search.t
=> FAIL: it fails
- Apply the patch
- Run
  $ prove -v t/db_dependent/Search.t
=> SUCCESS: it passes
- Re-run the search, notice the 'see from' doesn't show anymore on the facets.
- Sign off :-D

Edit: minor stylistic change

Regards
To+

Sponsored-by: Universidad Nacional de Cordoba
Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Signed-off-by: David Cook <dcook@prosentient.com.au>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBub 12788: (regression test) refactor facet extraction code to allow testing
Tomas Cohen Arazi [Wed, 20 Aug 2014 03:33:40 +0000 (00:33 -0300)]
Bub 12788: (regression test) refactor facet extraction code to allow testing

This patch refactors the facet extraction loop into a proper function.
The loop is changed so the MARC::Record objects are created only once
instead of the old/current behaviour: once for each defined facet (in
C4::Koha::getFacets).

To test:
- Apply the patch
=> SUCCESS: verify facets functionality remains unchanged.
- Run:
  $ prove -v t/db_dependent/Search.t
=> SUCCESS: tests for _get_facets_data_from_record fail, because
  100$a is considered for fields with indicator 1=z (field added
  by IncludeSeeFromInSearches syspref).
- Sign off :-D

Sponsored-by: Universidad Nacional de Cordoba
Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Signed-off-by: David Cook <dcook@prosentient.com.au>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, passes tests and QA script.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12582: DBRev 3.17.00.021
Tomas Cohen Arazi [Fri, 5 Sep 2014 15:18:34 +0000 (12:18 -0300)]
Bug 12582: DBRev 3.17.00.021

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12582 - Control of branch displayed in search results linked to HomeOrHoldingBranch
Kyle M Hall [Wed, 16 Jul 2014 11:44:12 +0000 (07:44 -0400)]
Bug 12582 - Control of branch displayed in search results linked to HomeOrHoldingBranch

Some libraries would like the ability choose to display the home branch
on search results while having circulation rules based on the holding
branch. This is currently impossible because both the display of the
branch in search results, and the selection of the home or holding
branch for circulation rules are controlled by the same system
preference: HomeOrHoldingBranch. This preference is described as being
used only for circulation rules, and makes no mention of its use for
display control. The display control should be split off into a separate
preference.

Test Plan:
1) Apply this patch
2) Run updatedatabase.pl
3) Note the value of the new system preference StaffSearchResultsDisplayBranch
   matches the current value of HomeOrHoldingBranch
4) Set the preference to home branch
5) Perform a staff catalog search with results having items with differing home and
   holding branches.
6) Note the home branch displays
7) Set the preference to holding branch
8) Repeat step 5
9) Note the holding branch displays

Signed-off-by: Jason Robb <jrobb@sekls.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, logic is now tied to a new system preference.
Passes tests and QA script.

10 years agoBug 12330: [QA Follow-up] Consistency between opac-search and staff client
Marcel de Rooy [Wed, 9 Jul 2014 13:56:54 +0000 (15:56 +0200)]
Bug 12330: [QA Follow-up] Consistency between opac-search and staff client

In opac-search two regexes are added for the results of splitting the
pref AdvancedSearchTypes, allowing spaces before and after the names.
The same regexes should be added to catalogue/search.pl to have the
same behavior there.

Test plan:
Set AdvancedSearchTypes to "itemtypes  |  ccode |  loc " (no quotes).
Check OPAC and staff with and without this patch.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12330: OpacHiddenItems not affecting Advanced Search
Mark Tompsett [Thu, 29 May 2014 22:31:03 +0000 (18:31 -0400)]
Bug 12330: OpacHiddenItems not affecting Advanced Search

This patch affects only the area displayed on Advanced Search
by setting the AdvancedSearchTypes OPAC system preference
accordingly. Prior to this patch, no filtering based on
OpacHiddenItems was done. This patch determines if itemtypes,
collections, or shelving locations are hidden and prevents
them from being shown.

TEST PLAN
---------
 1) Back up your DB (always handy)
 2) Set the Searching system preference AdvancedSearchTypes to:
    'itemtypes|ccode|loc' (without the single quotes).
 3) Set the OPAC system preference OpacHiddenItems to include the
    lines:
     itype: [{list of itemtype codes separated by commas}]
     location: [{list of comma delimited shelving location codes}]
     ccode: [{list of comma delimited collection codes}]
    Make sure to exclude one value for each, so there will be at
    least one known thing on each tab.
 4) Open the OPAC.
 5) Click on 'Advanced Search'
    -- three tabs appear: Itemtypes, Collection, Shelving location
    -- Everything is visible
 6) Set the Searching system preference AdvancedSearchTypes to:
    'itemtypes | ccode |     loc' (without the single quotes).
 7) Refresh the OPAC.
    -- There will be three tabs, but ugliness ensues.
 8) Apply the patch
 9) Refresh the OPAC.
    -- You will see: Itemtypes, Collection, Shelving location
    -- Only excluded values from OpacHiddenItems will be seen.
10) Run koha qa test tools

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
All tests pass

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Small rewording in comment only.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12864: Add LOC SRU server to sample z3950servers.sql
Marcel de Rooy [Wed, 3 Sep 2014 11:43:50 +0000 (13:43 +0200)]
Bug 12864: Add LOC SRU server to sample z3950servers.sql

Add LOC SRU server to sample servers installed.
This patch modifies four language files: de, en, es and nb.
For the other languages this may not be useful.

Test plan:
You could test this by using the webinstaller.
But you could also (backup) and delete your records in z3950servers, and
run the sample_z3950_servers.sql file from the command line.
Check if you see the new SRU server.

Tested using mysql. I truncated the table (to reset the IDs as well) and then ran the file. Works as expected.
Signed-off-by: Marc Véron <veron@veron.ch>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 8735: DBRev 3.17.00.020
Tomas Cohen Arazi [Fri, 5 Sep 2014 14:51:53 +0000 (11:51 -0300)]
Bug 8735: DBRev 3.17.00.020

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 8735 [QA Followup 2] - Remove extra period, add bug number to db update
Kyle M Hall [Wed, 4 Jun 2014 18:22:23 +0000 (14:22 -0400)]
Bug 8735 [QA Followup 2] - Remove extra period, add bug number to db update

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 8735 [QA Followup] - Add Unit Tests
Kyle M Hall [Thu, 17 Apr 2014 18:33:16 +0000 (14:33 -0400)]
Bug 8735 [QA Followup] - Add Unit Tests

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 8735 - Expire holds waiting only on days the library is open - Followup - Switch...
Kyle M Hall [Tue, 23 Apr 2013 16:32:03 +0000 (12:32 -0400)]
Bug 8735 - Expire holds waiting only on days the library is open - Followup - Switch from C4::Calendar to Koha::Calendar

Test Plan:
 1) Set ExpireReservesMaxPickUpDelay
 2) Set ReservesMaxPickUpDelay to 1
 3) Place a hold, set it to waiting
 4) Using the MySQL console, modify the waiting date and set it to the
    day before yesterday.
 5) Set today as a holiday for the pickup branch in question.
 6) Run misc/cronjobs/holds/cancel_expired_holds.pl
 7) The hold should remain unchanged
 8) Remove today as a holiday
 9) Run misc/cronjobs/holds/cancel_expired_holds.pl again
10) The hold should now be canceled

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 8735 - Expire holds waiting only on days the library is open
Kyle M Hall [Thu, 6 Sep 2012 15:15:21 +0000 (11:15 -0400)]
Bug 8735 - Expire holds waiting only on days the library is open

Signed-off-by: Leila <koha.aixmarseille@gmail.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 6149: (qa followup) sysprefs should be alphabetically sorted
Tomas Cohen Arazi [Fri, 5 Sep 2014 14:33:27 +0000 (11:33 -0300)]
Bug 6149: (qa followup) sysprefs should be alphabetically sorted

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 6149: DBRev 3.17.00.019
Tomas Cohen Arazi [Fri, 5 Sep 2014 13:57:52 +0000 (10:57 -0300)]
Bug 6149: DBRev 3.17.00.019

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 6149 - Follow-up [syspref] - Stopwords for Result Highlighting
Martin Renvoize [Tue, 11 Mar 2014 09:05:51 +0000 (09:05 +0000)]
Bug 6149 - Follow-up [syspref] - Stopwords for Result Highlighting

- global replaced HightedStopWords syspref with NotHightedWords for
  clarification purposes. Hopefully this is a bit better and still
  not too verbose.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, fixed some tabs and a typo.

10 years agoBug 6149: Follow-up [grouped] - Stopwords for Result Highlighting
Martin Renvoize [Tue, 19 Nov 2013 12:07:59 +0000 (12:07 +0000)]
Bug 6149: Follow-up [grouped] - Stopwords for Result Highlighting

- Added functionaility to opac-results-grouped page

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
10 years agoBug 6149: Follow-up [bootstrap] - Stopwords for Result Highlighting
Martin Renvoize [Mon, 28 Oct 2013 12:12:30 +0000 (12:12 +0000)]
Bug 6149: Follow-up [bootstrap] - Stopwords for Result Highlighting

- Added functionality to new bootstrap theme

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
10 years agoBug 6149: Follow-up [defaults] - Stopwords for Result Highlighting
Katrin Fischer [Sun, 27 Oct 2013 14:34:49 +0000 (15:34 +0100)]
Bug 6149: Follow-up [defaults] - Stopwords for Result Highlighting

- changed sample words to and|or|not in sysprefs.sql and
  updatedatabase
- changed systempreference name from OpacHighlightedStopwords
  to HighlightedStopwords, as it also works for the staff
  results page

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
10 years agoBug 6149 - Stopwords for Result Highlighting
Martin Renvoize [Fri, 9 Aug 2013 13:42:00 +0000 (14:42 +0100)]
Bug 6149 - Stopwords for Result Highlighting

This patch adds a system preference to control stopwords used to
prevent operators and other 'minor' words from being highlighted
in search results when the OpacHighlightWords syspref is enabled

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Works as advertised, I fixed the updatedatabase to use XXX so it won't
clash as much

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Fixed a conflict in updatedatabase.pl.
Also fixed tabs in opac-detail.tt

Feature passes all tests and QA script now.
Tested in intranet and OPAC with different words and searches.
Works on results pages and OPAC detail page.
Highlighting and unhighlighting still works correctly.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Passes all tests and QA script, works as expected.

10 years agoBug 7498 - Cloning a budget, enable change of description
charles [Wed, 6 Aug 2014 20:44:19 +0000 (16:44 -0400)]
Bug 7498 - Cloning a budget, enable change of description

Patch sponsored by the CCSR ( http://www.ccsr.qc.ca )

This new patch builds on the work of Bug 12164 by allowing the
user to enter a new name (budget_period_description) for the
cloned budget.

A test was added to t/db_dependent/Budgets.t.

Sponsored-by: CCSR
Signed-off-by: Paola Rossi <paola.rossi@cineca.it>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, passes tests and QA script.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Edit: added the sponsored-by line.

10 years agoBug 12790: QA-follow-up: Mention required version in online Help
Marcel de Rooy [Thu, 4 Sep 2014 06:40:05 +0000 (08:40 +0200)]
Bug 12790: QA-follow-up: Mention required version in online Help

For those who would not understand the two version numbers and come up
with the idea to check the Help page, some text is added.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12790 - Perl modules: Display required version numbers on about page
Marc Véron [Thu, 7 Aug 2014 05:45:34 +0000 (07:45 +0200)]
Bug 12790 - Perl modules: Display required version numbers on about page

This patch adds information about required perl module versions to the
'About' page.

To test:
Apply patch.
In staff client, go to 'About', tab 'Perl modules'
The required version numbers should display in small print next to or
underneath the module names.

Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12869 - update the version in build-git-snapshot
Robin Sheat [Thu, 4 Sep 2014 04:30:34 +0000 (16:30 +1200)]
Bug 12869 - update the version in build-git-snapshot

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12866: Fixes Display of Email in Alternate Address
Janet McGowan [Wed, 3 Sep 2014 16:05:58 +0000 (16:05 +0000)]
Bug 12866: Fixes Display of Email in Alternate Address

Incorrect display of Email Address in Alternate Address Patron Details

Without patch, mail from primary address is displayed in Alternate address
With patch, correct mail address is displayed
Signed-off-by: Marc Véron <veron@veron.ch>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, no problems found.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12205: (regression tests) HoldsInNoissuesCharge controls GetMemberAccountBalance...
Tomas Cohen Arazi [Tue, 2 Sep 2014 13:40:32 +0000 (10:40 -0300)]
Bug 12205: (regression tests) HoldsInNoissuesCharge controls GetMemberAccountBalance behaviour regarding hold charges

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12205: DBRev 3.17.00.018
Tomas Cohen Arazi [Mon, 1 Sep 2014 15:58:22 +0000 (12:58 -0300)]
Bug 12205: DBRev 3.17.00.018

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12205 : Add a system preference allowing hold charges to block circ
Chris Cormack [Tue, 6 May 2014 22:15:20 +0000 (10:15 +1200)]
Bug 12205 : Add a system preference allowing hold charges to block circ

Currently, if you have holds charges, they are not taken into
consideration when circulating items.

Manual Invoices, and rental charges are governed by a systempreference

Holds are never counted

And all other charges (overdues, lost items etc) are counted

This patch adds a systempreference to allow Hold charges to be counted
as well.

To test

1/ Set a borrower category to have holds charges
2/ Place a hold for a borrower in that category
3/ Go to checkout, notice that charge is not showing or blocking on
that screen
4/ apply patch
5/ notice that charge now shows on checkout

Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12705 - News not possible for one sublanguage if two sublanguages are installed
Marc Véron [Sun, 27 Jul 2014 06:03:23 +0000 (08:03 +0200)]
Bug 12705 - News not possible for one sublanguage if two sublanguages are installed

To test:
- Install two language variants (e.g. de-DE and de-CH)
- Create OPAC news for each of them
- Verify that they are displayed in OPAC as appropriate

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
Works Ok, no koha-qa errors

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works ok, all installed languages now show up in the pull downs
and display for the selected language in the OPAC.
Passes tests and QA script.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12309: Adapt zipcode wording to match the other patron pages
Jonathan Druart [Fri, 6 Jun 2014 14:36:46 +0000 (16:36 +0200)]
Bug 12309: Adapt zipcode wording to match the other patron pages

Replace "Zip code" with "Zip/Postal code" which is used in
members/memberentry.pl and members/moremember.pl.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12309 - Add more fields to batch patron modification tool
Owen Leonard [Thu, 22 May 2014 13:23:45 +0000 (09:23 -0400)]
Bug 12309 - Add more fields to batch patron modification tool

It would be helpful to be able to use the batch patron modification tool
to update city, state, zipcode, and country. This patch adds this
functionality.

This patch corrects an error in the batch modification script which
would have incorrectly marked first name as required if surname was
required.

This patch also converts the table sorting configuration to use table
header class names instead of index-based sorting for columns containing
dates.

To test, apply the patch and submit a batch of patrons to the batch
patron modification tool (Tools -> Batch patron modification).

In the list of patrons which will be modified you should see columns for
city, state, zipcode, and country. Table sorting should work correctly
without errors, including the registration and expiry date columns.

In the list of fields which can be batch modified you should see inputs
for city, state zipcode, and country. These fields should correctly
reflect mandatory/not mandatory based on the BorrowerMandatoryField
system preference.

Submit batch changes to a variety of fields and confirm that changes are
applied correctly. On the page of results the table should correctly
show all fields and be sorted correctly.

Signed-off-by: cachan <pierre.lorgue@ens-cachan.fr>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12061 - tmpl_process3.pl - Include/exclude file by name
Bernardo Gonzalez Kriegel [Thu, 10 Apr 2014 01:30:35 +0000 (22:30 -0300)]
Bug 12061 - tmpl_process3.pl - Include/exclude file by name

This patch adds options to include/exclude files by matching their names.
Also modifies current code to check only filenames (not dirs)
Checking is case insensitive !!

NOTE: The difference between -f and -m is subtle, but important.
They differs mainly on update,: -f do a merge, -m a replace

To test after patch:

A) Include only 'normarc'
1. create
cd misc/translator
perl tmpl_process3.pl create -i ../../koha-tmpl/opac-tmpl/bootstrap/en -s normarc.po -r -m normarc
- check provenance of strings
egrep "^#:" normarc.po | cut -d":" -f2 | sort | uniq
- only files with normarc in their names must be present

2. update
perl tmpl_process3.pl update -i ../../koha-tmpl/opac-tmpl/bootstrap/en -s normarc.po -r -m normarc
- repeat check

3. install
mkdir test
perl tmpl_process3.pl install -i ../../koha-tmpl/opac-tmpl/bootstrap/en -o ./test -s normarc.po -r -m normarc
- check name of created files

rm -rf test normarc.po

B) Exclude
4. create
perl tmpl_process3.pl create -i ../../koha-tmpl/opac-tmpl/bootstrap/en -s xnormarc.po -r -n normarc
- check provenance
egrep "^#:" xnormarc.po | cut -d":" -f2 | sort | uniq | grep -i normarc
- there must be no results

5. update
perl tmpl_process3.pl update -i ../../koha-tmpl/opac-tmpl/bootstrap/en -s xnormarc.po -r -n normarc
- check provenance

6. install
mkdir test
perl tmpl_process3.pl install -i ../../koha-tmpl/opac-tmpl/bootstrap/en -o ./test -s xnormarc.po -r -n normarc
- check files
find test | grep -i normarc
- there must be no results

You can also try another combination, use for example "-m patron -m user -m bottom" (or use -n)
or mixed "-m marc -n normarc", do create/install and look filenames

Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
10 years agoBug 12061 - tmpl_process3.pl - Multiple source dir
Bernardo Gonzalez Kriegel [Mon, 7 Apr 2014 00:39:08 +0000 (21:39 -0300)]
Bug 12061 - tmpl_process3.pl - Multiple source dir

This patch adds the option to use multiple source
dirs for tmpl_process3.pl on create/update actions.

To test after patch:
A) Old
1. install
cd misc/translator
perl tmpl_process3.pl create -i ../../koha-tmpl/opac-tmpl/bootstrap/en -s bootstrap.po -r

2. update
perl tmpl_process3.pl update -i ../../koha-tmpl/opac-tmpl/bootstrap -s bootstrap.po -r

3. install
mkdir test
perl tmpl_process3.pl install -i ../../koha-tmpl/opac-tmpl/bootstrap -s bootstrap.po -r -o ./test -q
- check same number of files
ls -lR ../../koha-tmpl/opac-tmpl/bootstrap | wc -l
ls -lR ./test | wc -l

rm -f bootstrap.po

B) New
4. create from multiple sources
perl tmpl_process3.pl create -i ../../koha-tmpl/opac-tmpl/prog/en -i ../../koha-tmpl/opac-tmpl/bootstrap/en -i ../../koha-tmpl/opac-tmpl/ccsr/en -s opac.po -r

5. update from multiple source
perl tmpl_process3.pl update -i ../../koha-tmpl/opac-tmpl/prog/en -i ../../koha-tmpl/opac-tmpl/bootstrap/en -i ../../koha-tmpl/opac-tmpl/ccsr/en -s opac.po -r

6. install (must give an error)
perl tmpl_process3.pl install -i ../../koha-tmpl/opac-tmpl/prog/en -i ../../koha-tmpl/opac-tmpl/bootstrap -s opac.po -r -o ./test

7. updated help
perl tmpl_process3.pl --help

Any other functionality must not be affected

Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>