Add comment/warning about simple links for changes to app/db
Kennedy Clark [Fri, 23 Jan 2009 08:07:25 +0000 (08:07 +0000)]
lib/Catalyst/Manual/Tutorial/BasicCRUD.pod

index 72be8e1..9d6f6b5 100644 (file)
@@ -394,9 +394,17 @@ and 2) the four lines for the Delete link near the bottom).
     [% END -%]
     </table>
 
-The additional code is obviously designed to add a new column to the
-right side of the table with a C<Delete> "button" (for simplicity, links
-will be used instead of full HTML buttons).
+The additional code is obviously designed to add a new column to the 
+right side of the table with a C<Delete> "button" (for simplicity, 
+links will be used instead of full HTML buttons).
+
+B<Note:> 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).
 
 
 =head2 Add a Delete Action to the Controller