X-Git-Url: http://git.ieval.ro/?a=blobdiff_plain;f=lib%2FPlack%2FApp%2FGruntmaster.pm;h=007ad6bd754369773e356cab360cfdde552c0e4c;hb=19e5c235f31cd0eb71aff0440874c914120016f3;hp=b011ffa048e766bdfc6ac80a87f654953ae58eb5;hpb=b8a0fa7145b52cf9a2263602352802d5323bde90;p=plack-app-gruntmaster.git

diff --git a/lib/Plack/App/Gruntmaster.pm b/lib/Plack/App/Gruntmaster.pm
index b011ffa..007ad6b 100644
--- a/lib/Plack/App/Gruntmaster.pm
+++ b/lib/Plack/App/Gruntmaster.pm
@@ -79,6 +79,7 @@ sub response {
 		$title = 'No title';
 	}
 	$params->{time} = time;
+	$params->{args} = {%_};
 	bless {template => $template, title => $title, params => $params, maxage => ($maxage // 1)}, __PACKAGE__.'::Response'
 }
 
@@ -112,7 +113,7 @@ sub dispatch_request{
 
 		sub (/src/:job) {
 			return NOT_FOUND if !job;
-			forbid job->private || job->problem->private || job->contest && job->contest->private;
+			forbid job->private || job->problem->private || job->contest && job->contest->is_running;
 			my @headers = ('X-Forever' => 1, 'Cache-Control' => 'public, max-age=604800', 'Content-Type' => CONTENT_TYPES->{job->format});
 			[200, \@headers, [job->source]]
 		},
@@ -144,19 +145,29 @@ sub dispatch_request{
 		sub (/st/:contest) {
 			response st => 'Standings', {
 				st => [ contest->standings ],
-				problems => [map { $_->problem } contest->contest_problems],
+				problems => [
+					map { [$_->id, $_->name] }
+					sort { $a->value <=> $b->value }
+					map { $_->problem } contest->contest_problems],
 			}, 10
 		},
 
+		sub (/login) {
+			forbid !remote_user;
+
+			my $return = $env->{HTTP_REFERER} // '/';
+			[303, ['Set-Cookie' => "username=".remote_user->id, Location => $return], []]
+		},
+
 		sub (/ct/:contest/log/st) { redirect "/st/$_{contest}" },
 
 		sub (/us/)                                        { response us => 'Users', {us => db->user_list} },
 		sub (/ct/  + ?:owner~)                            { response ct => 'Contests', db->contest_list(%_) },
 		sub (/log/ + ?:contest~&:owner~&:page~&:problem~&:private~) {
 			forbid $_{private};
-			response log => 'Job list', {%{db->job_list(%_)}, maybe contest => $_{contest}}
+			response log => 'Job list', {%{db->job_list(%_)}, maybe contest => $_{contest},}
 		},
-		sub (/pb/  + ?:owner~&:contest~&private~)                   {
+		sub (/pb/  + ?:owner~&:contest~&:private~)                  {
 			forbid $_{private};
 			response pb => 'Problems', {%{db->problem_list(%_)}, maybe contest => $_{contest}}
 		},
@@ -172,7 +183,11 @@ sub dispatch_request{
 			$_{contest} = $contest;
 			return NOT_FOUND if !contest && !problem->is_in_archive || contest && !db->contest_problems->find($_{contest}, $_{problem});
 			forbid problem->is_private;
-			response pb_entry => problem->name, db->problem_entry($_{problem}, $_{contest}, remote_user && remote_user->id);
+			if (contest && contest->is_running) {
+				forbid !remote_user;
+				$privacy = 'private';
+			}
+			response pb_entry => problem->name, {%{db->problem_entry($_{problem}, $_{contest}, remote_user && remote_user->id)}, maybe contest => $_{contest}};
 		},
 		sub (/sol/:problem) {
 			forbid !problem->is_in_archive;
@@ -180,17 +195,18 @@ sub dispatch_request{
 		},
 
 		sub (/) { redispatch_to '/index' },
+		sub (/favicon.ico) { redirect '/static/favicon.ico' },
 		sub (/:article) { [200, ['Content-Type' => 'text/html', 'Cache-Control' => 'public, max-age=60', 'X-Forever' => 1], [render_article $_{article}, 'en']] }
 	},
 
 	sub (POST) {
-		sub (/action/register + %:username=&:password=&:confirm_password=&:name=&:email=&:phone=&:town=&:university=&:level=) {
+		sub (/action/register + %:username=&:password=&:confirm_password=&:name=&:email=&:phone=&:town=&:university=&:country=&:level=) {
 			return reply 'Parameter too long' if grep { length > 200 } values %_;
 			return reply 'Bad username. Allowed characters are letters, digits and underscores, and the username must be between 2 and 20 characters long.' unless $_{username} =~ USER_REGEX;
 			return reply 'Username already in use' if db->user($_{username});
 			return reply 'The two passwords do not match' unless $_{password} eq $_{confirm_password};
 
-			db->users->create({id => $_{username}, name => $_{name}, email => $_{email}, phone => $_{phone}, town => $_{town}, university => $_{university}, level => $_{level}});
+			db->users->create({id => $_{username}, name => $_{name}, email => $_{email}, phone => $_{phone}, town => $_{town}, university => $_{university}, country => $_{country}, level => $_{level}});
 			db->user($_{username})->set_passphrase($_{password});
 
 			purge '/us/';
@@ -210,15 +226,16 @@ sub dispatch_request{
 			forbid !remote_user;
 			return reply 'This contest has finished' if contest && contest->is_finished;
 			return reply 'This contest has not yet started' if !admin && contest && contest->is_pending;
+			return reply 'This problem does not belong to this contest' if !contest && !problem->is_in_archive || contest && !db->contest_problems->find($_{contest}, $_{problem});
 			return reply 'Maximum source size is 10KB' if ($prog ? $prog->size : length $_{source_code}) > 10 * 1024;
 			return reply 'You must wait 30 seconds between jobs' if !admin && time <= remote_user->lastjob + 30;
 			remote_user->update({lastjob => time});
 
 			my $source = $prog ? read_file $prog->path : $_{source_code};
 			unlink $prog->path if $prog;
-			db->jobs->create({
+			my $newjob = db->jobs->create({
 				maybe contest => $_{contest},
-				maybe private => problem->private,
+				maybe private => problem->private && !$_{contest},
 				date => time,
 				extension => FORMAT_EXTENSION->{$_{prog_format}},
 				format => $_{prog_format},
@@ -228,7 +245,7 @@ sub dispatch_request{
 			});
 
 			purge '/log/';
-			redirect $_{contest} ? "/log/?contest=$_{contest}" : '/log/';
+			[303, [Location => '/log/' . $newjob->id], []]
 		}
 	}
 }