From 9b43918fe89054c91b5d3d31e338b3ca9800777a Mon Sep 17 00:00:00 2001 From: thepurpleblob Date: Wed, 6 Jul 2005 05:27:00 +0000 Subject: [PATCH] Ahem - data_submitted() remark, now considerably toned down in line with Petr Skoda's (actual) recommendation. --- lang/en/docs/coding.html | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/lang/en/docs/coding.html b/lang/en/docs/coding.html index fe37f7407..a58cecc7b 100755 --- a/lang/en/docs/coding.html +++ b/lang/en/docs/coding.html @@ -232,8 +232,7 @@ GOOD: $assignments (for an array of objects)
value for a variable, use the set_default() function.
  • Do not use the require_variable() function. Use the required_param() function instead. Pick the correct PARAM_XXXX value for the data type you expect.
  • -
  • Do not use data_submitted(), if possible, as this bypasses the data cleaning - functions of required_param() and optional_param(). +
  • Use data_submitted(), with care. Data must still be cleaned before use.
  • Do not use $_GET, $_POST or $_REQUEST. Use the appropriate required_param() or optional_param() appropriate to your need.
  • Do not check for an action using something like if (isset($_GET['something'])). -- 2.11.4.GIT