View | Details | Raw Unified | Return to bug 24718
Collapse All | Expand All

(-)a/installer/data/mysql/atomicupdate/bug_24718_-_use_datetime_for_reserves.perl (+13 lines)
Line 0 Link Here
1
$DBversion = 'XXX';
2
if( CheckVersion( $DBversion ) ) {
3
    if( column_exists( 'reserves', 'reservedate' ) ) {
4
        $dbh->do(q{ALTER TABLE reserves MODIFY column reservedate datetime});
5
    }
6
7
    if( column_exists( 'reserves', 'expirationdate' ) ) {
8
        $dbh->do(q{ALTER TABLE reserves MODIFY column expirationdate datetime});
9
    }
10
11
    SetVersion( $DBversion );
12
    print "Upgrade to $DBversion done (Bug 24718 - Use datetime for reserves.reservedate and reserves.expirationdate)\n";
13
}
(-)a/installer/data/mysql/kohastructure.sql (-5 / +4 lines)
Lines 4471-4477 DROP TABLE IF EXISTS `old_reserves`; Link Here
4471
CREATE TABLE `old_reserves` (
4471
CREATE TABLE `old_reserves` (
4472
  `reserve_id` int(11) NOT NULL COMMENT 'primary key',
4472
  `reserve_id` int(11) NOT NULL COMMENT 'primary key',
4473
  `borrowernumber` int(11) DEFAULT NULL COMMENT 'foreign key from the borrowers table defining which patron this hold is for',
4473
  `borrowernumber` int(11) DEFAULT NULL COMMENT 'foreign key from the borrowers table defining which patron this hold is for',
4474
  `reservedate` date DEFAULT NULL COMMENT 'the date the hold was places',
4474
  `reservedate` datetime DEFAULT NULL COMMENT 'the date the hold was places',
4475
  `biblionumber` int(11) DEFAULT NULL COMMENT 'foreign key from the biblio table defining which bib record this hold is on',
4475
  `biblionumber` int(11) DEFAULT NULL COMMENT 'foreign key from the biblio table defining which bib record this hold is on',
4476
  `item_group_id` int(11) DEFAULT NULL COMMENT 'foreign key from the item_groups table defining if this is an item group level hold',
4476
  `item_group_id` int(11) DEFAULT NULL COMMENT 'foreign key from the item_groups table defining if this is an item group level hold',
4477
  `branchcode` varchar(10) DEFAULT NULL COMMENT 'foreign key from the branches table defining which branch the patron wishes to pick this hold up at',
4477
  `branchcode` varchar(10) DEFAULT NULL COMMENT 'foreign key from the branches table defining which branch the patron wishes to pick this hold up at',
Lines 4486-4492 CREATE TABLE `old_reserves` ( Link Here
4486
  `timestamp` timestamp NOT NULL DEFAULT current_timestamp() ON UPDATE current_timestamp() COMMENT 'the date and time this hold was last updated',
4486
  `timestamp` timestamp NOT NULL DEFAULT current_timestamp() ON UPDATE current_timestamp() COMMENT 'the date and time this hold was last updated',
4487
  `itemnumber` int(11) DEFAULT NULL COMMENT 'foreign key from the items table defining the specific item the patron has placed on hold or the item this hold was filled with',
4487
  `itemnumber` int(11) DEFAULT NULL COMMENT 'foreign key from the items table defining the specific item the patron has placed on hold or the item this hold was filled with',
4488
  `waitingdate` date DEFAULT NULL COMMENT 'the date the item was marked as waiting for the patron at the library',
4488
  `waitingdate` date DEFAULT NULL COMMENT 'the date the item was marked as waiting for the patron at the library',
4489
  `expirationdate` date DEFAULT NULL COMMENT 'the date the hold expires (usually the date entered by the patron to say they don''t need the hold after a certain date)',
4489
  `expirationdate` datetime DEFAULT NULL COMMENT 'the date the hold expires (usually the date entered by the patron to say they don''t need the hold after a certain date)',
4490
  `patron_expiration_date` date DEFAULT NULL COMMENT 'the date the hold expires - usually the date entered by the patron to say they don''t need the hold after a certain date',
4490
  `patron_expiration_date` date DEFAULT NULL COMMENT 'the date the hold expires - usually the date entered by the patron to say they don''t need the hold after a certain date',
4491
  `lowestPriority` tinyint(1) NOT NULL DEFAULT 0 COMMENT 'has this hold been pinned to the lowest priority in the holds queue (1 for yes, 0 for no)',
4491
  `lowestPriority` tinyint(1) NOT NULL DEFAULT 0 COMMENT 'has this hold been pinned to the lowest priority in the holds queue (1 for yes, 0 for no)',
4492
  `suspend` tinyint(1) NOT NULL DEFAULT 0 COMMENT 'in this hold suspended (1 for yes, 0 for no)',
4492
  `suspend` tinyint(1) NOT NULL DEFAULT 0 COMMENT 'in this hold suspended (1 for yes, 0 for no)',
Lines 4920-4926 DROP TABLE IF EXISTS `reserves`; Link Here
4920
CREATE TABLE `reserves` (
4920
CREATE TABLE `reserves` (
4921
  `reserve_id` int(11) NOT NULL AUTO_INCREMENT COMMENT 'primary key',
4921
  `reserve_id` int(11) NOT NULL AUTO_INCREMENT COMMENT 'primary key',
4922
  `borrowernumber` int(11) NOT NULL DEFAULT 0 COMMENT 'foreign key from the borrowers table defining which patron this hold is for',
4922
  `borrowernumber` int(11) NOT NULL DEFAULT 0 COMMENT 'foreign key from the borrowers table defining which patron this hold is for',
4923
  `reservedate` date DEFAULT NULL COMMENT 'the date the hold was placed',
4923
  `reservedate` datetime DEFAULT NULL COMMENT 'the date the hold was placed',
4924
  `biblionumber` int(11) NOT NULL DEFAULT 0 COMMENT 'foreign key from the biblio table defining which bib record this hold is on',
4924
  `biblionumber` int(11) NOT NULL DEFAULT 0 COMMENT 'foreign key from the biblio table defining which bib record this hold is on',
4925
  `item_group_id` int(11) DEFAULT NULL COMMENT 'foreign key from the item_groups table defining if this is an item group level hold',
4925
  `item_group_id` int(11) DEFAULT NULL COMMENT 'foreign key from the item_groups table defining if this is an item group level hold',
4926
  `branchcode` varchar(10) NOT NULL COMMENT 'foreign key from the branches table defining which branch the patron wishes to pick this hold up at',
4926
  `branchcode` varchar(10) NOT NULL COMMENT 'foreign key from the branches table defining which branch the patron wishes to pick this hold up at',
Lines 4935-4941 CREATE TABLE `reserves` ( Link Here
4935
  `timestamp` timestamp NOT NULL DEFAULT current_timestamp() ON UPDATE current_timestamp() COMMENT 'the date and time this hold was last updated',
4935
  `timestamp` timestamp NOT NULL DEFAULT current_timestamp() ON UPDATE current_timestamp() COMMENT 'the date and time this hold was last updated',
4936
  `itemnumber` int(11) DEFAULT NULL COMMENT 'foreign key from the items table defining the specific item the patron has placed on hold or the item this hold was filled with',
4936
  `itemnumber` int(11) DEFAULT NULL COMMENT 'foreign key from the items table defining the specific item the patron has placed on hold or the item this hold was filled with',
4937
  `waitingdate` date DEFAULT NULL COMMENT 'the date the item was marked as waiting for the patron at the library',
4937
  `waitingdate` date DEFAULT NULL COMMENT 'the date the item was marked as waiting for the patron at the library',
4938
  `expirationdate` date DEFAULT NULL COMMENT 'the date the hold expires (calculated value)',
4938
  `expirationdate` datetime DEFAULT NULL COMMENT 'the date the hold expires (calculated value)',
4939
  `patron_expiration_date` date DEFAULT NULL COMMENT 'the date the hold expires - usually the date entered by the patron to say they don''t need the hold after a certain date',
4939
  `patron_expiration_date` date DEFAULT NULL COMMENT 'the date the hold expires - usually the date entered by the patron to say they don''t need the hold after a certain date',
4940
  `lowestPriority` tinyint(1) NOT NULL DEFAULT 0,
4940
  `lowestPriority` tinyint(1) NOT NULL DEFAULT 0,
4941
  `suspend` tinyint(1) NOT NULL DEFAULT 0,
4941
  `suspend` tinyint(1) NOT NULL DEFAULT 0,
4942
- 

Return to bug 24718