From: Kennedy Clark Date: Fri, 23 Jan 2009 08:07:25 +0000 (+0000) Subject: Add comment/warning about simple links for changes to app/db X-Git-Tag: v5.8005~220 X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=catagits%2FCatalyst-Manual.git;a=commitdiff_plain;h=fe01b24f85098a731125f36fc4de26a654ac7efc;hp=3c27830dc37c72a9383440b89effef0f80c795df Add comment/warning about simple links for changes to app/db --- diff --git a/lib/Catalyst/Manual/Tutorial/BasicCRUD.pod b/lib/Catalyst/Manual/Tutorial/BasicCRUD.pod index 72be8e1..9d6f6b5 100644 --- a/lib/Catalyst/Manual/Tutorial/BasicCRUD.pod +++ b/lib/Catalyst/Manual/Tutorial/BasicCRUD.pod @@ -394,9 +394,17 @@ and 2) the four lines for the Delete link near the bottom). [% END -%] -The additional code is obviously designed to add a new column to the -right side of the table with a C "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 "button" (for simplicity, +links will be used instead of full HTML buttons). + +B 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