Have has_one/might_have warn if set on nullable columns.
[dbsrgits/DBIx-Class.git] / lib / DBIx / Class / Relationship / HasOne.pm
CommitLineData
c0e7b4e5 1package # hide from PAUSE
2 DBIx::Class::Relationship::HasOne;
22b15c96 3
4use strict;
5use warnings;
dc571b76 6use Carp::Clan qw/^DBIx::Class/;
22b15c96 7
044e70c7 8our %_pod_inherit_config =
9 (
10 class_map => { 'DBIx::Class::Relationship::HasOne' => 'DBIx::Class::Relationship' }
11 );
12
07037f89 13sub might_have {
14 shift->_has_one('LEFT' => @_);
15}
16
22b15c96 17sub has_one {
e5d98edd 18 shift->_has_one(undef() => @_);
07037f89 19}
20
21sub _has_one {
503536d5 22 my ($class, $join_type, $rel, $f_class, $cond, $attrs) = @_;
99be059e 23 unless (ref $cond) {
fd4df975 24 $class->ensure_class_loaded($f_class);
e36de82e 25
dc571b76 26 my $pri = $class->_get_primary_key;
27
e36de82e 28 $class->throw_exception(
29 "might_have/has_one needs a primary key to infer a join; ".
30 "${class} has none"
31 ) if !defined $pri && (!defined $cond || !length $cond);
32
103647d5 33 my $f_class_loaded = eval { $f_class->columns };
dc571b76 34 my ($f_key,$too_many,$guess);
dcf8330b 35 if (defined $cond && length $cond) {
99be059e 36 $f_key = $cond;
dcf8330b 37 $guess = "caller specified foreign key '$f_key'";
103647d5 38 } elsif ($f_class_loaded && $f_class->has_column($rel)) {
07037f89 39 $f_key = $rel;
dcf8330b 40 $guess = "using given relationship '$rel' for foreign key";
07037f89 41 } else {
dc571b76 42 $f_key = $class->_get_primary_key($f_class);
dcf8330b 43 $guess = "using primary key of foreign class for foreign key";
22b15c96 44 }
fd4df975 45 $class->throw_exception(
46 "No such column ${f_key} on foreign class ${f_class} ($guess)"
47 ) if $f_class_loaded && !$f_class->has_column($f_key);
503536d5 48 $cond = { "foreign.${f_key}" => "self.${pri}" };
07037f89 49 }
dc571b76 50 $class->_validate_cond($cond);
07037f89 51 $class->add_relationship($rel, $f_class,
52 $cond,
503536d5 53 { accessor => 'single',
07037f89 54 cascade_update => 1, cascade_delete => 1,
55 ($join_type ? ('join_type' => $join_type) : ()),
503536d5 56 %{$attrs || {}} });
07037f89 57 1;
22b15c96 58}
59
dc571b76 60sub _get_primary_key {
61 my ( $class, $target_class ) = @_;
62 $target_class ||= $class;
63 my ($pri, $too_many) = $target_class->primary_columns;
64 $class->throw_exception(
65 "might_have/has_one can only infer join for a single primary key; ".
66 "${class} has more"
67 ) if $too_many;
68 return $pri;
69}
70
71sub _validate_cond {
72 my ($class, $cond ) = @_;
73
74 return if $ENV{DBIC_DONT_VALIDATE_RELS};
75 return unless 'HASH' eq ref $cond;
76 foreach my $foreign_id ( keys %$cond ) {
77 my $self_id = $cond->{$foreign_id};
78
79 # we can ignore a bad $self_id because add_relationship handles this
80 # warning
81 return unless $self_id =~ /^self\.(.*)$/;
82 my $key = $1;
83 my $column_info = $class->column_info($key);
84 if ( $column_info->{is_nullable} ) {
85 carp(qq'"might_have/has_one" must not be on columns with is_nullable set to true ($class/$key) ');
86 }
87 }
88}
89
22b15c96 901;