X-Git-Url: http://git.ieval.ro/?a=blobdiff_plain;f=lib%2FPlack%2FApp%2FGruntmaster%2FHTML.pm;h=b67ef296cf544a0164e7ecadb175a33485680d6c;hb=dd962dd2b75089dabe9f01ddad0a51529e78f3ec;hp=9f6f2ec5b27c6454686be8a1916a3b95df8b018e;hpb=f8a4fa095f622e783f2e27bfba86c0234a5983cf;p=plack-app-gruntmaster.git

diff --git a/lib/Plack/App/Gruntmaster/HTML.pm b/lib/Plack/App/Gruntmaster/HTML.pm
index 9f6f2ec..b67ef29 100644
--- a/lib/Plack/App/Gruntmaster/HTML.pm
+++ b/lib/Plack/App/Gruntmaster/HTML.pm
@@ -9,6 +9,8 @@ use HTML::TreeBuilder;
 use POSIX qw//;
 use Data::Dumper qw/Dumper/;
 
+my $optional_end_tags = {%HTML::Tagset::optionalEndTag, tr => 1, td => 1, th => 1};
+
 sub ftime ($)   { POSIX::strftime '%c', localtime shift }
 sub literal ($) {
 	my ($html) = @_;
@@ -16,7 +18,7 @@ sub literal ($) {
 	my $b = HTML::TreeBuilder->new;
 	$b->ignore_unknown(0);
 	$b->parse($html);
-	HTML::Element::Library::super_literal $b->guts->as_HTML;
+	HTML::Element::Library::super_literal $b->guts->as_HTML(undef, undef, $optional_end_tags);
 }
 
 sub HTML::Element::edit_href {
@@ -62,13 +64,22 @@ sub render {
 	my ($tmpl, $lang, %args) = @_;
 	$lang //= 'en';
 	my $meat = _render($tmpl, $lang, %args);
-	_render('skel', $lang, %args, meat => $meat)
+	my $html = _render('skel', $lang, %args, meat => $meat);
+	if ($tmpl eq 'pb_entry') { # Move sidebar to correct position
+		my $builder = HTML::TreeBuilder->new;
+		$builder->ignore_unknown(0);
+		my $tree = $builder->parse_content($html);
+		$tree->fid('content')->postinsert($tree->fid('sidebar'));
+		$html = $tree->as_HTML(undef, undef, $optional_end_tags)
+	}
+	$html
 }
 
 sub render_article {
 	my ($art, $lang, %args) = @_;
 	$lang //= 'en';
 	my $title = read_file "a/$art.$lang.title";
+	chomp $title;
 	my $meat  = read_file "a/$art.$lang";
 	_render('skel', $lang, title => $title , meat => $meat, %args)
 }
@@ -80,10 +91,8 @@ sub _render {
 	$tree->defmap(smap => \%args);
 	my $process = __PACKAGE__->can("process_$tmpl");
 	$process->($tree, %args) if $process;
-	$_->detach for $tree->look_down(static => $args{static} ? 'no' : 'yes');
-	$_->attr('static', undef) for $tree->look_down(sub {$_[0]->attr('static')});
 	$_->attr('smap', undef) for $tree->look_down(sub {$_[0]->attr('smap')});
-	$tree->as_HTML;
+	$tree->as_HTML(undef, undef, $optional_end_tags);
 }
 
 sub process_skel {
@@ -182,18 +191,13 @@ sub process_pb_entry {
 		$tree->fid('job_log')->edit_href(sub{$_ .= "&private=$args{private}"}) if $args{private};
 		$tree->fid('solution')->detach unless $args{solution};
 		$_->detach for $tree->fclass('rc'); # requires contest
-		$tree->fid('solution_modal')->fclass('modal-body')->replace_content(literal $args{solution});
-	}
-	if ($args{cansubmit}) {
-		$tree->fid('nosubmit')->detach;
-		$tree->look_down(name => 'problem')->attr(value => $args{id});
-		my $contest = $tree->look_down(name => 'contest');
-		$contest->attr(value => $args{args}{contest}) if $args{args}{contest};
-		$contest->detach unless $args{args}{contest}
-	} else {
-		$tree->fid('nosubmit')->find('a')->edit_href(sub{s/id/$args{id}/});
-		$tree->fid('submit')->detach
+		$tree->fid('solution_modal')->replace_content(literal $args{solution});
 	}
+
+	$tree->look_down(name => 'problem')->attr(value => $args{id});
+	my $contest = $tree->look_down(name => 'contest');
+	$contest->attr(value => $args{args}{contest}) if $args{args}{contest};
+	$contest->detach unless $args{args}{contest}
 }
 
 sub process_sol {
@@ -274,10 +278,6 @@ sub process_log {
 		$elem->find('a')->edit_href(sub{s/$/&$str/}) if $str;
 	}
 	$tree->fclass('current')->replace_content("Page $args{current_page} of $args{last_page}");
-
-	my @detach;
-	push @detach, $args{args}{$_} ? $tree->fclass($_) : () for qw/problem contest owner/;
-	$_->detach for @detach;
 }
 
 sub process_st {