--- trunk/lib/Frey/ClassLoader.pm	2008/07/11 19:19:42	100
+++ trunk/lib/Frey/ClassLoader.pm	2008/11/17 19:21:06	382
@@ -1,69 +1,127 @@
 package Frey::ClassLoader;
 use Moose;
 
+=head1 DESCRIPTION
+
+Load L<Frey> classes
+
+=cut
+
 extends 'Frey';
 
 use Data::Dump qw/dump/;
 use File::Find;
 
-has 'classes' => (
-	is => 'ro',
-#	isa => 'HashRef[Str]',
-	default => sub {
-		my $self = shift;
-		# FIXME there must be better way to do this in Moose style
-		my $classes;
-		finddepth({ no_chdir => 1, wanted => sub {
-			return unless s/\.pm$//;
-			my @a = split(m!/!,$_);
-			my $package = join('::', @a[ 1 .. $#a ]);
-			warn ">> $_ ",dump( @a ), " >> $package\n" if $self->debug;
-			push @$classes, { $package => "$_.pm" };
-		} }, 'lib');
-		warn "## classes = ",dump( $classes ) if $self->debug;
-		$classes;
-	},
-	lazy => 1,
-);
+our $class_path;
+our @classes;
+
+=head2 classes
+
+Return all local classes by reading from disk
+
+  my @classes = $o->classes;
+
+=cut
+
+sub classes {
+	my $self = shift;
+	return @classes if @classes;
+
+	# FIXME there must be better way to do this in Moose style
+	finddepth({ no_chdir => 1, wanted => sub {
+		return unless s/\.pm$//;
+		my @a = split(m!/!,$_);
+		my $class = join('::', @a[ 1 .. $#a ]);
+		warn ">> $_ ",dump( @a ), " >> $class\n" if $self->debug;
+		$class_path->{ $class } = "$_.pm";
+	} }, 'lib');
+	warn "## class_path = ",dump( $class_path ) if $self->debug;
+
+	@classes = sort keys %$class_path;
+}
 
-=head2 load_package
+=head2 class_path
 
-  my ( $class, $meta, $is_role ) = $o->load_package( 'Some::Package' );
+Return any local or loaded class
+
+  $path = $o->class_path( $class );
 
 =cut
 
-sub load_package {
-	my ( $self, $package ) = @_;
+sub class_path {
+	my ( $self, $class ) = @_;
+	$self->classes unless $class_path;
+	if ( ! defined $class_path->{$class} ) {
+		my $path = $class;
+		$path =~ s{::}{/}g;
+		$path .= '.pm';
+		$path = $INC{$path};
+		warn "# $class from INC $path";
+		$class_path->{$class} = $path;
+	}
+	return $class_path->{$class};
+}
 
-	#intercept role application so we can accurately generate
-	#method and attribute information for the parent class.
-	#this is fragile, but there is not better way that i am aware of
-	my $rmeta = Moose::Meta::Role->meta;
-	$rmeta->make_mutable if $rmeta->is_immutable;
-	my $original_apply = $rmeta->get_method("apply")->body;
-	$rmeta->remove_method("apply");
-	my @roles_to_apply;
-	$rmeta->add_method("apply", sub{push(@roles_to_apply, [@_])});
-	#load the package with the hacked Moose::Meta::Role
-
-	#eval { Class::MOP::load_class($package); };
-	#confess "Failed to load package ${package} $@" if $@;
-	Class::MOP::load_class($package);
+=head2 loaded_classes
 
-	my $meta = $package->meta;
+  my $available = $o->loaded_classes;
+  $available->{'Frey'} # true
 
-	my ($class, $is_role);
-	if($package->meta->isa('Moose::Meta::Role')){
+=cut
+
+our $loaded_class;
+sub loaded_classes { $loaded_class };
+
+=head2 class_meta
+
+  my ( $meta, $is_role ) = $o->class_meta( 'Some::Class' );
+
+=cut
+
+sub class_meta {
+	my ( $self, $class ) = @_;
+
+	$class ||= $self->class if $self->can('class');
+	warn "# class_meta $class";
+
+	$self->load_class($class);
+
+	if ( ! $class->can('meta') ) {
+		my $instance = Moose::Meta::Class->create_anon_class;
+		warn "class $class isn't Moose, faking anon class";
+		return ( $instance, 0 );
+	}
+
+	my $meta;
+	my $is_role = 0;
+	my $instance;
+
+	if($class->meta->isa('Moose::Meta::Role')){
 		$is_role = 1;
-		# we need to apply the role to a class to be able to properly introspect it
-		$class = Moose::Meta::Class->create_anon_class;
-		$original_apply->($meta, $class);
+		$instance = Moose::Meta::Class->create_anon_class;
+		$class->meta->apply( $instance );
+		$meta = $class->meta;
+		die $@ if $@;
 	} else {
-		#roles don't have superclasses ...
-		$class = $meta;
+		$meta = $class->meta;
 	}
-	return ( $class, $meta, $is_role );
+	return ( $meta, $is_role );
 }
 
+sub load_class {
+	my ( $self, $class ) = @_;
+	eval {
+		Class::MOP::load_class($class) if ! $loaded_class->{$class}++;
+	};
+	warn $@ if $@; # && $@ !~ m/role/;
+	warn "# load_class $class" if $self->debug && $loaded_class->{$class} == 1;
+}
+
+sub load_all_classes {
+	my $self = shift;
+	warn "# loaded_class = ",dump( $loaded_class ) if $self->debug;
+	$self->load_class( $_ ) foreach ( $self->classes );
+	$loaded_class;
+}
 
 1;