Merge lp:~seancron/mvhub/unique-webpage-title into lp:mvhub

[Sean Cronin]

[Dan MacNeil (omacneil)]

looks good, I didn't run the tests but trust you did

review: Approve

[lp:~seancron/mvhub/unique-webpage-title]

563. By Sean Cronin <email address hidden> on 2011-06-24

Escapes special characters in the title now.

=== modified file 'lib-mvhub/lib/MVHub/Guide.pm'
--- lib-mvhub/lib/MVHub/Guide.pm	2011-06-06 01:12:44 +0000
+++ lib-mvhub/lib/MVHub/Guide.pm	2011-06-24 20:35:14 +0000
@@ -156,7 +156,7 @@
     );
 
  # Before returning our page body, let's add some custom content to its title:
-    $self->param( 'title', $heading_name );
+    $self->param( 'title', CGI::escapeHTML($heading_name) );
 
     $html
         .= "<!-- stop generated by MVHub::Guide::browse_one_heading() -->\n";
@@ -308,6 +308,8 @@
     # the search box in the search bar of the ensuing page:
     $tmpl->param( $MVHub::Page::SEARCH_PHRASE_TAG, $search_phrase );
 
+    $self->param( 'title', CGI::escapeHTML($search_phrase) );
+
     return $tmpl->output();
 }
 
@@ -394,7 +396,7 @@
     $tmpl->param( $MVHub::Page::SEARCH_PHRASE_TAG, $search_phrase );
 
  # Before returning our page body, let's add some custom content to its title:
-    $self->param( 'title', $category_name );
+    $self->param( 'title', CGI::escapeHTML($category_name) );
 
     # needed by search_bar.tmpl
     $self->param( 'site_location', $self->get_config_param('SITE.location') );
@@ -476,6 +478,8 @@
         $tmpl->param( $SA_PROGRAMS_LOOP_TAG, $programs_aref );
     }
 
+    $self->param( 'title', CGI::escapeHTML( $agency_href->{'agency_name'} ) );
+
     return $tmpl->output();
 }
 
@@ -526,6 +530,9 @@
         $tmpl->param( $_, $program_href->{$_} );
     }
 
+    $self->param( 'title',
+        CGI::escapeHTML( $program_href->{'program_name'} ) );
+
     return $tmpl->output();
 }