Commit graph

355 commits

Author SHA1 Message Date
0700a8b732 bug 4896: granular permissions now always on (DB rev 138)
Per the following koha-devel thread, the use of
staff user subpermissions, AKA granular permissions, is
now the default behavior in Koha.  This patch removes
the GranularPermissions system preference.

[1] http://lists.koha-community.org/pipermail/koha-devel/2010-February/033670.html

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2010-06-19 07:56:03 -04:00
340f8f0725 bug 4508: fix crash when editing patron attributes or message prefs
When editing patron attributes or messaging preferences on the
patron editor, Koha would crash with the following error:

Software error:
Can't use an undefined value as an ARRAY reference at
/home/koha/kohaclone/C4/SQLHelper.pm line 186.

This would occur only when editing the specific attribute
or message preferences section, not when editing the entire
patron record.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2010-05-26 10:52:44 -04:00
ef58c445d3 removed needless imports of the YAML module
Removed instances of 'use YAML' that were either completely
unnecessary or which were used only in debug code.  Also
removed a needless import of Data::Dumper.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2010-05-25 23:30:48 -04:00
Matthias Meusburger
6710641eee MT2582: Fix user deletion without permission
Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2010-05-11 08:39:19 -04:00
257a179c3e Bug 4470 Patron search result pagination bar
Obviously, Koha global pagination function could be improved. This patch
do something which belongs to this function. A CPAN module like
Data::Page could help or be an inspiration... 3.4?

There is also room for improvement in they way resultset are
constructed. The whole set is retrieved and just a subset is displayed.
It impacts performances for libraries with a great number of patrons.
The right solution would be:

  - to find the resultset size with SELECT COUNT(*)
  - to retrieved the exact subset with LIMIT and OFFSET SQL arguments

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2010-05-11 06:09:36 -04:00
Kyle M Hall
3a5b075c90 Fixed Patron Search Results
By default, the patron search was pulling results in *Descending* order,
then reordering the results on the page using javascript in *Ascending* order.

This was causing screwy results where the entire list of results of multiple
pages was not in complete alphabetical order.

Fixed by changing the SQL to pull in Ascending order.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2010-05-05 08:57:04 -04:00
Colin Campbell
cb031bace9 Bug 2889 Remove toggle in boraccount display
Also remove the reverse column if no contents to be displayed
( otherwise looks like a markup error )
Rationalize the main loop now toggle not required

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2010-05-03 09:20:50 -04:00
Matthias Meusburger
e4a6abbf1a MT3440: Display only borrower's messages in borrower's profile
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2010-05-01 09:11:44 -04:00
Kyle M Hall
db8bda86ea Search Patrons - Make patron search more forgiving.
Patron search has only been searching fields for names
that either match or begin with the given search terms.

This is too strict a search.
For example, take the name "Billie Jo Robinson", "Billie Jo"
being the firstname, and "Robinson" being the last name. If one
were to search for "Billie Jo", this patron would not appear in
the search results.

 To remedy this, the search
type has been changed from 'begins_with' to 'contain'.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2010-04-29 22:45:29 -04:00
Colin Campbell
e7bdbd0583 Bug 4248 Rework Cities dropdown for members entry
Garry Collum's patch correctly identified the core flaw in operation
looking closer revealed some other flaws (2 "blank" options, some vars
and documentation belonging to a superceded version, etc.
Changed GetCities to return a structure rather than the messy two returns
Let the template do the concatenation and removed the CGI::Popup

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2010-04-29 22:20:36 -04:00
Henri-Damien LAURENT
c1f91af476 MT3212 : members modification wipeout dateexpiry
Members modification when not in Library information mode would wipe out dateexpiry information

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2010-04-29 21:33:51 -04:00
Donovan Jones
0718cc8b6d Bug 2505 - Add commented use warnings where missing in the members/ directory 2010-04-21 20:19:50 +12:00
d20305cca1 tweak display of issuing library on staff reading history
Following idea in patch submitted by Kyle Hall on 1 April 2010,
use issues.branchcode instead of issues.issuingbranch to set
displayed issuing library.  Also tweaked wording.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2010-04-07 10:57:47 -04:00
Paul Poulain
b7ce86240e [MT3191] follow-up, removing warn
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2010-04-07 09:57:11 -04:00
Paul Poulain
c6e04e8b40 [MT3191] various perl errors in script (merge conflict)
and SQL error solved

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2010-04-07 09:57:03 -04:00
Jean-André Santoni
732e1f954d [MT2195] Display issuing branch in reading record
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2010-04-07 09:56:54 -04:00
Nahuel ANGELINETTI
c5c83ed8d7 (MT #2920) fix reading record scripts
This patch fix some perl ugly code to be more readable and lesser complex.
And it allow in intranet to order by issues.timestamp.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2010-04-07 09:56:33 -04:00
Jean-André Santoni
576e544d02 [MT2358] Display issue date in member reading record
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2010-04-07 09:56:23 -04:00
Lars Wirzenius
e8df566734 Fix FSF address in directory members/
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2010-03-16 20:17:55 -04:00
Chris Nighswonger
7e3e572436 Enhancement: This patch adds a 'cardnumber' parameter and checks
to see if it is present when the script is called. If it is, the patron
information is retrieved and borrowernumber set from that. Otherwise,
the patron information is retrieved by borrowernumber.

The changes in this patch do not require porting to any other Koha code.

Sponsored by www.digital-loom.com

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2010-02-02 18:47:21 -05:00
Henri-Damien LAURENT
e154413382 Revert "MT2232 : members/notices.pl and members/messaging.pl are doing the same thing"
This reverts commit 653de3c295b5b21ddfe11f0597b4ba7092f1effc.
Should remove members/notices.pl instead
2010-01-28 15:11:57 +01:00
Henri-Damien LAURENT
1e1c87a1af (followup MT2690) search on first letter
followup commit 5a669826fa8e6cfbf2c53aa40271e0a2f8465a6c
letters were not introduced in first page
2010-01-28 15:11:57 +01:00
Nahuel ANGELINETTI
d639b9b79d (MT2690) fix search by letter
This patch fix searching by name that "begins" by one letter, using now the quicksearch arg.
And improve a bit the template, now it won't list all the alphabet...
2010-01-28 15:11:55 +01:00
Nahuel ANGELINETTI
ed9d4873d4 Fix ordering in member search
This fix the default ordering when the librarian browser by "last name". It must be ordered by "family name".
2010-01-28 15:11:55 +01:00
Henri-Damien LAURENT
7540dda2eb Bug Fixing Search By initials incorrect
Adding a research scope : field_start_with
Using this kind of research if searching by initial
2010-01-28 15:11:55 +01:00
Henri-Damien LAURENT
6a3579b2f3 MT 2285 : enhancement for patronSearch 2010-01-28 15:11:51 +01:00
Nahuel ANGELINETTI
05b61e4f3a (bug #3745) fix deletion of borrowers rights detection
This patch fix the rights checking for borrower deletion.
2010-01-28 15:11:49 +01:00
Matthias Meusburger
20df800fae MT2232 : members/notices.pl and members/messaging.pl are doing the same thing 2010-01-28 15:11:45 +01:00
Nicole Engard
3c832b0375 Bug 2662: make rand password consider minPasswordLength pref 2009-12-22 00:01:03 +01:00
Henri-Damien LAURENT
151ea5a06b Adding userid to search in member.pl 2009-11-06 12:37:01 +01:00
Henri-Damien LAURENT
ce8c3275a6 Removing default selection of userbranch for research 2009-11-06 12:33:15 +01:00
Henri-Damien LAURENT
a2d34166d6 Improving patron search in serials routing
Adding category and library search
Unifying with central member search
2009-10-29 11:56:08 +01:00
Henri-Damien LAURENT
3c8607b0e1 MT 1716 : Followup : search by initial was broken
Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
2009-10-07 23:27:54 +02:00
Henri-Damien LAURENT
cf4ad3ba64 Bug Fixing : New Lists Nothing in categorycode
Problem with New when PatronList categorycode
2009-10-07 17:14:33 +02:00
Henri-Damien LAURENT
6a6f65c41c Upgrading C4/Members in regards with C4/SQLHelper
Adding cardnumber as a search criteria
2009-09-30 11:30:37 +02:00
Henri-Damien LAURENT
9d9d933d3e Improving and Bug Fixing Membersearch
Now search can take borrowertype and branchcode
search is correctly positioned in page
2009-09-30 11:30:37 +02:00
Henri-Damien LAURENT
ff9319aea6 Adding More filters for member.pl page :
Adding Search Function on Members which is a search on all the fields in Members
Adding search.tmpl which adds a filter possiblity on Member page
Adding the management and the use of Search function in C4::Members
2009-09-30 11:30:34 +02:00
Henri-Damien LAURENT
4271bbb738 Modifying Members : Add Mod and GetMember
This update the way Member is added and editing so that import and Edition
 could be best automatized
GetMember evolves and allow ppl to serach on a hash of data

Adding SQLHelper A new package to deal with INSERT UPDATE and SELECT

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2009-09-30 11:29:23 +02:00
Henri-Damien LAURENT
984e197e5d Adding notices.pl which enables staff to see what notices have been sent to user
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2009-09-30 11:22:23 +02:00
Garry Collum
023ff5181b Bug 2505: Enabled warnings in members/messaging.pl and readingrec.pl.
Fixed any warnings that appeared in the logs.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2009-09-29 19:11:57 -04:00
Garry Collum
aae0efd051 Bug 2505: Enabled warnings in members/patronimage.pl and setdebar.pl.
Fixed resulting warnings after warnings was enabled.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2009-09-29 19:11:41 -04:00
Garry Collum
7f25142410 Bug 2505: Added warnings to members/members-home.pl and members/setstatus.pl
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2009-09-29 19:11:34 -04:00
Garry Collum
e033a41f06 Bug 2505: Enables warnings in members/mancredit.pl and maninvoice.pl
Fixes resulting warnings.

Also corrects a couple of minor typos.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2009-09-29 19:11:25 -04:00
Garry Collum
832e9abf64 Bug 2505: Enables warnings in member-flags.pl, member-password.pl, and member-picupload.pl.
No warnings were generated in testing after warnings were enabled.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2009-09-29 19:11:08 -04:00
PTFS
c84b638d01 Add predefined fee types
Ability in system administration to define additional fee and fine options that appear in dropdown box
on the create manual invoice tab in patron fines.

To use create authorised values with the category MANUAL_INV,
Authorized Value is the description, and Description is the default fee, if any.

This work sponsered by East Brunswick Public Library, East Brunswick, NJ, USA

Signed-off-by: Colin Campbell <colin.campbell@ptfs-europe.com>
2009-09-16 07:25:52 -04:00
e50bc421c4 Changes to make checkouts/holds display the same on circulation and patron detail screens (Bug 3106)
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2009-09-06 14:37:10 -04:00
Katrin Fischer
c36098e316 Bug 3464: Added altcontactcountry and B_address2 to borrowers and deletedborrowers
- added altcontactcountry and B_address2 to tables borrowers and deletedborrowers so that all addresses offer the same fields
- changed B_country and country to work with syspref BorrowerMandatoryField
- changed display of Alternate Address on borrower detail page, added labels, lines and edit button to match the other sections

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2009-09-06 10:01:51 -04:00
Garry Collum
d3a8f77011 Bug 2505: Enabled warnings in boraccount.pl and pay.pl.
Enabled warnings in members/boraccount.pl and members/pay.pl.  Fixed resulting warnings.
2009-09-04 15:17:39 -04:00
82f4badf51 bug 3464 followup - fix glitches
* Fixed 'counry' typo
* added country and B_country to deleteborrowers

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2009-08-23 18:38:56 -04:00
Sébastien Hinderer
f1f833c965 bug 3464: Takes columns country and B_country of table borrowers into account in related operations.
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
2009-08-23 18:19:20 -04:00