--- trunk/lib/Frey/Introspect.pm	2008/11/05 08:20:51	278
+++ trunk/lib/Frey/Introspect.pm	2008/11/05 19:57:49	310
@@ -10,7 +10,7 @@
 
 use lib 'lib';
 
-extends 'Frey';
+extends 'Frey::ClassLoader';
 with 'Frey::Web';
 
 has 'class' => (
@@ -32,7 +32,7 @@
 sub joose {
 	my ($self) = @_;
 
-	my ( $meta, $is_role ) = $self->load_class;
+	my ( $meta, $is_role ) = $self->class_meta;
 
 	if ( ! $is_role ) {
 		my @superclasses = map{ $_->meta->name }
@@ -77,7 +77,7 @@
 	$out .= "\t\t},\n\t\tmeta: Frey.HTML,
 		classMethods: {
 			renderHTML: function () {
-				return new Joose.SimpleRequest().getText(\"/Frey-Introspect?class=" . $self->class . "\")
+				return new Joose.SimpleRequest().getText(\"/" . $self->class . "\")
 			},\n";
 
 	$out .= "\t\t},\n";
@@ -109,7 +109,7 @@
 sub methods {
 	my $self = shift;
 
-	my ( $meta, $is_role ) = $self->load_class;
+	my ( $meta, $is_role ) = $self->class_meta;
 
 	my $attr;
 	$attr->{$_}++ foreach $meta->get_attribute_list;
@@ -119,14 +119,6 @@
 	return sort @methods;
 }
 
-use Frey::ClassLoader;
-
-sub load_class {
-	my $self = shift;
-	return Frey::ClassLoader->load_package( $self->class );
-}
-
-
 =head1 OUTPUT GENERATION
 
 =head2 markup
@@ -140,12 +132,51 @@
 
 	$self->add_head( 'static/introspect.css' );
 
-	my ( $meta, $is_role ) = $self->load_class;
+	my ( $meta, $is_role ) = $self->class_meta;
 
 	my $class = $self->class;
 
+	my ( $superclasses, $roles ) = ( '<b>Role</b>', '' );
+	if ( ! $is_role ) {
+		if ( $meta->superclasses ) {
+			$superclasses = 'Superclasses: ' .
+				join(', ',
+					map {
+						my $name = $_->meta->name;
+						qq|<a class="frey-popdown" href="/$name">$name<code>| . $_->meta->dump(2) . qq|</code></a>|;
+					}
+					#grep { $_ ne 'Moose::Object' }
+					$meta->superclasses
+				);
+		}
+	}
+
+	my $role_method;
+
+	if ( $meta->can('roles') ) {
+		my $role_nr = 1;
+		$roles = join(' ',
+			grep { ! m/\Q$class\E/ }	# skip me
+			map {
+				my $name = $_->name;
+				$role_method->{ $_ }->{$name} = $role_nr foreach $_->get_method_list;
+				qq|<a class="frey-popdown" href="/$name">$name<code>| . $name->meta->dump(2) . qq|</code></a><sup>| . $role_nr++ . qq|</sup>|;
+			}
+			$meta->calculate_all_roles
+		);
+		$roles = qq| with roles: $roles| if $roles;
+	}
+	warn "# role_method ",dump( $role_method );
+
 	my @methods;
-	@methods = map { qq|<td class="m">$_</td>| } $self->methods;
+	@methods = map {
+		my $method = $_;
+		if ( $role_method ) {
+			my ( $name, $nr ) = each %{ $role_method->{$_} };
+			$method .= qq|<sup title="$name">$nr</sup>|;
+		}
+		qq|<td class="m">$method</td>|
+	} $self->methods;
 
 	my @attributes;
 	if ( $meta->get_attribute_list ) {
@@ -154,7 +185,7 @@
 			my ( $before, $title, $after ) = ( '', '', '' );
 			( $before, $title, $after ) = ( '<b>', ' title="required"', '</b>' ) if $attr->is_required;
 warn $attr->dump(3);
-			foreach my $check ( qw/has_type_constraint has_handles is_weak_ref is_required is_lazy should_coerce should_auto_deref has_trigger has_documentation has_applied_traits/ ) {
+			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 $getter;
 				
 				$getter = $check;
@@ -165,17 +196,19 @@
 						$after .= "$check";
 					} else {
 						$after .= qq{<span class="frey-popdown">$check};
-						$after .= '<code>' . $attr->$getter->dump . '</code>' if $getter ne $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>';
 					}
 				}
 				$after .= ' ';
 			}
-			qq|<td class="a">$before $_</td><td>$after</td>|
+			my $type = $attr->has_type_constraint ? $attr->type_constraint->name : '';
+			qq|<td class="a">$before $_</td><td class="t">$type</td><td>$after</td>|
 		} sort $meta->get_attribute_list
 	}
 
-	my $table = qq|<table class="frey-object-browser"><tr><th class="m">Methods</th><th class="a">Attributes</th><th class="p">Properties</th></tr>|;
+	my $table = qq|<table class="frey-object-browser"><tr><th class="m">Methods</th><th class="a">Attributes</th><th>Type</th><th class="p">Properties</th></tr>|;
 	while ( @methods || @attributes ) {
 		my ($m,$a) = ( shift @methods, shift @attributes );
 		$m ||= '<td></td>';
@@ -184,33 +217,6 @@
 	}
 	$table .= qq|</table>|;
 
-	my ( $superclasses, $roles ) = ( '<b>Role</b>', '' );
-	if ( ! $is_role ) {
-		if ( $meta->superclasses ) {
-			$superclasses = 'Superclasses: ' .
-				join(', ',
-					map {
-						my $name = $_->meta->name;
-						qq|<a class="frey-popdown" href="/Frey-Introspect?class=$name">$name<code>| . $_->meta->dump(2) . qq|</code></a>|;
-					}
-					#grep { $_ ne 'Moose::Object' }
-					$meta->superclasses
-				);
-		}
-	}
-
-	if ( $meta->can('roles') ) {
-		$roles = join(', ',
-			grep { ! m/\Q$class\E/ }	# skip me
-			map {
-				my $name = $_->name;
-				qq|<a class="frey-popdown" href="/Frey-Introspect?class=$name">$name<code>| . $name->meta->dump(2) . qq|</code></a>|;
-			}
-			$meta->calculate_all_roles
-		);
-		$roles = " with roles: $roles" if $roles;
-	}
-
 	my $pod = Frey::Pod->new( class => $class )->markup;
 
 	use Frey::Run;
@@ -220,9 +226,9 @@
 	my $html = $self->page(
 		title => "Introspect $class",
 		body => qq|<h1>$class</h1>|
-			. qq|$superclasses\n$roles\n$execute\n|
+			. qq|<div class="frey-object-browser">$superclasses\n$roles\n$execute\n|
 			. ( $pod ? qq|<a href="#___top" title="Skip to POD" style="font-size: 80%; color: #aaa;">&darr;pod&darr</a>| : '' )
-			. qq|$table\n$pod|
+			. qq|$table\n$pod</div>|
 
 	);