[Koha-cvs] koha/updater updatedatabase [rel_3_0]

[paul poulain]

CVSROOT:        /sources/koha
Module name:    koha
Branch:         rel_3_0
Changes by:     paul poulain <tipaul>   06/10/11 15:22:23

Modified files:
        updater        : updatedatabase 

Log message:
        - adding some missing fields, coming from dev_week :
        * lcsort & ccode in biblioitems table. lcsort is used for loc 
callnumbers & ccode is used to have a item level circulation rules. Ccode means 
C<irc>code
        * onloan & issue_date in items table. They are filled by the 
misc/update_items.pl script, with circulation values. NOW, onloan & issue_date 
in items are NOT set by circulation, this decision speed up a lot the 
circulation rate (with the Date::Manip removal). The price for this is to have 
the status of the item not real time updated in zebra, but that's worth the 
price.
        * cutterextra in items, that we should ask tumer to understand what it 
does  ;-)

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/koha/updater/updatedatabase?cvsroot=koha&only_with_tag=rel_3_0&r1=1.157.2.6&r2=1.157.2.7

Patches:
Index: updatedatabase
===================================================================
RCS file: /sources/koha/koha/updater/updatedatabase,v
retrieving revision 1.157.2.6
retrieving revision 1.157.2.7
diff -u -b -r1.157.2.6 -r1.157.2.7
--- updatedatabase      10 Oct 2006 11:25:40 -0000      1.157.2.6
+++ updatedatabase      11 Oct 2006 15:22:23 -0000      1.157.2.7
@@ -1,6 +1,6 @@
 #!/usr/bin/perl
 
-# $Id: updatedatabase,v 1.157.2.6 2006/10/10 11:25:40 btoumi Exp $
+# $Id: updatedatabase,v 1.157.2.7 2006/10/11 15:22:23 tipaul Exp $
 
 # Database Updater
 # This script checks for required updates to the database.
@@ -189,12 +189,12 @@
 
     notifys    => "(
                          notify_id int(11) NOT NULL default '0',
-                         borrowernumber` int(11) NOT NULL default '0',
-                         itemnumber` int(11) NOT NULL default '0',
-                         notify_date` date NOT NULL default '0000-00-00',
-                         notify_send_date` date default NULL,
-                         notify_level` int(1) NOT NULL default '0',
-                         method` varchar(20) NOT NULL default '',
+                         `borrowernumber` int(11) NOT NULL default '0',
+                         `itemnumber` int(11) NOT NULL default '0',
+                         `notify_date` date NOT NULL default '0000-00-00',
+                         `notify_send_date` date default NULL,
+                         `notify_level` int(1) NOT NULL default '0',
+                         `method` varchar(20) NOT NULL default '',
                          KEY `borrowernumber` (`borrowernumber`,`itemnumber`)
                     )",
 
@@ -216,7 +216,7 @@
        aqbookfund => { 'branchcode' => 'varchar(4) NULL'},
        aqbudget => { 'branchcode' => 'varchar(4) NULL'},
        auth_header => { 'marc' => 'BLOB NOT NULL', 'linkid' => 'BIGINT(20) 
NULL'},
-       auth_subfield_structure =>{ 'hidden' => 'TINYINT(3) NOT NULL UNSIGNED 
ZEROFILL', 'kohafield' => 'VARCHAR(45) NOT NULL', 'linkid' =>  'TINYINT(1) NOT 
NULL UNSIGNED', 'isurl' => 'TINYINT(1) UNSIGNED'},
+       auth_subfield_structure =>{ 'hidden' => 'TINYINT(3) NOT NULL default 
0', 'kohafield' => 'VARCHAR(45) NOT NULL', 'linkid' =>  'TINYINT(1) NOT NULL', 
'isurl' => 'TINYINT(1)'},
        serial =>{ 'publisheddate' => 'date', 'itemnumber'=>'text 
NULL','routingnotes'=>'text NULL',},
     statistics => { 'associatedborrower' => 'integer'},
 #    tablename        => { 'field' => 'fieldtype' },
@@ -626,6 +626,68 @@
                        extra   => '',
                },
        ],
+       biblioitems =>  [
+               {
+                       field   => 'lcsort',
+                       type    => 'varchar(25)',
+                       null    => 'NULL',
+                       key             => '',
+                       default => '',
+                       extra   => '',
+               },
+               {
+                       field   => 'ccode',
+                       type    => 'varchar(4)',
+                       null    => 'NULL',
+                       key             => '',
+                       default => '',
+                       extra   => '',
+               },
+    ],
+    items => [
+               {
+                       field   => 'onloan',
+                       type    => 'date',
+                       null    => 'NULL',
+                       key             => '',
+                       default => '0000-00-00',
+                       extra   => '',
+               },
+               {
+                       field   => 'cutterextra',
+                       type    => 'varchar(45)',
+                       null    => 'NULL',
+                       key             => '',
+                       default => '',
+                       extra   => '',
+               },
+               {
+                       field   => 'issue_date',
+                       type    => 'date',
+                       null    => 'NULL',
+                       key             => '',
+                       default => '',
+                       extra   => '',
+               },
+       ],
+       systempreferences =>  [
+               {
+                       field   => 'value',
+                       type    => 'text',
+                       null    => 'NULL',
+                       key             => '',
+                       default => '',
+                       extra   => '',
+               },
+               {
+                       field   => 'explanation',
+                       type    => 'text',
+                       null    => 'NULL',
+                       key             => '',
+                       default => '',
+                       extra   => '',
+               },
+       ],
 );
 
 my %indexes = (
@@ -1290,7 +1352,7 @@
 # synch table and deletedtable.
 foreach my $table (('borrowers','items','biblio','biblioitems')) {
        my %deletedborrowers;
-       print "synch'ing $table\n";
+       print "synch'ing $table and deleted$table\n";
        $sth = $dbh->prepare("show columns from deleted$table");
        $sth->execute;
        while ( my ( $column, $type, $null, $key, $default, $extra ) = 
$sth->fetchrow ) {
@@ -1307,7 +1369,7 @@
                        } else {
                                $newcol .= " NOT NULL ";
                        }
-                       $newcol .= "default $default" if $default;
+                       $newcol .= "default ".$dbh->quote($default) if $default;
                        $newcol .= " after $previous" if $previous;
                        $previous=$column;
                        print "creating column $column\n";
@@ -1728,6 +1790,12 @@
 exit;
 
 # $Log: updatedatabase,v $
+# Revision 1.157.2.7  2006/10/11 15:22:23  tipaul
+# - adding some missing fields, coming from dev_week :
+# * lcsort & ccode in biblioitems table. lcsort is used for loc callnumbers & 
ccode is used to have a item level circulation rules. Ccode means C<irc>code
+# * onloan & issue_date in items table. They are filled by the 
misc/update_items.pl script, with circulation values. NOW, onloan & issue_date 
in items are NOT set by circulation, this decision speed up a lot the 
circulation rate (with the Date::Manip removal). The price for this is to have 
the status of the item not real time updated in zebra, but that's worth the 
price.
+# * cutterextra in items, that we should ask tumer to understand what it does  
;-)
+#
 # Revision 1.157.2.6  2006/10/10 11:25:40  btoumi
 # add two tables : notifys , charges
 # modify accountlines tables add two fields (notify_id and notify_level)