--- trunk/lib/Frey/Web.pm 2008/11/19 17:57:48 460
+++ trunk/lib/Frey/Web.pm 2008/11/20 15:23:13 482
@@ -1,6 +1,8 @@
package Frey::Web;
use Moose::Role;
+with 'Frey::Backtrace';
+
use Frey::Types;
use Continuity::Widget::DomNode;
@@ -22,8 +24,8 @@
isa => 'ArrayRef[HashRef[Str]]',
lazy => 1,
default => sub { [
- { 'Bookmarklets' => Frey::Bookmarklet->new->as_markup },
{ 'ClassBrowser' => Frey::ClassBrowser->new( usage_on_top => 0 )->as_markup },
+ { 'Bookmarklets' => Frey::Bookmarklet->new->as_markup },
] },
);
@@ -47,6 +49,14 @@
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
@@ -156,7 +166,7 @@
if ( ref($content) ) {
$content = '' . dump($content) . '';
my $l = length($content);
- $content = qq|$l bytes| if $l > 1024;
+ $content = qq|$l bytes| if $l > $self->dump_max_bytes;
} else {
$content = qq|$content|;
}
@@ -178,6 +188,39 @@
$body = '';
}
+ my $warn_colors = {
+ '#' => '#444',
+ '##' => '#888',
+ };
+
+ $status_line
+ .= qq||
+ 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|
+
+ $url
+ $exit
+
+ |;
+
my $html = join("\n",
qq||,
$self->_head_html,
@@ -189,8 +232,8 @@
$body
Frey $Frey::VERSION
-
$url
$status_line
+ $right
|,
@@ -201,34 +244,56 @@
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;
+ $line ||= 1;
+ qq|$class|;
+}
+
+=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 $1 line $2}gsm;
+
+ $error =~ s{(via package ")([\w:]+)(")}
+ {$1$2$3}gsm;
+
+ return $error;
+}
+
sub error {
my $self = shift;
my $error = join(" ", @_);
- my @backtrace;
- foreach ( 0 .. 5 ) {
- my @caller = caller($_) or last;
- my @description = ( qw/
- package filename line
- subroutine hasargs
- wantarray evaltext is_require
- hints bitmask hinthash
- /);
- push @backtrace, join(' ',
- map {
- $description[$_] . ': ' . dump $caller[$_]
- } ( 0 .. $#caller )
- );
- }
- if ( @backtrace ) {
- warn "# append backtrace: ", dump( @backtrace );
- $error .= "\n\t" . join( "\n\t", @backtrace );
- }
+ my @backtrace = $self->backtrace;
+ $error .= "\n\t" . join( "\n\t", @backtrace ) if @backtrace;
warn "ERROR: $error\n";
- $error =~ s{at\s+(\S+)\s+line\s+(\d+)}{at $1 line $2}gsm;
- $error =~ s{(via package ")([\w:]+)(")}{$1$2$3}gsm;
- return qq|$error
|;
+ return
+ qq||
+ . $self->editor_links( $error ) .
+ qq|
|
+ ;
}
1;