X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FDBIx%2FClass%2FStorage%2FDBI%2FSybase.pm;h=77b77e2273395f0e72f815bc7c463cd6dca44770;hb=4225194590a09e29451ba825c34483f98c1a0c03;hp=0a2cfb8bd9b8a603d331624a8f665a1f9da24489;hpb=2ad62d97476187e57d20ee49b39d7a7af864bb20;p=dbsrgits%2FDBIx-Class.git diff --git a/lib/DBIx/Class/Storage/DBI/Sybase.pm b/lib/DBIx/Class/Storage/DBI/Sybase.pm index 0a2cfb8..77b77e2 100644 --- a/lib/DBIx/Class/Storage/DBI/Sybase.pm +++ b/lib/DBIx/Class/Storage/DBI/Sybase.pm @@ -3,57 +3,127 @@ package DBIx::Class::Storage::DBI::Sybase; use strict; use warnings; -use base qw/ - DBIx::Class::Storage::DBI::Sybase::Base - DBIx::Class::Storage::DBI::NoBindVars -/; -use mro 'c3'; +use base qw/DBIx::Class::Storage::DBI/; + +=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; - - my $dbtype = eval { @{$self->dbh->selectrow_arrayref(qq{sp_server_info \@attribute_id=1})}[2] }; - unless ( $@ ) { - $dbtype =~ s/\W/_/gi; - my $subclass = "DBIx::Class::Storage::DBI::Sybase::${dbtype}"; - if ($self->load_optional_class($subclass) && !$self->isa($subclass)) { - bless $self, $subclass; - $self->_rebless; - } + my $self = shift; + + 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 $@; + + if ($dbtype) { + $dbtype =~ s/\W/_/gi; + + # 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; } + } } -sub _dbh_last_insert_id { - my ($self, $dbh, $source, $col) = @_; - return ($dbh->selectrow_array('select @@identity'))[0]; +sub _ping { + my $self = shift; + + my $dbh = $self->_dbh or return 0; + + local $dbh->{RaiseError} = 1; + local $dbh->{PrintError} = 0; + + if ($dbh->{syb_no_child_con}) { +# if extra connections are not allowed, then ->ping is reliable + my $ping = eval { $dbh->ping }; + return $@ ? 0 : $ping; + } + + eval { +# XXX if the main connection goes stale, does opening another for this statement +# really determine anything? + $dbh->do('select 1'); + }; + + return $@ ? 0 : 1; } -1; +sub _set_max_connect { + my $self = shift; + my $val = shift || 256; -=head1 NAME + my $dsn = $self->_dbi_connect_info->[0]; -DBIx::Class::Storage::DBI::Sybase - Storage::DBI subclass for Sybase + return if ref($dsn) eq 'CODE'; -=head1 SYNOPSIS + if ($dsn !~ /maxConnect=/) { + $self->_dbi_connect_info->[0] = "$dsn;maxConnect=$val"; + my $connected = defined $self->_dbh; + $self->disconnect; + $self->ensure_connected if $connected; + } +} -This subclass supports L for real Sybase databases. If -you are using an MSSQL database via L, see -L. +=head2 using_freetds -=head1 CAVEATS +Whether or not L was compiled against FreeTDS. If false, it means +the Sybase OpenClient libraries were used. -This storage driver uses L as a base. -This means that bind variables will be interpolated (properly quoted of course) -into the SQL query itself, without using bind placeholders. +=cut -More importantly this means that caching of prepared statements is explicitly -disabled, as the interpolation renders it useless. +sub using_freetds { + my $self = shift; -=head1 AUTHORS + return $self->_get_dbh->{syb_oc_version} =~ /freetds/i; +} + +=head2 set_textsize + +When using FreeTDS and/or MSSQL, C<< $dbh->{LongReadLen} >> is not available, +use this function instead. It does: + + $dbh->do("SET TEXTSIZE $bytes"); + +Takes the number of bytes, or uses the C value from your +L if omitted, lastly falls back to the C<32768> which +is the L default. -Brandon L Black +=cut + +sub set_textsize { + my $self = shift; + my $text_size = shift || + eval { $self->_dbi_connect_info->[-1]->{LongReadLen} } || + 32768; # the DBD::Sybase default + + return unless defined $text_size; + + $self->_dbh->do("SET TEXTSIZE $text_size"); +} + +1; + +=head1 AUTHORS -Justin Hunter +See L. =head1 LICENSE