--- trunk/lib/Frey/Web.pm	2008/11/19 03:00:46	445
+++ trunk/lib/Frey/Web.pm	2008/11/26 04:26:43	529
@@ -1,15 +1,18 @@
 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',
@@ -17,16 +20,6 @@
 	default => sub { [ 'static/frey.css' ] },
 );
 
-has 'status' => (
-	is => 'rw',
-	isa => 'ArrayRef[HashRef[Str]]',
-	lazy => 1,
-	default => sub { [
-		{ 'Bookmarklets' => Frey::Bookmarklet->new->markup },
-		{ 'ClassBrowser' => Frey::ClassBrowser->new( usage_on_top => 0 )->markup },
-	] },
-);
-
 has 'request_url' => (
 	is => 'rw',
 	isa => 'Uri', coerce => 1,
@@ -43,6 +36,20 @@
 	},
 );
 
+has 'content_type' => (
+	is => 'rw',
+	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
 
 Inline JavaScript and CSS smaller than this size into page reducing
@@ -79,7 +86,7 @@
 			$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{<.+>} ) {
+		} elsif ( $path =~ m{<.+>}s ) {
 			$out .= $path;
 		} else {
 			confess "don't know how to render $path";
@@ -104,7 +111,7 @@
 	return if ! defined $path || $path eq '';
 	$path =~ s!^/!!;
 
-	if ( $path =~ m{<.*>} ) {
+	if ( $path =~ m{<.*>}s ) {
 		push @{ $self->head }, $path;
 	} elsif ( -e $path ) {
 		if ( $path =~ m/\.(?:js|css)$/ ) {
@@ -132,25 +139,31 @@
 
 =cut
 
+our @status;
+sub status { @status };
+
+our $icon_html;
+
 sub page {
 	my $self = shift;
 	my $a = {@_};
 
+	warn "## page ",dump($a);
+
 	$reload_counter++;
 
 	my $status_line = '';
-	foreach my $part ( @{ $self->status } ) {
-		if ( ref($part) ne 'HASH' ) {
-			warn "part not hash ",dump( $part ) ;
-			#$self->status( $part );
-			next;
-		}
+
+	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>';
 				my $l = length($content);
-				$content = qq|<span>$l bytes</span>| if $l > 1024;
+				$content = qq|<span>$l bytes</span>| if $l > $self->dump_max_bytes;
 			} else {
 				$content = qq|<span>$content</span>|;
 			}
@@ -162,19 +175,69 @@
 	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+)};
+
+	$self->add_icon unless $icon_html;
+
 	my $html = join("\n",
 		qq|<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01//EN"><html><head>|,
 		$self->_head_html,
 		'<title>' . ( $self->title || $a->{title} || ref($self) ) . '</title>',
 		'<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">',
+		( $icon_html || '<!-- no icon -->' ),
 		( $a->{head} || '' ),
-		'</head><body>',
-		( $a->{body} || $self->markup || '<!-- no body -->' ),
 		qq|
+		</head><body>
+		$body
 		<div class="frey-status-line">
-			<a href="/">Frey</a> $Frey::VERSION
-			<a href="?reload=$reload_counter"><code>$url</code></a>
+			<a href="/">Frey</a> $Frey::VERSION $revision
 			$status_line
+			$right
 		</div>
 	    </body></html>
 		|,
@@ -185,15 +248,126 @@
 	return $html;
 }
 
+=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 ) = @_;
+
+	$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 ($package, $filename, $line) = caller;
-	$error .= " at $filename line $line" if $error !~ m{ at };
-	warn "WARN: $error\n";
-	$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 qq|<pre class="frey-error">$error</pre>|;
+
+	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 = ();
+	$icon_html = '';
+}
+
+sub status_parts {
+	warn "## status parts ", dump( map { keys %$_ } @status );
+}
+
+sub DEMOLISH {
+	my ( $self ) = @_;
+	warn "## $self DEMOLISH status ", $#status + 1, " elements ", dump( map { keys %$_ } @status ) if @status;
+}
+
+=head2 add_icon
+
+  Frey::Foo->add_icon;            # /static/icons/Frey/Foo.png
+  Frey::Foo->add_icon('warning'); # /static/icons/Frey/Foo/warning.png
+
+=cut
+
+sub icon_path {
+	my ($self,$class,$variant) = @_;
+	my $icon = $class;
+	$icon =~ s{::}{/}g;
+	$icon .= "/$variant" if $variant;
+	my $path = 'static/icons/' . $icon . '.png';
+	if ( -e $path ) {
+		warn "# $class from $self icon_path $path";
+		return $path;
+	} else {
+		warn "TODO: add $path icon for $class";
+		return undef;
+	}
+}
+
+sub add_icon {
+	my ($self,$variant) = @_;
+
+	my $class = ref($self);
+	$class = $self->class if $self->can('class');
+	my $icon_path = $self->icon_path( $class, $variant ) || return;
+
+	$icon_html .= qq|<link rel="icon" type="image/png" href="/$icon_path">|;
+	warn "# using icon $icon_path";
+
+=for later
+
+	# FIXME http://en.wikipedia.org/wiki/Favicon suggest just rel="icon" but that doesn't seem to work!
+	my $ico_path = $icon_path;
+	$ico_path =~ s{png$}{ico};
+	if ( ! -e $ico_path ) {
+		system "convert $icon_path $ico_path";
+		warn "# convert $icon_path $ico_path : $@";
+	}
+	$icon_html .= qq|<link rel="shortcut icon" type="image/x-icon" href="/$ico_path">| if -e $ico_path;
+
+=cut
+
 }
 
 1;