[Catalyst-commits] r9416 - in Catalyst-Manual/5.70/trunk: . lib/Catalyst lib/Catalyst/Manual/Tutorial

hkclark at dev.catalyst.perl.org hkclark at dev.catalyst.perl.org
Tue Mar 3 02:48:48 GMT 2009


Author: hkclark
Date: 2009-03-03 02:48:48 +0000 (Tue, 03 Mar 2009)
New Revision: 9416

Modified:
   Catalyst-Manual/5.70/trunk/Changes
   Catalyst-Manual/5.70/trunk/lib/Catalyst/Manual.pm
   Catalyst-Manual/5.70/trunk/lib/Catalyst/Manual/Tutorial/BasicCRUD.pod
Log:
Reword warning about not using GET for delete based on input from kd
Change wording of "heading" for starter links in C::M

Modified: Catalyst-Manual/5.70/trunk/Changes
===================================================================
--- Catalyst-Manual/5.70/trunk/Changes	2009-03-03 01:19:03 UTC (rev 9415)
+++ Catalyst-Manual/5.70/trunk/Changes	2009-03-03 02:48:48 UTC (rev 9416)
@@ -1,8 +1,10 @@
 Revision history for Catalyst-Manual
 
-5.7XXX
+5.7018  2 Mar 2009
         - Suggestions and fixes with thanks to Murray Walker
-        - Misc updates and fixes
+        - Fix misplaced "=over 4" in previous release
+        - Reword warning about not using GET for delete based on input from kd
+        - A few minor updates and fixes
 
 5.7017  28 Feb 2009
         - Main change = adding Chained dispatch starting in BasicCRUD (Part 4)

Modified: Catalyst-Manual/5.70/trunk/lib/Catalyst/Manual/Tutorial/BasicCRUD.pod
===================================================================
--- Catalyst-Manual/5.70/trunk/lib/Catalyst/Manual/Tutorial/BasicCRUD.pod	2009-03-03 01:19:03 UTC (rev 9415)
+++ Catalyst-Manual/5.70/trunk/lib/Catalyst/Manual/Tutorial/BasicCRUD.pod	2009-03-03 02:48:48 UTC (rev 9416)
@@ -658,13 +658,9 @@
 
 =back
 
-B<Note:> In general, you should use more than just a simple link with 
-your applications. Consider using some sort of of confirmation page 
-(typically with unique actions in your controller for both the 
-confirmation and the actual delete operation).  Also, you should try 
-to use an HTTP POST operation (versus the GET used here) for 
-operations that change the state of your application (e.g., the 
-database).
+B<Note:> In practice you should B<never> use a GET request to delete a 
+record -- always use POST for actions that will modify data.  We are 
+doing it here for illustrative and simplicity purposes only.
 
 
 =head2 Add a Common Method to Retrieve a Book for the Chain

Modified: Catalyst-Manual/5.70/trunk/lib/Catalyst/Manual.pm
===================================================================
--- Catalyst-Manual/5.70/trunk/lib/Catalyst/Manual.pm	2009-03-03 01:19:03 UTC (rev 9415)
+++ Catalyst-Manual/5.70/trunk/lib/Catalyst/Manual.pm	2009-03-03 02:48:48 UTC (rev 9416)
@@ -23,7 +23,7 @@
 L<Catalyst::Manual::Tutorial::Intro|Catalyst::Manual::Tutorial::Intro>
 for more information on installation options.
 
-Documentation Map:
+Some "Getting Started" Links:
 
 =over 4
 




More information about the Catalyst-commits mailing list