Go to file
Owen Leonard 7f441da181 Bug 7493 [REVISED] Deleting a record with comments breaks "Recent Comments"
Adding foreign key constraint on biblio and borrowers:

- delete reviews when bibliographic records are deleted
- set set reviews.borrowernumber to NULL when patron records are deleted.

Before these constraints are set the database update script will
clean up existing instances of these problems by deleting reviews of
deleted biblios and setting to NULL reviews.borrowernumber for deleted
patrons.

In comments moderation, the list of comments will indicate "(deleted patron")
if the review was submitted by a patron who has since been deleted.

In the OPAC display of comments will omit the patron name altogether
if the patron has since been deleted.

To test:

1. CONFIRM THAT THE DATABASE UPDATE RUNS CORRECTLY

Before applying the patch:

 - delete the record for a title which has a review
 - delete a patron who has submitted a review (on a different title).

Viewing /cgi-bin/koha/opac-showreviews.pl should result in an error.

Apply the patch and make sure the database update runs. Viewing
the recent comments page in the OPAC should no longer result in an
error. The title with the comment from a deleted patron should
show no patron name on these pages:

 - Recent comments (opac-showreviews.pl)
 - Recent comments RSS (opac-showreviews.pl?format=rss)
 - Detail page (opac-detail.pl)

Comments from other patrons should show correctly.

2. CONFIRM THAT NEW DELETIONS PERFORM CORRECTLY

After applying the patch:

 - delete the record for a title which has a review
 - delete a patron who has submitted a review (on a different title).

Viewing /cgi-bin/koha/opac-showreviews.pl should NOT result in an error.
The review of the title which you deleted should no longer appear in the
list of recent comments.

The title with the comment from a deleted patron should
show no patron name on these pages:

 - Recent comments (opac-showreviews.pl)
 - Recent comments RSS (opac-showreviews.pl?format=rss)
 - Detail page (opac-detail.pl)

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
2012-05-08 17:48:49 -04:00
C4 bug 7284: fix package build problem, move BEGIN into new 2012-05-08 17:48:48 -04:00
Koha/Template/Plugin Bug 929 : Followup fixing date formatting 2012-01-16 15:59:41 -05:00
acqui Bug 7284: Authority matching improvements 2012-05-08 17:48:47 -04:00
admin bug 7450 small follow-up, removing warn 2012-02-19 21:00:09 -05:00
authorities Bug 7284: Authority matching improvements 2012-05-08 17:48:47 -04:00
basket Bug 6050 Make calls to GetItemsInfo consistent 2011-06-14 14:12:02 +12:00
catalogue Bug 7345: Enable exporting records sans private fields 2012-02-19 21:00:19 -05:00
cataloguing Bug 7284: Authority matching improvements 2012-05-08 17:48:47 -04:00
circ bug_7201: Holds reports: add copynumber and enumchron 2012-02-19 21:00:35 -05:00
debian Revert "Bug 7238 - make packages deal with SIP config" 2012-02-21 07:50:47 -05:00
docs Adding Jorgia Kelsey and Peter Lorimer as contributors 2012-02-19 21:00:03 -05:00
errors Housekeeping in errors scripts 2010-05-12 07:29:03 -04:00
etc Bug 7284: Authority matching improvements 2012-05-08 17:48:47 -04:00
install_misc Bug 7532 - remove dependency on Date::ICal 2012-02-19 21:00:38 -05:00
installer Bug 7493 [REVISED] Deleting a record with comments breaks "Recent Comments" 2012-05-08 17:48:49 -04:00
koha-tmpl Bug 7493 [REVISED] Deleting a record with comments breaks "Recent Comments" 2012-05-08 17:48:49 -04:00
labels Bug 6679 : Adding license statement and stopping declaring variables in a conditional 2011-12-08 14:43:05 -05:00
members Bug 6718 No manager_id saved for writeoff fines. 2012-02-19 21:00:29 -05:00
misc Bug 7284: Authority matching improvements 2012-05-08 17:48:47 -04:00
offline_circ Bug 5449: JSON malformed in Koha - Blocker with jQuery 1.4.x 2011-03-12 08:53:41 +13:00
opac Revert "Bug 6875 cleaning opac-main.pl" 2012-02-22 14:43:43 -05:00
patroncards bug_6253: Unified member Search() 2011-10-19 16:23:02 +13:00
reports Bug 6984 - Holds statistics doesn't work. 2012-02-19 21:00:10 -05:00
reserve Bug 5674: patron name sort ignoring case when placing holds. 2012-02-19 21:00:13 -05:00
reviews Bug 1623 - Provide view of approved comments 2012-01-16 15:47:29 -05:00
rotating_collections Bug 6553 : Follow up adding license statements 2011-08-13 19:54:38 +12:00
selenium Adding selenium tests for filterMembers 2009-09-30 11:30:37 +02:00
serials Revert "Bug 7546 - cannot view subscription detail" 2012-02-22 14:40:41 -05:00
skel installer: fixed chown invocation; added skel for KOHA_LOG_DIR 2007-12-17 09:13:53 -06:00
sms Bug 2505 - Add commented use warnings where missing in the sms/ directory 2010-04-21 20:25:08 +12:00
suggestion Bug 6894: Order from suggestion defaulting to wrong currency 2011-12-02 15:47:24 -05:00
svc Bug 6752: Be stricter with utf-8 encoding of output 2012-02-19 15:51:19 -05:00
t bug 7284: fix package build problem, move BEGIN into new 2012-05-08 17:48:48 -04:00
tags Fix for Bug 2377 - Review tags: format date according to system preference 2010-10-06 09:31:56 -04:00
test Bug 5449: JSON malformed in Koha - Blocker with jQuery 1.4.x 2011-03-12 08:53:41 +13:00
tmp/modified_authorities changing DO_NOT_REMOVE to README.txt 2007-10-21 19:14:41 -05:00
tools Bug 7505 Superlibrarian can't export all biblios when IndependantBranches is enabled 2012-02-19 21:00:17 -05:00
virtualshelves Fix for Bug 6957, authors disappearing when emailing lists 2011-10-20 11:34:39 +13:00
xt Bug 7476 Remove executable bit from files that probably should not be executed 2012-02-19 21:00:11 -05:00
.htaccess Fix file permissions: if it is not a script, it should not be executable. 2010-04-16 00:40:34 -04:00
.mailmap Bug 7439 Follow up for mailmap 2012-01-18 10:11:06 -05:00
INSTALL Updating main INSTALL instructions 2011-10-22 09:04:19 +13:00
INSTALL.debian Updating distro specific install files 2011-10-22 09:12:15 +13:00
INSTALL.fedora7 Bug 7532 - remove dependency on Date::ICal 2012-02-19 21:00:38 -05:00
INSTALL.opensuse Bug 7356 - Fix various typos and mis-spellings 2012-01-18 10:10:53 -05:00
INSTALL.ubuntu Updating distro specific install files 2011-10-22 09:12:15 +13:00
INSTALL.ubuntu.lucid Updating distro specific install files 2011-10-22 09:12:15 +13:00
LICENSE Update LICENSE with a fresh copy from upstream. This updates the FSF address, and refers to the LGPL with its current name, and changes a few other minor things of the typographical sort. No semantic changes. 2010-03-16 20:17:48 -04:00
MANIFEST.SKIP Bug Fixing : 3334 2009-06-19 06:33:34 -05:00
Makefile.PL Bug 7359 - Begin migration to a new "Koha" namespace from the old "C4" namespace 2012-02-19 15:54:00 -05:00
README updated links in README 2010-05-24 08:14:16 -04:00
README.robots Bug 6411 add another example to README.robots 2011-07-05 14:48:05 +12:00
about.pl Bug 7164 follow-up history.txt and perlcritic compliance 2011-12-14 12:44:39 -05:00
changelanguage.pl Bug 6755 Problems with switching languages 2011-09-23 09:47:09 +12:00
edithelp.pl Bug 7038 Contextual help is always in English 2011-10-18 16:01:25 +13:00
fix-perl-path.PL installer: improvements to fix-path-perl.PL on Win32 2007-12-20 19:20:12 -06:00
help.pl Bug 6628 : Stopping a potential vulnerability 2011-11-28 20:11:39 -05:00
install-CPAN.pl Bug 5370: Fix all the references to koha.org 2010-11-08 09:41:49 +13:00
koha_perl_deps.pl Bug 6914 - fixes for the color option of koha_perl_deps.pl 2011-12-03 07:13:03 -05:00
kohaversion.pl Bug 7284 follow-up: Update DBRev 2012-05-08 17:48:48 -04:00
mainpage.pl Revert "Bug 6875 cleaning mainpage.pl" 2012-02-22 14:44:06 -05:00
rewrite-config.PL Bug 5370: Fix all the references to koha.org 2010-11-08 09:41:49 +13:00

README

Koha is a free software integrated library system.

Koha is distributed under the GNU GPL version 2 or later.
Please read the file LICENSE for more details.

To install or upgrade Koha, please see the INSTALL file appropriate
to your platform.

Report bugs at http://bugs.koha-community.org/

Visit the Koha Project website at http://www.koha-community.org/