Fix annoying warnings on innocent looking MSSQL code
[dbsrgits/DBIx-Class.git] / lib / DBIx / Class / Storage / DBI / MSSQL.pm
CommitLineData
75d07914 1package DBIx::Class::Storage::DBI::MSSQL;
3885cff6 2
75d07914 3use strict;
4use warnings;
3885cff6 5
fabbd5cc 6use base qw/
7 DBIx::Class::Storage::DBI::UniqueIdentifier
8 DBIx::Class::Storage::DBI::IdentityInsert
9/;
2ad62d97 10use mro 'c3';
fabbd5cc 11
ed7ab0f4 12use Try::Tiny;
e8f23a77 13use DBIx::Class::_Util qw( sigwarn_silencer );
6298a324 14use List::Util 'first';
fd323bf1 15use namespace::clean;
3885cff6 16
7b1b2582 17__PACKAGE__->mk_group_accessors(simple => qw/
25d3127d 18 _identity _identity_method _no_scope_identity_query
7b1b2582 19/);
20
d5dedbd6 21__PACKAGE__->sql_maker_class('DBIx::Class::SQLMaker::MSSQL');
ac93965c 22
2b8cc2f2 23__PACKAGE__->sql_quote_char([qw/[ ]/]);
24
6f7a118e 25__PACKAGE__->datetime_parser_type (
26 'DBIx::Class::Storage::DBI::MSSQL::DateTime::Format'
27);
28
40d8d018 29__PACKAGE__->new_guid('NEWID()');
30
5a77aa8b 31sub _prep_for_execute {
32 my $self = shift;
0e773352 33 my ($op, $ident, $args) = @_;
5a77aa8b 34
35# cast MONEY values properly
36 if ($op eq 'insert' || $op eq 'update') {
37 my $fields = $args->[0];
5a77aa8b 38
52416317 39 my $colinfo = $ident->columns_info([keys %$fields]);
40
5a77aa8b 41 for my $col (keys %$fields) {
1537084d 42 # $ident is a result source object with INSERT/UPDATE ops
52416317 43 if (
44 $colinfo->{$col}{data_type}
45 &&
46 $colinfo->{$col}{data_type} =~ /^money\z/i
47 ) {
5a77aa8b 48 my $val = $fields->{$col};
49 $fields->{$col} = \['CAST(? AS MONEY)', [ $col => $val ]];
50 }
51 }
52 }
53
54 my ($sql, $bind) = $self->next::method (@_);
55
fabbd5cc 56 # SELECT SCOPE_IDENTITY only works within a statement scope. We
4a0eed52 57 # must try to always use this particular idiom first, as it is the
fabbd5cc 58 # only one that guarantees retrieving the correct id under high
59 # concurrency. When this fails we will fall back to whatever secondary
60 # retrieval method is specified in _identity_method, but at this
61 # point we don't have many guarantees we will get what we expected.
62 # http://msdn.microsoft.com/en-us/library/ms190315.aspx
63 # http://davidhayden.com/blog/dave/archive/2006/01/17/2736.aspx
25d3127d 64 if ($self->_perform_autoinc_retrieval and not $self->_no_scope_identity_query) {
384b8bce 65 $sql .= "\nSELECT SCOPE_IDENTITY()";
5a77aa8b 66 }
67
68 return ($sql, $bind);
69}
70
71sub _execute {
72 my $self = shift;
5a77aa8b 73
fabbd5cc 74 # always list ctx - we need the $sth
0e773352 75 my ($rv, $sth, @bind) = $self->next::method(@_);
1537084d 76
fabbd5cc 77 if ($self->_perform_autoinc_retrieval) {
5a77aa8b 78
25d3127d 79 # attempt to bring back the result of SELECT SCOPE_IDENTITY() we tacked
1537084d 80 # on in _prep_for_execute above
25d3127d 81 my $identity;
82
83 # we didn't even try on ftds
84 unless ($self->_no_scope_identity_query) {
85 ($identity) = try { $sth->fetchrow_array };
86 $sth->finish;
87 }
ed8de058 88
1537084d 89 # SCOPE_IDENTITY failed, but we can do something else
90 if ( (! $identity) && $self->_identity_method) {
91 ($identity) = $self->_dbh->selectrow_array(
92 'select ' . $self->_identity_method
93 );
94 }
7b1b2582 95
1537084d 96 $self->_identity($identity);
7b1b2582 97 }
98
1537084d 99 return wantarray ? ($rv, $sth, @bind) : $rv;
7b1b2582 100}
5a77aa8b 101
7b1b2582 102sub last_insert_id { shift->_identity }
5a77aa8b 103
f0bd60fc 104#
e74c68ce 105# MSSQL is retarded wrt ordered subselects. One needs to add a TOP
6a247f33 106# to *all* subqueries, but one also *can't* use TOP 100 PERCENT
e74c68ce 107# http://sqladvice.com/forums/permalink/18496/22931/ShowThread.aspx#22931
f0bd60fc 108#
109sub _select_args_to_query {
b928a9d5 110 #my ($self, $ident, $select, $cond, $attrs) = @_;
f0bd60fc 111 my $self = shift;
b928a9d5 112 my $attrs = $_[3];
f0bd60fc 113
b928a9d5 114 my $sql_bind = $self->next::method (@_);
f0bd60fc 115
b8d88d9b 116 # see if this is an ordered subquery
aca481d8 117 if (
b928a9d5 118 $$sql_bind->[0] !~ /^ \s* \( \s* SELECT \s+ TOP \s+ \d+ \s+ /xi
119 and
bac358c9 120 scalar $self->_extract_order_criteria ($attrs->{order_by})
aca481d8 121 ) {
6de07ea3 122 $self->throw_exception(
e705f529 123 'An ordered subselect encountered - this is not safe! Please see "Ordered Subselects" in DBIx::Class::Storage::DBI::MSSQL'
124 ) unless $attrs->{unsafe_subselect_ok};
b928a9d5 125
126 $$sql_bind->[0] =~ s/^ \s* \( \s* SELECT (?=\s) / '(SELECT TOP ' . $self->sql_maker->__max_int /exi;
f0bd60fc 127 }
128
b928a9d5 129 $sql_bind;
f0bd60fc 130}
131
132
4c0f4206 133# savepoint syntax is the same as in Sybase ASE
134
90d7422f 135sub _exec_svp_begin {
4c0f4206 136 my ($self, $name) = @_;
137
90d7422f 138 $self->_dbh->do("SAVE TRANSACTION $name");
4c0f4206 139}
140
141# A new SAVE TRANSACTION with the same name releases the previous one.
90d7422f 142sub _exec_svp_release { 1 }
4c0f4206 143
90d7422f 144sub _exec_svp_rollback {
4c0f4206 145 my ($self, $name) = @_;
146
90d7422f 147 $self->_dbh->do("ROLLBACK TRANSACTION $name");
4c0f4206 148}
149
eb0323df 150sub sqlt_type { 'SQLServer' }
151
6a247f33 152sub sql_limit_dialect {
50772633 153 my $self = shift;
eb0323df 154
6a247f33 155 my $supports_rno = 0;
ff153e24 156
6a247f33 157 if (exists $self->_server_info->{normalized_dbms_version}) {
158 $supports_rno = 1 if $self->_server_info->{normalized_dbms_version} >= 9;
159 }
160 else {
161 # User is connecting via DBD::Sybase and has no permission to run
162 # stored procedures like xp_msver, or version detection failed for some
163 # other reason.
164 # So, we use a query to check if RNO is implemented.
165 try {
166 $self->_get_dbh->selectrow_array('SELECT row_number() OVER (ORDER BY rand())');
167 $supports_rno = 1;
168 };
50772633 169 }
e76e7b5c 170
6a247f33 171 return $supports_rno ? 'RowNumberOver' : 'Top';
ed8de058 172}
3885cff6 173
ecdf1ac8 174sub _ping {
175 my $self = shift;
176
177 my $dbh = $self->_dbh or return 0;
178
e8f23a77 179 try {
180 local $dbh->{RaiseError} = 1;
181 local $dbh->{PrintError} = 0;
ecdf1ac8 182 $dbh->do('select 1');
52b420dd 183 1;
e8f23a77 184 }
185 catch {
186 # MSSQL is *really* annoying wrt multiple active resultsets,
187 # and this may very well be the reason why the _ping failed
188 #
189 # Proactively disconnect, while hiding annoying warnings if the case
190 #
191 # The callchain is:
192 # < check basic retryability prerequisites (e.g. no txn) >
193 # ->retry_handler
194 # ->storage->connected()
195 # ->ping
196 # So if we got here with the in_handler bit set - we won't break
197 # anything by a disconnect
198 if( $self->{_in_do_block_retry_handler} ) {
199 local $SIG{__WARN__} = sigwarn_silencer qr/disconnect invalidates .+? active statement/;
200 $self->disconnect;
201 }
202
203 # RV of _ping itself
52b420dd 204 0;
ecdf1ac8 205 };
ecdf1ac8 206}
207
fb95dc4d 208package # hide from PAUSE
209 DBIx::Class::Storage::DBI::MSSQL::DateTime::Format;
210
fd323bf1 211my $datetime_format = '%Y-%m-%d %H:%M:%S.%3N'; # %F %T
fb95dc4d 212my $smalldatetime_format = '%Y-%m-%d %H:%M:%S';
213
214my ($datetime_parser, $smalldatetime_parser);
215
216sub parse_datetime {
217 shift;
218 require DateTime::Format::Strptime;
219 $datetime_parser ||= DateTime::Format::Strptime->new(
220 pattern => $datetime_format,
221 on_error => 'croak',
222 );
223 return $datetime_parser->parse_datetime(shift);
224}
225
226sub format_datetime {
227 shift;
228 require DateTime::Format::Strptime;
229 $datetime_parser ||= DateTime::Format::Strptime->new(
230 pattern => $datetime_format,
231 on_error => 'croak',
232 );
233 return $datetime_parser->format_datetime(shift);
234}
235
236sub parse_smalldatetime {
237 shift;
238 require DateTime::Format::Strptime;
239 $smalldatetime_parser ||= DateTime::Format::Strptime->new(
240 pattern => $smalldatetime_format,
241 on_error => 'croak',
242 );
243 return $smalldatetime_parser->parse_datetime(shift);
244}
245
246sub format_smalldatetime {
247 shift;
248 require DateTime::Format::Strptime;
249 $smalldatetime_parser ||= DateTime::Format::Strptime->new(
250 pattern => $smalldatetime_format,
251 on_error => 'croak',
252 );
253 return $smalldatetime_parser->format_datetime(shift);
254}
255
75d07914 2561;
3885cff6 257
75d07914 258=head1 NAME
3885cff6 259
5a77aa8b 260DBIx::Class::Storage::DBI::MSSQL - Base Class for Microsoft SQL Server support
261in DBIx::Class
3885cff6 262
75d07914 263=head1 SYNOPSIS
3885cff6 264
5a77aa8b 265This is the base class for Microsoft SQL Server support, used by
266L<DBIx::Class::Storage::DBI::ODBC::Microsoft_SQL_Server> and
267L<DBIx::Class::Storage::DBI::Sybase::Microsoft_SQL_Server>.
eb0323df 268
5a77aa8b 269=head1 IMPLEMENTATION NOTES
eb0323df 270
fd05d10a 271=head2 IDENTITY information
272
5a77aa8b 273Microsoft SQL Server supports three methods of retrieving the IDENTITY
274value for inserted row: IDENT_CURRENT, @@IDENTITY, and SCOPE_IDENTITY().
275SCOPE_IDENTITY is used here because it is the safest. However, it must
276be called is the same execute statement, not just the same connection.
eb0323df 277
5a77aa8b 278So, this implementation appends a SELECT SCOPE_IDENTITY() statement
279onto each INSERT to accommodate that requirement.
eb0323df 280
7b1b2582 281C<SELECT @@IDENTITY> can also be used by issuing:
282
283 $self->_identity_method('@@identity');
284
08cdc412 285it will only be used if SCOPE_IDENTITY() fails.
286
287This is more dangerous, as inserting into a table with an on insert trigger that
288inserts into another table with an identity will give erroneous results on
289recent versions of SQL Server.
7b1b2582 290
c84189e1 291=head2 identity insert
fd05d10a 292
293Be aware that we have tried to make things as simple as possible for our users.
c84189e1 294For MSSQL that means that when a user tries to create a row, while supplying an
295explicit value for an autoincrementing column, we will try to issue the
296appropriate database call to make this possible, namely C<SET IDENTITY_INSERT
297$table_name ON>. Unfortunately this operation in MSSQL requires the
298C<db_ddladmin> privilege, which is normally not included in the standard
299write-permissions.
fd05d10a 300
d74f2da9 301=head2 Ordered Subselects
6de07ea3 302
d74f2da9 303If you attempted the following query (among many others) in Microsoft SQL
304Server
6de07ea3 305
6de07ea3 306 $rs->search ({}, {
6de07ea3 307 prefetch => 'relation',
308 rows => 2,
309 offset => 3,
310 });
311
d74f2da9 312You may be surprised to receive an exception. The reason for this is a quirk
313in the MSSQL engine itself, and sadly doesn't have a sensible workaround due
314to the way DBIC is built. DBIC can do truly wonderful things with the aid of
315subselects, and does so automatically when necessary. The list of situations
316when a subselect is necessary is long and still changes often, so it can not
317be exhaustively enumerated here. The general rule of thumb is a joined
318L<has_many|DBIx::Class::Relationship/has_many> relationship with limit/group
319applied to the left part of the join.
320
321In its "pursuit of standards" Microsft SQL Server goes to great lengths to
322forbid the use of ordered subselects. This breaks a very useful group of
323searches like "Give me things number 4 to 6 (ordered by name), and prefetch
324all their relations, no matter how many". While there is a hack which fools
325the syntax checker, the optimizer may B<still elect to break the subselect>.
326Testing has determined that while such breakage does occur (the test suite
327contains an explicit test which demonstrates the problem), it is relative
328rare. The benefits of ordered subselects are on the other hand too great to be
329outright disabled for MSSQL.
6de07ea3 330
331Thus compromise between usability and perfection is the MSSQL-specific
69a8b315 332L<resultset attribute|DBIx::Class::ResultSet/ATTRIBUTES> C<unsafe_subselect_ok>.
6de07ea3 333It is deliberately not possible to set this on the Storage level, as the user
48580715 334should inspect (and preferably regression-test) the return of every such
d74f2da9 335ResultSet individually. The example above would work if written like:
336
337 $rs->search ({}, {
69a8b315 338 unsafe_subselect_ok => 1,
d74f2da9 339 prefetch => 'relation',
340 rows => 2,
341 offset => 3,
342 });
6de07ea3 343
344If it is possible to rewrite the search() in a way that will avoid the need
345for this flag - you are urged to do so. If DBIC internals insist that an
d74f2da9 346ordered subselect is necessary for an operation, and you believe there is a
48580715 347different/better way to get the same result - please file a bugreport.
6de07ea3 348
a2bd3796 349=head1 FURTHER QUESTIONS?
3885cff6 350
a2bd3796 351Check the list of L<additional DBIC resources|DBIx::Class/GETTING HELP/SUPPORT>.
3885cff6 352
a2bd3796 353=head1 COPYRIGHT AND LICENSE
3885cff6 354
a2bd3796 355This module is free software L<copyright|DBIx::Class/COPYRIGHT AND LICENSE>
356by the L<DBIx::Class (DBIC) authors|DBIx::Class/AUTHORS>. You can
357redistribute it and/or modify it under the same terms as the
358L<DBIx::Class library|DBIx::Class/COPYRIGHT AND LICENSE>.