[Catalyst-commits] r9227 - in Catalyst-View-ContentNegotiation-XHTML/1.000/trunk: . t/lib t/lib/TestApp/Controller t/lib/TestApp/View

[t0m at dev.catalyst.perl.org]

Author: t0m
Date: 2009-02-05 23:59:37 +0000 (Thu, 05 Feb 2009)
New Revision: 9227

Modified:
   Catalyst-View-ContentNegotiation-XHTML/1.000/trunk/MANIFEST.SKIP
   Catalyst-View-ContentNegotiation-XHTML/1.000/trunk/README
   Catalyst-View-ContentNegotiation-XHTML/1.000/trunk/t/lib/TestApp.pm
   Catalyst-View-ContentNegotiation-XHTML/1.000/trunk/t/lib/TestApp/Controller/Root.pm
   Catalyst-View-ContentNegotiation-XHTML/1.000/trunk/t/lib/TestApp/View/XHTML.pm
Log:
Checking in changes prior to tagging of version 1.100.  Changelog diff is:



Modified: Catalyst-View-ContentNegotiation-XHTML/1.000/trunk/MANIFEST.SKIP
===================================================================
--- Catalyst-View-ContentNegotiation-XHTML/1.000/trunk/MANIFEST.SKIP	2009-02-05 23:27:03 UTC (rev 9226)
+++ Catalyst-View-ContentNegotiation-XHTML/1.000/trunk/MANIFEST.SKIP	2009-02-05 23:59:37 UTC (rev 9227)
@@ -7,3 +7,5 @@
 Makefile$
 Makefile.old$
 .shipit
+\B\.svn\b
+MANIFEST.SKIP

Modified: Catalyst-View-ContentNegotiation-XHTML/1.000/trunk/README
===================================================================
--- Catalyst-View-ContentNegotiation-XHTML/1.000/trunk/README	2009-02-05 23:27:03 UTC (rev 9226)
+++ Catalyst-View-ContentNegotiation-XHTML/1.000/trunk/README	2009-02-05 23:59:37 UTC (rev 9227)
@@ -1,7 +1,6 @@
 NAME
-    Catalyst::View::ContentNegotiation::XHTML - A Moose Role to apply to
-    Catalyst views adjusts the response Content-Type header to
-    application/xhtml+xml content if the browser accepts it.
+    Catalyst::View::ContentNegotiation::XHTML - Adjusts the response
+    Content-Type header to application/xhtml+xml if the browser accepts it.
 
 SYNOPSIS
         package Catalyst::View::TT;
@@ -15,18 +14,23 @@
         1;
 
 DESCRIPTION
-    This is a very simple Role which uses a method modifier to run after the
-    "process" method, and sets the response "Content-Type" to be
+    This is a simple Role which sets the response "Content-Type" to be
     "application/xhtml+xml" if the users browser sends an "Accept" header
     indicating that it is willing to process that MIME type.
 
-    Changing the "Content-Type" causes browsers to interpret the page as
-    XML, meaning that the markup must be well formed.
+    Changing the "Content-Type" to "application/xhtml+xml" causes browsers
+    to interpret the page as XML, meaning that your markup must be well
+    formed.
 
+CAVEATS
     This is useful when you're developing your application, as you know that
     all pages you view are parsed as XML, so any errors caused by your
     markup not being well-formed will show up at once.
 
+    Whilst this module is has been tested against most popular browsers
+    including Internet Explorer, it may cause unexpected results on browsers
+    which do not properly support the "application/xhtml+xml" MIME type.
+
 METHOD MODIFIERS
   after process
     Changes the response "Content-Type" if appropriate (from the requests
@@ -56,18 +60,18 @@
     negotiation RFC.
 
 BUGS
-    Will only work with Views which implement a process method.
-
     Should be split into a base ContentNegotiation role which is consumed by
     ContentNegotiation::XHTML.
 
 AUTHOR
-    Tomas Doran (t0m) "<bobtfish at bobtfish.net>"
+    Original author and maintainer - Tomas Doran (t0m)
+    "<bobtfish at bobtfish.net>"
 
 CONTRIBUTORS
-    David Dorward - test patches and */* pragmatism.
+    David Dorward - test patches and */* pragmatism to make it work for
+    browsers which aren't firefox.
     Florian Ragwitz (rafl) "<rafl at debian.org>" - Conversion into a Moose
-    Role
+    Role, which is what the module should have been originally.
 
 COPYRIGHT
     This module itself is copyright (c) 2008 Tomas Doran and is licensed

Modified: Catalyst-View-ContentNegotiation-XHTML/1.000/trunk/t/lib/TestApp/Controller/Root.pm
===================================================================
--- Catalyst-View-ContentNegotiation-XHTML/1.000/trunk/t/lib/TestApp/Controller/Root.pm	2009-02-05 23:27:03 UTC (rev 9226)
+++ Catalyst-View-ContentNegotiation-XHTML/1.000/trunk/t/lib/TestApp/Controller/Root.pm	2009-02-05 23:59:37 UTC (rev 9227)
@@ -1,4 +1,5 @@
-package TestApp::Controller::Root;
+package # Hide from PAUSE
+    TestApp::Controller::Root;
 use strict;
 use warnings;
 

Modified: Catalyst-View-ContentNegotiation-XHTML/1.000/trunk/t/lib/TestApp/View/XHTML.pm
===================================================================
--- Catalyst-View-ContentNegotiation-XHTML/1.000/trunk/t/lib/TestApp/View/XHTML.pm	2009-02-05 23:27:03 UTC (rev 9226)
+++ Catalyst-View-ContentNegotiation-XHTML/1.000/trunk/t/lib/TestApp/View/XHTML.pm	2009-02-05 23:59:37 UTC (rev 9227)
@@ -1,4 +1,5 @@
-package TestApp::View::XHTML;
+package # Hide from PAUSE
+    TestApp::View::XHTML;
 use strict;
 use warnings;
 use base qw/Catalyst::View::TT::XHTML/;

Modified: Catalyst-View-ContentNegotiation-XHTML/1.000/trunk/t/lib/TestApp.pm
===================================================================
--- Catalyst-View-ContentNegotiation-XHTML/1.000/trunk/t/lib/TestApp.pm	2009-02-05 23:27:03 UTC (rev 9226)
+++ Catalyst-View-ContentNegotiation-XHTML/1.000/trunk/t/lib/TestApp.pm	2009-02-05 23:59:37 UTC (rev 9227)
@@ -1,4 +1,5 @@
-package TestApp;
+package # Hide from PAUSE
+    TestApp;
 use strict;
 use warnings;