X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FDBIx%2FClass%2FStorage%2FDBI%2FSybase.pm;h=8c5f988dcfaf00cf57cc28708a40f6a37cde6076;hb=fecb38cba68186d579ef96f89bf6ce557e55ab1c;hp=fe383596feaac32866df8d35f01fea10b28b237e;hpb=f0070d01c1ebc3895551ff52580c544e180e7bcf;p=dbsrgits%2FDBIx-Class-Historic.git diff --git a/lib/DBIx/Class/Storage/DBI/Sybase.pm b/lib/DBIx/Class/Storage/DBI/Sybase.pm index fe38359..8c5f988 100644 --- a/lib/DBIx/Class/Storage/DBI/Sybase.pm +++ b/lib/DBIx/Class/Storage/DBI/Sybase.pm @@ -5,111 +5,128 @@ use warnings; use base qw/DBIx::Class::Storage::DBI/; -use Carp::Clan qw/^DBIx::Class/; +=head1 NAME + +DBIx::Class::Storage::DBI::Sybase - Base class for drivers using +L + +=head1 DESCRIPTION + +This is the base class/dispatcher for Storage's designed to work with +L + +=head1 METHODS + +=cut sub _rebless { my $self = shift; - if (ref($self) eq 'DBIx::Class::Storage::DBI::Sybase') { - my $dbtype = eval { - @{$self->dbh->selectrow_arrayref(qq{sp_server_info \@attribute_id=1})}[2] - } || ''; + my $dbtype = eval { + @{$self->_get_dbh->selectrow_arrayref(qq{sp_server_info \@attribute_id=1})}[2] + }; + + $self->throw_exception("Unable to estable connection to determine database type: $@") + if $@; - my $exception = $@; + if ($dbtype) { $dbtype =~ s/\W/_/gi; - my $subclass = "DBIx::Class::Storage::DBI::Sybase::${dbtype}"; - if (!$exception && $dbtype && $self->load_optional_class($subclass)) { + # saner class name + $dbtype = 'ASE' if $dbtype eq 'SQL_Server'; + + my $subclass = __PACKAGE__ . "::$dbtype"; + if ($self->load_optional_class($subclass)) { bless $self, $subclass; $self->_rebless; - } elsif (not $self->dbh->{syb_dynamic_supported}) { - bless $self, 'DBIx::Class::Storage:DBI::Sybase::NoBindVars'; - $self->_rebless; } } } -{ - my $old_dbd_warned = 0; +sub _ping { + my $self = shift; - sub _populate_dbh { - my $self = shift; - $self->next::method(@_); - my $dbh = $self->_dbh; + my $dbh = $self->_dbh or return 0; - if ($dbh->can('syb_date_fmt')) { - $dbh->syb_date_fmt('ISO_strict'); - } elsif (not $old_dbd_warned) { - carp "Your DBD::Sybase is too old to support ". - "DBIx::Class::InflateColumn::DateTime, please upgrade!"; - $old_dbd_warned = 1; - } + local $dbh->{RaiseError} = 1; + local $dbh->{PrintError} = 0; - $dbh->do('set dateformat mdy'); - - 1; + if ($dbh->{syb_no_child_con}) { +# if extra connections are not allowed, then ->ping is reliable + my $ping = eval { $dbh->ping }; + return $@ ? 0 : $ping; } -} -sub _dbh_last_insert_id { - my ($self, $dbh, $source, $col) = @_; + eval { +# XXX if the main connection goes stale, does opening another for this statement +# really determine anything? + $dbh->do('select 1'); + }; - # sorry, there's no other way! - my $sth = $dbh->prepare_cached("select max($col) from ".$source->from); - return ($dbh->selectrow_array($sth))[0]; + return $@ ? 0 : 1; } -sub datetime_parser_type { "DBIx::Class::Storage::DBI::Sybase::DateTime" } +sub _set_max_connect { + my $self = shift; + my $val = shift || 256; -1; + my $dsn = $self->_dbi_connect_info->[0]; -=head1 NAME + return if ref($dsn) eq 'CODE'; -DBIx::Class::Storage::DBI::Sybase - Storage::DBI subclass for Sybase + if ($dsn !~ /maxConnect=/) { + $self->_dbi_connect_info->[0] = "$dsn;maxConnect=$val"; + my $connected = defined $self->_dbh; + $self->disconnect; + $self->ensure_connected if $connected; + } +} -=head1 SYNOPSIS +=head2 using_freetds -This subclass supports L for real Sybase databases. If -you are using an MSSQL database via L, see -L. +Whether or not L was compiled against FreeTDS. If false, it means +the Sybase OpenClient libraries were used. -=head1 CAVEATS +=cut + +sub using_freetds { + my $self = shift; -If your version of Sybase does not support placeholders, then this storage -driver uses L as a base, + return $self->_get_dbh->{syb_oc_version} =~ /freetds/i; +} -In which case, bind variables will be interpolated (properly quoted of course) -into the SQL query itself, without using bind placeholders. +=head2 set_textsize -More importantly this means that caching of prepared statements is explicitly -disabled, as the interpolation renders it useless. +When using FreeTDS and/or MSSQL, C<< $dbh->{LongReadLen} >> is not available, +use this function instead. It does: -If your version of Sybase B support placeholders (check -C<<$dbh->{syb_dynamic_supported}>> then unfortunately there's no way to get the -C without doing a C