From: jeff Date: Sat, 4 Apr 2009 16:22:19 +0000 (+0000) Subject: correct bad schema bug in cust_bill_pkg_detail X-Git-Tag: root_of_svc_elec_features~1304 X-Git-Url: http://git.freeside.biz/gitweb/?p=freeside.git;a=commitdiff_plain;h=e6f3d047e87488badad86b71508b1f4d07c981f9 correct bad schema bug in cust_bill_pkg_detail --- diff --git a/FS/FS/Schema.pm b/FS/FS/Schema.pm index 59abbda8a..e4076b13d 100644 --- a/FS/FS/Schema.pm +++ b/FS/FS/Schema.pm @@ -531,7 +531,7 @@ sub tables_hashref { 'invnum', 'int', 'NULL', '', '', '', # deprecated 'amount', @money_typen, '', '', 'format', 'char', 'NULL', 1, '', '', - 'classnum', 'char', 'NULL', 1, '', '', + 'classnum', 'int', 'NULL', '', '', '', 'detail', 'varchar', '', 255, '', '', ], 'primary_key' => 'detailnum', diff --git a/FS/FS/cust_bill_pkg_detail.pm b/FS/FS/cust_bill_pkg_detail.pm index 8a4888894..f40ce5869 100644 --- a/FS/FS/cust_bill_pkg_detail.pm +++ b/FS/FS/cust_bill_pkg_detail.pm @@ -1,8 +1,8 @@ package FS::cust_bill_pkg_detail; use strict; -use vars qw( @ISA $me $DEBUG ); -use FS::Record qw( qsearch qsearchs dbdef ); +use vars qw( @ISA $me $DEBUG %GetInfoType ); +use FS::Record qw( qsearch qsearchs dbdef dbh ); use FS::cust_bill_pkg; @ISA = qw(FS::Record); @@ -121,6 +121,48 @@ sub _upgrade_data { # class method warn "$me upgrading $class\n" if $DEBUG; + my $columndef = dbdef->table($class->table)->column('classnum'); + unless ($columndef->type eq 'int4') { + + my $dbh = dbh; + if ( $dbh->{Driver}->{Name} eq 'Pg' ) { + + eval "use DBI::Const::GetInfoType;"; + die $@ if $@; + + my $major_version = 0; + $dbh->get_info( $GetInfoType{SQL_DBMS_VER} ) =~ /^(\d{2})/ + && ( $major_version = sprintf("%d", $1) ); + + if ( $major_version > 7 ) { + + # ideally this would be supported in DBIx-DBSchema and friends + + foreach my $table ( qw( cust_bill_pkg_detail h_cust_bill_pkg_detail ) ){ + + warn "updating $table column classnum to integer\n" if $DEBUG; + my $sql = "ALTER TABLE $table ALTER classnum TYPE int USING ". + "int4(classnum)"; + my $sth = $dbh->prepare($sql) or die $dbh->errstr; + $sth->execute or die $sth->errstr; + + } + + } else { + + die "cust_bill_pkg_detail classnum upgrade unsupported for this Pg version\n"; + + } + + } else { + + die "cust_bill_pkg_detail classnum upgrade only supported for Pg 8+\n"; + + } + + } + + if ( defined( dbdef->table($class->table)->column('billpkgnum') ) && defined( dbdef->table($class->table)->column('invnum') ) && defined( dbdef->table($class->table)->column('pkgnum') )