summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGeekologist <Conver@users.noreply.github.com>2016-07-01 15:29:25 +0200
committerGitHub <noreply@github.com>2016-07-01 15:29:25 +0200
commita141c41ba2a94ee67ff689aa4eb688d77b41d48a (patch)
tree76186c092148deffb7df9f40f9929074f098d3ad
parenta00269fdb8961f57f14abcfc99c69819fee55352 (diff)
parent210c3ec90ae9627844167163c03c45df8ae4d70c (diff)
downloadPHPAuth-a141c41ba2a94ee67ff689aa4eb688d77b41d48a.zip
PHPAuth-a141c41ba2a94ee67ff689aa4eb688d77b41d48a.tar.gz
PHPAuth-a141c41ba2a94ee67ff689aa4eb688d77b41d48a.tar.bz2
Merge pull request #192 from nurtext/master
Changed visibility of functions and variables from "private" to "protected"
-rwxr-xr-xAuth.php70
-rwxr-xr-xConfig.php16
-rw-r--r--languages/de_DE.php4
3 files changed, 45 insertions, 45 deletions
diff --git a/Auth.php b/Auth.php
index 306d9e1..5dcbcca 100755
--- a/Auth.php
+++ b/Auth.php
@@ -12,7 +12,7 @@ use PHPMailer\PHPMailer\PHPMailer;
class Auth
{
- private $dbh;
+ protected $dbh;
public $config;
public $lang;
@@ -196,7 +196,7 @@ class Auth
$return['message'] = $addUser['message'];
return $return;
}
-
+
$return['error'] = false;
$return['message'] = ($sendmail == true ? $this->lang["register_success"] : $this->lang['register_success_emailmessage_suppressed'] );
@@ -351,7 +351,7 @@ class Auth
* @return array $data
*/
- private function addSession($uid, $remember)
+ protected function addSession($uid, $remember)
{
$ip = $this->getIp();
$user = $this->getBaseUser($uid);
@@ -391,7 +391,7 @@ class Auth
* @return boolean
*/
- private function deleteExistingSessions($uid)
+ protected function deleteExistingSessions($uid)
{
$query = $this->dbh->prepare("DELETE FROM {$this->config->table_sessions} WHERE uid = ?");
$query->execute(array($uid));
@@ -405,7 +405,7 @@ class Auth
* @return boolean
*/
- private function deleteSession($hash)
+ protected function deleteSession($hash)
{
$query = $this->dbh->prepare("DELETE FROM {$this->config->table_sessions} WHERE hash = ?");
$query->execute(array($hash));
@@ -510,7 +510,7 @@ class Auth
* @return int $uid
*/
- private function addUser($email, $password, $params = array(), &$sendmail)
+ protected function addUser($email, $password, $params = array(), &$sendmail)
{
$return['error'] = true;
@@ -539,16 +539,16 @@ class Auth
} else {
$isactive = 1;
}
-
+
$password = $this->getHash($password);
-
+
if (is_array($params)&& count($params) > 0) {
$customParamsQueryArray = Array();
-
+
foreach($params as $paramKey => $paramValue) {
$customParamsQueryArray[] = array('value' => $paramKey . ' = ?');
}
-
+
$setParams = ', ' . implode(', ', array_map(function ($entry) {
return $entry['value'];
}, $customParamsQueryArray));
@@ -576,7 +576,7 @@ class Auth
* @return array $data
*/
- private function getBaseUser($uid)
+ protected function getBaseUser($uid)
{
$query = $this->dbh->prepare("SELECT email, password, isactive FROM {$this->config->table_users} WHERE id = ?");
$query->execute(array($uid));
@@ -594,7 +594,7 @@ class Auth
$data['uid'] = $uid;
return $data;
}
-
+
/**
* Gets public user data for a given UID and returns an array, password is not returned
* @param int $uid
@@ -619,7 +619,7 @@ class Auth
$data['uid'] = $uid;
unset($data['password']);
return $data;
- }
+ }
/**
* Allows a user to delete their account
@@ -701,31 +701,31 @@ class Auth
* @return boolean
*/
- private function addRequest($uid, $email, $type, &$sendmail)
+ protected function addRequest($uid, $email, $type, &$sendmail)
{
$return['error'] = true;
if($type != "activation" && $type != "reset") {
$return['message'] = $this->lang["system_error"] . " #08";
return $return;
- }
-
+ }
+
// if not set manually, check config data
if($sendmail === NULL)
{
- $sendmail = true;
+ $sendmail = true;
if($type == "reset" && $this->config->emailmessage_suppress_reset === true ) {
$sendmail = false;
$return['error'] = false;
return $return;
- }
+ }
if ($type == "activation" && $this->config->emailmessage_suppress_activation === true ) {
$sendmail = false;
$return['error'] = false;
return $return;
}
- }
-
+ }
+
$query = $this->dbh->prepare("SELECT id, expire FROM {$this->config->table_requests} WHERE uid = ? AND type = ?");
$query->execute(array($uid, $type));
@@ -762,7 +762,7 @@ class Auth
if($sendmail === true)
{
- // Check configuration for SMTP parameters
+ // Check configuration for SMTP parameters
$mail = new PHPMailer;
if($this->config->smtp) {
$mail->isSMTP();
@@ -773,19 +773,19 @@ class Auth
$mail->Password = $this->config->smtp_password;
}
$mail->Port = $this->config->smtp_port;
-
+
if(!is_null($this->config->smtp_security)) {
$mail->SMTPSecure = $this->config->smtp_security;
}
}
-
+
$mail->From = $this->config->site_email;
$mail->FromName = $this->config->site_name;
$mail->addAddress($email);
$mail->isHTML(true);
-
+
if($type == "activation") {
-
+
$mail->Subject = sprintf($this->lang['email_activation_subject'], $this->config->site_name);
$mail->Body = sprintf($this->lang['email_activation_body'], $this->config->site_url, $this->config->site_activation_page, $key);
$mail->AltBody = sprintf($this->lang['email_activation_altbody'], $this->config->site_url, $this->config->site_activation_page, $key);
@@ -795,10 +795,10 @@ class Auth
$mail->Body = sprintf($this->lang['email_reset_body'], $this->config->site_url, $this->config->site_password_reset_page, $key);
$mail->AltBody = sprintf($this->lang['email_reset_altbody'], $this->config->site_url, $this->config->site_password_reset_page, $key);
}
-
+
if(!$mail->send()) {
$this->deleteRequest($request_id);
-
+
$return['message'] = $this->lang["system_error"] . " #10";
return $return;
}
@@ -856,7 +856,7 @@ class Auth
* @return boolean
*/
- private function deleteRequest($id)
+ protected function deleteRequest($id)
{
$query = $this->dbh->prepare("DELETE FROM {$this->config->table_requests} WHERE id = ?");
return $query->execute(array($id));
@@ -868,7 +868,7 @@ class Auth
* @return array $return
*/
- private function validatePassword($password) {
+ protected function validatePassword($password) {
$return['error'] = true;
if (strlen($password) < (int)$this->config->verify_password_min_length ) {
@@ -886,7 +886,7 @@ class Auth
* @return array $return
*/
- private function validateEmail($email) {
+ protected function validateEmail($email) {
$return['error'] = true;
if (strlen($email) < (int)$this->config->verify_email_min_length ) {
@@ -1249,7 +1249,7 @@ class Auth
* @param string $captcha
* @return boolean
*/
- private function checkCaptcha($captcha)
+ protected function checkCaptcha($captcha)
{
return true;
}
@@ -1259,7 +1259,7 @@ class Auth
* @return boolean
*/
- private function addAttempt()
+ protected function addAttempt()
{
$ip = $this->getIp();
@@ -1277,7 +1277,7 @@ class Auth
* @return boolean
*/
- private function deleteAttempts($ip, $all = false)
+ protected function deleteAttempts($ip, $all = false)
{
if($all==true)
{
@@ -1322,7 +1322,7 @@ class Auth
* @return string $ip
*/
- private function getIp()
+ protected function getIp()
{
if(isset($_SERVER['HTTP_X_FORWARDED_FOR']) && $_SERVER['HTTP_X_FORWARDED_FOR'] != '') {
return $_SERVER['HTTP_X_FORWARDED_FOR'];
@@ -1330,7 +1330,7 @@ class Auth
return $_SERVER['REMOTE_ADDR'];
}
}
-
+
/**
* Returns is user logged in
* @return boolean
diff --git a/Config.php b/Config.php
index 2e4288c..dd17b56 100755
--- a/Config.php
+++ b/Config.php
@@ -6,9 +6,9 @@ namespace PHPAuth;
*/
class Config
{
- private $dbh;
- private $config;
- private $config_table = 'config';
+ protected $dbh;
+ protected $config;
+ protected $config_table = 'config';
/**
*
@@ -37,7 +37,7 @@ class Config
/**
* Config::__get()
- *
+ *
* @param mixed $setting
* @return string
*/
@@ -48,7 +48,7 @@ class Config
/**
* Config::__set()
- *
+ *
* @param mixed $setting
* @param mixed $value
* @return bool
@@ -60,13 +60,13 @@ class Config
if($query->execute(array($value, $setting))) {
$this->config[$setting] = $value;
return true;
- }
+ }
return false;
}
/**
* Config::override()
- *
+ *
* @param mixed $setting
* @param mixed $value
* @return bool
@@ -84,7 +84,7 @@ class Config
* Set default values.
* REQUIRED FOR USERS THAT DOES NOT UPDATE THEIR `config` TABLES.
*/
- private function setForgottenDefaults()
+ protected function setForgottenDefaults()
{
// verify* values.
diff --git a/languages/de_DE.php b/languages/de_DE.php
index c9ec4ff..afd2741 100644
--- a/languages/de_DE.php
+++ b/languages/de_DE.php
@@ -10,7 +10,7 @@ $lang['email_password_incorrect'] = "E-Mail-Adresse / Passwort nicht korrekt.";
$lang['remember_me_invalid'] = "Das Angemeldet bleiben-Feld ist ungültig.";
$lang['password_short'] = "Passwort ist zu kurz.";
-$lang['password_weak'] = "Password is too weak.";
+$lang['password_weak'] = "Passwort ist zu einfach.";
$lang['password_nomatch'] = "Passwörter stimmen nicht überein";
$lang['password_changed'] = "Passwort wurde erfolgreich geändert.";
$lang['password_incorrect'] = "Aktuelles Passwort ist nicht korrekt.";
@@ -69,4 +69,4 @@ $lang['email_reset_body'] = 'Hallo,<br/><br/>um dein Passwort zurückzusetzen, k
$lang['email_reset_altbody'] = 'Hallo,' . "\n\n" . 'um dein Passwort zurückzusetzen, klicke bitte auf folgenden Link oder kopiere ihn manuell in die Adresszeile des Browsers:' . "\n" . '%1$s/%2$s' . "\n\n" . 'Du benötigst folgenden Sicherheitsschlüssel: %3$s' . "\n\n" . 'Wenn dein Passwort auf %1$s gar nicht zurücksetzen willst, dann ignoriere diese E-Mail bitte einfach.';
$lang['account_deleted'] = "Benutzerkonto wurde erfolgreich gelöscht.";
-$lang['function_disabled'] = "Diese Funktion wurde deaktiviert."; \ No newline at end of file
+$lang['function_disabled'] = "Diese Funktion wurde deaktiviert.";