From eeb792decae28e85e487ebeeb6d56f1994b56c62 Mon Sep 17 00:00:00 2001 From: Jesse Luehrs Date: Mon, 21 Feb 2011 01:07:16 -0600 Subject: handle type checks and auto_deref on values returned from services --- lib/MooseX/Bread/Board/Meta/Role/Accessor.pm | 26 ++++++++++++++++++++++ lib/MooseX/Bread/Board/Meta/Role/Attribute.pm | 31 +++++++++++++++++++++------ 2 files changed, 51 insertions(+), 6 deletions(-) create mode 100644 lib/MooseX/Bread/Board/Meta/Role/Accessor.pm (limited to 'lib') diff --git a/lib/MooseX/Bread/Board/Meta/Role/Accessor.pm b/lib/MooseX/Bread/Board/Meta/Role/Accessor.pm new file mode 100644 index 0000000..10748e0 --- /dev/null +++ b/lib/MooseX/Bread/Board/Meta/Role/Accessor.pm @@ -0,0 +1,26 @@ +package MooseX::Bread::Board::Meta::Role::Accessor; +use Moose::Role; + +around _inline_get => sub { + my $orig = shift; + my $self = shift; + my ($instance) = @_; + + my $attr = $self->associated_attribute; + + return 'do {' . "\n" + . 'my $val;' . "\n" + . 'if (' . $self->_inline_has($instance) . ') {' . "\n" + . '$val = ' . $self->$orig($instance) . ';' . "\n" + . '}' . "\n" + . 'else {' . "\n" + . '$val = ' . $instance . '->get_service(\'' . $attr->name . '\')->get;' . "\n" + . $self->_inline_check_constraint('$val') + . '}' . "\n" + . '$val' . "\n" + . '}'; +}; + +no Moose::Role; + +1; diff --git a/lib/MooseX/Bread/Board/Meta/Role/Attribute.pm b/lib/MooseX/Bread/Board/Meta/Role/Attribute.pm index fe98bb7..9f4c16e 100644 --- a/lib/MooseX/Bread/Board/Meta/Role/Attribute.pm +++ b/lib/MooseX/Bread/Board/Meta/Role/Attribute.pm @@ -91,17 +91,36 @@ around get_value => sub { return $self->$orig($instance) if $self->has_value($instance); - return $instance->get_service($self->name)->get; + my $val = $instance->get_service($self->name)->get; + + $self->verify_against_type_constraint($val, instance => $instance) + if $self->has_type_constraint; + + if ($self->should_auto_deref) { + if (ref($val) eq 'ARRAY') { + return wantarray ? @$val : $val; + } + elsif (ref($val) eq 'HASH') { + return wantarray ? %$val : $val; + } + else { + die 'XXX'; + } + } + else { + return $val; + } }; -around inline_get => sub { +around accessor_metaclass => sub { my $orig = shift; my $self = shift; - my ($instance) = @_; - return 'do { (' . $self->inline_has($instance) . ')' . "\n" - . '? (' . $self->$orig($instance) . ')' . "\n" - . ': (' . $instance . '->get_service(\'' . $self->name . '\')->get) }'; + return Moose::Meta::Class->create_anon_class( + superclasses => [ $self->$orig(@_) ], + roles => [ 'MooseX::Bread::Board::Meta::Role::Accessor' ], + cache => 1 + )->name; }; no Moose::Role; -- cgit v1.2.3-54-g00ecf