[koha-commits] main Koha release repository branch master updated. v3.12.00-beta1-1077-g46bbfa1
Git repo owner
gitmaster at git.koha-community.org
Sun Sep 8 21:07:09 CEST 2013
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "main Koha release repository".
The branch, master has been updated
via 46bbfa12a1d20cfaea2c61ad9bf1fdb47a25435a (commit)
via 3d8f462cce3779fd1963d623b051dbc79d2f799a (commit)
from 6434f9264ac0415e1e52904dc953a00ee0806491 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
commit 46bbfa12a1d20cfaea2c61ad9bf1fdb47a25435a
Author: Galen Charlton <gmc at esilibrary.com>
Date: Sun Sep 8 07:37:04 2013 +0000
Bug 9924: (follow-up) ensure that correct error message is displayed
(Apparently) unlike HTML::Template::Pro, Template Toolkit doesn't like
template variables that are entirely numeric -- in conditionals, it
consider them integers, most of which are Perl true.
This patch changes this by setting the error variable to the error
value.
To test:
[1] Run the test plan from the previous patch. In each
case, verify that the error message is specifically applicable
to the test. For example, if you try uplaoading a patron image
that is larger than 500KB, the error message displayed should
specifically say so.
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Thank you Galen for catching this. Error messages showing up
now are much more specific and according to the error codes given.
I testd uploading a file larger then 500KB and triggered several
error messages giving the error code in the URL:
/cgi-bin/koha/patroncards/manage.pl?card_element=profile&error=201
All tests and QA script pass.
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
commit 3d8f462cce3779fd1963d623b051dbc79d2f799a
Author: Owen Leonard <oleonard at myacpl.org>
Date: Mon Mar 25 12:22:33 2013 -0400
Bug 9924 - Simplify and rename patron card creator error message include
The patron card creator error message include uses a non-standard method
for displaying error messages, and is poorly-named.
This patch converts the method of displaying error messages for various
patron card creator options to the standard one ('<div class="dialog
alert">') and renames the include file to make it clear that it relates
only to patron card creator operations.
To test, perform various operations:
- Go to 'manage images' and try to upload a file which exceeds the
500KBfile size limit
- Go to the edit batch page and manually append an error code to the
URL: /cgi-bin/koha/patroncards/edit-batch.pl?op=new&error=403
- Go to one of the manage pages and manually append an error code to the
URL:
/cgi-bin/koha/patroncards/manage.pl?card_element=profile&error=201
Correct display of an error message indicates that the include file is
being found.
Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Passes test plan, test suite and QA script.
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
-----------------------------------------------------------------------
Summary of changes:
.../prog/en/includes/error-messages.inc | 108 --------------------
.../prog/en/includes/patroncards-errors.inc | 35 +++++++
.../prog/en/modules/patroncards/edit-batch.tt | 2 +-
.../prog/en/modules/patroncards/image-manage.tt | 2 +-
.../prog/en/modules/patroncards/manage.tt | 2 +-
patroncards/edit-batch.pl | 3 +-
patroncards/image-manage.pl | 15 +--
patroncards/manage.pl | 3 +-
8 files changed, 45 insertions(+), 125 deletions(-)
delete mode 100644 koha-tmpl/intranet-tmpl/prog/en/includes/error-messages.inc
create mode 100644 koha-tmpl/intranet-tmpl/prog/en/includes/patroncards-errors.inc
hooks/post-receive
--
main Koha release repository
More information about the koha-commits
mailing list