SF.net SVN: postfixadmin:[426] trunk/functions.inc.php
Brought to you by:
christian_boltz,
gingerdog
|
From: <Gin...@us...> - 2008-07-29 19:32:22
|
Revision: 426
http://postfixadmin.svn.sourceforge.net/postfixadmin/?rev=426&view=rev
Author: GingerDog
Date: 2008-07-29 19:32:32 +0000 (Tue, 29 Jul 2008)
Log Message:
-----------
functions.inc.php: reindent (as per inline vim settings) and add in flash_error calls when validating an email address (I suspect the strings need changing as per translation issues)
Modified Paths:
--------------
trunk/functions.inc.php
Modified: trunk/functions.inc.php
===================================================================
--- trunk/functions.inc.php 2008-07-28 15:18:54 UTC (rev 425)
+++ trunk/functions.inc.php 2008-07-29 19:32:32 UTC (rev 426)
@@ -17,7 +17,7 @@
*/
if (ereg ("functions.inc.php", $_SERVER['PHP_SELF']))
-{
+{
header ("Location: login.php");
exit;
}
@@ -183,7 +183,7 @@
function language_selector()
{
global $supported_languages; # from languages/languages.php
-
+
$current_lang = check_language();
$selector = '<select name="lang" xml:lang="en" dir="ltr">';
@@ -260,49 +260,47 @@
$ce_email = preg_replace("/#/", '@', $ce_email);
}
- if (
- isset($CONF['emailcheck_resolve_domain'])
- && 'YES'==$CONF['emailcheck_resolve_domain']
- && 'WINDOWS'!=(strtoupper(substr(php_uname('s'), 0, 7)))
- ) {
+ if (isset($CONF['emailcheck_resolve_domain']) && 'YES' == $CONF['emailcheck_resolve_domain'] && 'WINDOWS'!=(strtoupper(substr(php_uname('s'), 0, 7))))
+ {
- // Perform non-domain-part sanity checks
- if (!preg_match ('/^[-!#$%&\'*+\\.\/0-9=?A-Z^_{|}~]+' . '@' . '[^@]+$/i', trim ($ce_email)))
- {
- return false;
- }
+ // Perform non-domain-part sanity checks
+ if (!preg_match ('/^[-!#$%&\'*+\\.\/0-9=?A-Z^_{|}~]+' . '@' . '[^@]+$/i', trim ($ce_email)))
+ {
+ return false;
+ }
- // Determine domain name
- $matches=array();
- if (!preg_match('|@(.+)$|',$ce_email,$matches))
- {
- return false;
- }
- $domain=$matches[1];
+ // Determine domain name
+ $matches=array();
+ if (!preg_match('|@(.+)$|',$ce_email,$matches))
+ {
+ return false;
+ }
+ $domain=$matches[1];
- // Look for an AAAA, A, or MX record for the domain
+ // Look for an AAAA, A, or MX record for the domain
- if(function_exists('checkdnsrr')) {
- // AAAA (IPv6) is only available in PHP v. >= 5
- if (version_compare(phpversion(), "5.0.0", ">="))
- {
- if (checkdnsrr($domain,'AAAA')) return true;
- }
- if (checkdnsrr($domain,'A')) return true;
- if (checkdnsrr($domain,'MX')) return true;
- }
- # TODO: different error message for non-existing domains (instead of "email is invalid")
- return false;
+ if(function_exists('checkdnsrr')) {
+ // AAAA (IPv6) is only available in PHP v. >= 5
+ if (version_compare(phpversion(), "5.0.0", ">="))
+ {
+ if (checkdnsrr($domain,'AAAA')) return true;
}
+ if (checkdnsrr($domain,'A')) return true;
+ if (checkdnsrr($domain,'MX')) return true;
+ flash_error("Invalid domain, and/or not discoverable in DNS");
+ return false;
+ }
+ else {
+ flash_error("emailcheck_resolve_domain is enabled, but function (checkdnsrr) missing!");
+ }
+ }
if (preg_match ('/^[-!#$%&\'*+\\.\/0-9=?A-Z^_{|}~]+' . '@' . '([-0-9A-Z]+\.)+' . '([0-9A-Z]){2,6}$/i', trim ($ce_email)))
{
return true;
}
- else
- {
- return false;
- }
+ flash_error("Invalid email address, fails regexp check");
+ return false;
}
@@ -375,9 +373,9 @@
* @return String
*/
function safeget ($param, $default="") {
- $retval=$default;
- if (isset($_GET[$param])) $retval=$_GET[$param];
- return $retval;
+ $retval=$default;
+ if (isset($_GET[$param])) $retval=$_GET[$param];
+ return $retval;
}
/**
@@ -389,9 +387,9 @@
* same as safeget, but for $_POST
*/
function safepost ($param, $default="") {
- $retval=$default;
- if (isset($_POST[$param])) $retval=$_POST[$param];
- return $retval;
+ $retval=$default;
+ if (isset($_POST[$param])) $retval=$_POST[$param];
+ return $retval;
}
/**
@@ -402,9 +400,9 @@
* @return String value from $_SERVER[$param] or $default
*/
function safeserver ($param, $default="") {
- $retval=$default;
- if (isset($_SERVER[$param])) $retval=$_SERVER[$param];
- return $retval;
+ $retval=$default;
+ if (isset($_SERVER[$param])) $retval=$_SERVER[$param];
+ return $retval;
}
/**
@@ -415,9 +413,9 @@
* @return String value from $_COOKIE[$param] or $default
*/
function safecookie ($param, $default="") {
- $retval=$default;
- if (isset($_COOKIE[$param])) $retval=$_COOKIE[$param];
- return $retval;
+ $retval=$default;
+ if (isset($_COOKIE[$param])) $retval=$_COOKIE[$param];
+ return $retval;
}
@@ -851,7 +849,7 @@
{
global $table_domain;
- $result = db_query ("SELECT 1 FROM $table_domain WHERE domain='$domain'");
+ $result = db_query("SELECT 1 FROM $table_domain WHERE domain='$domain'");
if ($result['rows'] != 1)
{
return false;
@@ -1592,7 +1590,7 @@
foreach($timestamp as $key) {
$values[$key] = "now()";
}
-
+
$sql_values = "(" . implode(",",escape_string(array_keys($values))).") VALUES (".implode(",",$values).")";
$result = db_query ("INSERT INTO $table $sql_values");
@@ -2104,7 +2102,7 @@
$error = 0;
$tMessage = '';
$pAdminCreate_admin_username_text = '';
- $pAdminCreate_admin_password_text = '';
+ $pAdminCreate_admin_password_text = '';
if (!check_email ($fUsername))
{
@@ -2117,24 +2115,24 @@
$error = 1;
$pAdminCreate_admin_username_text = $PALANG['pAdminCreate_admin_username_text_error2'];
}
-
+
if (empty ($fPassword) or empty ($fPassword2) or ($fPassword != $fPassword2))
{
if (empty ($fPassword) and empty ($fPassword2) and $CONF['generate_password'] == "YES" && $no_generate_password == 0)
{
- $fPassword = generate_password ();
+ $fPassword = generate_password ();
}
else
{
- $error = 1;
- $pAdminCreate_admin_username_text = $PALANG['pAdminCreate_admin_username_text'];
- $pAdminCreate_admin_password_text = $PALANG['pAdminCreate_admin_password_text_error'];
+ $error = 1;
+ $pAdminCreate_admin_username_text = $PALANG['pAdminCreate_admin_username_text'];
+ $pAdminCreate_admin_password_text = $PALANG['pAdminCreate_admin_password_text_error'];
}
}
if ($error != 1)
{
- $password = pacrypt($fPassword);
+ $password = pacrypt($fPassword);
$pAdminCreate_admin_username_text = $PALANG['pAdminCreate_admin_username_text'];
$result = db_query ("INSERT INTO " . table_by_key('admin') . " (username,password,created,modified) VALUES ('$fUsername','$password',NOW(),NOW())");
@@ -2152,24 +2150,24 @@
$result = db_query ("INSERT INTO " . table_by_key ('domain_admins') . " (username,domain,created) VALUES ('$fUsername','$domain',NOW())");
}
}
- $tMessage = $PALANG['pAdminCreate_admin_result_success'] . "<br />($fUsername";
- if ($CONF['generate_password'] == "YES" && $no_generate_password == 0)
- {
- $tMessage .= " / $fPassword)</br />";
- }
- else
- {
- if ($CONF['show_password'] == "YES" && $no_generate_password == 0)
- {
- $tMessage .= " / $fPassword)</br />";
- }
- else
- {
- $tMessage .= ")</br />";
- }
- }
- }
- }
+ $tMessage = $PALANG['pAdminCreate_admin_result_success'] . "<br />($fUsername";
+ if ($CONF['generate_password'] == "YES" && $no_generate_password == 0)
+ {
+ $tMessage .= " / $fPassword)</br />";
+ }
+ else
+ {
+ if ($CONF['show_password'] == "YES" && $no_generate_password == 0)
+ {
+ $tMessage .= " / $fPassword)</br />";
+ }
+ else
+ {
+ $tMessage .= ")</br />";
+ }
+ }
+ }
+ }
# TODO: should we log creation, editing and deletion of admins?
# Note: needs special handling in viewlog, because domain is empty
@@ -2179,7 +2177,7 @@
$error,
$tMessage,
$pAdminCreate_admin_username_text,
- $pAdminCreate_admin_password_text
+ $pAdminCreate_admin_password_text
);
This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.
|