michael.dollinger/mlmmj-light-web-ecg
1
0
Fork 0
forked from services/mlmmj-light-web-ecg
Code Issues Pull Requests Projects Releases Wiki Activity

Added more descriptive error messages for three cases.

Browse Source
This commit is contained in:
Christian Süßenguth
2022-04-13 13:49:22 +02:00
parent 9bff04b518
commit 85f5824f7b
3 changed files with 12 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
edit_list.php
View File

@@ -13,16 +13,16 @@ if (!isset($_SESSION["auth"]) || $_SESSION["auth"] != 1)
exit(); exit();
} }
// We do not print any error in the next three cases, because a legitimate
// user will never produce such results, even with disabled javascript
if ( preg_match("/[^a-z0-9_-]/", $list_name) ) if ( preg_match("/[^a-z0-9_-]/", $list_name) )
{ {
$_SESSION["error_code"] = 14;
header("Location: error.php"); header("Location: error.php");
exit(); exit();
} }
if ( strlen($list_name) > 50 ) if ( strlen($list_name) > 50 )
{ {
$_SESSION["error_code"] = 13;
header("Location: error.php"); header("Location: error.php");
exit(); exit();
} }
@@ -30,6 +30,7 @@ if ( strlen($list_name) > 50 )
// Test list existence // Test list existence
if( !is_dir("$lists_path/$domain/$list_name") || $list_name == "" ) if( !is_dir("$lists_path/$domain/$list_name") || $list_name == "" )
{ {
$_SESSION["error_code"] = 12;
header("Location: error.php"); header("Location: error.php");
exit(); exit();
} }

6
misc/smarty/templates_en/error.tpl
View File

@@ -36,6 +36,12 @@
There is an incorrect email in the moderators list. There is an incorrect email in the moderators list.
{elseif $error_code == 11} {elseif $error_code == 11}
You do not own this list. You do not own this list.
{elseif $error_code == 12}
The list does not exist within the mlmmj working folder.
{elseif $error_code == 13}
The list name exceeds the maximum length of 50 chars.
{elseif $error_code == 14}
The list name contains chars which are not allowed.
{else} {else}
Unknown error. Unknown error.
{/if} {/if}

5
save_list.php
View File

@@ -32,16 +32,16 @@ if ( !isset($_SESSION["auth"]) || $_SESSION["auth"] != 1 )
$domain = $_SESSION["domain"]; $domain = $_SESSION["domain"];
// We do not print any error in the next four cases, because a legitimate
// user will never produce such results, even with disabled javascript
if ( preg_match("/[^a-z0-9_-]/", $list_name) ) if ( preg_match("/[^a-z0-9_-]/", $list_name) )
{ {
$_SESSION["error_code"] = 14;
header("Location: error.php"); header("Location: error.php");
exit(); exit();
} }
if ( strlen($list_name) > 50 ) if ( strlen($list_name) > 50 )
{ {
$_SESSION["error_code"] = 13;
header("Location: error.php"); header("Location: error.php");
exit(); exit();
} }
@@ -49,6 +49,7 @@ if ( strlen($list_name) > 50 )
// Test list existence // Test list existence
if( !is_dir("$lists_path/$domain/$list_name") || $list_name == "" ) if( !is_dir("$lists_path/$domain/$list_name") || $list_name == "" )
{ {
$_SESSION["error_code"] = 12;
header("Location: error.php"); header("Location: error.php");
exit(); exit();
} }