From: Kennedy Clark <hkclark@gmail.com>
Date: Tue, 3 Mar 2009 02:48:48 +0000 (+0000)
Subject: Reword warning about not using GET for delete based on input from kd
X-Git-Tag: v5.8005~195
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=catagits%2FCatalyst-Manual.git;a=commitdiff_plain;h=c5d94181a3ea5ce8c06b9c33c11954d8514eb120

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
--- 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
--- 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
--- 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