koha.git
6 years agoBug 13364: Use SQL placeholder
Jonathan Druart [Fri, 17 Apr 2015 10:06:49 +0000 (12:06 +0200)]
Bug 13364: Use SQL placeholder

6 years agoBug 13364: Remove unused JS functions
Jonathan Druart [Fri, 17 Apr 2015 10:04:24 +0000 (12:04 +0200)]
Bug 13364: Remove unused JS functions

6 years agoBug 13364: Fix table id + software error
Jonathan Druart [Fri, 17 Apr 2015 09:53:42 +0000 (11:53 +0200)]
Bug 13364: Fix table id + software error

change table id in css file
+ C4::Output is used (Undefined subroutine
CGI::output_html_with_http_headers)

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
6 years agoBug 13364: Add a call number browser in item edition page
simith [Fri, 27 Feb 2015 13:09:09 +0000 (08:09 -0500)]
Bug 13364: Add a call number browser in item edition page

When editing an item, the call number browser search for existing call number. If it shows a line is green between two calli numbers, the call numbers searched is available. If the call number is highlighted in red, it means that it already exists. This can be used to avoid repeated  call numbers in your collection.

The call number browser also shows its position relative to the rest of the collection.

Sponsored-by: CCSR
To test :

                   1) Go to MARC bibliographic framework
                   Home › Administration › MARC frameworks
                   2) Click in MARC structure (Default framework)
                   3) Search field 952 and click in subfields
                   4) Edit subfield o (Full call number)
                   5) Click Display more constraints
                   6) In Others Options, Plugin, Choose cn_browser.pl and save changes
                   7) Search a record and edit its items
                   Validate : three points (…) beside Full call number input
                   8)  Click in …
                   Validate : if it can not find the given call number, a green line is shown. Otherwise,  the given call number is highlighted in red.

Followed test plan. Works as expected.
Signed-off-by: Marc Véron <veron@veron.ch>
Signed-off-by: Magnus <magnus@enger.priv.no>
Works as advertised. QA scripts pass.

Tested together with other patches. Works as expected.
Signed-off-by: Marc Véron <veron@veron.ch>
Bug 13364 - fix datatables.inc

http://bugs.koha-community.org/show_bug.cgi?id=13364
Signed-off-by: Magnus <magnus@enger.priv.no>
Works as advertised. QA scripts pass.

Tested together with other patches, works as expected.
Signed-off-by: Marc Véron <veron@veron.ch>
Bug 13364 - Add a call number browser in item edition page - QA fixed

Tested together with other patches.
Signed-off-by: Marc Véron <veron@veron.ch>
Bug 13364 - Add a call number browser in item edition page - QA fixed II

Tested together with oter patches.
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>
6 years agoBug 14066: Removing more Readonly
Katrin Fischer [Sun, 3 May 2015 20:14:26 +0000 (22:14 +0200)]
Bug 14066: Removing more Readonly

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 14066: Correct and optimize
Mark Tompsett [Mon, 27 Apr 2015 22:03:55 +0000 (18:03 -0400)]
Bug 14066: Correct and optimize

As far as I can tell $sunday is never used. And it seemed to be
scoped to only _init.

The remaining hashiness was cleaned up for the $return_by_hour.

TEST PLAN
---------
1) Apply both patches
2) (Skip this one - file does not exist / MV) prove -v t/db_dependent/Calendar.t
3) prove -v t/db_dependent/Holidays.t
4) prove -v t/Calendar.t
5) perlcritic -3 Koha/Calendar.pm
   -- the message corresponds to a comment in the code already
      and not this change.
6) run koha qa test tools

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
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>
6 years agoBug 14066: Remove the Readonly[::XS] dependency
Jonathan Druart [Mon, 27 Apr 2015 11:42:20 +0000 (13:42 +0200)]
Bug 14066: Remove the Readonly[::XS] dependency

The Readonly and Readonly::XS package are only used in Koha::Calendar.
IMO the added value of there packages does not justify their purpose.

Test plan:
1/ Remove Readonly and Readonly::XS
2/ Open the calendar and confirm you don't get any error concerning missing
dependency

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
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>
6 years agoBug 14057: Inventory is painfully slow
Mark Tompsett [Fri, 24 Apr 2015 04:40:46 +0000 (00:40 -0400)]
Bug 14057: Inventory is painfully slow

This patch is a rough start. I believe it runs exponentially
faster, but its equality to the previous version needs to be
tested before I clean it up to acceptable standards.

Nested hashes of hashes was being a debugging nightmare.

Moved the SQL select to C4::Koha.

Changed the GetItemsForInventory to have a hashref parameter.
Added interface, in case there is a need for 'opac' vs. 'staff'.

Added t/db_dependent/Items/GetItemsForInventory.t
Added t/db_dependent/Koha/GetKohaAuthorisedValuesMapping.t

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 14056: Small punctuation error in description for deleting a holiday
Aleisha [Mon, 27 Apr 2015 23:54:37 +0000 (23:54 +0000)]
Bug 14056: Small punctuation error in description for deleting a holiday

This patch just inserts a semi-colon into the description so that it makes more grammatical sense.

Test plan:
1) Go to Tools --> Calendar
2) Create or find a holiday, click on it so the editing options come up
3) Click on the question mark link to show the description for the option "Delete the exceptions on a range"
4) Confirm that this now has a semi-colon and reads "... Be careful about your scope range; if it is oversized you could slow down Koha."

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 12192: Template toolkit code in HTML attributes is picked up into po-files
Bernardo Gonzalez Kriegel [Tue, 28 Apr 2015 21:24:27 +0000 (18:24 -0300)]
Bug 12192: Template toolkit code in HTML attributes is picked up into po-files

This patch gives a partial solution for this problem.
It ignores strings like "[% something %]", but not
"[% IF ( value )  %][% value %][% END %]"

We get 100+ strings less to translate

To test:
1) Update translation files for your preferred language
2) Apply the patch
3) Update again
4) Compare, you must find 100+ strings removed from
translation files

eg.
-msgid "[% SEARCH_RESULT.biblionumber |url %]"
-msgid "[% accepteddate | $KohaDates %]"
-msgid "[% amountoutstanding | format('%.2f') %]"
-msgid "[% authtypetext |html %]"
-msgid "[% barcode_llx |html %]"
-msgid "[% barcode_lly |html %]"
-msgid "[% biblio.quantity.length ? biblio.quantity : 1 %]"
-msgid "[% billingdate | $KohaDates %]"
-msgid "[% borname |html %]"
...

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Verified all strings removed from the po files were
pure TT.
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 14119: Missing de-DE DISCHARGE message (German wording)
Marc Véron [Sat, 2 May 2015 07:25:56 +0000 (09:25 +0200)]
Bug 14119: Missing de-DE DISCHARGE message (German wording)

This patch adds DISCHARGE sample message to de-DE sample_notices.pl
(in GErman wording)

To test:
1) prove xt/sample_notices.t, it fails
2) apply the patch
3) prove again, this time no errors

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
Right wording :)
Fixed version

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 14119: Missing de-DE DISCHARGE message
Bernardo Gonzalez Kriegel [Sat, 2 May 2015 02:40:50 +0000 (23:40 -0300)]
Bug 14119: Missing de-DE DISCHARGE message

This patch adds DISCHARGE sample message to de-DE sample_notices.pl

To test:
1) prove xt/sample_notices.t, it fails
2) apply the patch
3) prove again, this time no errors

As original Bug 8007 sample message this is not translated,
so a German counter patch is welcome.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
As noted, a German counter patch is preferred. :)

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 14122: Patron card: Software error when creating using borrower_number as paramet...
Marc Véron [Sat, 2 May 2015 19:47:01 +0000 (21:47 +0200)]
Bug 14122: Patron card: Software error when creating using borrower_number as parameter from URL

To test prepare an URL like the following:

...cgi-bin/koha/patroncards/create-pdf.pl?borrower_number=61&template_id=24&layout_id=21&start_card=1

Change the values for borrower_number, template_id and layout_id to something that exists in your system.

Without patch, the URL leads to a software error:
Can't use an undefined value as a HASH reference at /usr/share/kohaclone/patroncards/create-pdf.pl line 118.

With patch, a PDF with the patron card is created.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
NOTE: This does NOT deal with the case where borrowernumber,
template_id, or layout_id are invalid. This only deals with
the case where they do exist.

Invalid borrowernumber still gives same error.
Invalid template_id or layout_id gives error on line 111.
Those are beyond the scope of this bug.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 14127: Typo in message preference form - "mesage"
Marc Véron [Mon, 4 May 2015 09:45:45 +0000 (11:45 +0200)]
Bug 14127: Typo in message preference form - "mesage"

This path fixes the typo.

To reproduce and test:
Go to a patron's detail page
The message appears as hint in column-header 'Digests only (i)'

Sponsored-by: Universidad de El Salvador
Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 14033: Capitalization: confirmation message on deleting an authority type
Marc Véron [Wed, 22 Apr 2015 06:56:34 +0000 (08:56 +0200)]
Bug 14033: Capitalization: confirmation message on deleting an authority type

This bug fixes capitalization of "No, Do Not Delete" in following templates:

koha-tmpl/intranet-tmpl/prog/en/modules/admin/authtypes.tt
koha-tmpl/intranet-tmpl/prog/en/modules/admin/classsources.tt
koha-tmpl/intranet-tmpl/prog/en/modules/admin/currency.tt
koha-tmpl/intranet-tmpl/prog/en/modules/admin/marctagstructure.tt
koha-tmpl/intranet-tmpl/prog/en/modules/admin/matching-rules.tt
koha-tmpl/intranet-tmpl/prog/en/modules/admin/patron-attr-types.tt
koha-tmpl/intranet-tmpl/prog/en/modules/admin/stopwords.tt

To test: Verify that the strings are properly replaced.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Fixed some tabs.
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 14126: Typo on order receive page
Hector Eduardo Castro Avalos [Mon, 4 May 2015 03:13:08 +0000 (21:13 -0600)]
Bug 14126: Typo on order receive page

A typo found while doing translation:

To test
1) Go to dir 'intranet-tmpl/prog/en/modules/acqui/neworderempty.tt'
   line 268 and check the typo "To notify on reveiving:"
2) Apply the patch
3) Repeat step 1 and check if the typo is fixed

Sponsored-by: Universidad de El Salvador
Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 14044: Don't escape html characters
Jonathan Druart [Thu, 30 Apr 2015 08:06:25 +0000 (10:06 +0200)]
Bug 14044: Don't escape html characters

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 14044: Escape json strings for patron searches
Jonathan Druart [Wed, 22 Apr 2015 13:39:55 +0000 (09:39 -0400)]
Bug 14044: Escape json strings for patron searches

In a couple of patron searches, the address is displayed.
If the address contain invalid json characters, DataTables won't manage
to parse the results and display them.
This patch makes the templates more readable by creating a BLOCK instead
of a long line and escape the invalid json characters calling the TT
plugin (To.json).

Test plan:
Edit a patron and add something like:
    patron <address> \123 ☎ ⚑
in the address field.

Search for this patron using the main patron search (members/member.pl)
and the guarantor search.
Confirm that a result with this patron is correctly displayed.

Regression test: Since Koha::Template::Plugin::To::json is modified,
confirm that the add users to basket search (on acqui/basket.pl?basketno=X
click on the "Add user" button) still works as expected.

Signed-off-by: Christopher Brannon <cbrannon@cdalibary.org>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 11574 - Clean up Printing Lists
Mark Tompsett [Sat, 4 Apr 2015 20:43:40 +0000 (16:43 -0400)]
Bug 11574 - Clean up Printing Lists

Removes extraneous stuff from prints - toolbars, breadcrumbs, etc.

To test:

1. make a list
2. print it to a file (from OPAC), observe wasted paper
3. apply patch
4. print list, compare
5. if you like it, sign off. ;)

Patch cleans up OPAC list printing as expected.

Signed-off-by: Marc Véron <veron@veron.ch>
NOTE: Made functional again. Turns out my issues were a result of
      cached CSS pages. Tested in IE and Firefox successfully.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 14006: about.pl checks the wrong zebra index mode
Tomas Cohen Arazi [Mon, 20 Apr 2015 18:39:45 +0000 (15:39 -0300)]
Bug 14006: about.pl checks the wrong zebra index mode

When setting zebra_bib_index_mode to grs1 I get two warnings when not applying the patch:

"The <zebra_bib_index_mode> entry is set to grs1. GRS-1 support is now deprecated and will be removed in future releases. Please use DOM instead by setting <zebra_bib_index_mode> to dom (full reindex required)."

"You have set <use_zebra_facets> but the <zebra_bib_index_mode> is not set to dom. Falling back to legacy facet calculation."

When applying the patch a third warning appears in addition to the two previous ones:

"The <zebra_bib_index_mode> entry is set to dom, but your system still appears to be set up for grs1 indexing."

Seems like the patch does what it should to me regarding the configuration mismatch warning.

Signed-off-by: Eivin Giske Skaaren <eskaaren@yahoo.no>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 12671: Guess next serial date when there are several issues per unit. Added suppo...
Olli-Antti Kivilahti [Tue, 23 Dec 2014 07:55:06 +0000 (09:55 +0200)]
Bug 12671: Guess next serial date when there are several issues per unit. Added support for real month and year lenghts.

When getting the next frequency number, use the real number of days for each month and each year.

Unit tests updated.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 12671: Guess next serial date when there are several issues per unit
Julian Maurice [Tue, 22 Jul 2014 12:43:16 +0000 (14:43 +0200)]
Bug 12671: Guess next serial date when there are several issues per unit

This guess uses the following algorithm
1/ Get the number of days remaining from first issue of this unit of
time until the end of the unit of time (using 7 days for week, 28 for
month and 365 for year)
2/ Divide it by the number of issues per unit
3/ Take the integer part of the result, this is the number of days to
add to previous serial published date

This patch also try to guess the date of the first issue of a unit of
time. Before that, it was:
- for week: monday of week
- for month: 1st day of month
- for year: 1st day of year
Now it uses firstacquidate.

Example:
  firstacquidate = 1970-01-07 (wednesday)
  unit = week
  issuesperunit = 2 (2 issues per week)

  From wednesday to sunday, there are 4 days so we add 2 days to get the
  next date.
  The next dates should be:
  - 1970-01-09 (+ 2 days)
  - 1970-01-14 (wednesday of next week)
  - 1970-01-16 (+ 2 days)
  - ... and so on

Test plan:
1/ Go to numbering pattern creation page (Serials -> Manager numbering
patterns -> New numbering pattern)
2/ In "Test prediction pattern" fieldset, set subscription length to 50
issues (this avoid to compute the next 1000 dates, which can take time)
3/ Fill first issue publication date
4/ Choose the frequency you want to test
5/ click on "Test pattern" button
6/ Check that the dates are correct
7/ Repeat steps 3-7 until you think you have tested enough cases

Interesting frequencies to test:
- 1/day
- 1/week
- 2/week
- 6/week (firstacquidate should be monday or tuesday, or the generated
  dates will be the same for all the week)
- 1/month
- 2/month
- 3/month
- 1/year
- 2/year
- 3/year

----------------
Sign off report:
----------------
-Followed the test plan. Unit tests work. Also trying some more strange
 frequencies with our librarians:
 5/year, 43/year, 10/year.
-Received all serials for a subscription and found no errors.
-Received some serials with frequency 2/month pre-patch, getting publication
 date as 1st of every month for each magazine. Having applied the patch,
 publication dates started working mid-subscription for new magazines.
-Works well and fine!
Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 12671: Refactor GetNextDate
Julian Maurice [Mon, 21 Jul 2014 15:06:33 +0000 (17:06 +0200)]
Bug 12671: Refactor GetNextDate

A much needed refactoring. Thanks for seeing the effort through.
Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 9089: show suggestion number
Nicolas Legrand [Thu, 26 Mar 2015 11:15:33 +0000 (12:15 +0100)]
Bug 9089: show suggestion number

Show suggestion number on suggestion management.

Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 10388: Does not display 'null' after editing an item on receiving
Jonathan Druart [Tue, 7 Apr 2015 15:03:46 +0000 (17:03 +0200)]
Bug 10388: Does not display 'null' after editing an item on receiving

After editing an item on receiving, 'null' is displayed if no value is
defined for a field. It should be blank.

Test plan:
0/ Set AcqCreateItem to 'ordering'
1/ Go on the receipt page page
2/ Edit an item
3/ Does not fill all values
4/ Confirm that undefined values are replaced with an empty string

NOTE: I think you meant receive. Editing requires at least clicking
      the dropdown value, even if you don't change it before
      clicking save. null's appeared. Post patch application was
      able to get nulls to disappear. :)

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 12412: Add ability for plugins to convert arbitrary files to MARC from record...
Kyle M Hall [Thu, 12 Jun 2014 16:18:41 +0000 (12:18 -0400)]
Bug 12412: Add ability for plugins to convert arbitrary files to MARC from record staging tool

Many libraries would like to be able to import various types of files as
MARC records ( citations, csv files, etc ). We can add a new function to
the plugins system to allow that kind of behavior at a very custom
level.

Test Plan:
1) Ensure you have plugins enabled and configured correctly
2) Installed the attached version 2.00 of the Kitchen Sink plugin
3) Download the attached text file
4) Browse to "Stage MARC records for import"
5) Select the downloaded text file for staging
6) After uploading, you should see a new area "Transform file to MARC:",
   select "Example Kitchen-Sink Plugin" from the pulldown menu
7) Click 'Stage for import"
8) Click 'Manage staged records"
9) You should now see two new MARC records!

Signed-off-by: Aleisha <aleishaamohia@hotmail.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described - interesting new feature.
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13421: DBRev 3.19.00.038
Tomas Cohen Arazi [Mon, 4 May 2015 17:22:31 +0000 (14:22 -0300)]
Bug 13421: DBRev 3.19.00.038

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13421: (QA followup) Change creation_time to created_on
Kyle M Hall [Fri, 19 Dec 2014 15:20:22 +0000 (10:20 -0500)]
Bug 13421: (QA followup) Change creation_time to created_on

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13421: Add the creation time for virtual shelves
Jonathan Druart [Mon, 8 Dec 2014 15:31:03 +0000 (16:31 +0100)]
Bug 13421: Add the creation time for virtual shelves

Test plan:
1/ Execute the updatedb entry
2/ Create a list
3/ Go on the shelve list and confirm that the creation and last
modification time are now displayed.
4/ Confirm that you are able to sort the list by creation/modification
time.

Applied on top of 13419 (rebased updatedatabase.pl)
Works as expected.
Signed-off-by: Marc Veron <veron@veron.ch>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13773: add stock number to details and items display
Dobrica Pavlinusic [Wed, 18 Feb 2015 14:15:33 +0000 (15:15 +0100)]
Bug 13773: add stock number to details and items display

This patch adds stocknumber (named Inventory number in interface)
to details table and items display if it exists.

Test scenario:
1. find item(s) which have inventory number assigned
2. verify that you don't see inventory number in items display
3. apply this patch
4. verify that inventory number is now available

Signed-off-by: Nicole <nicole@bywatersolutions.com>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 14011: Make the jQuery selector explicit
Jonathan Druart [Mon, 20 Apr 2015 13:13:44 +0000 (15:13 +0200)]
Bug 14011: Make the jQuery selector explicit

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 14011: Triggering slip print with enter key collides with 'barcode submitted'
Marc Véron [Sun, 19 Apr 2015 18:24:05 +0000 (20:24 +0200)]
Bug 14011: Triggering slip print with enter key collides with 'barcode submitted'

To reproduce:

- Log into staff client
- Find a client with 0 checkouts
- Print a slip (or cancel printing it)
- Attempt to type in a barcode to checkout
  -- annoying modal screen pops up
     'barcode submitted' message will be triggered each time
     you try

To test:
- apply patch
- try to reproduce issue

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>
6 years agoBug 13810: DBRev 3.19.00.037
Tomas Cohen Arazi [Mon, 4 May 2015 14:41:33 +0000 (11:41 -0300)]
Bug 13810: DBRev 3.19.00.037

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13810: Change collate for tagsubfield (utf8_bin)
Jonathan Druart [Mon, 27 Apr 2015 10:38:40 +0000 (12:38 +0200)]
Bug 13810: Change collate for tagsubfield (utf8_bin)

Before bug 11944, the marc_subfield_structure.tagsubfield column had a
specific collate: utf8_bin. It has been introduced by
  commit 67e20d82ffdfcd69344ec30696bebc51c00d863c
  Date:   Sun Jun 22 16:35:48 2008 -0500
      DB Bump 094 - bug 2268 -- allow mixed case subfield labels in MARC21
      by changing db column collation.

This change should be kept and Koha should continue to allow and create
subfields with the same letter but different case sensitivity.

What does this patch:
1/ To prevent the updatedatabase entry 3.19.00.006 to fail if subfields
with different case sensitivity already exist in the DB, the table is
managed separately from others.
2/ To update DB which have already pass this dbrev, a new entry will be
create to update the specific collate for this column.

Test plan:
1/ a. With a 3.18 DB, create subfield 'a' and 'A' for the same field
   b. Execute the updatedatabase.pl script. 3.19.00.006 should not fail
      anymore
2/ a. With a master DB (3.19.00.006 has already been executed), create
subfields 'a' and 'A'. You should get an error.
   b. Apply this patch, execute the DB entry and try again 2/a. you
   should be able to create the second subfield.

Signed-off-by: Nicolas Legrand <nicolas.legrand@bulac.fr>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 14069: Drop existing primary key on items if exists
Jonathan Druart [Wed, 29 Apr 2015 08:15:50 +0000 (10:15 +0200)]
Bug 14069: Drop existing primary key on items if exists

On old databases, the issues table has a primary key on itemnumber.
The DBrev 3.19.00.028 (Bug 13790 - Add unique id issue_id to issues and
oldissues tables) has to remove it before adding the new primary key

Test plan:
1/ Make sure you have an old DB, or add primary key (issues.itemnumber)manually
on a DB created before 3.19.00.028.
2/ Execute the updatedatabase.pl script
3/ You should not get any error and the primary key on itemnumber should
have been removed and the new one created as expected.

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
No koha-qa errors

Tested using a 3.4 db (3.0404000)
Interestingly, it gives a similar error on the same table but
on another (older $DBversion = "3.05.00.009";) update

1) Pre-patch, 3.4 DB, updatedatabase from master, errors
(older)
DBD::mysql::db do failed: Multiple primary key defined at installer/data/mysql/updatedatabase.pl line 4435.
(13790)
DBD::mysql::db do failed: Multiple primary key defined at installer/data/mysql/updatedatabase.pl line 10166.
DBD::mysql::db do failed: Unknown column 'issue_id' in 'order clause' at installer/data/mysql/updatedatabase.pl line 10170.
DBD::mysql::st execute failed: Unknown column 'me.issue_id' in 'field list' at /usr/share/perl5/DBIx/Class/Storage/DBI.pm line 1593.
DBIx::Class::ResultSetColumn::max(): Unknown column 'me.issue_id' in 'field list' at installer/data/mysql/updatedatabase.pl line 10174

2) Post-patch, same conditions
(only older)
DBD::mysql::db do failed: Multiple primary key defined at installer/data/mysql/updatedatabase.pl line 4435.

There are other warnings, but nor related with keys.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13887: Display the due date as a due date + sort on info
Jonathan Druart [Wed, 22 Apr 2015 08:47:15 +0000 (10:47 +0200)]
Bug 13887: Display the due date as a due date + sort on info

The due dates should be displayed as due dates :)
i.e not displayed with 23:59

On the way, this patch fixes the sort on the info column.
The column is now sorted using the due dates

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13887: Move the caption into a h2
Jonathan Druart [Wed, 22 Apr 2015 08:39:19 +0000 (10:39 +0200)]
Bug 13887: Move the caption into a h2

With the DT filter, the caption is between the table and the filter.
It looks better with the caption as a h2 on top of the table.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13887: Fix sDom to display what is needed in the table
Jonathan Druart [Wed, 22 Apr 2015 08:39:09 +0000 (10:39 +0200)]
Bug 13887: Fix sDom to display what is needed in the table

1/ Add the information 'showing x to y of z'
2/ Remove some lost pixels

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13887: Tab fix and "intranet-tmpl" to "[% interface %]"
Nick Clemens [Wed, 22 Apr 2015 01:07:59 +0000 (21:07 -0400)]
Bug 13887: Tab fix and "intranet-tmpl" to "[% interface %]"

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>
6 years agoBug 13887: Add datatables to item specific holds table
Nick Clemens [Sat, 21 Mar 2015 17:41:21 +0000 (13:41 -0400)]
Bug 13887: Add datatables to item specific holds table

This patch adds datatables functionality to the item specific holds table

To test:
1 - Apply patch
2 - Attempt to place an item specific hold for a patron
3 - Note that columns are now sortable and searchable
4 - Ensure that you can place both specific and next available holds as before

Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
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>
6 years agoBug 13889: DBRev 3.19.00.036
Tomas Cohen Arazi [Mon, 4 May 2015 14:16:33 +0000 (11:16 -0300)]
Bug 13889: DBRev 3.19.00.036

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13889: (QA followup) Supress warning, set object column to undef instead of 0
Kyle M Hall [Thu, 23 Apr 2015 11:51:30 +0000 (07:51 -0400)]
Bug 13889: (QA followup) Supress warning, set object column to undef instead of 0

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13889: Provide parameter $infos in cronlogaction()
Marc Véron [Tue, 21 Apr 2015 15:19:06 +0000 (17:19 +0200)]
Bug 13889: Provide parameter $infos in cronlogaction()

(Alternative for patch Bug 13889: Log with parameters)

To test:

- Apply patches interactive and skip following patch:  Bug 13889: Log with parameters
- Tweak a cron job in order to call cronlogaction twice, one with a param at the beginning of the cron ob, e.g.
cronlogaction("This is my message") and one without param.
- Run this cron job.
Result: You should have two entries in the log, one with and one without additional information after the path and name of the calling script.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13889: Add unit tests
Mark Tompsett [Mon, 20 Apr 2015 08:32:06 +0000 (04:32 -0400)]
Bug 13889: Add unit tests

Previous commit was lacking test coverage of new function.

TEST PLAN
---------
1) Apply all the patches
2) prove -v t/db_dependent/Log.t
   -- should be success.
3) koha qa test tools

Signed-off-by: Marc Véron <veron@veron.ch>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13889: Add information about cron jobs to system log
Marc Véron [Sun, 22 Mar 2015 12:55:00 +0000 (13:55 +0100)]
Bug 13889: Add information about cron jobs to system log

This patch adds information about cron jobs performed and make it viewable
under Home > Tools > Logs ("Browse system logs")

To test:

Apply patch
- Got to system preferences and set 'CronjobLog' to:  [Log] information from
  cron jobs.
- Run some cron jobs
- Go to Home > Tools > Logs
- Verify that you have a selection 'Cron jobs' in drop-down 'Module'. Select it
  with Action "All" and Submit.
- Output should show Date/time and info about Cron jobs

Rebased to work on top of Bug 6911 (conflict in viewlog.tt)  /MV

Rebased after applying patch for Bug 6911 /MV
Conflicts resolved:
misc/cronjobs/overdue_notices.pl
misc/cronjobs/cleanup_database.pl

Signed-off-by: Frederic Demians <f.demians@tamil.fr>
  - Merge both patches, and fix updatedatabase.pl
  - Works as described. Provide intersting feedback from cronjob scripts.

--
Modified version taking in account syspref CronJobLog. Handling simplified by introducing a convenience sub cronlogaction in C4/Log.pm /MV
Amended to take in account comments #11, #12, #13 /MV

http://bugs.koha-community.org/show_bug.cgi?id=13899

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13886: (QA followup) only show merge button on search results
Tomas Cohen Arazi [Mon, 4 May 2015 14:01:55 +0000 (11:01 -0300)]
Bug 13886: (QA followup) only show merge button on search results

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13886: (QA followup) Remove tabs
Nicole [Sun, 29 Mar 2015 20:15:16 +0000 (15:15 -0500)]
Bug 13886: (QA followup) Remove tabs

Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13886: Add ability to merge records directly from cataloging search
Nick Clemens [Sat, 21 Mar 2015 15:31:14 +0000 (11:31 -0400)]
Bug 13886: Add ability to merge records directly from cataloging search

This patch adds a merge button and checkboxes to the cataloging search results screen to allow librarians to merge directly from these reults

To test:
1 - Apply patch
2 - Perform a cataloging search
3 - Try to select 1 or more than 2 records to merge and check that you get an error
4 - Select 2 records to merge and confirm the correct records are passed to the merge tool

Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 9528: Add delivery branch to the place hold display
Owen Leonard [Tue, 30 Sep 2014 15:35:56 +0000 (11:35 -0400)]
Bug 9528: Add delivery branch to the place hold display

When viewing the list of a patron's holds from the circulation or patron
detail page the pickup library is not listed. This patch adds a column
to the table of holds which shows the pickup branch.

This patch also removes some unnecessary markup from the generation of
the table and corrects an instance where the term "reserve" was used
instead of "hold."

This patch also modifies the language describing an item which is marked
waiting at the current library: "Item is waiting here" instead of "Item
waiting."

To test, add several holds to a patron's account with various pickup
locations.

- Confirm that those pickup locations are correctly displayed
  under the Holds tab in Circulation.

- Check in and confirm a hold which is to be picked up at the current
  branch. Confirm that the revised language appears.

- Confirm that table sorting works correctly.

Signed-off-by: Nick <Nick@quechelibrary.org>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13442: Facet links broken when collection code limit is used
Kyle M Hall [Thu, 11 Dec 2014 14:47:01 +0000 (09:47 -0500)]
Bug 13442: Facet links broken when collection code limit is used

Any search limits including a ccode will break the search facts.

Test Plan:
1) Run an advanced search using a ccode limit
2) Try using any of the facet links of the left
3) Note they are broken
4) Apply this patch
5) Refresh the results
6) Note the facet links are no longer broken

Note: I have not been able to reproduce this issue on my own test
system, but have noted the problem on at least a dozen Koha servers.

I could not reproduce the bug either, but I saw it on the Bywater Demo (comment #1).
Applied patch and tested facets, no problems found, signing off
Signed-off-by: Marc Véron <veron@veron.ch>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Can not reproduce the problem, but I can also not find a regression.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 6059: Don't permit to change the quantity when items are created on ordering
Jonathan Druart [Wed, 8 Apr 2015 08:04:40 +0000 (10:04 +0200)]
Bug 6059: Don't permit to change the quantity when items are created on ordering

If items are created on ordering, the quantity for uncertain prices
should not be editable.

Test plan:
0/ Set AcqCreateItem to "on ordering"
1/ Create a basket and add an order with 1+ items
2/ Mark it as uncertain prices
3/ Go on the uncertain prices page for this vendor
(acqui/uncertainprice.pl?booksellerid=X)
4/ Confirm you are not allowed to update the quantity
5/ Change the price and save
6/ Confirm the quantity is still correct

Set the pref to another value and confirm the behavior has not changed.

NOTE: Tested with 'on placing order' and 'on receiving order'.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 8007: (followup) double quotes...
Tomas Cohen Arazi [Thu, 30 Apr 2015 19:15:10 +0000 (16:15 -0300)]
Bug 8007: (followup) double quotes...

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 7843: (QA followup) Make news selection persistant
Kyle M Hall [Fri, 6 Mar 2015 09:37:04 +0000 (10:37 +0100)]
Bug 7843: (QA followup) Make news selection persistant

This patch makes the following changes:
1) If there is no logged in patron, the RSS link states it is for
   system-wide news.
2) If a patron is logged in, the RSS link states it is for system-wide
   news *and* news for the patron's home library.
3) The patron's home branch code is embedded in the RSS feed url so
   he or she no longer needs to be logged in for us to know what branch's
   news to pull.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 7843: Followup - fix broken url
Bernardo Gonzalez Kriegel [Tue, 3 Mar 2015 11:53:04 +0000 (08:53 -0300)]
Bug 7843: Followup - fix broken url

This patch fixes an invalid URL when clicking
on RSS feed.

To test:
1) Using Nicole words :)
If you visit : http://mykoha/cgi-bin/koha/opac-main.pl
then the RSS feed works.
If you're on http://mykoha and you click the rss icon you
get : http://mykoha/opac-news-rss.pl which is a 404 page.

2) Apply the patch

3) Repeat 1, error is now fixed

Signed-off-by: Pierre <tredok.pierre@gmail.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 7843: (follow up) Improve RSS feed language
Nicole [Tue, 3 Mar 2015 11:05:38 +0000 (03:05 -0800)]
Bug 7843: (follow up) Improve RSS feed language

The RSS button used to read ' RSS for the librarys general newsfeed.'
This included one grammatical error and I think included more words
than most average people would read.  This patch simplifies it to say
'RSS for Library News.'

To test:
* Apply all patches
* Review RSS feed button's text on main page under news

http://bugs.koha-community.org/show_bug.cgi?id=7843

Signed-off-by: Pierre <tredok.pierre@gmail.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 7843: (qa-followup) Create an RSS feed for news in Opac
Jonathan Druart [Thu, 19 Feb 2015 11:48:44 +0000 (12:48 +0100)]
Bug 7843: (qa-followup) Create an RSS feed for news in Opac

1/ use strict and warnings are useless, Modern::Perl is used
2/ Prefer to use the interface and theme TT var instead of the hard
coded path

Signed-off-by: Pierre <tredok.pierre@gmail.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 7843: Remove the useless use of casAuthentication
Jonathan Druart [Thu, 19 Feb 2015 11:48:16 +0000 (12:48 +0100)]
Bug 7843: Remove the useless use of casAuthentication

Certainly a c/p error from opac-main.pl

Signed-off-by: Pierre <tredok.pierre@gmail.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 7843: Create an RSS feed for news in Opac
Viktor Sarge [Sun, 8 Feb 2015 19:41:55 +0000 (19:41 +0000)]
Bug 7843: Create an RSS feed for news in Opac

Test plan:
* Install the patch
* Make shure there is news in the Opac
* Go to the Opac (opac-main.pl)
* Make shure you see an RSS icon below the news and a short text.
* Click the RSS icon and verify that you get an RSS feed that validates.

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
I took the liberty of fixing the copyright statement when signing it off

Signed-off-by: Pierre <tredok.pierre@gmail.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 11814: Fix existing tests
Jonathan Druart [Fri, 19 Dec 2014 14:48:49 +0000 (15:48 +0100)]
Bug 11814: Fix existing tests

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 11814: Fix sql parameters order
Jonathan Druart [Tue, 25 Nov 2014 09:01:05 +0000 (10:01 +0100)]
Bug 11814: Fix sql parameters order

Oops, bad paramaters passed on executing the sql query produced
unexpected behaviors.

Signed-off-by: Paola Rossi <paola.rossi@cineca.it>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 11814: (follow-up) Use constants to describe statuses
Jonathan Druart [Thu, 13 Nov 2014 09:07:15 +0000 (10:07 +0100)]
Bug 11814: (follow-up) Use constants to describe statuses

This patch deals with all hard-coded status codes in the C4::Serials
module.

Test plan:
Test a complete workflow in the serial module (create, order, receive,
generate next) trying to use all statuses.

Signed-off-by: Paola Rossi <paola.rossi@cineca.it>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 11814: Use constants to describe statuses
Kyle M Hall [Fri, 21 Feb 2014 14:19:09 +0000 (09:19 -0500)]
Bug 11814: Use constants to describe statuses

Make the code a bit more understandable by using constants to define
the status values and reference those. Adds self-documentation.

Signed-off-by: Paola Rossi <paola.rossi@cineca.it>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 1487: Store the "display only used tags/subf" value in a cookie
Jonathan Druart [Wed, 8 Apr 2015 09:15:02 +0000 (11:15 +0200)]
Bug 1487: Store the "display only used tags/subf" value in a cookie

On the marc framework page, the checkbox to display only used
tags/subfields is always unchecked.
It should be stored into a cookie to always display the same view.

Test plan:
1/ Go on the marc framework page
2/ Check the checkbox
3/ Go somewhere else on the staff interface
4/ Back to the marc framework page, the checkbox should be checked

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Works as advertised, getting datatables error when using feature
but the error exists pre-patch
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 8007: (followup) manually add the <body> tag
Tomas Cohen Arazi [Thu, 30 Apr 2015 18:12:13 +0000 (15:12 -0300)]
Bug 8007: (followup) manually add the <body> tag

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 8007: (QA followup) <body> was missing id and class
Tomas Cohen Arazi [Thu, 30 Apr 2015 15:58:15 +0000 (12:58 -0300)]
Bug 8007: (QA followup) <body> was missing id and class

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 8007: (QA followup) fix charset collate
Tomas Cohen Arazi [Thu, 30 Apr 2015 15:05:22 +0000 (12:05 -0300)]
Bug 8007: (QA followup) fix charset collate

This bug was started before the changes on default collation. This
patch fixes that.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 8007: DBRev 3.19.00.035
Tomas Cohen Arazi [Wed, 29 Apr 2015 13:11:44 +0000 (10:11 -0300)]
Bug 8007: DBRev 3.19.00.035

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 8007: (QA followup) Add error handling when generating the pdf
Jonathan Druart [Tue, 28 Apr 2015 08:59:42 +0000 (10:59 +0200)]
Bug 8007: (QA followup) Add error handling when generating the pdf

If error occurs when generating the pdf, it would be better to get an
encapsulated error instead of the "software error" message in the pdf
file.
To test this patch I added this change:

b/Koha/Borrower/Discharge.pm
-115,6 +115,7 @@ sub generate_as_pdf {
     say $html_fh $html_content;
     close $html_fh;
     my $pdf = PDF::FromHTML->new( encoding => 'utf-8' );
+    $html_path .= "poeut";
     $pdf->load_file( $html_path );
     $pdf->convert;

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 8007: (QA followup) wording issue on sysprefs
Tomas Cohen Arazi [Mon, 27 Apr 2015 19:45:46 +0000 (16:45 -0300)]
Bug 8007: (QA followup) wording issue on sysprefs

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 8007: (QA followup) fix kohastructure to follow guidelines
Tomas Cohen Arazi [Mon, 27 Apr 2015 19:45:24 +0000 (16:45 -0300)]
Bug 8007: (QA followup) fix kohastructure to follow guidelines

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 8007: (RM followup) fix DBIx schema checksum
Tomas Cohen Arazi [Mon, 27 Apr 2015 19:09:42 +0000 (16:09 -0300)]
Bug 8007: (RM followup) fix DBIx schema checksum

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 8007: Compatibility with bug 11944
Jonathan Druart [Thu, 18 Dec 2014 14:39:46 +0000 (15:39 +0100)]
Bug 8007: Compatibility with bug 11944

This patch should be applied to work with bug 11944.
If you don't have 11944 yet, just don't apply this patch.

Note that the line
  binmode(STDOUT);
in pl files is mandatory.

It comes from bug 11944 that adds the line
  binmode( STDOUT, ":utf8" );
in C4::Templates::gettemplate.

Signed-off-by: Lucie <lucie.rousseaux@dracenie.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 8007: Replace HTML::HTMLDoc with PDF::FromHTML
Jonathan Druart [Thu, 18 Dec 2014 14:39:46 +0000 (15:39 +0100)]
Bug 8007: Replace HTML::HTMLDoc with PDF::FromHTML

Signed-off-by: Lucie <lucie.rousseaux@dracenie.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 8007: Discharge - Adding Perl dependencie
Yohann Dufour [Wed, 20 Aug 2014 08:24:17 +0000 (10:24 +0200)]
Bug 8007: Discharge - Adding Perl dependencie

HTML::HTMLDoc is required in order to generate the discharges in PDF format.

Signed-off-by: Lucie <lucie.rousseaux@dracenie.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 8007: Discharge - Add unit tests
Yohann Dufour [Mon, 4 Aug 2014 14:09:53 +0000 (16:09 +0200)]
Bug 8007: Discharge - Add unit tests

Signed-off-by: Lucie <lucie.rousseaux@dracenie.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 8007: Discharge management
Yohann Dufour [Mon, 4 Aug 2014 14:09:42 +0000 (16:09 +0200)]
Bug 8007: Discharge management

This patch is the main patch. It adds new package and files for the new
pages (opac-discharge, members/discharge and members/discharges).

At the intranet, it is now possible to generate a discharge for a patron.
At the opac, a patron can request a discharge and a discharge if it has
been validated by a librarian.

Requirements:
    The perl module PDF::FromHTML

New sysprefs:
 - useDischarge: Allows librarians to discharge borrowers and borrowers
   to request a discharge

New letter with a letter_code DISCHARGE.

Test plan:
- Switch on the syspref useDischarge.
- Verify a new tab appears in the patron page (intranet and opac).
- Verify the discharge cannot be generated if the patron has issues.
- Verify the patron can request a discharge from it's opac area.
- The request appears on the main page (intranet).
- Generate the discharge from the intranet.
- Try to download it (from the opac and the intranet).

Signed-off-by: Lucie <lucie.rousseaux@dracenie.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 8007: Discharge - Glue
Yohann Dufour [Mon, 4 Aug 2014 14:09:35 +0000 (16:09 +0200)]
Bug 8007: Discharge - Glue

This patch adds:
- links to the new pages.
- syspref description
- links on the main page (intranet)
- the DISCHARGE type for debarment

Signed-off-by: Lucie <lucie.rousseaux@dracenie.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 8007: Discharge - DBIx changes
Yohann Dufour [Mon, 4 Aug 2014 14:09:27 +0000 (16:09 +0200)]
Bug 8007: Discharge - DBIx changes

Signed-off-by: Lucie <lucie.rousseaux@dracenie.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 8007: Discharge - DB changes
Yohann Dufour [Mon, 4 Aug 2014 14:09:14 +0000 (16:09 +0200)]
Bug 8007: Discharge - DB changes

This patch adds:
- syspref useDischarge
- the new table discharges
- the discharge notice

Signed-off-by: Lucie <lucie.rousseaux@dracenie.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 7939: (QA followup) make the koha-translate script test the new filenames
Tomas Cohen Arazi [Thu, 30 Apr 2015 14:37:20 +0000 (11:37 -0300)]
Bug 7939: (QA followup) make the koha-translate script test the new filenames

The debian/scripts/koha-translate script was still testing for the old naming
schema files. I also made sure all files are tested, to warn the user if
any of them is absent.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 7939: (followup) Update translation files
Bernardo Gonzalez Kriegel [Wed, 29 Apr 2015 19:28:00 +0000 (16:28 -0300)]
Bug 7939: (followup) Update translation files

Create/rename/update/fix translation files

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 7939: Separate po files for different MARC dialects
Bernardo Gonzalez Kriegel [Sun, 23 Mar 2014 22:01:43 +0000 (19:01 -0300)]
Bug 7939: Separate po files for different MARC dialects

This patch implements separate PO files for
different MARC dialects.

It depends on correct filenames, i.e. it will build PO
files using files with/without "unimarc|normarc|marc21"
on their names.

Changes:
A) LangInstaller.pm
Added definitions to create or update xx-YY-{MARCFLAVOR}.po,
minor change to create and install procedure, and modification
of install procedure to handle multiple target dirs.
Updated documentation.

B) Standarization of filenames
STAFF po file is now xx-YY-staff-prog.po
MARC dialects po files are xx-YY-marc-{MARCFLAVOUR}.po

To test:
1) Update po files for your preferred language, ej. nn-NO
cd misc/translator
perl translate update nn-NO

2) Do some copying/renaming
cp po/nn-NO-i-staff-t-prog-v-3006000.po po/nn-NO-marc-UNIMARC.po
cp po/nn-NO-i-staff-t-prog-v-3006000.po po/nn-NO-marc-NORMARC.po
cp po/nn-NO-i-staff-t-prog-v-3006000.po po/nn-NO-marc-MARC21.po
mv po/nn-NO-i-staff-t-prog-v-3006000.po po/nn-NO-staff-prog.po

(most MARC dialect strings are on staff, so we use that as basis)

3) Apply the patch

4) Update again to fix translation files, verbose
perl translate update nn-NO -v

5) Install language, verbose, verify translations
perl translate install nn-NO -v

6) Create translation files
rm po/nn-NO*
perl translate create nn-NO

we must have this list:
po/nn-NO-marc-MARC21.po
po/nn-NO-marc-NORMARC.po
po/nn-NO-marc-UNIMARC.po
po/nn-NO-opac-bootstrap.po
po/nn-NO-pref.po
po/nn-NO-staff-help.po
po/nn-NO-staff-prog.po

Additional tests:
7) Number of msgids
7.a) Before patch and after upgrade, extract and count msgids
for i in $(ls po/nn-NO-*po); \
do msginit -i $i -o nn-old.po --no-translator --no-wrap --locale=nn_NO; \
egrep ^msgid nn-old.po >> old; \
done
sort old | uniq | tee s-old | wc -l > n-old
s-old: have all msgids
n-old: number of msgids

7.b) After patch and after creation of new files
Repeat procedure, diferent files (s-new, n-new)

7.c) Compare (diff s-old snew), they are the same
(save for a strange UNIMARC char in my case, but
it's present on corresponding PO file)

8) Installed dirs/files
8.a) List of EN dirs/files
cd koha-tmpl
find | egrep "/en/" > en

8.b) List of nn-NO dirs/files. After patch and language install
cd koha-tmpl
find | egrep "/nn-NO/" | sed 's|/nn-NO/|/en/|' > nn

8.c) Compare (diff en nn), they are the same

Signed-off-by: Magnus Enger <magnus@enger.priv.no>
Followed the steps outlined by Bernardo, and everything works as
expected. I think the most important points are that "perl translate
create nn-NO" produces the right files, and translating anything in
them, then doing "translate install" makes the translations show
up in the interface. The numbers msgids in the nn-NO correspond
well wit the number of msgids in other sets of .po files.

I bet y'all will be happy when you don't have to see the stupid
Norwegian strings when you translate! ;-)

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13893: [QA Follow-up] Move print statement, sort files in loop
Marcel de Rooy [Thu, 16 Apr 2015 13:34:25 +0000 (15:34 +0200)]
Bug 13893: [QA Follow-up] Move print statement, sort files in loop

Just moving the print statement for DEV updates to show them too for
the new perl format.
In the loop opendir..readdir is used; this might not be always in the
expected order. I added a sort to force the order.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13893: Add ability to execute perl scripts in atomicupdates
Kyle M Hall [Mon, 23 Mar 2015 13:19:03 +0000 (09:19 -0400)]
Bug 13893: Add ability to execute perl scripts in atomicupdates

Bug 13068 adds the ability to execute sql files from atomicupdates for
in process developments to reduce merge conflicts. We need to extend
this ability further to be able to handle perl code.

Revised Test Plan
-----------------
1) Run updatedatabase.pl
   -- Expect output to be similar to:
   $ ./installer/data/mysql/updatedatabase.pl
   DEV atomic update : test.perl
   Upgrade to XXX done (Bug 13893 - Add ability to execute perl scripts in atomicupdates)
2) Run koha qa test tools.
   -- only need to check last commit (-c 1), because first commit is a
      sample file which shouldn't be pushed.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13984: DBRev 3.19.00.034
Tomas Cohen Arazi [Wed, 29 Apr 2015 18:23:43 +0000 (15:23 -0300)]
Bug 13984: DBRev 3.19.00.034

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13984: Add atomic update - Insert pref CardnumberLength if not exists
Jonathan Druart [Mon, 13 Apr 2015 11:30:03 +0000 (13:30 +0200)]
Bug 13984: Add atomic update - Insert pref CardnumberLength if not exists

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13984: CardnumberLength missing in sysprefs.sql
Fridolin Somers [Mon, 13 Apr 2015 11:09:16 +0000 (13:09 +0200)]
Bug 13984: CardnumberLength missing in sysprefs.sql

Test plan :
- create a fresh database with installer
- check the presence of the syspref : select value from systempreferences where variable like 'CardnumberLength'

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 14068: fix preinst for fresh package installs
Robin Sheat [Tue, 28 Apr 2015 03:19:30 +0000 (15:19 +1200)]
Bug 14068: fix preinst for fresh package installs

Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Works as expected. Tested both upgrading and on a new install.

6 years agoBug 11165: DBRev 3.19.00.033
Tomas Cohen Arazi [Wed, 29 Apr 2015 18:09:27 +0000 (15:09 -0300)]
Bug 11165: DBRev 3.19.00.033

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 11165: (RM followup) DBIx updates
Tomas Cohen Arazi [Wed, 29 Apr 2015 18:07:56 +0000 (15:07 -0300)]
Bug 11165: (RM followup) DBIx updates

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 11165: Automatically save the authority last modification time
Jonathan Druart [Thu, 11 Dec 2014 15:49:34 +0000 (16:49 +0100)]
Bug 11165: Automatically save the authority last modification time

This patch rename the column auth_header.datemodified with modification_time.
Now this column will be automatically updated when a row will be
updated.

Test plan:
0/ Verify that la major part of your auth_header.datemodified is set to
null
1/ Execute the updatedb entry
2/ Verify that the column is renamed and the values have not been
changed.
3/ Update an authority and verify (using the sql cli) that this value
has been updated.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
In the absence of a coding guideline stipulating a preferred column
name for a timestamp, this modification_time works for me.
Null values are replaced by current time after db rev; other values
are left alone.
NOTE FOR RM: Please run update_dbix_class_files.pl.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13885: (QA followup) Remove tab, remove unnecessary permission check
Kyle M Hall [Fri, 17 Apr 2015 11:31:13 +0000 (07:31 -0400)]
Bug 13885: (QA followup) Remove tab, remove unnecessary permission check

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 13885: Cataloging search header should include more options
Nick Clemens [Sat, 21 Mar 2015 14:51:24 +0000 (10:51 -0400)]
Bug 13885: Cataloging search header should include more options

This patch adds the check in and standard catalog search to cataloging-search.inc

To test:
1 - Apply patch
2 - Go to More->Cataloging
3 - Search in each tab (Cataloging search / Check out / Check in / Standard
    search) and ensure that all work as expected

Signed-off-by: Frederic Demians <f.demians@tamil.fr>
  - Added search options work properly, and are consistently named.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 1917 [QA Followup] - Don't limit the list results for RSS feeds
Kyle M Hall [Wed, 22 Apr 2015 11:36:51 +0000 (07:36 -0400)]
Bug 1917 [QA Followup] - Don't limit the list results for RSS feeds

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 1917 [QA Followup] - Remove unnecessary use of html filter
Kyle M Hall [Tue, 21 Apr 2015 13:52:49 +0000 (09:52 -0400)]
Bug 1917 [QA Followup] - Remove unnecessary use of html filter

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 1917 [QA Followup] - Use html filter, only show 'by' if author exists, change...
Kyle M Hall [Mon, 20 Apr 2015 16:09:27 +0000 (12:09 -0400)]
Bug 1917 [QA Followup] - Use html filter, only show 'by' if author exists, change link title

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 1917 - Add RSS Feeds for Lists
Kyle M Hall [Fri, 17 Apr 2015 18:02:15 +0000 (14:02 -0400)]
Bug 1917 - Add RSS Feeds for Lists

Test Plan:
1) Apply this patch
2) Create one or more public lists
3) View the list in the opac
4) Note the new RSS icon next to the list name
5) Open the link in FireFox or an RSS reader
6) You should see an RSS feed of your list with the title
   and author of each item as a hyperlink to the record details

Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Followed test plan. Works as expected.
Signed-off-by: Marc Véron <veron@veron.ch>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoRevert "Bug 8992: Use the existing greybox plugin in interface/theme"
Jonathan Druart [Wed, 29 Apr 2015 07:33:41 +0000 (09:33 +0200)]
Revert "Bug 8992: Use the existing greybox plugin in interface/theme"

This reverts commit 986b03e1c602b057275e343e3251a5cfbcc7cca6.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 4041: (QA followup) fix issues raised by koha-qa.pl
Tomas Cohen Arazi [Wed, 29 Apr 2015 15:04:07 +0000 (12:04 -0300)]
Bug 4041: (QA followup) fix issues raised by koha-qa.pl

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
6 years agoBug 4041: DBRev 3.19.00.032
Tomas Cohen Arazi [Wed, 29 Apr 2015 14:30:34 +0000 (11:30 -0300)]
Bug 4041: DBRev 3.19.00.032

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>