From 51b920c87509d2bfd221ca4b09d990ca2b22da5f Mon Sep 17 00:00:00 2001 From: Eelco Dolstra Date: Tue, 17 Apr 2012 16:53:11 +0200 Subject: [PATCH] Bootstrapify the Hydra forms (except the project and jobset edit pages) Plus lots of other tweaks. --- src/lib/Hydra/Controller/Admin.pm | 67 ++++++++++--- src/lib/Hydra/Controller/Project.pm | 30 +++--- src/lib/Hydra/Controller/Release.pm | 2 + src/lib/Hydra/Controller/View.pm | 16 +-- src/root/admin.tt | 2 +- src/root/all.tt | 6 +- src/root/build.tt | 11 +- src/root/channel-contents.tt | 2 +- src/root/clone-build.tt | 2 +- src/root/common.tt | 25 +++-- src/root/edit-release.tt | 84 ++++++++-------- src/root/edit-view.tt | 72 ++++++++------ src/root/error.tt | 2 +- src/root/errors.tt | 2 +- src/root/jobset-eval.tt | 2 +- src/root/jobset-evals.tt | 4 +- src/root/jobset.tt | 17 ++-- src/root/jobstatus.tt | 4 +- src/root/layout.tt | 21 ++-- src/root/log.tt | 2 +- src/root/login.tt | 45 +++++---- src/root/machine.tt | 149 +++++++++++++++------------- src/root/machines.tt | 9 +- src/root/news.tt | 32 +++--- src/root/plain.tt | 2 +- src/root/project.tt | 44 ++++---- src/root/queue.tt | 4 +- src/root/release.tt | 8 +- src/root/releases.tt | 8 +- src/root/status.tt | 2 +- src/root/timeline.tt | 2 +- src/root/topbar.tt | 12 ++- src/root/user.tt | 99 +++++++++--------- src/root/users.tt | 10 +- src/root/view-result.tt | 6 +- src/root/view.tt | 10 +- 36 files changed, 456 insertions(+), 359 deletions(-) diff --git a/src/lib/Hydra/Controller/Admin.pm b/src/lib/Hydra/Controller/Admin.pm index e8898597..c09d6cf1 100644 --- a/src/lib/Hydra/Controller/Admin.pm +++ b/src/lib/Hydra/Controller/Admin.pm @@ -15,6 +15,7 @@ use Email::Sender::Simple qw(sendmail); use Email::Sender::Transport::SMTP; use Config::General; + sub nixMachines { my ($c) = @_; my $result = "# GENERATED BY HYDRA\n"; @@ -32,6 +33,7 @@ sub nixMachines { return $result; } + sub saveNixMachines { my ($c) = @_; @@ -42,12 +44,14 @@ sub saveNixMachines { close (NIXMACHINES); } + sub admin : Chained('/') PathPart('admin') CaptureArgs(0) { my ($self, $c) = @_; requireAdmin($c); $c->stash->{admin} = 1; } + sub index : Chained('admin') PathPart('') Args(0) { my ($self, $c) = @_; $c->stash->{machines} = [$c->model('DB::BuildMachines')->search( @@ -62,7 +66,8 @@ sub index : Chained('admin') PathPart('') Args(0) { , order_by => [ 'machine', 'stepnr' ] } ) ]; $c->stash->{template} = 'admin.tt'; - } +} + sub updateUser { my ($c, $user) = @_; @@ -88,6 +93,16 @@ sub updateUser { } } + +sub create_user : Chained('admin') PathPart('create-user') Args(0) { + my ($self, $c) = @_; + + requireAdmin($c); + + error($c, "Not implemented yet!"); # FIXME +} + + sub user : Chained('admin') PathPart('user') CaptureArgs(1) { my ($self, $c, $username) = @_; @@ -99,6 +114,7 @@ sub user : Chained('admin') PathPart('user') CaptureArgs(1) { $c->stash->{user} = $user; } + sub users : Chained('admin') PathPart('users') Args(0) { my ($self, $c) = @_; $c->stash->{users} = [$c->model('DB::Users')->search({}, {order_by => "username"})]; @@ -106,6 +122,7 @@ sub users : Chained('admin') PathPart('users') Args(0) { $c->stash->{template} = 'users.tt'; } + sub user_edit : Chained('user') PathPart('edit') Args(0) { my ($self, $c) = @_; @@ -113,16 +130,23 @@ sub user_edit : Chained('user') PathPart('edit') Args(0) { $c->stash->{edit} = 1; } + sub user_edit_submit : Chained('user') PathPart('submit') Args(0) { my ($self, $c) = @_; requirePost($c); txn_do($c->model('DB')->schema, sub { - updateUser($c, $c->stash->{user}) ; + if (($c->request->params->{submit} || "") eq "delete") { + $c->stash->{user}->delete; + } else { + updateUser($c, $c->stash->{user}); + } }); + $c->res->redirect("/admin/users"); } + sub sendemail { my ($to, $subject, $body) = @_; @@ -141,6 +165,7 @@ sub sendemail { sendmail($email); } + sub reset_password : Chained('user') PathPart('reset-password') Args(0) { my ($self, $c) = @_; @@ -168,6 +193,7 @@ sub reset_password : Chained('user') PathPart('reset-password') Args(0) { $c->res->redirect("/admin/users"); } + sub machines : Chained('admin') PathPart('machines') Args(0) { my ($self, $c) = @_; $c->stash->{machines} = [$c->model('DB::BuildMachines')->search({}, {order_by => "hostname"})]; @@ -176,7 +202,8 @@ sub machines : Chained('admin') PathPart('machines') Args(0) { $c->stash->{nixMachinesWritable} = (-e "/etc/nix.machines" && -w "/etc/nix.machines"); $c->stash->{template} = 'machines.tt'; - } +} + sub machine : Chained('admin') PathPart('machine') CaptureArgs(1) { my ($self, $c, $machineName) = @_; @@ -189,6 +216,7 @@ sub machine : Chained('admin') PathPart('machine') CaptureArgs(1) { $c->stash->{machine} = $machine; } + sub machine_edit : Chained('machine') PathPart('edit') Args(0) { my ($self, $c) = @_; $c->stash->{template} = 'machine.tt'; @@ -196,19 +224,27 @@ sub machine_edit : Chained('machine') PathPart('edit') Args(0) { $c->stash->{edit} = 1; } + sub machine_edit_submit : Chained('machine') PathPart('submit') Args(0) { my ($self, $c) = @_; requirePost($c); txn_do($c->model('DB')->schema, sub { - updateMachine($c, $c->stash->{machine}) ; + if (($c->request->params->{submit} || "") eq "delete") { + $c->stash->{machine}->delete; + } else { + updateMachine($c, $c->stash->{machine}); + } }); + saveNixMachines($c); + $c->res->redirect("/admin/machines"); } + sub updateMachine { - my ($c, $machine) = @_; + my ($c, $machine) = @_; my $hostname = trim $c->request->params->{"hostname"}; my $username = trim $c->request->params->{"username"}; @@ -240,6 +276,7 @@ sub updateMachine { } } + sub create_machine : Chained('admin') PathPart('create-machine') Args(0) { my ($self, $c) = @_; @@ -269,16 +306,6 @@ sub create_machine_submit : Chained('admin') PathPart('create-machine/submit') A $c->res->redirect("/admin/machines"); } -sub machine_delete : Chained('machine') PathPart('delete') Args(0) { - my ($self, $c) = @_; - requirePost($c); - - txn_do($c->model('DB')->schema, sub { - $c->stash->{machine}->delete; - }); - saveNixMachines($c); - $c->res->redirect("/admin/machines"); -} sub machine_enable : Chained('machine') PathPart('enable') Args(0) { my ($self, $c) = @_; @@ -287,6 +314,7 @@ sub machine_enable : Chained('machine') PathPart('enable') Args(0) { $c->res->redirect("/admin/machines"); } + sub machine_disable : Chained('machine') PathPart('disable') Args(0) { my ($self, $c) = @_; $c->stash->{machine}->update({ enabled => 0}); @@ -294,6 +322,7 @@ sub machine_disable : Chained('machine') PathPart('disable') Args(0) { $c->res->redirect("/admin/machines"); } + sub clear_queue_non_current : Chained('admin') Path('clear-queue-non-current') Args(0) { my ($self, $c) = @_; # !!! Mark the builds as cancelled instead. @@ -301,6 +330,7 @@ sub clear_queue_non_current : Chained('admin') Path('clear-queue-non-current') A $c->res->redirect("/admin"); } + sub clear_queue : Chained('admin') Path('clear-queue') Args(0) { my ($self, $c) = @_; # !!! Mark the builds as cancelled instead. @@ -308,6 +338,7 @@ sub clear_queue : Chained('admin') Path('clear-queue') Args(0) { $c->res->redirect("/admin"); } + sub clearfailedcache : Chained('admin') Path('clear-failed-cache') Args(0) { my ($self, $c) = @_; @@ -316,6 +347,7 @@ sub clearfailedcache : Chained('admin') Path('clear-failed-cache') Args(0) { $c->res->redirect("/admin"); } + sub clearvcscache : Chained('admin') Path('clear-vcs-cache') Args(0) { my ($self, $c) = @_; @@ -334,6 +366,7 @@ sub clearvcscache : Chained('admin') Path('clear-vcs-cache') Args(0) { $c->res->redirect("/admin"); } + sub managenews : Chained('admin') Path('news') Args(0) { my ($self, $c) = @_; @@ -342,6 +375,7 @@ sub managenews : Chained('admin') Path('news') Args(0) { $c->stash->{template} = 'news.tt'; } + sub news_submit : Chained('admin') Path('news/submit') Args(0) { my ($self, $c) = @_; @@ -359,6 +393,7 @@ sub news_submit : Chained('admin') Path('news/submit') Args(0) { $c->res->redirect("/admin/news"); } + sub news_delete : Chained('admin') Path('news/delete') Args(1) { my ($self, $c, $id) = @_; @@ -371,6 +406,7 @@ sub news_delete : Chained('admin') Path('news/delete') Args(1) { $c->res->redirect("/admin/news"); } + sub force_eval : Chained('admin') Path('eval') Args(2) { my ($self, $c, $projectName, $jobsetName) = @_; @@ -387,4 +423,5 @@ sub force_eval : Chained('admin') Path('eval') Args(2) { $c->res->redirect("/project/$projectName"); } + 1; diff --git a/src/lib/Hydra/Controller/Project.pm b/src/lib/Hydra/Controller/Project.pm index 18597942..050a7d40 100644 --- a/src/lib/Hydra/Controller/Project.pm +++ b/src/lib/Hydra/Controller/Project.pm @@ -45,6 +45,11 @@ sub submit : Chained('project') PathPart Args(0) { requireProjectOwner($c, $c->stash->{project}); requirePost($c); + if (($c->request->params->{submit} || "") eq "delete") { + $c->stash->{project}->delete; + $c->res->redirect($c->uri_for("/")); + } + txn_do($c->model('DB')->schema, sub { updateProject($c, $c->stash->{project}); }); @@ -65,6 +70,7 @@ sub hide : Chained('project') PathPart Args(0) { $c->res->redirect($c->uri_for("/")); } + sub unhide : Chained('project') PathPart Args(0) { my ($self, $c) = @_; @@ -77,19 +83,6 @@ sub unhide : Chained('project') PathPart Args(0) { $c->res->redirect($c->uri_for("/")); } -sub delete : Chained('project') PathPart Args(0) { - my ($self, $c) = @_; - - requireProjectOwner($c, $c->stash->{project}); - requirePost($c); - - txn_do($c->model('DB')->schema, sub { - $c->stash->{project}->delete; - }); - - $c->res->redirect($c->uri_for("/")); -} - sub requireMayCreateProjects { my ($c) = @_; @@ -171,12 +164,7 @@ sub create_jobset_submit : Chained('project') PathPart('create-jobset/submit') A sub updateProject { my ($c, $project) = @_; - my $projectName = trim $c->request->params->{name}; - error($c, "Invalid project name: ‘$projectName’") if $projectName !~ /^$projectNameRE$/; - my $displayName = trim $c->request->params->{displayname}; - error($c, "Invalid display name: $displayName") if $displayName eq ""; - my $owner = $project->owner; if ($c->check_user_roles('admin')) { $owner = trim $c->request->params->{owner}; @@ -184,6 +172,12 @@ sub updateProject { unless defined $c->model('DB::Users')->find({username => $owner}); } + my $projectName = trim $c->request->params->{name}; + error($c, "Invalid project name: ‘$projectName’") if $projectName !~ /^$projectNameRE$/; + + my $displayName = trim $c->request->params->{displayname}; + error($c, "Invalid display name: $displayName") if $displayName eq ""; + $project->update( { name => $projectName , displayname => $displayName diff --git a/src/lib/Hydra/Controller/Release.pm b/src/lib/Hydra/Controller/Release.pm index da282a57..89cb2103 100644 --- a/src/lib/Hydra/Controller/Release.pm +++ b/src/lib/Hydra/Controller/Release.pm @@ -52,6 +52,8 @@ sub edit : Chained('release') PathPart('edit') Args(0) { my ($self, $c) = @_; requireProjectOwner($c, $c->stash->{project}); $c->stash->{template} = 'edit-release.tt'; + $c->stash->{members} = [$c->stash->{release}->releasemembers->search({}, + {order_by => ["description"]})]; } diff --git a/src/lib/Hydra/Controller/View.pm b/src/lib/Hydra/Controller/View.pm index a9cb1e05..aa3789cc 100644 --- a/src/lib/Hydra/Controller/View.pm +++ b/src/lib/Hydra/Controller/View.pm @@ -116,6 +116,11 @@ sub edit : Chained('view') PathPart('edit') Args(0) { sub submit : Chained('view') PathPart('submit') Args(0) { my ($self, $c) = @_; requireProjectOwner($c, $c->stash->{project}); + if (($c->request->params->{submit} || "") eq "delete") { + $c->stash->{view}->delete; + $c->res->redirect($c->uri_for($c->controller('Project')->action_for('view'), + [$c->stash->{project}->name])); + } txn_do($c->model('DB')->schema, sub { updateView($c, $c->stash->{view}); }); @@ -123,17 +128,6 @@ sub submit : Chained('view') PathPart('submit') Args(0) { } -sub delete : Chained('view') PathPart('delete') Args(0) { - my ($self, $c) = @_; - requireProjectOwner($c, $c->stash->{project}); - txn_do($c->model('DB')->schema, sub { - $c->stash->{view}->delete; - }); - $c->res->redirect($c->uri_for($c->controller('Project')->action_for('view'), - [$c->stash->{project}->name])); -} - - sub latest : Chained('view') PathPart('latest') { my ($self, $c, @args) = @_; diff --git a/src/root/admin.tt b/src/root/admin.tt index d2fab0fa..42c044b2 100644 --- a/src/root/admin.tt +++ b/src/root/admin.tt @@ -1,7 +1,7 @@ [% WRAPPER layout.tt title="Admin" %] [% PROCESS common.tt %] -

Machine status

+

Machine status

[% FOREACH m IN machines %] diff --git a/src/root/all.tt b/src/root/all.tt index ff146515..f0f0b9a6 100644 --- a/src/root/all.tt +++ b/src/root/all.tt @@ -1,10 +1,10 @@ -[% WRAPPER layout.tt title="All Builds" %] +[% WRAPPER layout.tt title="All builds" %] [% PROCESS common.tt %] -

All Builds +

All builds [% IF job %]for Job [% project.name %]:[% jobset.name %]:[% job.name %] [% ELSIF jobset %]for Jobset [% project.name %]:[% jobset.name %] - [% ELSIF project %] for Project [% project.name %][% END %]

+ [% ELSIF project %] for Project [% project.name %][% END %]

Showing builds [% (page - 1) * resultsPerPage + 1 %] - [% (page - 1) * resultsPerPage + builds.size %] out of [% total %] in order of descending timestamp.

diff --git a/src/root/build.tt b/src/root/build.tt index a6b51c8c..99de759b 100644 --- a/src/root/build.tt +++ b/src/root/build.tt @@ -140,9 +140,14 @@
[% IF c.user_exists && available %] -
-

Add to release: -

+ +
+ +
+ + +
+
[% END %] diff --git a/src/root/channel-contents.tt b/src/root/channel-contents.tt index 911c4767..0f0387e2 100644 --- a/src/root/channel-contents.tt +++ b/src/root/channel-contents.tt @@ -2,7 +2,7 @@ [% PROCESS common.tt %] [% USE HTML %] -

Channel [% channelName %]

+

Channel [% channelName %]

This page provides a channel for the Nix package manager. If you have Nix diff --git a/src/root/clone-build.tt b/src/root/clone-build.tt index e44da753..44b6ccb7 100644 --- a/src/root/clone-build.tt +++ b/src/root/clone-build.tt @@ -3,7 +3,7 @@ [% USE HTML %] [% edit=1 %] -

Clone Build

+

Clone Build

Cloning allows you to perform a build with modified inputs.

diff --git a/src/root/common.tt b/src/root/common.tt index 1f900282..c6b14a5e 100644 --- a/src/root/common.tt +++ b/src/root/common.tt @@ -189,7 +189,7 @@ [% BLOCK maybeLink -%] - [% IF uri %] uri) %][% IF confirmmsg %]onclick="javascript:return confirm('[% confirmmsg %]')"[% END %]>[% content %][% ELSE; content; END -%] + [% IF uri %] uri, class => class) %][% IF confirmmsg %]onclick="javascript:return confirm('[% confirmmsg %]')"[% END %]>[% content %][% ELSE; content; END -%] [% END -%] [% BLOCK maybeButton -%] @@ -198,11 +198,24 @@ [% BLOCK renderSelection %] [% IF edit %] - + [% IF radiobuttons %] +
+ [% FOREACH name IN options.keys.sort %] + + [% END %] +
+ [% ELSE %] + + [% END %] [% ELSE %] [% options.$curValue %] [% END %] diff --git a/src/root/edit-release.tt b/src/root/edit-release.tt index 63e29b51..7f8e86dc 100644 --- a/src/root/edit-release.tt +++ b/src/root/edit-release.tt @@ -1,57 +1,63 @@ -[% WRAPPER layout.tt title=(create ? "New Release" : "Edit Release") %] +[% WRAPPER layout.tt title=(create ? "New release" : "Edit release") %] [% PROCESS common.tt %] [% USE HTML %] -

[% IF create %]New Release[% ELSE %]Release [% release.name %][% END %]

+

[% IF create %]New release[% ELSE %]Release [% release.name %][% END %]

-
+ - - - - - - - - - -
Identifier: release.name) %] />
Description: release.description) %] />
- -

Release Members

+
+ +
+ +
+ release.name) %]> +
+
+ +
+ +
+ release.description) %]> +
+
+ +

Release members

Note: to add a build to this release, go to the build’s information page and click on “Add to release”.

- [% FOREACH m IN release.releasemembers %] - -
- -

Build [% m.build.id %]

- - - - - - -
Label: m.description) %] />
+ [% FOREACH m IN members %] +
+ +
+ m.description) %]> + +
- + [% END %] -
- -

- +

+ [% IF !create %] - - + + [% END %] -

+
+ +
diff --git a/src/root/edit-view.tt b/src/root/edit-view.tt index 98439ee0..57e238e1 100644 --- a/src/root/edit-view.tt +++ b/src/root/edit-view.tt @@ -1,15 +1,15 @@ -[% WRAPPER layout.tt title=(create ? "New View" : "View ‘$project.name:$view.name’") %] +[% WRAPPER layout.tt title=(create ? "New view" : "View ‘$project.name:$view.name’") %] [% PROCESS common.tt %] [% USE HTML %] -

[% IF create %]New View[% ELSE %]View [% project.name %]:[% view.name %][% END %]

+

[% IF create %]New view[% ELSE %]View [% project.name %]:[% view.name %][% END %]

[% BLOCK renderJob %] - - - - - - - - - - - -
Identifier: view.name) %] />
Description: view.description) %] />
+
+
+ +
+ +
+ view.name) %]> +
+
+ +
+ +
+ view.description) %]> +
+
+ @@ -51,12 +56,28 @@ [% n = n + 1 %] [% END %] - +
-

+
+ + [% IF !create %] + + + [% END %] +
@@ -83,20 +104,5 @@ }); - -[% IF !create %] - -
-

-
- - - -[% END %] - [% END %] diff --git a/src/root/error.tt b/src/root/error.tt index 1538415d..c4434f22 100644 --- a/src/root/error.tt +++ b/src/root/error.tt @@ -1,7 +1,7 @@ [% WRAPPER layout.tt title="Error" %] [% USE HTML %] -

[% IF httpStatus %][% httpStatus %][% ELSE %]Error[% END %]

+

[% IF httpStatus %][% httpStatus %][% ELSE %]Error[% END %]

I'm very sorry, but the following error(s) occurred:

diff --git a/src/root/errors.tt b/src/root/errors.tt index 0333d154..df6d8aa4 100644 --- a/src/root/errors.tt +++ b/src/root/errors.tt @@ -1,7 +1,7 @@ [% WRAPPER layout.tt title="Errors" %] [% PROCESS common.tt %] -

Errors

+

Errors

This page provides a quick way to see how FUBARed your packages are. It shows job expressions that don’t evaluate properly and jobs diff --git a/src/root/jobset-eval.tt b/src/root/jobset-eval.tt index cbdbf44f..0d8b4620 100644 --- a/src/root/jobset-eval.tt +++ b/src/root/jobset-eval.tt @@ -1,7 +1,7 @@ [% WRAPPER layout.tt title="Jobset $project.name:$jobset.name evaluation $eval.id" %] [% PROCESS common.tt %] -

Jobset [% project.name %]:[% jobset.name %] evaluation [% eval.id %]

+

Jobset [% project.name %]:[% jobset.name %] evaluation [% eval.id %]

[%- IF otherEval -%]

Comparisons are relative to [% INCLUDE renderFullJobsetName diff --git a/src/root/jobset-evals.tt b/src/root/jobset-evals.tt index c8bfab1b..c75509ec 100644 --- a/src/root/jobset-evals.tt +++ b/src/root/jobset-evals.tt @@ -1,9 +1,9 @@ [% WRAPPER layout.tt title="Jobset ‘$project.name:$jobset.name’ Evaluations" %] [% PROCESS common.tt %] -

Evaluations of Jobset [% INCLUDE renderLink +

Evaluations of Jobset [% INCLUDE renderLink uri = c.uri_for(c.controller('Project').action_for('view'), [project.name]) - title = project.name %]:[% jobset.name %]

+ title = project.name %]:[% jobset.name %]

Showing evaluations [% (page - 1) * resultsPerPage + 1 %] - [% (page - 1) * resultsPerPage + evals.size %] out of [% total %].

diff --git a/src/root/jobset.tt b/src/root/jobset.tt index 66f2badf..f2aa3c7f 100644 --- a/src/root/jobset.tt +++ b/src/root/jobset.tt @@ -2,7 +2,7 @@ [% PROCESS common.tt %] [% IF edit %] -
+ [% END %] @@ -13,7 +13,7 @@ [% BLOCK renderInputAlt %] [% IF edit %] - + [% INCLUDE maybeEditString param=param value=alt.value %]
[% ELSE %] @@ -26,7 +26,7 @@ - [% IF edit %][% END -%] + [% IF edit %][% END -%] [% INCLUDE maybeEditString param="$baseName-name" value=input.name extraClass="shortString" %] @@ -42,7 +42,7 @@ [% END %] [% END %] - [% IF edit %][% END %] + [% IF edit %][% END %] @@ -62,7 +62,7 @@ [% END %] [% IF edit %] - + [% END %] @@ -79,6 +79,7 @@ [% END %]
  • Setup
    • +
    [% IF !edit && evals.size() > 0 -%] @@ -151,13 +152,13 @@ Enabled: - [% INCLUDE renderSelection param="enabled" curValue=jobset.enabled options={"1" = "Yes", "0" = "No"} %] + [% INCLUDE renderSelection param="enabled" curValue=jobset.enabled radiobuttons=1 options={"1" = "Yes", "0" = "No"} %] Enable email notification: - [% INCLUDE renderSelection param="enableemail" curValue=jobset.enableemail options={"1" = "Yes", "0" = "No"} %] + [% INCLUDE renderSelection param="enableemail" curValue=jobset.enableemail radiobuttons=1 options={"1" = "Yes", "0" = "No"} %] @@ -254,7 +255,7 @@ }); -

    +

    diff --git a/src/root/jobstatus.tt b/src/root/jobstatus.tt index e39447e1..b37fa3c2 100644 --- a/src/root/jobstatus.tt +++ b/src/root/jobstatus.tt @@ -1,7 +1,7 @@ -[% WRAPPER layout.tt title="Job Status" %] +[% WRAPPER layout.tt title="Job status" %] [% PROCESS common.tt %] -

    Job Status[% IF project %] of [% project.name %][% IF jobset %]:[% jobset.name%][% END %][% IF job %]:[% job.name%][% END %][% END %]

    +

    Job status[% IF project %] of [% project.name %][% IF jobset %]:[% jobset.name%][% END %][% IF job %]:[% job.name%][% END %][% END %]

    Below are the latest builds for each job. It is ordered by the status change time (the timestamp of the last build that had a different diff --git a/src/root/layout.tt b/src/root/layout.tt index 2247dcca..60e13f0e 100644 --- a/src/root/layout.tt +++ b/src/root/layout.tt @@ -108,17 +108,16 @@ [% content %] -

    - -
    - diff --git a/src/root/log.tt b/src/root/log.tt index 7ec61e55..edc53b61 100644 --- a/src/root/log.tt +++ b/src/root/log.tt @@ -1,7 +1,7 @@ [% WRAPPER layout.tt title="Logfile" %] [% PROCESS common.tt %] -

    Build log of [% INCLUDE renderFullJobNameOfBuild %] build [% build.id %][%IF step %] step [% step.stepnr %][% END %]

    +

    Build log of [% INCLUDE renderFullJobNameOfBuild %] build [% build.id %][%IF step %] step [% step.stepnr %][% END %]

    This is the build log of path [% IF step; step.outpath; ELSE; build.outpath; END %]. diff --git a/src/root/login.tt b/src/root/login.tt index 81f5fd3a..7f1d62ae 100644 --- a/src/root/login.tt +++ b/src/root/login.tt @@ -1,36 +1,39 @@ [% WRAPPER layout.tt title="Login to Hydra" %] [% PROCESS common.tt %] -

    Login

    +

    Login

    + +[% IF errorMsg %] +

    Error: [% errorMsg %]

    +[% END %] [% IF c.user_exists %] -

    +

    You are already logged in as [% c.user.username %]. You can logout here.

    [% ELSE %] -[% IF errorMsg %] -

    Error: [% errorMsg %]

    -[% END %] +
    - +
    +
    + +
    + +
    +
    - - - - - - - - - - - - -
    Username:
    Password:
    - -
    +
    + +
    + +
    +
    + +
    + +
    diff --git a/src/root/machine.tt b/src/root/machine.tt index 4d0bcfcf..b1f4ce98 100644 --- a/src/root/machine.tt +++ b/src/root/machine.tt @@ -1,77 +1,90 @@ [% WRAPPER layout.tt title=(create ? "New machine" : "Editing machine '$machine.hostname'") %] [% PROCESS common.tt %] -
    +

    [% IF create %]New machine[% ELSE %]Machine [% machine.hostname %][% END %]

    + + -

    Machine[% IF ! create %] '[% machine.hostname %]'[% END %]

    - - +
    [% IF create %] -
    - - - +
    + +
    + +
    +
    [% END %] - - - - - - - - - - - - - - - - - - - - - - - - -
    Hostname:[% INCLUDE maybeEditString param="hostname" value=machine.hostname %]
    Username:[% INCLUDE maybeEditString param="username" value=machine.username %]
    SSH key location:[% INCLUDE maybeEditString param="ssh_key" value=machine.ssh_key %]
    Options: - [% INCLUDE maybeEditString param="options" value=machine.options %] -
    Max concurrent builds:[% INCLUDE maybeEditString param="maxconcurrent" value=machine.maxconcurrent %]
    Speed factor: - [% INCLUDE maybeEditString param="speedfactor" value=machine.speedfactor %] -
    Systems: - -
    + +
    + +
    + +
    +
    + +
    + +
    + +
    +
    + +
    + +
    + +
    +
    + +
    + +
    + +
    +
    + +
    + +
    + +
    +
    + +
    + +
    + +
    +
    -

    - -
    - -[% IF !create %] - -
    -

    -
    - - -[% END %] +
    + + [% IF !create %] + + + [% END %] +
    + +
    [% END %] diff --git a/src/root/machines.tt b/src/root/machines.tt index 8f0273f8..9a509794 100644 --- a/src/root/machines.tt +++ b/src/root/machines.tt @@ -1,7 +1,7 @@ [% WRAPPER layout.tt title="Machines" %] [% PROCESS common.tt %] -

    Build machines

    +

    Build machines

    @@ -9,7 +9,7 @@ - + [% FOREACH s IN systems %][% END %] @@ -17,7 +17,7 @@ [% FOREACH m IN machines %] - + [% FOREACH s IN systems %] @@ -36,8 +36,7 @@
    Enabled Machine Max concurrentSpeedfactorSpeed factor[% s.system %]
    [% INCLUDE maybeLink uri = c.uri_for(c.controller('Admin').action_for('machine_edit'), [m.hostname]) content = m.hostname %][% m.hostname %] [% m.maxconcurrent %] [% m.speedfactor %]
    -

    [ Add a new machine ]

    - +

    Add a new machine

    Resulting /etc/nix.machines [% IF nixMachinesWritable == 0 %](note: file is not writable!)[% END%]: diff --git a/src/root/news.tt b/src/root/news.tt index f9ac80bf..cffc6caa 100644 --- a/src/root/news.tt +++ b/src/root/news.tt @@ -2,7 +2,7 @@ [% PROCESS common.tt %] [% USE String %] -

    News items

    +

    News items

    [% IF newsItems.size == 0 %]

    No news items

    @@ -15,25 +15,29 @@ [% INCLUDE renderDateTime timestamp=i.createtime %] [% contents.replace('\n','
    \n') %] - [ [% INCLUDE maybeLink uri = c.uri_for(c.controller('Admin').action_for('news_delete') i.id) content = "Delete" confirmmsg = "Are you sure you want to delete this news item?" %] ] + [% INCLUDE maybeLink uri = c.uri_for(c.controller('Admin').action_for('news_delete') i.id) content = "Delete" confirmmsg = "Are you sure you want to delete this news item?" class = "btn btn-mini btn-danger" %] [% END %] [% END %] -
    - -

    Add news item

    -

    - -

    -

    - -

    - + +
    + Add news item +
    + +
    + +
    +
    +
    +
    + +
    - - [% END %] diff --git a/src/root/plain.tt b/src/root/plain.tt index 1975756a..110074e8 100644 --- a/src/root/plain.tt +++ b/src/root/plain.tt @@ -5,7 +5,7 @@ [% jobset = build.jobset %] [% job = build.job %] -

    [% title %]

    +

    [% title %]

    [% contents -%] diff --git a/src/root/project.tt b/src/root/project.tt index da1e7b9f..c913697c 100644 --- a/src/root/project.tt +++ b/src/root/project.tt @@ -133,33 +133,35 @@ Enabled: - [% INCLUDE renderSelection param="enabled" curValue=project.enabled options={"1" = "Yes", "0" = "No"} %] + [% INCLUDE renderSelection param="enabled" curValue=project.enabled radiobuttons=1 options={"1" = "Yes", "0" = "No"} %] [% IF edit %] -

    +

    + + [% IF !create %] + + + [% END %] +

    - - - [% IF !create %] - -
    -

    -
    - - - - [% END %] - [% END %] + +
    [% IF !edit %] @@ -175,7 +177,7 @@ [% FOREACH view IN views %]
  • [% view.name %] - [Edit] + Edit
    • [% END %] @@ -186,7 +188,9 @@ [% END %] -

    [Create a new view]

    +

    + Create a new view +

    [% END %] diff --git a/src/root/queue.tt b/src/root/queue.tt index 0ab13a0e..1c6d3425 100644 --- a/src/root/queue.tt +++ b/src/root/queue.tt @@ -1,9 +1,9 @@ [% WRAPPER layout.tt title="Queue" %] [% PROCESS common.tt %] -

    Hydra Queue

    +

    Hydra queue

    -

    [ Running build steps ]

    +

    Running build steps

    [% IF flashMsg %]

    [% flashMsg %]

    diff --git a/src/root/release.tt b/src/root/release.tt index d8ca9d37..eb7fbaea 100644 --- a/src/root/release.tt +++ b/src/root/release.tt @@ -3,8 +3,8 @@ [% PROCESS "product-list.tt" %] [% USE HTML %] -

    Release [% release.name %] [Edit]

    +

    Release [% release.name %] +

    Released on [% INCLUDE renderDateTime timestamp = release.timestamp %].

    @@ -17,11 +17,11 @@ release.timestamp %].

    [% FOREACH m IN members %] -

    +

    [% HTML.escape(m.description) %] -

    + [% INCLUDE renderProductList build=m.build %] diff --git a/src/root/releases.tt b/src/root/releases.tt index 9fe7061b..c0fc6a38 100644 --- a/src/root/releases.tt +++ b/src/root/releases.tt @@ -2,7 +2,7 @@ [% PROCESS common.tt %] [% USE HTML %] -

    Releases for Project [% project.name %]

    +

    Releases for Project [% project.name %]

    [% IF releases.size == 0 %] @@ -33,9 +33,9 @@ [% END %] [% IF c.user_exists %] -

    -[Create a release] -

    +

    + Create a release +

    [% END %] [% END %] diff --git a/src/root/status.tt b/src/root/status.tt index 20677e4a..ee820c2e 100644 --- a/src/root/status.tt +++ b/src/root/status.tt @@ -1,7 +1,7 @@ [% WRAPPER layout.tt title="Hydra status" %] [% PROCESS common.tt %] -

    Hydra Status

    +

    Hydra status

    [% INCLUDE hydraStatus %] diff --git a/src/root/timeline.tt b/src/root/timeline.tt index 96344590..6b35d09b 100644 --- a/src/root/timeline.tt +++ b/src/root/timeline.tt @@ -4,7 +4,7 @@ [% PROCESS common.tt %] -

    Hydra timeline of last 24 hours

    +

    Hydra timeline of last 24 hours

    + [% END %] +
    +

    -[% IF !create %] - -
    -

    -
    - - -[% END %] - [% END %] diff --git a/src/root/users.tt b/src/root/users.tt index 01203245..7894155d 100644 --- a/src/root/users.tt +++ b/src/root/users.tt @@ -1,7 +1,7 @@ [% WRAPPER layout.tt title="Users" %] [% PROCESS common.tt %] -

    Users

    +

    Users

    @@ -17,18 +17,20 @@ [% FOREACH u IN users %] - + - + [% END %]
    [% INCLUDE maybeLink uri = c.uri_for(c.controller('Admin').action_for('user_edit'), [u.username]) content = u.username %][% u.username %] [% u.fullname %] [% u.emailaddress %] [% FOREACH r IN u.userroles %][% r.role %] [% END %] [% IF u.emailonerror %]Yes[% ELSE %]No[% END %][ [% INCLUDE maybeLink uri = c.uri_for(c.controller('Admin').action_for('reset_password'), [u.username]) content = "Reset password" confirmmsg = "Are you sure you want to reset the password for this user?" %] ][% INCLUDE maybeLink uri = c.uri_for(c.controller('Admin').action_for('reset_password'), [u.username]) content = "Reset password" confirmmsg = "Are you sure you want to reset the password for this user?" class = "btn btn-mini" %]
    -

    [ Add a new user ]

    +

    + Add a new user +

    [% END %] diff --git a/src/root/view-result.tt b/src/root/view-result.tt index 8ee2e958..96fb80ba 100644 --- a/src/root/view-result.tt +++ b/src/root/view-result.tt @@ -4,7 +4,7 @@ [% PROCESS "product-list.tt" %] [% USE HTML %] -

    View [% view.project.name %]:[% view.name %] result [% result.id %][% IF result.releasename %] ([% result.releasename %])[% END %]

    +

    View [% view.project.name %]:[% view.name %] result [% result.id %][% IF result.releasename %] ([% result.releasename %])[% END %]

    Finished building on [% INCLUDE renderDateTime timestamp = result.timestamp %].

    @@ -46,11 +46,13 @@ [% END %] +
    + [% END %] [% IF c.user_exists %]

    -[Release] +Release

    [% END %] diff --git a/src/root/view.tt b/src/root/view.tt index cc314541..05eefd2b 100644 --- a/src/root/view.tt +++ b/src/root/view.tt @@ -2,11 +2,11 @@ [% PROCESS common.tt %] [% USE HTML %] -

    View [% view.project.name %]:[% view.name %]

    +

    View [% view.project.name %]:[% view.name %]

    -[Edit] -[Latest] + Edit + Latest

    Showing results [% (page - 1) * resultsPerPage + 1 %] - [% (page - 1) * resultsPerPage + results.size %] out of [% totalResults %].

    @@ -65,8 +65,12 @@
      + [% IF page > 1 %] + [% END %] + [% IF page * resultsPerPage < totalResults %] + [% END %]