--- trunk/lib/Frey/Web.pm	2008/11/18 00:55:23	392
+++ trunk/lib/Frey/Web.pm	2008/11/25 17:15:18	519
@@ -1,33 +1,53 @@
 package Frey::Web;
 use Moose::Role;
 
+with 'Frey::Backtrace';
+
 use Frey::Types;
 
 use Continuity::Widget::DomNode;
 use Data::Dump qw/dump/;
-use Carp qw/confess/;
+use Carp qw/confess cluck/;
 use File::Slurp;
 
+use Frey::Bookmarklet;
+use Frey::ClassBrowser;
+use Frey::SVK;
+
 has 'head' => (
 	is => 'rw',
 	isa => 'ArrayRef[Str]',
 	default => sub { [ 'static/frey.css' ] },
 );
 
-has 'status' => (
+has 'request_url' => (
 	is => 'rw',
-	isa => 'ArrayRef[HashRef[Str]]',
+	isa => 'Uri', coerce => 1,
+	default => '/',
+);
+
+has 'title' => (
+	is => 'rw',
+	isa => 'Str',
 	lazy => 1,
-	default => sub { [
-		{ 'Bookmarklets' => Frey::Bookmarklet->markup },
-		{ 'ClassBrowser' => Frey::ClassBrowser->markup },
-	] },
+	default => sub {
+		my ($self) = @_;
+		ref($self);
+	},
 );
 
-has 'request_url' => (
+has 'content_type' => (
 	is => 'rw',
-	isa => 'Uri', coerce => 1,
-	default => '/',
+	isa => 'Str',
+	default => 'text/html',
+	documentation => 'Content-type header',
+);
+
+has 'dump_max_bytes' => (
+	is => 'rw',
+	isa => 'Int',
+	default => 4096,
+	documentation => 'Maximum dump size sent to browser before truncation',
 );
 
 =head2 inline_smaller_than
@@ -66,6 +86,8 @@
 			$out .= $self->_inline_path( $path ) ?
 				qq|<!-- $path --><style type="text/css">\n| . read_file( $path ) . qq|\n</style>| :
 				qq|<link type="text/css" rel="stylesheet" href="/$path" media="screen">|;
+		} elsif ( $path =~ m{<.+>}s ) {
+			$out .= $path;
 		} else {
 			confess "don't know how to render $path";
 		}
@@ -80,6 +102,8 @@
 
   my $size = $o->add_head( 'path/to/external.css' );
 
+  $o->add_head( '<!-- html content -->' );
+
 =cut
 
 sub add_head {
@@ -87,17 +111,19 @@
 	return if ! defined $path || $path eq '';
 	$path =~ s!^/!!;
 
-	if ( -e $path ) {
+	if ( $path =~ m{<.*>}s ) {
+		push @{ $self->head }, $path;
+	} elsif ( -e $path ) {
 		if ( $path =~ m/\.(?:js|css)$/ ) {
 			push @{ $self->head }, $path;
 		} else {
 			confess "can't add_head( $path ) it's not js or css";
 		}
+		return -s $path;
 	} else {
 		confess "can't find $path: $!";
 	}
 
-	return -s $path;
 }
 
 our $reload_counter = 0;
@@ -113,42 +139,100 @@
 
 =cut
 
-use Frey::Bookmarklet;
-use Frey::ClassBrowser;
+our @status;
+sub status { @status };
 
 sub page {
 	my $self = shift;
 	my $a = {@_};
 
+	warn "## page ",dump($a);
+
 	$reload_counter++;
 
 	my $status_line = '';
-	foreach my $part ( @{ $self->status } ) {
+
+	unshift @status, { 'ClassBrowser' => Frey::ClassBrowser->new( usage_on_top => 0 )->as_markup };
+	unshift @status, { 'Bookmarklets' => Frey::Bookmarklet->new->as_markup };
+
+	foreach my $part ( @status ) {
 		foreach my $name ( keys %$part ) {
 			my $content = $part->{$name};
 			if ( ref($content) ) {
-				$content = '<code>' . dump( $content ) . '</code>';
+				$content = '<code>' . dump($content) . '</code>';
+				my $l = length($content);
+				$content = qq|<span>$l bytes</span>| if $l > $self->dump_max_bytes;
 			} else {
 				$content = qq|<span>$content</span>|;
 			}
 			warn "### part [$name] = ", length( $content ), " bytes" if $self->debug;
-			$status_line .= qq|<span class="frey-popup">$name $content</span>|;
+			$status_line .= qq|<span class="frey-popup">$name $content</span>\n|;
 		}
 	}
 
+	my $url = $self->request_url;
+	$url =~ s{\?reload=\d+}{};
+
+	my $body = $a->{body};
+	$body ||= $self->as_markup if $self->can('as_markup');
+	if ( $self->content_type !~ m{html} ) {
+		warn "# return only $self body ", $self->content_type;
+		return $body
+	} elsif ( ! defined $body ) {
+		warn "# no body";
+		$body = '<!-- no body -->';
+	}
+
+	my $warn_colors = {
+		'#'  => '#444',
+		'##' => '#888',
+	};
+
+	$status_line
+		.= qq|<span class="frey-popup">warn<span>|
+		. $self->editor_links(
+			join("", map {
+				warn "# $_";
+				my $style = '';
+				$style = $warn_colors->{$1}
+					? ' style="color:' . $warn_colors->{$1} . '"'
+					: ''
+					if m{^(#+)};
+				qq|<tt$style>$_</tt><br/>|; # XXX <tt> should be <code> but CSS hates me
+			} $self->warnings )
+		)
+		. qq|</span></span>|
+		if $self->warnings;
+
+	my	($exit,$description) = ('exit','stop server');
+		($exit,$description) = ('restart','restart server')
+		if $ENV{FREY_RESTART}; # tune labels on exit link
+
+	my $right =
+		qq|
+			<span class="right">
+			<a title="reload $url"  href="/reload$url">reload</a>
+			<a title="$description" href="/exit$url">$exit</a>
+			</span>
+		|;
+
+	my $info = Frey::SVK->info;
+	my $revision = Frey::SVK->info->{Revision} || '';
+	$revision = $1 if $info->{'Mirrored From'} =~ m{Rev\.\s+(\d+)};
+
 	my $html = join("\n",
 		qq|<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01//EN"><html><head>|,
 		$self->_head_html,
-		'<title>' . ( $a->{title} || ref($self) ) . '</title>',
+		'<title>' . ( $self->title || $a->{title} || ref($self) ) . '</title>',
 		'<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">',
 		( $a->{head} || '' ),
-		'</head><body>',
-		( $a->{body} || '<!-- no body -->' ),
 		qq|
+		</head><body>
+		$body
 		<div class="frey-status-line">
-			<a href="/">Frey</a> $Frey::VERSION
-			<a href="?reload=$reload_counter"><code>| . $self->request_url . qq|</code></a>
+			<a href="/">Frey</a> $Frey::VERSION $revision
 			$status_line
+			$right
 		</div>
 	    </body></html>
 		|,
@@ -159,13 +243,78 @@
 	return $html;
 }
 
-sub error {
+=head2 editor
+
+Create HTML editor link with optional line and title
+
+  my $html = $self->editor( $class, $line, $title );
+
+=cut
+
+sub editor {
+	my ( $self, $class, $line, $title ) = @_;
+	confess "need class" unless $class;
+	if ( ! defined $title ) {
+		$title  = "edit $class";
+		$title .= " line $line" if $line;
+	}
+	$line  ||= 1;
+	qq|<a target="editor" href="/editor+$class+$line"| .
+	( $title ? qq| title="$title"| : '' ) .
+	qq|>$class</a>|;
+}
+
+=head2 editor_links
+
+Create HTML links to editor for perl error message
+
+  my $html = $self->editor_links( $error )
+
+=cut
+
+sub editor_links {
 	my ( $self, $error ) = @_;
-	my ($package, $filename, $line) = caller;
-	$error .= " at $filename line $line";
-	warn "WARN: $error\n";
-	$error =~ s{at\s+(\S+)\s+line\s+(\d+)}{at <a href="/editor$1+$2" target="editor">$1</a> line $2}gsm;
-	return qq|<pre class="frey-error">$error</pre>|;
+
+	$error =~ s{at\s+(\S+)\s+line\s+(\d+)}
+		{at <a target="editor" href="/editor+$1+$2">$1</a> line $2}gsm;
+
+	$error =~ s{(via package ")([\w:]+)(")}
+		{$1<a target="editor" href="/editor+$2+1">$2</a>$3}gsm;
+
+	return $error;
+}
+
+sub error {
+	my $self = shift;
+	my $error = join(" ", @_);
+
+	my @backtrace = $self->backtrace;
+	$error .= "\n\t" . join( "\n\t", @backtrace ) if @backtrace;
+
+	warn "ERROR: $error\n";
+	return
+		qq|<pre class="frey-error">|
+		. $self->editor_links( $error ) .
+		qq|</pre>|
+		;
+}
+
+sub add_status {
+	my ( $self, $data ) = @_;
+	push @status, $data;
+}
+
+sub clean_status {
+	@status = ();
+}
+
+sub status_parts {
+	warn "## status parts ", dump( map { keys %$_ } @status );
+}
+
+sub DEMOLISH {
+	my ( $self ) = @_;
+	cluck "## DEMOLISH status ", $#status + 1, " elements ", dump( map { keys %$_ } @status ) if @status;
 }
 
 1;