X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FDBIx%2FClass%2FStorage%2FDBI%2FSybase.pm;h=9072b38696fdb2adead8647088c287f33f88e5c6;hb=ddcc02d14d03169c54c65db9f0f446836483ba55;hp=3fb9045dc0b4528ecf7720e00df4d35b0108bbe8;hpb=6b1f5ef74ba9728a8587863b5034a43d7fbe705e;p=dbsrgits%2FDBIx-Class.git diff --git a/lib/DBIx/Class/Storage/DBI/Sybase.pm b/lib/DBIx/Class/Storage/DBI/Sybase.pm index 3fb9045..9072b38 100644 --- a/lib/DBIx/Class/Storage/DBI/Sybase.pm +++ b/lib/DBIx/Class/Storage/DBI/Sybase.pm @@ -2,115 +2,146 @@ package DBIx::Class::Storage::DBI::Sybase; use strict; use warnings; +use DBIx::Class::_Util 'dbic_internal_try'; +use Try::Tiny; +use namespace::clean; use base qw/DBIx::Class::Storage::DBI/; -use Carp::Clan qw/^DBIx::Class/; +=head1 NAME -sub _rebless { - my $self = shift; +DBIx::Class::Storage::DBI::Sybase - Base class for drivers using +L - 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 $exception = $@; - $dbtype =~ s/\W/_/gi; - my $subclass = "DBIx::Class::Storage::DBI::Sybase::${dbtype}"; - - if (!$exception && $dbtype && $self->load_optional_class($subclass)) { - bless $self, $subclass; - $self->_rebless; - } elsif (not $self->dbh->{syb_dynamic_supported}) { -# probably real Sybase - bless $self, 'DBIx::Class::Storage:DBI::Sybase::NoBindVars'; - $self->_rebless; - } - } -} +=head1 DESCRIPTION -{ - my $old_dbd_warned = 0; +This is the base class/dispatcher for Storage's designed to work with +L - sub _populate_dbh { - my $self = shift; - $self->next::method(@_); - my $dbh = $self->_dbh; +=head1 METHODS - 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; - } +=cut - $dbh->do('set dateformat mdy'); +sub _rebless { shift->_determine_connector_driver('Sybase') } - 1; - } -} +sub _get_rdbms_name { + my $self = shift; + + dbic_internal_try { + my $name = $self->_get_dbh->selectrow_arrayref('sp_server_info @attribute_id=1')->[2]; + + if ($name) { + $name =~ s/\W/_/gi; -sub _dbh_last_insert_id { - my ($self, $dbh, $source, $col) = @_; + # saner class name + $name = 'ASE' if $name eq 'SQL_Server'; + } - # sorry, there's no other way! - my $sth = $dbh->prepare_cached("select max($col) from ".$source->from); - return ($dbh->selectrow_array($sth))[0]; + $name; # RV + } catch { + $self->throw_exception("Unable to establish connection to determine database type: $_") + }; } -sub datetime_parser_type { "DBIx::Class::Storage::DBI::Sybase::DateTime" } +sub _init { + # once the driver is determined see if we need to insert the DBD::Sybase w/ FreeTDS fixups + # this is a dirty version of "instance role application", \o/ DO WANT Moo \o/ + my $self = shift; + if (! $self->isa('DBIx::Class::Storage::DBI::Sybase::FreeTDS') and $self->_using_freetds) { + require DBIx::Class::Storage::DBI::Sybase::FreeTDS; -1; + my @isa = @{mro::get_linear_isa(ref $self)}; + my $class = shift @isa; # this is our current ref -=head1 NAME + my $trait_class = $class . '::FreeTDS'; + mro::set_mro ($trait_class, 'c3'); + no strict 'refs'; + @{"${trait_class}::ISA"} = ($class, 'DBIx::Class::Storage::DBI::Sybase::FreeTDS', @isa); -DBIx::Class::Storage::DBI::Sybase - Storage::DBI subclass for Sybase + bless ($self, $trait_class); -=head1 SYNOPSIS + Class::C3->reinitialize() if DBIx::Class::_ENV_::OLD_MRO; -This subclass supports L for real Sybase databases. If -you are using an MSSQL database via L, see -L. + $self->_init(@_); + } + + $self->next::method(@_); +} -=head1 CAVEATS +sub _ping { + my $self = shift; -If your version of Sybase does not support placeholders, then this storage -driver uses L as a base, + my $dbh = $self->_dbh or return 0; -In which case, bind variables will be interpolated (properly quoted of course) -into the SQL query itself, without using bind placeholders. + local $dbh->{RaiseError} = 1; + local $dbh->{PrintError} = 0; -More importantly this means that caching of prepared statements is explicitly -disabled, as the interpolation renders it useless. +# FIXME if the main connection goes stale, does opening another for this statement +# really determine anything? +# FIXME (2) THIS MAKES 0 SENSE!!! Need to test later + if ($dbh->{syb_no_child_con}) { + return dbic_internal_try { + $self->_connect->do('select 1'); + 1; + } + catch { + 0; + }; + } -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