[DBIx-Class-Devel] [dbix-class] Allow SQLT options to be passed to unique constraints (#88)
Peter Rabbitson
notifications at github.com
Tue Nov 3 15:18:41 GMT 2015
> @@ -149,13 +149,16 @@ sub parse {
>
> $table->primary_key(@primary) if @primary;
>
> - my %unique_constraints = $source->unique_constraints;
> - foreach my $uniq (sort keys %unique_constraints) {
> - if (!$source->_compare_relationship_keys($unique_constraints{$uniq}, \@primary)) {
> + my $unique_constraints = $source->unique_constraints_info;
> + foreach my $uniq (sort keys %$unique_constraints) {
> + my $cons = $unique_constraints->{$uniq};
*Lines get really long* is exclusively an aesthetic concern. On the other hand *author made the interpreter do extra work... but I can't see why* is more involved.
Either way - this was a minor nitpick as I was reading. I will fix it when I do the final review/rebase pass.
Ignore this particular comment.
---
Reply to this email directly or view it on GitHub:
https://github.com/dbsrgits/dbix-class/pull/88/files#r43759696
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.scsys.co.uk/pipermail/dbix-class-devel/attachments/20151103/efcae3aa/attachment.htm>
More information about the DBIx-Class-Devel
mailing list