throw an error on non-string builder (RT#78479)
[gitmo/Moo.git] / lib / Method / Generate / Accessor.pm
CommitLineData
51a3b106 1package Method::Generate::Accessor;
2
3use strictures 1;
b1eebd55 4use Moo::_Utils;
5use base qw(Moo::Object);
51a3b106 6use Sub::Quote;
7use B 'perlstring';
3202e039 8use Scalar::Util 'blessed';
ac34668b 9use overload ();
daa05b62 10BEGIN {
6e294299 11 our $CAN_HAZ_XS =
33d35735 12 !$ENV{MOO_XS_DISABLE}
13 &&
6e294299 14 _maybe_load_module('Class::XSAccessor')
15 &&
edb9977c 16 (eval { Class::XSAccessor->VERSION('1.07') })
6e294299 17 ;
daa05b62 18}
51a3b106 19
316917c9 20sub generate_method {
6f68f022 21 my ($self, $into, $name, $spec, $quote_opts) = @_;
9f10cfe0 22 $name =~ s/^\+//;
51a3b106 23 die "Must have an is" unless my $is = $spec->{is};
dcae37d3 24 if ($is eq 'ro') {
25 $spec->{reader} = $name unless exists $spec->{reader};
26 } elsif ($is eq 'rw') {
27 $spec->{accessor} = $name unless exists $spec->{accessor};
28 } elsif ($is eq 'lazy') {
dcae37d3 29 $spec->{reader} = $name unless exists $spec->{reader};
30 $spec->{lazy} = 1;
31 $spec->{builder} ||= '_build_'.$name unless $spec->{default};
2bb6aaa3 32 } elsif ($is eq 'rwp') {
33 $spec->{reader} = $name unless exists $spec->{reader};
34 $spec->{writer} = "_set_${name}" unless exists $spec->{writer};
dcae37d3 35 } elsif ($is ne 'bare') {
36 die "Unknown is ${is}";
37 }
2bb6aaa3 38 $spec->{builder} = '_build_'.$name if ($spec->{builder}||0) eq 1;
463df7d2 39 die "Invalid builder for $into->$name - not a string"
40 if exists $spec->{builder} and ref $spec->{builder};
2bb6aaa3 41 if (($spec->{predicate}||0) eq 1) {
42 $spec->{predicate} = $name =~ /^_/ ? "_has${name}" : "has_${name}";
43 }
44 if (($spec->{clearer}||0) eq 1) {
45 $spec->{clearer} = $name =~ /^_/ ? "_clear${name}" : "clear_${name}";
46 }
47 if (($spec->{trigger}||0) eq 1) {
48 $spec->{trigger} = quote_sub('shift->_trigger_'.$name.'(@_)');
49 }
17c4135a 50
51 for my $setting (qw( default coerce )) {
52 next if !exists $spec->{$setting};
53 my $value = $spec->{$setting};
54 my $invalid = "Invalid $setting '" . overload::StrVal($value)
3202e039 55 . "' for $into->$name - not a coderef";
56 die "$invalid or code-convertible object"
17c4135a 57 unless ref $value and (ref $value eq 'CODE' or blessed($value));
3202e039 58 die "$invalid and could not be converted to a coderef: $@"
17c4135a 59 if !eval { \&$value };
533797e1 60 }
61
dcae37d3 62 my %methods;
63 if (my $reader = $spec->{reader}) {
64 if (our $CAN_HAZ_XS && $self->is_simple_get($name, $spec)) {
65 $methods{$reader} = $self->_generate_xs(
873df570 66 getters => $into, $reader, $name, $spec
dcae37d3 67 );
68 } else {
e57f338d 69 $self->{captures} = {};
dcae37d3 70 $methods{$reader} =
71 quote_sub "${into}::${reader}"
33d35735 72 => ' die "'.$reader.' is a read-only accessor" if @_ > 1;'."\n"
73 .$self->_generate_get($name, $spec)
e57f338d 74 => delete $self->{captures}
dcae37d3 75 ;
76 }
77 }
78 if (my $accessor = $spec->{accessor}) {
79 if (
80 our $CAN_HAZ_XS
81 && $self->is_simple_get($name, $spec)
82 && $self->is_simple_set($name, $spec)
83 ) {
84 $methods{$accessor} = $self->_generate_xs(
873df570 85 accessors => $into, $accessor, $name, $spec
dcae37d3 86 );
87 } else {
e57f338d 88 $self->{captures} = {};
dcae37d3 89 $methods{$accessor} =
90 quote_sub "${into}::${accessor}"
91 => $self->_generate_getset($name, $spec)
e57f338d 92 => delete $self->{captures}
dcae37d3 93 ;
94 }
95 }
96 if (my $writer = $spec->{writer}) {
97 if (
98 our $CAN_HAZ_XS
99 && $self->is_simple_set($name, $spec)
100 ) {
101 $methods{$writer} = $self->_generate_xs(
873df570 102 setters => $into, $writer, $name, $spec
dcae37d3 103 );
51a3b106 104 } else {
e57f338d 105 $self->{captures} = {};
dcae37d3 106 $methods{$writer} =
107 quote_sub "${into}::${writer}"
108 => $self->_generate_set($name, $spec)
e57f338d 109 => delete $self->{captures}
dcae37d3 110 ;
51a3b106 111 }
dcae37d3 112 }
8f333548 113 if (my $pred = $spec->{predicate}) {
dcae37d3 114 $methods{$pred} =
115 quote_sub "${into}::${pred}" =>
ceea0e38 116 ' '.$self->_generate_simple_has('$_[0]', $name, $spec)."\n"
dcae37d3 117 ;
8f333548 118 }
119 if (my $cl = $spec->{clearer}) {
dcae37d3 120 $methods{$cl} =
121 quote_sub "${into}::${cl}" =>
ceea0e38 122 $self->_generate_simple_clear('$_[0]', $name, $spec)."\n"
dcae37d3 123 ;
901efe1a 124 }
4db3a740 125 if (my $hspec = $spec->{handles}) {
126 my $asserter = $spec->{asserter} ||= '_assert_'.$name;
127 my @specs = do {
128 if (ref($hspec) eq 'ARRAY') {
129 map [ $_ => $_ ], @$hspec;
130 } elsif (ref($hspec) eq 'HASH') {
131 map [ $_ => ref($hspec->{$_}) ? @{$hspec->{$_}} : $hspec->{$_} ],
132 keys %$hspec;
133 } elsif (!ref($hspec)) {
134 map [ $_ => $_ ], Role::Tiny->methods_provided_by($hspec);
135 } else {
136 die "You gave me a handles of ${hspec} and I have no idea why";
137 }
138 };
139 foreach my $spec (@specs) {
140 my ($proxy, $target, @args) = @$spec;
e57f338d 141 $self->{captures} = {};
4db3a740 142 $methods{$proxy} =
143 quote_sub "${into}::${proxy}" =>
144 $self->_generate_delegation($asserter, $target, \@args),
e57f338d 145 delete $self->{captures}
4db3a740 146 ;
147 }
148 }
149 if (my $asserter = $spec->{asserter}) {
e57f338d 150 $self->{captures} = {};
4db3a740 151 $methods{$asserter} =
152 quote_sub "${into}::${asserter}" =>
153 'do { '.$self->_generate_get($name, $spec).qq! }||die "Attempted to access '${name}' but it is not set"!,
e57f338d 154 delete $self->{captures}
4db3a740 155 ;
156 }
dcae37d3 157 \%methods;
51a3b106 158}
159
3a9a65a4 160sub is_simple_attribute {
161 my ($self, $name, $spec) = @_;
8f333548 162 # clearer doesn't have to be listed because it doesn't
163 # affect whether defined/exists makes a difference
901efe1a 164 !grep $spec->{$_},
82a5b146 165 qw(lazy default builder coerce isa trigger predicate weak_ref);
3a9a65a4 166}
167
daa05b62 168sub is_simple_get {
169 my ($self, $name, $spec) = @_;
901efe1a 170 !($spec->{lazy} and ($spec->{default} or $spec->{builder}));
daa05b62 171}
172
173sub is_simple_set {
174 my ($self, $name, $spec) = @_;
82a5b146 175 !grep $spec->{$_}, qw(coerce isa trigger weak_ref);
daa05b62 176}
177
4ced3a94 178sub has_eager_default {
179 my ($self, $name, $spec) = @_;
180 (!$spec->{lazy} and ($spec->{default} or $spec->{builder}));
181}
182
51a3b106 183sub _generate_get {
46389f86 184 my ($self, $name, $spec) = @_;
ceea0e38 185 my $simple = $self->_generate_simple_get('$_[0]', $name, $spec);
901efe1a 186 if ($self->is_simple_get($name, $spec)) {
187 $simple;
188 } else {
189 'do { '.$self->_generate_use_default(
190 '$_[0]', $name, $spec,
ceea0e38 191 $self->_generate_simple_has('$_[0]', $name, $spec),
ef21bc32 192 ).'; '
193 .($spec->{isa}
194 ?($self->_generate_isa_check($name, $simple, $spec->{isa}).'; ')
195 :''
196 )
197 .$simple.' }';
901efe1a 198 }
46389f86 199}
200
201sub _generate_simple_has {
202 my ($self, $me, $name) = @_;
203 "exists ${me}->{${\perlstring $name}}";
204}
205
02e9ef74 206sub _generate_simple_clear {
207 my ($self, $me, $name) = @_;
208 " delete ${me}->{${\perlstring $name}}\n"
209}
210
649ac264 211sub generate_get_default {
212 my $self = shift;
e57f338d 213 $self->{captures} = {};
649ac264 214 my $code = $self->_generate_get_default(@_);
e57f338d 215 ($code, delete $self->{captures});
649ac264 216}
217
218sub _generate_use_default {
219 my ($self, $me, $name, $spec, $test) = @_;
160c2c2c 220 my $get_value = $self->_generate_get_default($me, $name, $spec);
221 if ($spec->{coerce}) {
222 $get_value = $self->_generate_coerce(
b1f04da5 223 $name, $get_value,
160c2c2c 224 $spec->{coerce}
225 )
226 }
46389f86 227 $self->_generate_simple_set(
160c2c2c 228 $me, $name, $spec, $get_value
46389f86 229 ).' unless '.$test;
a16d301e 230}
231
649ac264 232sub _generate_get_default {
233 my ($self, $me, $name, $spec) = @_;
234 $spec->{default}
235 ? $self->_generate_call_code($name, 'default', $me, $spec->{default})
236 : "${me}->${\$spec->{builder}}"
237}
238
a16d301e 239sub generate_simple_get {
901efe1a 240 my ($self, @args) = @_;
241 $self->_generate_simple_get(@args);
a16d301e 242}
243
244sub _generate_simple_get {
245 my ($self, $me, $name) = @_;
246 my $name_str = perlstring $name;
247 "${me}->{${name_str}}";
51a3b106 248}
249
250sub _generate_set {
dcae37d3 251 my ($self, $name, $spec) = @_;
901efe1a 252 if ($self->is_simple_set($name, $spec)) {
32381de9 253 $self->_generate_simple_set('$_[0]', $name, $spec, '$_[1]');
6d377074 254 } else {
82a5b146 255 my ($coerce, $trigger, $isa_check) = @{$spec}{qw(coerce trigger isa)};
32381de9 256 my $simple = $self->_generate_simple_set('$self', $name, $spec, '$value');
e57f338d 257 my $code = "do { my (\$self, \$value) = \@_;\n";
82a5b146 258 if ($coerce) {
259 $code .=
260 " \$value = "
b1f04da5 261 .$self->_generate_coerce($name, '$value', $coerce).";\n";
82a5b146 262 }
901efe1a 263 if ($isa_check) {
264 $code .=
e57f338d 265 " ".$self->_generate_isa_check($name, '$value', $isa_check).";\n";
901efe1a 266 }
267 if ($trigger) {
e57f338d 268 my $fire = $self->_generate_trigger($name, '$self', '$value', $trigger);
901efe1a 269 $code .=
e57f338d 270 " ".$simple.";\n ".$fire.";\n"
901efe1a 271 ." \$value;\n";
272 } else {
273 $code .= " ".$simple.";\n";
274 }
275 $code .= " }";
276 $code;
a16d301e 277 }
a16d301e 278}
82a5b146 279
280sub generate_coerce {
281 my $self = shift;
282 $self->{captures} = {};
283 my $code = $self->_generate_coerce(@_);
284 ($code, delete $self->{captures});
285}
286
287sub _generate_coerce {
b1f04da5 288 my ($self, $name, $value, $coerce) = @_;
2d592c35 289 $self->_generate_die_prefix(
290 "coercion for ${\perlstring($name)} failed: ",
291 $self->_generate_call_code($name, 'coerce', "${value}", $coerce)
292 );
82a5b146 293}
294
a16d301e 295sub generate_trigger {
296 my $self = shift;
e57f338d 297 $self->{captures} = {};
a16d301e 298 my $code = $self->_generate_trigger(@_);
e57f338d 299 ($code, delete $self->{captures});
a16d301e 300}
301
302sub _generate_trigger {
303 my ($self, $name, $obj, $value, $trigger) = @_;
6d377074 304 $self->_generate_call_code($name, 'trigger', "${obj}, ${value}", $trigger);
305}
306
307sub generate_isa_check {
901efe1a 308 my ($self, @args) = @_;
e57f338d 309 $self->{captures} = {};
901efe1a 310 my $code = $self->_generate_isa_check(@args);
e57f338d 311 ($code, delete $self->{captures});
6d377074 312}
313
2d592c35 314sub _generate_die_prefix {
315 my ($self, $prefix, $inside) = @_;
316 "do {\n"
bc65d744 317 .' my $sig_die = $SIG{__DIE__} || sub { die $_[0] };'."\n"
318 .' local $SIG{__DIE__} = sub {'."\n"
2d592c35 319 .' $sig_die->(ref($_[0]) ? $_[0] : '.perlstring($prefix).'.$_[0]);'."\n"
bc65d744 320 .' };'."\n"
2d592c35 321 .$inside
bc65d744 322 ."}\n"
6d377074 323}
324
2d592c35 325sub _generate_isa_check {
326 my ($self, $name, $value, $check) = @_;
327 $self->_generate_die_prefix(
328 "isa check for ${\perlstring($name)} failed: ",
329 $self->_generate_call_code($name, 'isa_check', $value, $check)
330 );
331}
332
6d377074 333sub _generate_call_code {
334 my ($self, $name, $type, $values, $sub) = @_;
335 if (my $quoted = quoted_from_sub($sub)) {
625d6219 336 my $code = $quoted->[1];
8c6626cf 337 if (my $captures = $quoted->[2]) {
6d377074 338 my $cap_name = qq{\$${type}_captures_for_${name}};
8c6626cf 339 $self->{captures}->{$cap_name} = \$captures;
e57f338d 340 Sub::Quote::inlinify(
159d9c5b 341 $code, $values, Sub::Quote::capture_unroll($cap_name, $captures, 6), 1
e57f338d 342 );
343 } else {
159d9c5b 344 Sub::Quote::inlinify($code, $values, undef, 1);
8c6626cf 345 }
901efe1a 346 } else {
347 my $cap_name = qq{\$${type}_for_${name}};
348 $self->{captures}->{$cap_name} = \$sub;
349 "${cap_name}->(${values})";
625d6219 350 }
a16d301e 351}
352
3a9a65a4 353sub generate_populate_set {
354 my $self = shift;
e57f338d 355 $self->{captures} = {};
3a9a65a4 356 my $code = $self->_generate_populate_set(@_);
e57f338d 357 ($code, delete $self->{captures});
3a9a65a4 358}
359
360sub _generate_populate_set {
361 my ($self, $me, $name, $spec, $source, $test) = @_;
4ced3a94 362 if ($self->has_eager_default($name, $spec)) {
3a9a65a4 363 my $get_indent = ' ' x ($spec->{isa} ? 6 : 4);
d02da2bc 364 my $get_default = $self->_generate_get_default(
365 '$new', $_, $spec
366 );
3a9a65a4 367 my $get_value =
d02da2bc 368 defined($spec->{init_arg})
369 ? "(\n${get_indent} ${test}\n${get_indent} ? ${source}\n${get_indent} : "
370 .$get_default
371 ."\n${get_indent})"
372 : $get_default;
b1f04da5 373 if ($spec->{coerce}) {
374 $get_value = $self->_generate_coerce(
375 $name, $get_value,
376 $spec->{coerce}
377 )
7fe3b886 378 }
3a9a65a4 379 ($spec->{isa}
380 ? " {\n my \$value = ".$get_value.";\n "
5d349892 381 .$self->_generate_isa_check(
382 $name, '$value', $spec->{isa}
383 ).";\n"
32381de9 384 .' '.$self->_generate_simple_set($me, $name, $spec, '$value').";\n"
5d349892 385 ." }\n"
32381de9 386 : ' '.$self->_generate_simple_set($me, $name, $spec, $get_value).";\n"
3a9a65a4 387 )
388 .($spec->{trigger}
389 ? ' '
5d349892 390 .$self->_generate_trigger(
ceea0e38 391 $name, $me, $self->_generate_simple_get($me, $name, $spec),
5d349892 392 $spec->{trigger}
393 )." if ${test};\n"
3a9a65a4 394 : ''
395 );
396 } else {
397 " if (${test}) {\n"
82a5b146 398 .($spec->{coerce}
399 ? " $source = "
400 .$self->_generate_coerce(
b1f04da5 401 $name, $source,
82a5b146 402 $spec->{coerce}
403 ).";\n"
404 : ""
405 )
3a9a65a4 406 .($spec->{isa}
407 ? " "
5d349892 408 .$self->_generate_isa_check(
409 $name, $source, $spec->{isa}
410 ).";\n"
3a9a65a4 411 : ""
412 )
32381de9 413 ." ".$self->_generate_simple_set($me, $name, $spec, $source).";\n"
3a9a65a4 414 .($spec->{trigger}
5d349892 415 ? " "
416 .$self->_generate_trigger(
ceea0e38 417 $name, $me, $self->_generate_simple_get($me, $name, $spec),
5d349892 418 $spec->{trigger}
419 ).";\n"
420 : ""
3a9a65a4 421 )
422 ." }\n";
423 }
424}
425
873df570 426sub _generate_core_set {
427 my ($self, $me, $name, $spec, $value) = @_;
428 my $name_str = perlstring $name;
429 "${me}->{${name_str}} = ${value}";
430}
431
a16d301e 432sub _generate_simple_set {
32381de9 433 my ($self, $me, $name, $spec, $value) = @_;
a16d301e 434 my $name_str = perlstring $name;
2215d4b9 435
32381de9 436 if ($spec->{weak_ref}) {
6998abed 437 $value = '$preserve = '.$value;
438 my $simple = $self->_generate_core_set($me, $name, $spec, $value);
faa9ce11 439 require Scalar::Util;
2215d4b9 440
441 # Perl < 5.8.3 can't weaken refs to readonly vars
442 # (e.g. string constants). This *can* be solved by:
443 #
444 #Internals::SetReadWrite($foo);
445 #Scalar::Util::weaken ($foo);
446 #Internals::SetReadOnly($foo);
447 #
448 # but requires XS and is just too damn crazy
449 # so simply throw a better exception
6998abed 450 my $weak_simple = "my \$preserve; Scalar::Util::weaken(${simple})";
451 Moo::_Utils::lt_5_8_3() ? <<"EOC" : $weak_simple;
2215d4b9 452
6998abed 453 my \$preserve;
2215d4b9 454 eval { Scalar::Util::weaken($simple); 1 } or do {
455 if( \$@ =~ /Modification of a read-only value attempted/) {
faa9ce11 456 require Carp;
2215d4b9 457 Carp::croak( sprintf (
458 'Reference to readonly value in "%s" can not be weakened on Perl < 5.8.3',
459 $name_str,
460 ) );
461 } else {
462 die \$@;
463 }
464 };
465EOC
32381de9 466 } else {
6998abed 467 $self->_generate_core_set($me, $name, $spec, $value);
32381de9 468 }
51a3b106 469}
470
471sub _generate_getset {
a16d301e 472 my ($self, $name, $spec) = @_;
dcae37d3 473 q{(@_ > 1}."\n ? ".$self->_generate_set($name, $spec)
f537c364 474 ."\n : ".$self->_generate_get($name, $spec)."\n )";
51a3b106 475}
476
4db3a740 477sub _generate_delegation {
478 my ($self, $asserter, $target, $args) = @_;
479 my $arg_string = do {
480 if (@$args) {
481 # I could, I reckon, linearise out non-refs here using perlstring
482 # plus something to check for numbers but I'm unsure if it's worth it
483 $self->{captures}{'@curries'} = $args;
484 '@curries, @_';
485 } else {
486 '@_';
487 }
488 };
489 "shift->${asserter}->${target}(${arg_string});";
490}
491
daa05b62 492sub _generate_xs {
dcae37d3 493 my ($self, $type, $into, $name, $slot) = @_;
daa05b62 494 Class::XSAccessor->import(
e6f2e914 495 class => $into,
9f10cfe0 496 $type => { $name => $slot },
497 replace => 1,
daa05b62 498 );
901efe1a 499 $into->can($name);
daa05b62 500}
501
02e9ef74 502sub default_construction_string { '{}' }
503
51a3b106 5041;