[public_server] Move NULL_DATE to DBA::NUL_DATETIME
authorHypolite Petovan <hypolite@mrpetovan.com>
Sun, 21 Oct 2018 06:54:09 +0000 (02:54 -0400)
committerHypolite Petovan <hypolite@mrpetovan.com>
Sun, 21 Oct 2018 06:54:09 +0000 (02:54 -0400)
- Remove useless DBA::escape when used with database constant

public_server/public_server.php

index 4d55295..2b36f3f 100644 (file)
@@ -6,13 +6,13 @@
  * Author: Keith Fernie <http://friendika.me4.it/profile/keith>
  */
 
+use Friendica\App;
 use Friendica\BaseModule;
 use Friendica\Core\Addon;
 use Friendica\Core\Config;
 use Friendica\Core\L10n;
 use Friendica\Database\DBA;
 use Friendica\Util\DateTimeFormat;
-use Friendica\Util\Security;
 
 function public_server_install()
 {
@@ -32,7 +32,7 @@ function public_server_uninstall()
        Addon::unregisterHook('logged_in', 'addon/public_server/public_server.php', 'public_server_login');
 }
 
-function public_server_load_config(\Friendica\App $a)
+function public_server_load_config(App $a)
 {
        $a->loadConfigFile(__DIR__. '/config/public_server.ini.php');
 }
@@ -59,7 +59,7 @@ function public_server_cron($a, $b)
        $r = q("SELECT * FROM `user` WHERE `account_expires_on` < UTC_TIMESTAMP() + INTERVAL 5 DAY AND
                `account_expires_on` > '%s' AND
                `expire_notification_sent` <= '%s'",
-               DBA::escape(NULL_DATE), DBA::escape(NULL_DATE));
+               DBA::NULL_DATETIME, DBA::NULL_DATETIME);
 
        if (DBA::isResult($r)) {
                foreach ($r as $rr) {
@@ -83,7 +83,7 @@ function public_server_cron($a, $b)
        $nologin = Config::get('public_server', 'nologin', false);
        if ($nologin) {
                $r = q("SELECT `uid` FROM `user` WHERE NOT `account_expired` AND `login_date` <= '%s' AND `register_date` < UTC_TIMESTAMP() - INTERVAL %d DAY AND `account_expires_on` <= '%s'",
-                       DBA::escape(NULL_DATE), intval($nologin), DBA::escape(NULL_DATE));
+                       DBA::NULL_DATETIME, intval($nologin), DBA::NULL_DATETIME);
                if (DBA::isResult($r)) {
                        foreach ($r as $rr) {
                                $fields = ['account_expires_on' => DateTimeFormat::utc('now +6 days')];
@@ -95,7 +95,7 @@ function public_server_cron($a, $b)
        $flagusers = Config::get('public_server', 'flagusers', false);
        if ($flagusers) {
                $r = q("SELECT `uid` FROM `user` WHERE NOT `account_expired` AND `login_date` < UTC_TIMESTAMP() - INTERVAL %d DAY AND `account_expires_on` <= '%s' AND `page-flags` = 0",
-                       intval($flagusers), DBA::escape(NULL_DATE));
+                       intval($flagusers), DBA::NULL_DATETIME);
                if (DBA::isResult($r)) {
                        foreach ($r as $rr) {
                                $fields = ['account_expires_on' => DateTimeFormat::utc('now +6 days')];
@@ -108,7 +108,7 @@ function public_server_cron($a, $b)
        $flagpostsexpire = Config::get('public_server', 'flagpostsexpire');
        if ($flagposts && $flagpostsexpire) {
                $r = q("SELECT `uid` FROM `user` WHERE NOT `account_expired` AND `login_date` < UTC_TIMESTAMP() - INTERVAL %d DAY AND `account_expires_on` <= '%s' and `expire` = 0 AND `page-flags` = 0",
-                       intval($flagposts), DBA::escape(NULL_DATE));
+                       intval($flagposts), DBA::NULL_DATETIME);
                if (DBA::isResult($r)) {
                        foreach ($r as $rr) {
                                DBA::update('user', ['expire' => $flagpostsexpire], ['uid' => $rr['uid']]);
@@ -138,7 +138,7 @@ function public_server_login($a, $b)
        }
 
        $fields = ['account_expires_on' => DateTimeFormat::utc('now +' . $days . ' days')];
-       $condition = ["`uid` = ? AND `account_expires_on` > ?", local_user(), NULL_DATE];
+       $condition = ["`uid` = ? AND `account_expires_on` > ?", local_user(), DBA::NULL_DATETIME];
        DBA::update('user', $fields, $condition);
 }