rewriting ->component and friends to use BB. Completely broken
[catagits/Catalyst-Runtime.git] / lib / Catalyst.pm
index 1fabc37..dc24c14 100644 (file)
@@ -651,18 +651,30 @@ If you want to search for controllers, pass in a regexp as the argument.
 
 sub controller {
     my ( $c, $name, @args ) = @_;
+    my $container = $c->container->get_sub_container('controller');
 
-    my $appclass = ref($c) || $c;
+    my $appclass = ref $c || $c;
     if( $name ) {
-        unless ( ref($name) ) { # Direct component hash lookup to avoid costly regexps
-            my $check = $appclass."::Controller::".$name;
-            my $container = $c->container->get_sub_container('controller');
-            return $c->_filter_component( $container->resolve(service => "$check"), @args )
-                if $container->has_service($check);
+        if ( !ref $name ) { # Direct component hash lookup to avoid costly regexps
+            return $container->resolve(service => $name, parameters => { context => [ $c, @args ] } )
+                if $container->has_service($name);
+        }
+        elsif ( ref $name eq 'Regexp' ) {
+            my @comps = $container->get_service_list;
+            my @result;
+            for (@comps) {
+                push @result, $container->resolve( service => $_, parameters => { context => [ $c, @args ] } )
+                    if m/$name/;
+            }
+            return @result;
         }
-        my @result = $c->_comp_search_prefixes( $name, qw/Controller C/ );
-        return map { $c->_filter_component( $_, @args ) } @result if ref $name;
-        return $c->_filter_component( $result[ 0 ], @args );
+        else {
+            my $short_name = ref $name;
+            $short_name =~ s/^${appclass}::(C|Controller)//;
+            return $container->resolve( service => $short_name, parameters => { context => [ $c, @args ] } )
+                if $container->has_service($short_name);
+        }
+        return;
     }
 
     return $c->component( $c->action->class );
@@ -692,16 +704,29 @@ If you want to search for models, pass in a regexp as the argument.
 sub model {
     my ( $c, $name, @args ) = @_;
     my $appclass = ref($c) || $c;
+    my $container = $c->container->get_sub_container('model');
+
     if( $name ) {
-        unless ( ref($name) ) { # Direct component hash lookup to avoid costly regexps
-            my $check = $appclass."::Model::".$name;
-            my $container = $c->container->get_sub_container('model');
-            return $c->_filter_component( $container->resolve(service => "$check"), @args )
-                if $container->has_service($check);
+        if ( !ref $name ) { # Direct component hash lookup to avoid costly regexps
+            return $container->resolve( service => $name, parameters => { context => [ $c, @args ] } )
+                if $container->has_service($name);
         }
-        my @result = $c->_comp_search_prefixes( $name, qw/Model M/ );
-        return map { $c->_filter_component( $_, @args ) } @result if ref $name;
-        return $c->_filter_component( $result[ 0 ], @args );
+        elsif ( ref $name eq 'Regexp' ) {
+            my @comps = $container->get_service_list;
+            my @result;
+            for (@comps) {
+                push @result, $container->resolve( service => $_, parameters => { context => [ $c, @args ] } )
+                    if m/$name/;
+            }
+            return @result;
+        }
+        else {
+            my $short_name = ref $name;
+            $short_name =~ s/^${appclass}::(M|Model)//;
+            return $container->resolve( service => $short_name, parameters => { context => [ $c, @args ] } )
+                if $container->has_service($short_name);
+        }
+        return;
     }
 
     if (ref $c) {
@@ -713,7 +738,8 @@ sub model {
     return $c->model( $appclass->config->{default_model} )
       if $appclass->config->{default_model};
 
-    my( $comp, $rest ) = $c->_comp_search_prefixes( undef, qw/Model M/);
+# FIXME: will this still be mantained?
+    my( $comp, $rest ) = $container->get_service_list;
 
     if( $rest ) {
         $c->log->warn( Carp::shortmess('Calling $c->model() will return a random model unless you specify one of:') );
@@ -723,7 +749,7 @@ sub model {
         $c->log->warn( 'NB: in version 5.81, the "random" behavior will not work at all.' );
     }
 
-    return $c->_filter_component( $comp );
+    return $container->resolve( service => $comp, parameters => { context => [ $c, @args ] } );
 }
 
 
@@ -750,23 +776,35 @@ If you want to search for views, pass in a regexp as the argument.
 
 sub view {
     my ( $c, $name, @args ) = @_;
-
     my $appclass = ref($c) || $c;
-    if( $name ) {
-        unless ( ref($name) ) { # Direct component hash lookup to avoid costly regexps
-            my $check = $appclass."::View::".$name;
-            my $container = $c->container->get_sub_container('view');
-            if ($container->has_service($check)) {
+    my $container = $c->container->get_sub_container('view');
 
-                return $c->_filter_component( $container->resolve(service => $check), @args );
+    if( $name ) {
+        if ( !ref $name ) { # Direct component hash lookup to avoid costly regexps
+            if ( $container->has_service($name) ) {
+                return $container->resolve( service => $name, parameters => { context => [ $c, @args ] } );
             }
             else {
-                $c->log->warn( "Attempted to use view '$check', but does not exist" );
+                $c->log->warn( "Attempted to use view '$name', but does not exist" );
             }
         }
-        my @result = $c->_comp_search_prefixes( $name, qw/View V/ );
-        return map { $c->_filter_component( $_, @args ) } @result if ref $name;
-        return $c->_filter_component( $result[ 0 ], @args );
+        elsif ( ref $name eq 'Regexp' ) {
+            my @comps = $container->get_service_list;
+            my @result;
+            for (@comps) {
+                push @result, $container->resolve( service => $_, parameters => { context => [ $c, @args ] } )
+                    if m/$name/;
+            }
+            return @result;
+        }
+        else {
+            my $short_name = ref $name;
+            $short_name =~ s/^${appclass}::(V|View)//;
+            return $container->resolve( service => $short_name, parameters => { context => [ $c, @args ] } )
+                if $container->has_service($short_name);
+        }
+
+        return;
     }
 
     if (ref $c) {
@@ -778,7 +816,7 @@ sub view {
     return $c->view( $appclass->config->{default_view} )
       if $appclass->config->{default_view};
 
-    my( $comp, $rest ) = $c->_comp_search_prefixes( undef, qw/View V/);
+    my( $comp, $rest ) = $container->get_service_list;
 
     if( $rest ) {
         $c->log->warn( 'Calling $c->view() will return a random view unless you specify one of:' );
@@ -788,7 +826,7 @@ sub view {
         $c->log->warn( 'NB: in version 5.81, the "random" behavior will not work at all.' );
     }
 
-    return $c->_filter_component( $comp );
+    return $container->resolve( service => $comp, parameters => { context => [ $c, @args ] } );
 }
 
 =head2 $c->controllers
@@ -846,34 +884,30 @@ disable_component_resolution_regex_fallback to a true value.
 =cut
 
 sub component {
-    my ( $c, $name, @args ) = @_;
+    my ( $c, $component, @args ) = @_;
 
-    if( $name ) {
-        my $comps = $c->components;
-
-        if( !ref $name ) {
-            # is it the exact name?
-            return $c->_filter_component( $comps->{ $name }, @args )
-                       if exists $comps->{ $name };
+    my ($type, $name) = _get_component_type_name($component);
+    my $container = $c->container->get_sub_container($type);
 
-            # perhaps we just omitted "MyApp"?
-            my $composed = ( ref $c || $c ) . "::${name}";
-            return $c->_filter_component( $comps->{ $composed }, @args )
-                       if exists $comps->{ $composed };
+    if( $component ) {
+        my $comps = $c->components;
 
-            # search all of the models, views and controllers
-            my( $comp ) = $c->_comp_search_prefixes( $name, qw/Model M Controller C View V/ );
-            return $c->_filter_component( $comp, @args ) if $comp;
+        if( !ref $component ) {
+            return $container->resolve( service => $name, parameters => { context => [ $c, @args ] } )
+               if $container->has_service( $name );
         }
 
+        return
+            if $c->config->{disable_component_resolution_regex_fallback};
+
         # This is here so $c->comp( '::M::' ) works
-        my $query = ref $name ? $name : qr{$name}i;
+        my $query = ref $component ? $component : qr{$component}i;
 
         my @result = grep { m{$query} } keys %{ $c->components };
-        return map { $c->_filter_component( $_, @args ) } @result if ref $name;
+        return map { $c->_filter_component( $_, @args ) } @result if ref $component;
 
         if( $result[ 0 ] ) {
-            $c->log->warn( Carp::shortmess(qq(Found results for "${name}" using regexp fallback)) );
+            $c->log->warn( Carp::shortmess(qq(Found results for "${component}" using regexp fallback)) );
             $c->log->warn( 'Relying on the regexp fallback behavior for component resolution' );
             $c->log->warn( 'is unreliable and unsafe. You have been warned' );
             return $c->_filter_component( $result[ 0 ], @args );
@@ -1616,9 +1650,10 @@ around components => sub {
     for my $component ( keys %$comps ) {
         $components->{ $component } = $comps->{$component};
 
-        my $type = _get_component_type($component);
+        my ($type, $name) = _get_component_type_name($component);
 
-        $containers->{$type}->add_service(Bread::Board::BlockInjection->new( name => $component, block => sub { return $class->setup_component($component) } ));
+# FIXME: shouldn't the service name be $name?
+        $containers->{$type}->add_service(Catalyst::BlockInjection->new( name => $name, block => sub { return $class->setup_component($component) } ));
     }
 
     return $class->$orig($components);
@@ -2516,8 +2551,8 @@ sub setup_components {
 
     for my $component (@comps) {
         my $instance = $class->components->{ $component } = $class->setup_component($component);
-        if ( my $type = _get_component_type($component) ) {
-            $containers->{$type}->add_service(Bread::Board::BlockInjection->new( name => $component, block => sub { return $instance } ));
+        if ( my ($type, $name) = _get_component_type_name($component) ) {
+            $containers->{$type}->add_service(Catalyst::BlockInjection->new( name => $name, block => sub { return $instance } ));
         }
         my @expanded_components = $instance->can('expand_modules')
             ? $instance->expand_modules( $component, $config )
@@ -2530,8 +2565,8 @@ sub setup_components {
                 qq{Please switch your class names to ::Model::, ::View:: and ::Controller: as appropriate.\n}
             ) if $deprecatedcatalyst_component_names;
 
-            if (my $type = _get_component_type($component)) {
-                $containers->{$type}->add_service(Bread::Board::BlockInjection->new( name => $component, block => sub { return $class->setup_component($component) } ));
+            if (my ($type, $name) = _get_component_type_name($component)) {
+                $containers->{$type}->add_service(Catalyst::BlockInjection->new( name => $name, block => sub { return $class->setup_component($component) } ));
             }
 
             $class->components->{ $component } = $class->setup_component($component);
@@ -2539,14 +2574,19 @@ sub setup_components {
     }
 }
 
-sub _get_component_type {
+sub _get_component_type_name {
     my $component = shift;
     my @parts     = split /::/, $component;
 
-    for (@parts) {
-        return 'controller' if /^c|controller$/i;
-        return 'model'      if /^m|model$/i;
-        return 'view'       if /^v|view$/i;
+    while (my $type = shift @parts) {
+        return ('controller', join '::', @parts)
+            if $type =~ /^(c|controller)$/i;
+
+        return ('model', join '::', @parts)
+            if $type =~ /^(m|model)$/i;
+
+        return ('view', join '::', @parts)
+            if $type =~ /^(v|view)$/i;
     }
 }
 
@@ -2894,7 +2934,7 @@ the plugin name does not begin with C<Catalyst::Plugin::>.
         my $class = ref $proto || $proto;
 
         Class::MOP::load_class( $plugin );
-        $class->log->warn( "$plugin inherits from 'Catalyst::Component' - this is decated and will not work in 5.81" )
+        $class->log->warn( "$plugin inherits from 'Catalyst::Component' - this is deprecated and will not work in 5.81" )
             if $plugin->isa( 'Catalyst::Component' );
         $proto->_plugins->{$plugin} = 1;
         unless ($instant) {