Reword warning about not using GET for delete based on input from kd

Change wording of "heading" for starter links in C::M

diff --git a/Changes b/Changes
index 73a0a84..61108ba 100644 (file)
--- a/Changes
+++ b/Changes
@@ -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)

diff --git a/lib/Catalyst/Manual.pm b/lib/Catalyst/Manual.pm
index 5df4a86..310e8d9 100644 (file)
--- a/lib/Catalyst/Manual.pm
+++ b/lib/Catalyst/Manual.pm
@@ -23,7 +23,7 @@ Tutorial.  You can also refer to
 L<Catalyst::Manual::Tutorial::Intro|Catalyst::Manual::Tutorial::Intro>
 for more information on installation options.
 
-Documentation Map:
+Some "Getting Started" Links:
 
 =over 4
 

diff --git a/lib/Catalyst/Manual/Tutorial/BasicCRUD.pod b/lib/Catalyst/Manual/Tutorial/BasicCRUD.pod
index de04465..6a8278f 100644 (file)
--- a/lib/Catalyst/Manual/Tutorial/BasicCRUD.pod
+++ b/lib/Catalyst/Manual/Tutorial/BasicCRUD.pod
@@ -658,13 +658,9 @@ C<$c-E<gt>controller('_controller_name_')-E<gt>action_for('_method_name_')>.
 
 =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