X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FDBIx%2FClass%2FStorage%2FDBI%2FSybase.pm;h=abf15bfbcfd2b243b20c0a7adc7f27ae5050e5a3;hb=a85b7ebe9762ca64a08468f6c8f27a0ae583d38c;hp=8c5f988dcfaf00cf57cc28708a40f6a37cde6076;hpb=8384a713d9046e08314e06bf17c2a878cd215819;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 8c5f988..abf15bf 100644 --- a/lib/DBIx/Class/Storage/DBI/Sybase.pm +++ b/lib/DBIx/Class/Storage/DBI/Sybase.pm @@ -2,6 +2,8 @@ package DBIx::Class::Storage::DBI::Sybase; use strict; use warnings; +use Try::Tiny; +use namespace::clean; use base qw/DBIx::Class::Storage::DBI/; @@ -22,13 +24,13 @@ L sub _rebless { my $self = shift; - my $dbtype = eval { - @{$self->_get_dbh->selectrow_arrayref(qq{sp_server_info \@attribute_id=1})}[2] + my $dbtype; + try { + $dbtype = @{$self->_get_dbh->selectrow_arrayref(qq{sp_server_info \@attribute_id=1})}[2] + } catch { + $self->throw_exception("Unable to estable connection to determine database type: $_") }; - $self->throw_exception("Unable to estable connection to determine database type: $@") - if $@; - if ($dbtype) { $dbtype =~ s/\W/_/gi; @@ -53,17 +55,17 @@ sub _ping { if ($dbh->{syb_no_child_con}) { # if extra connections are not allowed, then ->ping is reliable - my $ping = eval { $dbh->ping }; - return $@ ? 0 : $ping; + return try { $dbh->ping } catch { 0; }; } - eval { + return try { # XXX if the main connection goes stale, does opening another for this statement # really determine anything? $dbh->do('select 1'); + 1; + } catch { + 0; }; - - return $@ ? 0 : 1; } sub _set_max_connect { @@ -110,8 +112,11 @@ back to the C<32768> which is the L default. sub set_textsize { my $self = shift; - my $text_size = shift || - eval { $self->_dbi_connect_info->[-1]->{LongReadLen} } || + my $text_size = + shift + || + try { $self->_dbi_connect_info->[-1]->{LongReadLen} } + || 32768; # the DBD::Sybase default return unless defined $text_size;