--- trunk/lib/Frey/Introspect.pm	2008/11/24 22:56:41	506
+++ trunk/lib/Frey/Introspect.pm	2008/12/06 00:43:28	733
@@ -12,9 +12,11 @@
 use PPI::HTML;
 
 use lib 'lib';
+use Frey::Pod;
 
 extends 'Frey::PPI';
 with 'Frey::Web';
+with 'Frey::Storage';
 
 has 'class' => (
 	is => 'rw',
@@ -107,6 +109,10 @@
 
 	my $class = $self->class;
 
+	my $introspect_path = "var/introspect/$class.yaml";
+	$self->mkbasepath( $introspect_path );
+	my $introspect = $self->load( $introspect_path );
+
 	my ( $superclasses, $roles ) = ( '<b>Role</b>', '' );
 	if ( ! $is_role ) {
 		if ( $meta->superclasses ) {
@@ -114,7 +120,8 @@
 				join(', ',
 					map {
 						my $name = $_->meta->name;
-						qq|<a class="frey-popdown" href="/$name">$name<code>| . $_->meta->dump(2) . qq|</code></a>|;
+						$introspect->{superclass}->{$name} = {};
+						$self->dropdown( $name, $_->meta )
 					}
 					#grep { $_ ne 'Moose::Object' }
 					$meta->superclasses
@@ -122,8 +129,8 @@
 		}
 	}
 
-	my $role_method;
-	my $role_attribute;
+	my $method_from_role;
+	my $attribute_from_role;
 
 	if ( $meta->can('roles') ) {
 		my $role_nr = 1;
@@ -131,36 +138,47 @@
 			grep { ! m/\Q$class\E/ }	# skip me
 			map {
 				my $name = $_->name;
-				$role_method->{    $_ }->{$name} = $role_nr foreach $_->get_method_list;
-				$role_attribute->{ $_ }->{$name} = $role_nr foreach $_->get_attribute_list;
-				qq|<a class="frey-popdown" href="/$name">$name<code>| . $name->meta->dump(2) . qq|</code></a><sup>| . $role_nr++ . qq|</sup>|;
+				$introspect->{roles}->{$name} = {};
+				$method_from_role->{    $_ }->{$name} = $role_nr foreach $_->get_method_list;
+				$attribute_from_role->{ $_ }->{$name} = $role_nr foreach $_->get_attribute_list;
+				$self->dropdown( $name, $name->meta ) . qq|<sup>| . $role_nr++ . qq|</sup>|;
 			}
 			$meta->calculate_all_roles
 		);
 		$roles = qq| with roles: $roles| if $roles;
 	}
-	warn "# role_method ",dump( $role_method );
+	warn "# method_from_role ",dump( $method_from_role );
 
 	my @methods;
 	@methods = map {
 		my $name = $_;
-		if ( $role_method->{$name} ) {
-			my ( $role_name, $nr ) = each %{ $role_method->{$name} };
+		if ( $method_from_role->{$name} ) {
+			my ( $role_name, $nr ) = each %{ $method_from_role->{$name} };
+			$introspect->{methods}->{$name}->{role} = $role_name;
 			$name .= qq|<sup title="$role_name">$nr</sup>|;
+		} else {
+			$introspect->{methods}->{$name} = {};
 		}
 		qq|<td class="m">$name</td>|
-	} $self->class_methods( $class );
+	} sort $self->class_methods( $class );
 
 	my @attributes;
 	if ( $meta->get_attribute_list ) {
 		@attributes = map {
 			my $name = $_;
+			$introspect->{attribute}->{$name} = {}; 
+			my $html_name = $name;
 			my $attr = $meta->get_attribute($name);
 			confess "$class attribute $name isn't blessed ",dump( $attr ) unless blessed $attr;
 			warn "## attr $name ref ",ref( $attr ) if $self->debug;
-			my ( $before, $title, $after ) = ( '', '', '' );
-			( $before, $title, $after ) = ( '<b>', ' title="required"', '</b>' ) if $attr->can('is_required') && $attr->is_required;
-			foreach my $check ( qw/has_type_constraint has_handles is_weak_ref is_required is_lazy should_coerce should_auto_deref has_default has_trigger has_documentation has_applied_traits/ ) {
+
+			my ( $title, $properties ) = ( '', '' );
+			if ( $attr->can('is_required') && $attr->is_required ) {
+				( $html_name, $title ) = ( "<b>$name</b>", ' title="required"' );
+				$introspect->{attribute}->{$name}->{required} = 1;
+			}
+
+			foreach my $check ( qw/has_type_constraint has_handles is_weak_ref is_required is_lazy should_coerce should_auto_deref has_default has_trigger has_applied_traits/ ) {
 				my $getter;
 				
 				$getter = $check;
@@ -168,25 +186,25 @@
 				
 				if ( $attr->can($check) && $attr->$check ) {
 					if ( $getter eq $check ) {
-						$after .= "$check";
+						$properties .= "$check";
 					} else {
-						$after .= qq{<span class="frey-popdown">$check};
-						# we need dump here instead of $attr->$getter->dump because default can return scalar
-						$after .= '<code>' . dump( $attr->$getter ) . '</code>' if $getter ne $check;
-						$after .= '</span>';
+						# we need our dump here instead of $attr->$getter->dump because default can return scalar
+						$properties .= $self->dropdown( $check, $attr->$getter );
 					}
 				}
-				$after .= ' ';
+				$properties .= ' ';
 			}
 			my $type = $attr->can('has_type_constraint') && $attr->has_type_constraint ? $attr->type_constraint->name : '';
 
-			if ( $role_attribute->{$name} ) {
-				my ( $role_name, $nr ) = each %{ $role_attribute->{$name} };
+			if ( $attribute_from_role->{$name} ) {
+				my ( $role_name, $nr ) = each %{ $attribute_from_role->{$name} };
 				$name .= qq|<sup title="$role_name">$nr</sup>|;
 			}
 
-			qq|<td class="a">$before $name</td><td class="t">$type</td><td>$after</td>|
-		} $meta->get_attribute_list
+			eval { $properties = $attr->documentation . ' ' . $properties };
+
+			qq|<td class="a">$html_name</td><td class="t">$type</td><td>$properties</td>|
+		} sort $meta->get_attribute_list
 	}
 
 	my $table = qq|<table class="frey-introspect"><tr><th class="m">Methods</th><th class="a">Attributes</th><th>Type</th><th class="p">Properties</th></tr>|;
@@ -198,10 +216,11 @@
 	}
 	$table .= qq|</table>|;
 
-	my $pod = Frey::Pod->new( class => $class )->as_markup;
-	$pod = $pod->{body} if ref($pod);
-
 	my $path = $self->class_path( $class );
+
+	my $pod = Frey::Pod->new( class => $class, request_url => $self->request_url )->as_markup;
+	return $pod if $path =~ m{\.pod};
+
 	my $Document = PPI::Document->new( $path );
 
 	# Create a reusable syntax highlighter
@@ -223,31 +242,36 @@
 #	$source =~ s{^.*<body[^>]+>}{}s;
 #	$source =~ s{</body.*$}{}s;
 
-
 	my $runnable = join("\n",
 		map {
-			qq|<a target="$_" href="/$class/$_">$_</a>|
+			$introspect->{runnable}->{$_} = {};
+			my $short = $_;
+			$short =~ s{_as_(?:markup|data|sponge)$}{};
+			qq|<a target="$class" href="/$class/$_" title="/$class/$_">$short</a>|
 		} $self->class_runnable( $class )
 	);
-	$runnable = " runnable: $runnable" if $runnable;
+	$runnable = "<br>run: $runnable" if $runnable;
+
+	$self->store( $introspect_path, $introspect );
 
 	$self->title( $class );
 
 	my $has_tests = '';
-	if ( my @tests = $self->has_tests ) {
+	my @tests = grep { defined $_ } $self->has_tests;
+	if ( @tests ) {
 		$has_tests = 
 		'<br/>test' . ( $#tests > 0 ? 's' : '' ) . ': ' .	
 		join("\n", map {
-			qq|<a target="test" href="/Frey::Test::Runner/as_markup?test=$_">$_</a>|
+			qq|<a target="$class" href="/Frey::Test::Runner/as_markup?test=$_">$_</a>|
 		} @tests );
 	}
 
 	return join("\n",
 		qq|<h1>$class</h1>|,
 		qq|<div class="frey-introspect">$superclasses\n$roles\n$runnable\n|,
+		$has_tests,
 		$pod    ? qq|<a class="frey-skip" href="#___top" title="Skip to POD"    >pod</a>|    : '',
 		$source ? qq|<a class="frey-skip" href="#source" title="Skip to source" >source</a>| : '',
-		$has_tests,
 		qq|$table\n$pod\n</div>\n|,
 		qq|<h1>Source</h1><a name="source"></a><div class="frey-source">$source</div>|,
 	);