The ReadingHistory sytem preference is no longer used, and possibly
was used by only one library anyway.  Removed references to it; note
that had it been turned on, a patron viewing checkout history would have
seen old loans showing up twice.

Signed-off-by: Galen Charlton <[email protected]>
---
 C4/Members.pm                                      |   28 --------------------
 admin/systempreferences.pl                         |    1 -
 installer/data/mysql/en/mandatory/sysprefs.sql     |    2 +-
 .../1-Obligatoire/unimarc_standard_systemprefs.sql |    1 -
 installer/data/mysql/pl-PL/mandatory/sysprefs.sql  |    2 +-
 ...m_preferences_full_optimal_for_install_only.sql |    5 +---
 ...m_preferences_full_optimal_for_install_only.sql |    5 +---
 7 files changed, 4 insertions(+), 40 deletions(-)

diff --git a/C4/Members.pm b/C4/Members.pm
index 4326af9..2395b72 100644
--- a/C4/Members.pm
+++ b/C4/Members.pm
@@ -1059,7 +1059,6 @@ sub GetAllIssues {
         $query .= " limit $limit";
     }
 
-    #print $query;
     my $sth = $dbh->prepare($query);
     $sth->execute($borrowernumber, $borrowernumber);
     my @result;
@@ -1068,33 +1067,6 @@ sub GetAllIssues {
         push @result, $data;
     }
 
-    # get all issued items for borrowernumber from oldissues table
-    # large chunk of older issues data put into table oldissues
-    # to speed up db calls for issuing items
-    if ( C4::Context->preference("ReadingHistory") ) {
-        # FIXME oldissues (not to be confused with old_issues) is
-        # apparently specific to HLT.  Not sure if the ReadingHistory
-        # syspref is still required, as old_issues by design
-        # is no longer checked with each loan.
-        my $query2 = "SELECT * FROM old_issues
-                      LEFT JOIN items ON items.itemnumber=old_issues.itemnumber
-                      LEFT JOIN biblio ON 
items.biblionumber=biblio.biblionumber
-                      LEFT JOIN biblioitems ON 
items.biblioitemnumber=biblioitems.biblioitemnumber
-                      WHERE borrowernumber=? 
-                      ORDER BY $order";
-        if ( $limit != 0 ) {
-            $limit = $limit - scalar(@result);
-            $query2 .= " limit $limit";
-        }
-
-        my $sth2 = $dbh->prepare($query2);
-        $sth2->execute($borrowernumber);
-
-        while ( my $data2 = $sth2->fetchrow_hashref ) {
-            push @result, $data2;
-        }
-    }
-
     return \...@result;
 }
 
diff --git a/admin/systempreferences.pl b/admin/systempreferences.pl
index 05ffd7b..6739cd9 100755
--- a/admin/systempreferences.pl
+++ b/admin/systempreferences.pl
@@ -199,7 +199,6 @@ $tabsysprefs{checkdigit}                   = "Patrons";
 $tabsysprefs{intranetreadinghistory}       = "Patrons";
 $tabsysprefs{NotifyBorrowerDeparture}      = "Patrons";
 $tabsysprefs{memberofinstitution}          = "Patrons";
-$tabsysprefs{ReadingHistory}               = "Patrons";
 $tabsysprefs{BorrowerMandatoryField}       = "Patrons";
 $tabsysprefs{borrowerRelationship}         = "Patrons";
 $tabsysprefs{BorrowersTitles}              = "Patrons";
diff --git a/installer/data/mysql/en/mandatory/sysprefs.sql 
b/installer/data/mysql/en/mandatory/sysprefs.sql
index 29e904c..a9d437b 100644
--- a/installer/data/mysql/en/mandatory/sysprefs.sql
+++ b/installer/data/mysql/en/mandatory/sysprefs.sql
@@ -220,7 +220,7 @@ INSERT INTO `systempreferences` 
(variable,value,options,explanation,type) VALUES
 INSERT INTO `systempreferences` (variable,value,options,explanation,type) 
VALUES('AllowOnShelfHolds', '0', '', 'Allow hold requests to be placed on items 
that are not on loan', 'YesNo');
 INSERT INTO `systempreferences` (variable,value,options,explanation,type) 
VALUES('AllowHoldsOnDamagedItems', '1', '', 'Allow hold requests to be placed 
on damaged items', 'YesNo');
 INSERT INTO `systempreferences` (variable,value,options,explanation,type) 
VALUES('OpacSuppression', '0', '', 'Turn ON the OPAC Suppression feature, 
requires further setup, ask your system administrator for details', 'YesNo');
--- FIXME: add FrameworksLoaded, noOPACUserLogin, ReadingHistory ?
+-- FIXME: add FrameworksLoaded, noOPACUserLogin?
 INSERT INTO `systempreferences` (variable,value,options,explanation,type) 
VALUES ('SMSSendDriver','','','Sets which SMS::Send driver is used to send SMS 
messages.','free');
 INSERT INTO `systempreferences` (variable,value,options,explanation,type) 
VALUES ('OrderPdfFormat','pdfformat::layout3pages','Controls what script is 
used for printing (basketgroups)','','free');
 INSERT INTO systempreferences (variable,value,explanation,options,type) VALUES 
('OrderPdfTemplate','','Uploads a PDF template to use for printing 
baskets','NULL','Upload');
diff --git 
a/installer/data/mysql/fr-FR/1-Obligatoire/unimarc_standard_systemprefs.sql 
b/installer/data/mysql/fr-FR/1-Obligatoire/unimarc_standard_systemprefs.sql
index e3ce7f3..8095198 100644
--- a/installer/data/mysql/fr-FR/1-Obligatoire/unimarc_standard_systemprefs.sql
+++ b/installer/data/mysql/fr-FR/1-Obligatoire/unimarc_standard_systemprefs.sql
@@ -106,7 +106,6 @@ INSERT INTO `systempreferences` 
(variable,value,explanation,options,type) VALUES
 INSERT INTO `systempreferences` (variable,value,explanation,options,type) 
VALUES('opacuserlogin', '1', 'si ce paramètre est activé, les adhérents 
peuvent s''identifier à l''OPAC. Sinon seule la consultation anonyme est 
possible', '', 'YesNo');
 INSERT INTO `systempreferences` (variable,value,explanation,options,type) 
VALUES('patronimages', 'jpg', 'Ce paramètre permet d''activer la gestion des 
photos des adhérents. Mettre une extension d''image (jpg) pour activer la 
chose', '', 'free');
 INSERT INTO `systempreferences` (variable,value,explanation,options,type) 
VALUES('printcirculationslips', '1', 'Active ou non l''impression de tickets de 
circulation', '', 'free');
-INSERT INTO `systempreferences` (variable,value,explanation,options,type) 
VALUES('ReadingHistory', '0', 'Active ou non l''affichage de l''historique de 
lecture ', NULL, 'YesNo');
 INSERT INTO `systempreferences` (variable,value,explanation,options,type) 
VALUES('ReceiveBackIssues', '5', 'Ce paramètre définit le nombre d''anciens 
bulletins à afficher lorsque l''on bulletine', '', '');
 INSERT INTO `systempreferences` (variable,value,explanation,options,type) 
VALUES('RequestOnOpac', '1', 'Active ou non les réservations à l''OPAC', '', 
'YesNo');
 INSERT INTO `systempreferences` (variable,value,explanation,options,type) 
VALUES('ReservesMaxPickUpDelay', '10', 'Délai maximum pour venir chercher un 
document réservé et mis de coté', '', 'Integer');
diff --git a/installer/data/mysql/pl-PL/mandatory/sysprefs.sql 
b/installer/data/mysql/pl-PL/mandatory/sysprefs.sql
index 77222c6..c6a8a41 100755
--- a/installer/data/mysql/pl-PL/mandatory/sysprefs.sql
+++ b/installer/data/mysql/pl-PL/mandatory/sysprefs.sql
@@ -219,7 +219,7 @@ INSERT INTO `systempreferences` 
(variable,value,options,explanation,type) VALUES
 INSERT INTO `systempreferences` (variable,value,options,explanation,type) 
VALUES('AllowOnShelfHolds', '0', '', 'Allow hold requests to be placed on items 
that are not on loan', 'YesNo');
 INSERT INTO `systempreferences` (variable,value,options,explanation,type) 
VALUES('AllowHoldsOnDamagedItems', '1', '', 'Allow hold requests to be placed 
on damaged items', 'YesNo');
 INSERT INTO `systempreferences` (variable,value,options,explanation,type) 
VALUES('OpacSuppression', '0', '', 'Turn ON the OPAC Suppression feature, 
requires further setup, ask your system administrator for details', 'YesNo');
--- FIXME: add FrameworksLoaded, noOPACUserLogin, ReadingHistory ?
+-- FIXME: add FrameworksLoaded, noOPACUserLogin?
 INSERT INTO `systempreferences` (variable,value,options,explanation,type) 
VALUES ('SMSSendDriver','','','Sets which SMS::Send driver is used to send SMS 
messages.','free');
 INSERT INTO `systempreferences` (variable,value,options,explanation,type) 
VALUES ('OrderPdfFormat','pdfformat::layout3pages','Controls what script is 
used for printing (basketgroups)','','free');
 INSERT INTO systempreferences (variable,value,explanation,options,type) VALUES 
('OrderPdfTemplate','','Uploads a PDF template to use for printing 
baskets','NULL','Upload');
diff --git 
a/installer/data/mysql/ru-RU/mandatory/system_preferences_full_optimal_for_install_only.sql
 
b/installer/data/mysql/ru-RU/mandatory/system_preferences_full_optimal_for_install_only.sql
index 5ad68b4..d6ee529 100755
--- 
a/installer/data/mysql/ru-RU/mandatory/system_preferences_full_optimal_for_install_only.sql
+++ 
b/installer/data/mysql/ru-RU/mandatory/system_preferences_full_optimal_for_install_only.sql
@@ -247,7 +247,7 @@ INSERT INTO `systempreferences` 
(variable,value,options,explanation,type) VALUES
 INSERT INTO `systempreferences` (variable,value,options,explanation,type) 
VALUES('AllowOnShelfHolds', '0', '', 'Allow hold requests to be placed on items 
that are not on loan', 'YesNo');
 INSERT INTO `systempreferences` (variable,value,options,explanation,type) 
VALUES('AllowHoldsOnDamagedItems', '1', '', 'Allow hold requests to be placed 
on damaged items', 'YesNo');
 INSERT INTO `systempreferences` (variable,value,options,explanation,type) 
VALUES('OpacSuppression', '0', '', 'Turn ON the OPAC Suppression feature, 
requires further setup, ask your system administrator for details', 'YesNo');
--- FIXME: add FrameworksLoaded, noOPACUserLogin, ReadingHistory ?
+-- FIXME: add FrameworksLoaded, noOPACUserLogin?
 INSERT INTO `systempreferences` (variable,value,options,explanation,type) 
VALUES ('SMSSendDriver','','','Sets which SMS::Send driver is used to send SMS 
messages.','free');
 INSERT INTO systempreferences (variable,value,explanation,options,type) 
VALUES('AllowRenewalLimitOverride', '0', 'if ON, allows renewal limits to be 
overridden on the circulation screen',NULL,'YesNo');
 INSERT INTO systempreferences (variable,value,explanation,options,type) 
VALUES('MergeAuthoritiesOnUpdate', '1', 'if ON, Updating authorities will 
automatically updates biblios',NULL,'YesNo');
@@ -272,9 +272,6 @@ INSERT INTO systempreferences 
(variable,value,explanation,options,type) VALUES (
 INSERT INTO systempreferences (variable,value,explanation,options,type) VALUES 
('OPACAmazonCoverImages', '0', 'Display cover images on OPAC from Amazon Web 
Services','','YesNo');
 --from 3.00.02.007
 
--- INSERT INTO `systempreferences` (variable,value,explanation,options,type) 
VALUES('ReadingHistory', '0', 'If on display reading history for user', NULL, 
'YesNo');
--- Removed unused ReadingHistory sypref
-
 INSERT INTO `systempreferences` (variable,value,explanation,options,type) 
VALUES('ReceiveBackIssues', '5', 'Number of Previous journals to display when 
on subscription detail', '', '');
 
 INSERT INTO systempreferences (variable,value,explanation,options,type) 
VALUES('AllowHoldPolicyOverride', '0', 'Allow staff to override hold policies 
when placing holds',NULL,'YesNo');
diff --git 
a/installer/data/mysql/uk-UA/mandatory/system_preferences_full_optimal_for_install_only.sql
 
b/installer/data/mysql/uk-UA/mandatory/system_preferences_full_optimal_for_install_only.sql
index cb6f96e..e22efcf 100755
--- 
a/installer/data/mysql/uk-UA/mandatory/system_preferences_full_optimal_for_install_only.sql
+++ 
b/installer/data/mysql/uk-UA/mandatory/system_preferences_full_optimal_for_install_only.sql
@@ -273,7 +273,7 @@ INSERT INTO `systempreferences` 
(variable,value,options,explanation,type) VALUES
 INSERT INTO `systempreferences` (variable,value,options,explanation,type) 
VALUES('AllowOnShelfHolds', '0', '', 'Allow hold requests to be placed on items 
that are not on loan', 'YesNo');
 INSERT INTO `systempreferences` (variable,value,options,explanation,type) 
VALUES('AllowHoldsOnDamagedItems', '1', '', 'Allow hold requests to be placed 
on damaged items', 'YesNo');
 INSERT INTO `systempreferences` (variable,value,options,explanation,type) 
VALUES('OpacSuppression', '0', '', 'Turn ON the OPAC Suppression feature, 
requires further setup, ask your system administrator for details', 'YesNo');
--- FIXME: add FrameworksLoaded, noOPACUserLogin, ReadingHistory ?
+-- FIXME: add FrameworksLoaded, noOPACUserLogin?
 INSERT INTO `systempreferences` (variable,value,options,explanation,type) 
VALUES ('SMSSendDriver','','','Sets which SMS::Send driver is used to send SMS 
messages.','free');
 INSERT INTO systempreferences (variable,value,explanation,options,type) 
VALUES('AllowRenewalLimitOverride', '0', 'if ON, allows renewal limits to be 
overridden on the circulation screen',NULL,'YesNo');
 INSERT INTO systempreferences (variable,value,explanation,options,type) 
VALUES('MergeAuthoritiesOnUpdate', '1', 'if ON, Updating authorities will 
automatically updates biblios',NULL,'YesNo');
@@ -298,9 +298,6 @@ INSERT INTO systempreferences 
(variable,value,explanation,options,type) VALUES (
 INSERT INTO systempreferences (variable,value,explanation,options,type) VALUES 
('OPACAmazonCoverImages', '0', 'Display cover images on OPAC from Amazon Web 
Services','','YesNo');
 --from 3.00.02.007
 
--- INSERT INTO `systempreferences` (variable,value,explanation,options,type) 
VALUES('ReadingHistory', '0', 'If on display reading history for user', NULL, 
'YesNo');
--- Removed unused ReadingHistory sypref
-
 INSERT INTO `systempreferences` (variable,value,explanation,options,type) 
VALUES('ReceiveBackIssues', '5', 'Number of Previous journals to display when 
on subscription detail', '', '');
 
 INSERT INTO systempreferences (variable,value,explanation,options,type) 
VALUES('AllowHoldPolicyOverride', '0', 'Allow staff to override hold policies 
when placing holds',NULL,'YesNo');
-- 
1.6.5

_______________________________________________
Koha-patches mailing list
[email protected]
http://lists.koha.org/mailman/listinfo/koha-patches

Reply via email to