--- trunk/lib/Frey/Web.pm 2008/12/19 14:10:03 881 +++ trunk/lib/Frey/Web.pm 2009/01/07 22:59:23 965 @@ -71,15 +71,31 @@ default => 250, ); +has 'wrap_in_page' => ( + documentation => 'wrap full html page with status bar around content', + is => 'rw', + isa => 'Bool', + default => 1, +); + my %escape = ('<'=>'<', '>'=>'>', '&'=>'&', '"'=>'"'); my $escape_re = join '|' => keys %escape; sub html_escape { my ( $self, $html ) = @_; + return '' unless defined $html; $html =~ s/($escape_re)/$escape{$1}/g; return $html; } +# from Mojo::ByteStream +sub url_escape { + my ( $self, $url, $pattern ) = @_; + $pattern ||= 'A-Za-z0-9\-\.\_\~'; + $url =~ s/([^$pattern])/sprintf('%%%02X',ord($1))/ge; + return $url; +} + sub html_dump { my ( $self, $dump ) = @_; $dump = dump( $dump ) if ref($dump); @@ -187,14 +203,14 @@ my ( $package, $path, $line ) = caller(1); - if ( $content =~ m{\.(js|css)} ) { - $content = "/$content" if -e $content; - if ( $content =~ $re_html ) { - $head = qq| - $content - - |; - } elsif ( $what eq 'js' ) { + $content = "/$content" if $content !~ m{[\n\r]} && -e $content; + if ( $content =~ $re_html ) { + $head = qq| + $content + + |; + } elsif ( $content =~ m{^(/\w+|https?://)} && $content !~ m{[\n\r]} ) { + if ( $what eq 'js' ) { $head = qq| <$tag type="$type" src="$content"> /* $what via $package at $path line $line */ @@ -230,9 +246,9 @@ our $reload_counter = 0; -=head2 page +=head2 html_page - $self->page( + $self->html_page( title => 'page title', head => '', body => 'Page Body', @@ -245,12 +261,10 @@ our $icon_html; -sub page { +sub html_page { my $self = shift; my $a = {@_}; - warn "## page ",dump($a); - $reload_counter++; my $status_line = ''; @@ -270,7 +284,7 @@ warn "# no body, invoke $self->$run on ", ref($self); $body = $self->$run; } - if ( $self->content_type !~ m{html} ) { + if ( $self->content_type !~ m{html} || ! $self->wrap_in_page ) { warn "# return only $self body ", $self->content_type; return $body } elsif ( ! defined $body ) { @@ -299,10 +313,18 @@ $self->add_icon unless $icon_html; + my $title = undef + || $a->{title} + || $self->title + || ref($self) + ; + +# $title =~ s{(\w)\w+::}{$1:}g; # XXX compress names of classes + my $html = join("\n", qq||, $self->_head_html, - '' . ( $self->title || $a->{title} || ref($self) ) . '', + qq|$title|, '', ( $icon_html || '' ), ( $a->{head} || '' ), @@ -433,8 +455,8 @@ sub add_status { my ( $self, $data ) = @_; - push @status, { 'X' => [ $self->backtrace ] }; - if ( ref($data) ) { + die "no data" unless $data; + if ( ref $data ) { push @status, $data; } else { if ( defined $status[ $#status ] ) {