X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FDBIx%2FClass%2FSchema.pm;h=ee4b936387f3fa9017efb50f09bbd6c9d5583acb;hb=382a78385abc2213ea09b3f6c6732694bf522e63;hp=0b92c116d5fd51a62430c003cc169b6199b42f8c;hpb=7a58f051d9b201fd50c1ee43c0ca9602523ca66b;p=dbsrgits%2FDBIx-Class-Historic.git diff --git a/lib/DBIx/Class/Schema.pm b/lib/DBIx/Class/Schema.pm index 0b92c11..ee4b936 100644 --- a/lib/DBIx/Class/Schema.pm +++ b/lib/DBIx/Class/Schema.pm @@ -285,23 +285,21 @@ sub load_classes { =back This is an alternative to L above which assumes an alternative -layout for automatic class loading. It assumes that all source-definition -classes are underneath a sub-namespace of the schema called C, any +layout for automatic class loading. It assumes that all result +classes are underneath a sub-namespace of the schema called C, any corresponding ResultSet classes are underneath a sub-namespace of the schema -called C, and any corresponding Result classes are underneath a -sub-namespace of the schema called C. +called C. -All of those sub-namespaces are configurable if you don't like the defaults, -via the options C, C, and -C, respectively. +Both of the sub-namespaces are configurable if you don't like the defaults, +via the options C and C. If (and only if) you specify the option C, any found -source-definition classes for which we do not find a corresponding +Result classes for which we do not find a corresponding ResultSet class will have their C set to C. -C takes care of calling C and/or -C for you where neccessary if you didn't do it for yourself. +C takes care of calling C for you where +neccessary if you didn't do it for yourself. All of the namespace and classname options to this method are relative to the schema classname by default. To specify a fully-qualified name, prefix @@ -309,34 +307,34 @@ it with a literal C<+>. Examples: - # load My::Schema::Source::CD, My::Schema::Source::Artist, + # load My::Schema::Result::CD, My::Schema::Result::Artist, # My::Schema::ResultSet::CD, etc... My::Schema->load_namespaces; - # Override everything... + # Override everything to use ugly names. + # In this example, if there is a My::Schema::Res::Foo, but no matching + # My::Schema::RSets::Foo, then Foo will have its + # resultset_class set to My::Schema::RSetBase My::Schema->load_namespaces( - source_namespace => 'Srcs', + result_namespace => 'Res', resultset_namespace => 'RSets', - result_namespace => 'Results', default_resultset_class => 'RSetBase', ); - # In the above, if there is a My::Schema::Srcs::Foo, but no matching - # My::Schema::RSets::Foo, then the Foo source will have its - # resultset_class set to My::Schema::RSetBase # Put things in other namespaces My::Schema->load_namespaces( - source_namespace => '+Some::Place::Sources', + result_namespace => '+Some::Place::Results', resultset_namespace => '+Another::Place::RSets', ); If you'd like to use multiple namespaces of each type, simply use an arrayref -of namespaces for that option. In the case that the same source-definition -(or resultset, or result) class exists in multiple namespaces, the latter -entries in your list of namespaces will override earlier ones. +of namespaces for that option. In the case that the same result +(or resultset) class exists in multiple namespaces, the latter entries in +your list of namespaces will override earlier ones. My::Schema->load_namespaces( - source_namespace => [ 'Sources_C', 'Sources_B', 'Sources_A' ], + # My::Schema::Results_C::Foo takes precedence over My::Schema::Results_B::Foo : + result_namespace => [ 'Results_A', 'Results_B', 'Results_C' ], resultset_namespace => [ '+Some::Place::RSets', 'RSets' ], ); @@ -373,9 +371,8 @@ sub _map_namespaces { sub load_namespaces { my ($class, %args) = @_; - my $source_namespace = delete $args{source_namespace} || 'Source'; - my $resultset_namespace = delete $args{resultset_namespace} || 'ResultSet'; my $result_namespace = delete $args{result_namespace} || 'Result'; + my $resultset_namespace = delete $args{resultset_namespace} || 'ResultSet'; my $default_resultset_class = delete $args{default_resultset_class}; $class->throw_exception('load_namespaces: unknown option(s): ' @@ -385,7 +382,7 @@ sub load_namespaces { $default_resultset_class = $class->_expand_relative_name($default_resultset_class); - for my $arg ($source_namespace, $resultset_namespace, $result_namespace) { + for my $arg ($result_namespace, $resultset_namespace) { $arg = [ $arg ] if !ref($arg) && $arg; $class->throw_exception('load_namespaces: namespace arguments must be ' @@ -395,9 +392,8 @@ sub load_namespaces { $_ = $class->_expand_relative_name($_) for (@$arg); } - my %sources = $class->_map_namespaces(@$source_namespace); - my %resultsets = $class->_map_namespaces(@$resultset_namespace); my %results = $class->_map_namespaces(@$result_namespace); + my %resultsets = $class->_map_namespaces(@$resultset_namespace); my @to_register; { @@ -405,49 +401,31 @@ sub load_namespaces { local *Class::C3::reinitialize = sub { }; use warnings 'redefine'; - foreach my $source (keys %sources) { - my $source_class = $sources{$source}; - $class->ensure_class_loaded($source_class); - $source_class->source_name($source) unless $source_class->source_name; + foreach my $result (keys %results) { + my $result_class = $results{$result}; + $class->ensure_class_loaded($result_class); + $result_class->source_name($result) unless $result_class->source_name; - my $rs_class = delete $resultsets{$source}; - my $rs_set = $source_class->resultset_class; + my $rs_class = delete $resultsets{$result}; + my $rs_set = $result_class->resultset_class; if($rs_set && $rs_set ne 'DBIx::Class::ResultSet') { if($rs_class && $rs_class ne $rs_set) { - warn "We found ResultSet class '$rs_class' for '$source', but it seems " - . "that you had already set '$source' to use '$rs_set' instead"; + warn "We found ResultSet class '$rs_class' for '$result', but it seems " + . "that you had already set '$result' to use '$rs_set' instead"; } } elsif($rs_class ||= $default_resultset_class) { $class->ensure_class_loaded($rs_class); - $source_class->resultset_class($rs_class); + $result_class->resultset_class($rs_class); } - my $r_class = delete $results{$source}; - if($r_class) { - my $r_set = $source_class->result_class; - if(!$r_set || $r_set eq $sources{$source}) { - $class->ensure_class_loaded($r_class); - $source_class->result_class($r_class); - } - elsif($r_set ne $r_class) { - warn "We found Result class '$r_class' for '$source', but it seems " - . "that you had already set '$source' to use '$r_set' instead"; - } - } - - push(@to_register, [ $source_class->source_name, $source_class ]); + push(@to_register, [ $result_class->source_name, $result_class ]); } } foreach (sort keys %resultsets) { warn "load_namespaces found ResultSet class $_ with no " - . 'corresponding source-definition class'; - } - - foreach (sort keys %results) { - warn "load_namespaces found Result class $_ with no " - . 'corresponding source-definition class'; + . 'corresponding Result class'; } Class::C3->reinitialize; @@ -615,8 +593,10 @@ sub setup_connection_class { Instantiates a new Storage object of type L and passes the arguments to -$storage->connect_info. Sets the connection in-place on the schema. See -L for more information. +$storage->connect_info. Sets the connection in-place on the schema. + +See L for DBI-specific syntax, +or L in general. =cut @@ -654,138 +634,83 @@ information. sub connect { shift->clone->connection(@_) } -=head2 txn_begin - -Begins a transaction (does nothing if AutoCommit is off). Equivalent to -calling $schema->storage->txn_begin. See -L for more information. +=head2 txn_do -=cut +=over 4 -sub txn_begin { shift->storage->txn_begin } +=item Arguments: C<$coderef>, @coderef_args? -=head2 txn_commit +=item Return Value: The return value of $coderef -Commits the current transaction. Equivalent to calling -$schema->storage->txn_commit. See L -for more information. +=back -=cut +Executes C<$coderef> with (optional) arguments C<@coderef_args> atomically, +returning its result (if any). Equivalent to calling $schema->storage->txn_do. +See L for more information. -sub txn_commit { shift->storage->txn_commit } +This interface is preferred over using the individual methods L, +L, and L below. -=head2 txn_rollback +=cut -Rolls back the current transaction. Equivalent to calling -$schema->storage->txn_rollback. See -L for more information. +sub txn_do { + my $self = shift; -=cut + $self->storage or $self->throw_exception + ('txn_do called on $schema without storage'); -sub txn_rollback { shift->storage->txn_rollback } + $self->storage->txn_do(@_); +} -=head2 txn_do +=head2 txn_begin -=over 4 +Begins a transaction (does nothing if AutoCommit is off). Equivalent to +calling $schema->storage->txn_begin. See +L for more information. -=item Arguments: C<$coderef>, @coderef_args? +=cut -=item Return Value: The return value of $coderef +sub txn_begin { + my $self = shift; -=back + $self->storage or $self->throw_exception + ('txn_begin called on $schema without storage'); -Executes C<$coderef> with (optional) arguments C<@coderef_args> atomically, -returning its result (if any). If an exception is caught, a rollback is issued -and the exception is rethrown. If the rollback fails, (i.e. throws an -exception) an exception is thrown that includes a "Rollback failed" message. + $self->storage->txn_begin; +} -For example, +=head2 txn_commit - my $author_rs = $schema->resultset('Author')->find(1); - my @titles = qw/Night Day It/; +Commits the current transaction. Equivalent to calling +$schema->storage->txn_commit. See L +for more information. - my $coderef = sub { - # If any one of these fails, the entire transaction fails - $author_rs->create_related('books', { - title => $_ - }) foreach (@titles); +=cut - return $author->books; - }; +sub txn_commit { + my $self = shift; - my $rs; - eval { - $rs = $schema->txn_do($coderef); - }; + $self->storage or $self->throw_exception + ('txn_commit called on $schema without storage'); - if ($@) { # Transaction failed - die "something terrible has happened!" # - if ($@ =~ /Rollback failed/); # Rollback failed + $self->storage->txn_commit; +} - deal_with_failed_transaction(); - } +=head2 txn_rollback -In a nested transaction (calling txn_do() from within a txn_do() coderef) only -the outermost transaction will issue a L on -the Schema's storage, and txn_do() can be called in void, scalar and list -context and it will behave as expected. +Rolls back the current transaction. Equivalent to calling +$schema->storage->txn_rollback. See +L for more information. =cut -sub txn_do { - my ($self, $coderef, @args) = @_; +sub txn_rollback { + my $self = shift; $self->storage or $self->throw_exception - ('txn_do called on $schema without storage'); - ref $coderef eq 'CODE' or $self->throw_exception - ('$coderef must be a CODE reference'); - - my (@return_values, $return_value); - - $self->txn_begin; # If this throws an exception, no rollback is needed - - my $wantarray = wantarray; # Need to save this since the context - # inside the eval{} block is independent - # of the context that called txn_do() - eval { - - # Need to differentiate between scalar/list context to allow for - # returning a list in scalar context to get the size of the list - if ($wantarray) { - # list context - @return_values = $coderef->(@args); - } elsif (defined $wantarray) { - # scalar context - $return_value = $coderef->(@args); - } else { - # void context - $coderef->(@args); - } - $self->txn_commit; - }; - - if ($@) { - my $error = $@; - - eval { - $self->txn_rollback; - }; - - if ($@) { - my $rollback_error = $@; - my $exception_class = "DBIx::Class::Storage::NESTED_ROLLBACK_EXCEPTION"; - $self->throw_exception($error) # propagate nested rollback - if $rollback_error =~ /$exception_class/; - - $self->throw_exception( - "Transaction aborted: $error. Rollback failed: ${rollback_error}" - ); - } else { - $self->throw_exception($error); # txn failed but rollback succeeded - } - } + ('txn_rollback called on $schema without storage'); - return $wantarray ? @return_values : $return_value; + $self->storage->txn_rollback; } =head2 clone @@ -803,7 +728,9 @@ copy. sub clone { my ($self) = @_; - my $clone = bless({ (ref $self ? %$self : ()) }, ref $self || $self); + my $clone = { (ref $self ? %$self : ()) }; + bless $clone, (ref $self || $self); + foreach my $moniker ($self->sources) { my $source = $self->source($moniker); my $new = $source->new($source); @@ -904,7 +831,7 @@ sub throw_exception { =over 4 -=item Arguments: $sqlt_args +=item Arguments: $sqlt_args, $dir =back @@ -920,9 +847,9 @@ produced include a DROP TABLE statement for each table created. =cut sub deploy { - my ($self, $sqltargs) = @_; + my ($self, $sqltargs, $dir) = @_; $self->throw_exception("Can't deploy without storage") unless $self->storage; - $self->storage->deploy($self, undef, $sqltargs); + $self->storage->deploy($self, undef, $sqltargs, $dir); } =head2 create_ddl_dir (EXPERIMENTAL) @@ -941,8 +868,7 @@ across all databases, or fully handle complex relationships. =cut -sub create_ddl_dir -{ +sub create_ddl_dir { my $self = shift; $self->throw_exception("Can't create_ddl_dir without storage") unless $self->storage; @@ -958,8 +884,7 @@ intended for direct end user use. =cut -sub ddl_filename -{ +sub ddl_filename { my ($self, $type, $dir, $version) = @_; my $filename = ref($self); @@ -980,4 +905,3 @@ Matt S. Trout You may distribute this code under the same terms as Perl itself. =cut -