diff --git a/Changes b/Changes index 8e8dab3..a63617b 100644 --- a/Changes +++ b/Changes @@ -1,5 +1,14 @@ Revision history for Smolder +1.52 + + Modifed subject line in smoke report e-mail to display more detail - P. Leonard, K. McDaniel + + Fixed problem where the AutoRedirectToProject configuration directive would redirect to a broken link - K. McDaniel + + Fixed broken link(s) for progess graphs on project listing pages - K. McDaniel + + Fixed broken link to the full report within the smoke report e-mail(s) - P. Leonard, K. McDaniel + + Fixed auto completion for platform and architecture fields when submitting a new smoke report - P. Leonard + + Fixed RT #55514 by removing references of developer_X and public_X within the URLs - P. Leonard + + Fixed RT #84077 by enabling the skip field to be charted on the progress graphs - K. McDaniel + 1.51 + Fixed problem on machines with multiple IP addresses and other services bound to port 80 on different IPs - M. Peters + Another go at using images in POD - M. Peters diff --git a/htdocs/js/smolder.js b/htdocs/js/smolder.js index 3a8a27d..b4c9f52 100644 --- a/htdocs/js/smolder.js +++ b/htdocs/js/smolder.js @@ -813,7 +813,7 @@ var myrules = { new Ajax.Autocompleter( 'platform', 'platform_auto_complete', - '/app/developer_projects/platform_options' + '/app/projects/platform_options' ); }, @@ -821,7 +821,7 @@ var myrules = { new Ajax.Autocompleter( 'architecture', 'architecture_auto_complete', - '/app/developer_projects/architecture_options' + '/app/projects/architecture_options' ); }, diff --git a/lib/Smolder/Control/Projects.pm b/lib/Smolder/Control/Projects.pm index 8435fad..2670eac 100644 --- a/lib/Smolder/Control/Projects.pm +++ b/lib/Smolder/Control/Projects.pm @@ -813,8 +813,7 @@ sub feed { foreach my $report (@reports) { my $link = Smolder::Util::url_base() . '/app/' - . ($project->public ? 'public' : 'developer') - . '_projects/smoke_report/' + . 'projects/smoke_report/' . $report->id; $feed->add_entry( title => '#' diff --git a/lib/Smolder/Control/Public.pm b/lib/Smolder/Control/Public.pm index 5dc87a7..79ffc47 100644 --- a/lib/Smolder/Control/Public.pm +++ b/lib/Smolder/Control/Public.pm @@ -42,7 +42,7 @@ sub welcome { my $self = shift; if (my $project_id = Smolder::Conf->get('AutoRedirectToProject')) { $self->header_type('redirect'); - $self->header_add( -uri => "/app/public_projects/smoke_reports/$project_id" ); + $self->header_add( -uri => "/app/projects/smoke_reports/$project_id" ); return "redirecting"; } else { diff --git a/lib/Smolder/DB/SmokeReport.pm b/lib/Smolder/DB/SmokeReport.pm index e20d252..6968f00 100644 --- a/lib/Smolder/DB/SmokeReport.pm +++ b/lib/Smolder/DB/SmokeReport.pm @@ -230,9 +230,18 @@ sub _slurp_file { sub _send_emails { my ($self, $results) = @_; + # determine the appropriate status for the e-mail + my $status = ( $self->failed() ? "FAILURE" : "SUCCESS" ); + $status = "WARNING" if( $self->skip > 0 ); + $status = "ABORTED" if( $self->skip == $self->pass ); # setup some stuff for the emails that we only need to do once - my $subject = - "[" . $self->project->name . "] new " . ($self->failed ? "failed " : '') . "Smolder report"; + my $subject = sprintf("Smolder - [%s] %i files ( %i/%i ) : %s", + $self->project->name, + $self->test_files, + int( $self->pass - $self->skip ), + $self->total, + $status ); + my $matrix = Smolder::TAPHTMLMatrix->new( smoke_report => $self, test_results => $results, diff --git a/lib/Smolder/Manual.pm b/lib/Smolder/Manual.pm index c7d425b..7841a31 100644 --- a/lib/Smolder/Manual.pm +++ b/lib/Smolder/Manual.pm @@ -182,7 +182,7 @@ Or you can use the C Perl library directly (and there's probably a way to do it just using C from the command line. LWP::UserAgent->new()->post( - 'http://smolder.project.org/app/developer_projects/add_report/$project_id', + 'http://smolder.project.org/app/projects/add_report/$project_id', Content_Type => 'form-data', Content => [ architecture => '386', diff --git a/templates/Email/smoke_report_full.tmpl b/templates/Email/smoke_report_full.tmpl index 8fda339..e1646db 100644 --- a/templates/Email/smoke_report_full.tmpl +++ b/templates/Email/smoke_report_full.tmpl @@ -18,7 +18,7 @@

Test Details #[% report.id %]

- [View Full Report] + [View Full Report]
diff --git a/templates/Email/smoke_report_link.tmpl b/templates/Email/smoke_report_link.tmpl index 93dba39..787947f 100644 --- a/templates/Email/smoke_report_link.tmpl +++ b/templates/Email/smoke_report_link.tmpl @@ -3,7 +3,7 @@ A new smoke report has been added to the [% report.project.name %] project at [% url_base %].
You can find the details of this test at -[% SET my_url = "$url_base/app/developer_projects/smoke_report/${report.id}" %] +[% SET my_url = "$url_base/app/projects/smoke_report/${report.id}" %] [% my_url %].
diff --git a/templates/Graphs/start.tmpl b/templates/Graphs/start.tmpl index 1d100bd..2ca474c 100644 --- a/templates/Graphs/start.tmpl +++ b/templates/Graphs/start.tmpl @@ -38,7 +38,7 @@ Fail Todo Skip - Duration + Duration [% IF project.platforms.size || project.architectures.size %] diff --git a/templates/Projects/public.tmpl b/templates/Projects/public.tmpl index b4e64b6..e508842 100644 --- a/templates/Projects/public.tmpl +++ b/templates/Projects/public.tmpl @@ -10,7 +10,7 @@