From 3ecdbb6a5965459caa2e10ee30fd3fc011ab212f Mon Sep 17 00:00:00 2001
From: Michael <heluecht@pirati.ca>
Date: Tue, 16 Jan 2018 23:20:07 +0000
Subject: [PATCH 1/5] Standards

---
 src/Model/Item.php | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/src/Model/Item.php b/src/Model/Item.php
index 06c625274..0c8b8b549 100644
--- a/src/Model/Item.php
+++ b/src/Model/Item.php
@@ -14,6 +14,7 @@ use dba;
 
 require_once 'include/tags.php';
 require_once 'include/threads.php';
+require_once 'include/items.php';
 
 class Item
 {
@@ -122,13 +123,11 @@ class Item
 
 		$item = dba::selectFirst('item', [], ['id' => $itemid]);
 
-		if (count($item) && ($item["allow_cid"] == '')  && ($item["allow_gid"] == '') &&
+		if (DBM::is_result($item) && ($item["allow_cid"] == '')  && ($item["allow_gid"] == '') &&
 			($item["deny_cid"] == '') && ($item["deny_gid"] == '')) {
 
 			if (!dba::exists('item', ['uri' => $item['uri'], 'uid' => 0])) {
 				// Preparing public shadow (removing user specific data)
-				require_once("include/items.php");
-
 				unset($item['id']);
 				$item['uid'] = 0;
 				$item['origin'] = 0;
@@ -183,8 +182,6 @@ class Item
 		}
 
 		// Preparing public shadow (removing user specific data)
-		require_once("include/items.php");
-
 		unset($item['id']);
 		$item['uid'] = 0;
 		$item['origin'] = 0;

From 78ac7afe9cb110fa92c0e600b15f8efdb76b2b52 Mon Sep 17 00:00:00 2001
From: Hypolite Petovan <mrpetovan@gmail.com>
Date: Mon, 15 Jan 2018 19:08:28 -0500
Subject: [PATCH 2/5] Move process functions to Model\Process

- Add start|endProcess functions to Core\Worker
---
 mod/worker.php        |  4 +--
 scripts/worker.php    |  2 +-
 src/App.php           | 50 ++-----------------------------
 src/Core/Worker.php   | 36 +++++++++++++++++++----
 src/Model/Process.php | 68 +++++++++++++++++++++++++++++++++++++++++++
 5 files changed, 104 insertions(+), 56 deletions(-)
 create mode 100644 src/Model/Process.php

diff --git a/mod/worker.php b/mod/worker.php
index d6ecbecc2..056416e79 100644
--- a/mod/worker.php
+++ b/mod/worker.php
@@ -29,7 +29,7 @@ function worker_init($a){
 		return;
 	}
 
-	$a->start_process();
+	Worker::startProcess();
 
 	logger("Front end worker started: ".getmypid());
 
@@ -50,7 +50,7 @@ function worker_init($a){
 
 	Worker::unclaimProcess();
 
-	$a->end_process();
+	Worker::endProcess();
 
 	logger("Front end worker ended: ".getmypid());
 
diff --git a/scripts/worker.php b/scripts/worker.php
index a3738038b..d69b089ae 100755
--- a/scripts/worker.php
+++ b/scripts/worker.php
@@ -57,7 +57,7 @@ Worker::processQueue($run_cron);
 
 Worker::unclaimProcess();
 
-$a->end_process();
+Worker::endProcess();
 
 killme();
 
diff --git a/src/App.php b/src/App.php
index fdd928ba1..62b5f12e2 100644
--- a/src/App.php
+++ b/src/App.php
@@ -2,20 +2,17 @@
 
 namespace Friendica;
 
-use Friendica\Core\System;
 use Friendica\Core\Cache;
 use Friendica\Core\Config;
 use Friendica\Core\PConfig;
-use Friendica\Database\DBM;
-
-use dba;
+use Friendica\Core\System;
 
 use Detection\MobileDetect;
 
 use Exception;
 
 require_once 'boot.php';
-require_once 'include/dba.php';
+require_once 'include/text.php';
 
 /**
  *
@@ -695,49 +692,6 @@ class App {
 		$this->callstack[$value][$callstack] += (float) $duration;
 	}
 
-	/**
-	 * @brief Log active processes into the "process" table
-	 */
-	function start_process() {
-		$trace = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS, 1);
-
-		$command = basename($trace[0]['file']);
-
-		$this->remove_inactive_processes();
-
-		dba::transaction();
-
-		$r = q('SELECT `pid` FROM `process` WHERE `pid` = %d', intval(getmypid()));
-		if (!DBM::is_result($r)) {
-			dba::insert('process', ['pid' => getmypid(), 'command' => $command, 'created' => datetime_convert()]);
-		}
-		dba::commit();
-	}
-
-	/**
-	 * @brief Remove inactive processes
-	 */
-	function remove_inactive_processes() {
-		dba::transaction();
-
-		$r = q('SELECT `pid` FROM `process`');
-		if (DBM::is_result($r)) {
-			foreach ($r AS $process) {
-				if (!posix_kill($process['pid'], 0)) {
-					dba::delete('process', ['pid' => $process['pid']]);
-				}
-			}
-		}
-		dba::commit();
-	}
-
-	/**
-	 * @brief Remove the active process from the "process" table
-	 */
-	function end_process() {
-		dba::delete('process', ['pid' => getmypid()]);
-	}
-
 	function get_useragent() {
 		return
 			FRIENDICA_PLATFORM . " '" .
diff --git a/src/Core/Worker.php b/src/Core/Worker.php
index da7425a61..478f01a96 100644
--- a/src/Core/Worker.php
+++ b/src/Core/Worker.php
@@ -4,11 +4,10 @@
  */
 namespace Friendica\Core;
 
-use Friendica\App;
-use Friendica\Core\System;
 use Friendica\Core\Config;
-use Friendica\Core\Worker;
+use Friendica\Core\System;
 use Friendica\Database\DBM;
+use Friendica\Model\Process;
 use Friendica\Util\Lock;
 
 use dba;
@@ -50,7 +49,7 @@ class Worker
 		}
 
 		// We now start the process. This is done after the load check since this could increase the load.
-		$a->start_process();
+		self::startProcess();
 
 		// Kill stale processes every 5 minutes
 		$last_cleanup = Config::get('system', 'poller_last_cleaned', 0);
@@ -915,7 +914,7 @@ class Worker
 			if (self::tooMuchWorkers()) {
 				// Cleaning dead processes
 				self::killStaleWorkers();
-				get_app()->remove_inactive_processes();
+				Process::deleteInactive();
 
 				return;
 			}
@@ -1092,4 +1091,31 @@ class Worker
 
 		return true;
 	}
+
+	/**
+	 * Log active processes into the "process" table
+	 *
+	 * @brief Log active processes into the "process" table
+	 */
+	public static function startProcess()
+	{
+		$trace = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS, 1);
+
+		$command = basename($trace[0]['file']);
+
+		Process::deleteInactive();
+
+		Process::insert($command);
+	}
+
+	/**
+	 * Remove the active process from the "process" table
+	 *
+	 * @brief Remove the active process from the "process" table
+	 * @return bool
+	 */
+	public static function endProcess()
+	{
+		return Process::deleteByPid();
+	}
 }
diff --git a/src/Model/Process.php b/src/Model/Process.php
new file mode 100644
index 000000000..6b2d2fb64
--- /dev/null
+++ b/src/Model/Process.php
@@ -0,0 +1,68 @@
+<?php
+/**
+ * @file src/Model/Process.php
+ */
+namespace Friendica\Model;
+
+use Friendica\BaseObject;
+use dba;
+
+require_once 'include/dba.php';
+
+/**
+ * @brief functions for interacting with a process
+ */
+class Process extends BaseObject
+{
+	/**
+	 * Insert a new process row. If the pid parameter is omitted, we use the current pid
+	 *
+	 * @param string $command
+	 * @param string $pid
+	 * @return bool
+	 */
+	public static function insert($command, $pid = null)
+	{
+		dba::transaction();
+
+		if (!dba::exists('process', ['pid' => getmypid()])) {
+			$return = dba::insert('process', ['pid' => $pid, 'command' => $command, 'created' => datetime_convert()]);
+		}
+
+		dba::commit();
+
+		return $return;
+	}
+
+	/**
+	 * Remove a process row by pid. If the pid parameter is omitted, we use the current pid
+	 *
+	 * @param string $pid
+	 * @return bool
+	 */
+	public static function deleteByPid($pid = null)
+	{
+		if ($pid === null) {
+			$pid = getmypid();
+		}
+
+		return dba::delete('process', ['pid' => $pid]);
+	}
+
+	/**
+	 * Clean the process table of inactive physical processes
+	 */
+	public static function deleteInactive()
+	{
+		dba::transaction();
+
+		$processes = dba::select('process', ['pid']);
+		while($process = dba::fetch($processes)) {
+			if (!posix_kill($process['pid'], 0)) {
+				self::deleteByPid($process['pid']);
+			}
+		}
+
+		dba::commit();
+	}
+}

From ad88b6e3af2489bf8da8f423142418b9c57d58ac Mon Sep 17 00:00:00 2001
From: Hypolite Petovan <mrpetovan@gmail.com>
Date: Mon, 15 Jan 2018 19:13:21 -0500
Subject: [PATCH 3/5] Fix issues in App.php

- Fix formatting
- Add method visibility
- Remove unused properties
- Remove unused methods
- Remove unused parameters
- Fix implode parameter order (?!)
---
 src/App.php | 270 +++++++++++++++++++++++++++-------------------------
 1 file changed, 139 insertions(+), 131 deletions(-)

diff --git a/src/App.php b/src/App.php
index 62b5f12e2..ae914ba60 100644
--- a/src/App.php
+++ b/src/App.php
@@ -27,8 +27,8 @@ require_once 'include/text.php';
  * before we spit the page out.
  *
  */
-class App {
-
+class App
+{
 	public $module_loaded = false;
 	public $module_class = null;
 	public $query_string;
@@ -114,12 +114,9 @@ class App {
 	];
 	private $scheme;
 	private $hostname;
-	private $db;
 	private $curl_code;
 	private $curl_content_type;
 	private $curl_headers;
-	private $cached_profile_image;
-	private $cached_profile_picdate;
 	private static $a;
 
 	/**
@@ -127,20 +124,18 @@ class App {
 	 *
 	 * @param string $basepath Path to the app base folder
 	 */
-	function __construct($basepath) {
-
+	public function __construct($basepath)
+	{
 		global $default_timezone;
 
-		$hostname = '';
-
-		if (! static::directory_usable($basepath, false)) {
+		if (!static::directory_usable($basepath, false)) {
 			throw new Exception('Basepath ' . $basepath . ' isn\'t usable.');
 		}
 
 		$this->basepath = rtrim($basepath, DIRECTORY_SEPARATOR);
 
-		if (file_exists($this->basepath.DIRECTORY_SEPARATOR.'.htpreconfig.php')) {
-			include $this->basepath.DIRECTORY_SEPARATOR.'.htpreconfig.php';
+		if (file_exists($this->basepath . DIRECTORY_SEPARATOR . '.htpreconfig.php')) {
+			include $this->basepath . DIRECTORY_SEPARATOR . '.htpreconfig.php';
 		}
 
 		$this->timezone = ((x($default_timezone)) ? $default_timezone : 'UTC');
@@ -204,10 +199,6 @@ class App {
 			}
 		}
 
-		if ($hostname != '') {
-			$this->hostname = $hostname;
-		}
-
 		set_include_path(
 			get_include_path() . PATH_SEPARATOR
 			. $this->basepath . DIRECTORY_SEPARATOR . 'include' . PATH_SEPARATOR
@@ -312,18 +303,19 @@ class App {
 	 *
 	 * @return string
 	 */
-	public function get_basepath() {
+	public function get_basepath()
+	{
 		$basepath = $this->basepath;
 
-		if (! $basepath) {
+		if (!$basepath) {
 			$basepath = Config::get('system', 'basepath');
 		}
 
-		if (! $basepath && x($_SERVER, 'DOCUMENT_ROOT')) {
+		if (!$basepath && x($_SERVER, 'DOCUMENT_ROOT')) {
 			$basepath = $_SERVER['DOCUMENT_ROOT'];
 		}
 
-		if (! $basepath && x($_SERVER, 'PWD')) {
+		if (!$basepath && x($_SERVER, 'PWD')) {
 			$basepath = $_SERVER['PWD'];
 		}
 
@@ -340,7 +332,8 @@ class App {
 	 * @param string $path The path that is about to be normalized
 	 * @return string normalized path - when possible
 	 */
-	public static function realpath($path) {
+	public static function realpath($path)
+	{
 		$normalized = realpath($path);
 
 		if (!is_bool($normalized)) {
@@ -350,7 +343,8 @@ class App {
 		}
 	}
 
-	function get_scheme() {
+	public function get_scheme()
+	{
 		return $this->scheme;
 	}
 
@@ -368,7 +362,8 @@ class App {
 	 * @param bool $ssl Whether to append http or https under SSL_POLICY_SELFSIGN
 	 * @return string Friendica server base URL
 	 */
-	function get_baseurl($ssl = false) {
+	public function get_baseurl($ssl = false)
+	{
 		$scheme = $this->scheme;
 
 		if (Config::get('system', 'ssl_policy') == SSL_POLICY_FULL) {
@@ -396,14 +391,15 @@ class App {
 	/**
 	 * @brief Initializes the baseurl components
 	 *
-	 * Clears the baseurl cache to prevent inconstistencies
+	 * Clears the baseurl cache to prevent inconsistencies
 	 *
 	 * @param string $url
 	 */
-	function set_baseurl($url) {
+	public function set_baseurl($url)
+	{
 		$parsed = @parse_url($url);
 
-		if ($parsed) {
+		if (x($parsed)) {
 			$this->scheme = $parsed['scheme'];
 
 			$hostname = $parsed['host'];
@@ -414,8 +410,8 @@ class App {
 				$this->path = trim($parsed['path'], '\\/');
 			}
 
-			if (file_exists($this->basepath.DIRECTORY_SEPARATOR.'.htpreconfig.php')) {
-				include $this->basepath.DIRECTORY_SEPARATOR.'.htpreconfig.php';
+			if (file_exists($this->basepath . DIRECTORY_SEPARATOR . '.htpreconfig.php')) {
+				include $this->basepath . DIRECTORY_SEPARATOR . '.htpreconfig.php';
 			}
 
 			if (Config::get('config', 'hostname') != '') {
@@ -428,7 +424,8 @@ class App {
 		}
 	}
 
-	function get_hostname() {
+	public function get_hostname()
+	{
 		if (Config::get('config', 'hostname') != '') {
 			$this->hostname = Config::get('config', 'hostname');
 		}
@@ -436,33 +433,30 @@ class App {
 		return $this->hostname;
 	}
 
-	function set_hostname($h) {
-		$this->hostname = $h;
-	}
-
-	function set_path($p) {
-		$this->path = trim(trim($p), '/');
-	}
-
-	function get_path() {
+	public function get_path()
+	{
 		return $this->path;
 	}
 
-	function set_pager_total($n) {
+	public function set_pager_total($n)
+	{
 		$this->pager['total'] = intval($n);
 	}
 
-	function set_pager_itemspage($n) {
+	public function set_pager_itemspage($n)
+	{
 		$this->pager['itemspage'] = ((intval($n) > 0) ? intval($n) : 0);
 		$this->pager['start'] = ($this->pager['page'] * $this->pager['itemspage']) - $this->pager['itemspage'];
 	}
 
-	function set_pager_page($n) {
+	public function set_pager_page($n)
+	{
 		$this->pager['page'] = $n;
 		$this->pager['start'] = ($this->pager['page'] * $this->pager['itemspage']) - $this->pager['itemspage'];
 	}
 
-	function init_pagehead() {
+	public function init_pagehead()
+	{
 		$interval = ((local_user()) ? PConfig::get(local_user(), 'system', 'update_interval') : 40000);
 
 		// If the update is 'deactivated' set it to the highest integer number (~24 days)
@@ -515,67 +509,70 @@ class App {
 
 		$tpl = get_markup_template('head.tpl');
 		$this->page['htmlhead'] = replace_macros($tpl, [
-				'$baseurl' => $this->get_baseurl(),
-				'$local_user' => local_user(),
-				'$generator' => 'Friendica' . ' ' . FRIENDICA_VERSION,
-				'$delitem' => t('Delete this item?'),
-				'$showmore' => t('show more'),
-				'$showfewer' => t('show fewer'),
-				'$update_interval' => $interval,
-				'$shortcut_icon' => $shortcut_icon,
-				'$touch_icon' => $touch_icon,
-				'$stylesheet' => $stylesheet,
-				'$infinite_scroll' => $invinite_scroll,
-			]) . $this->page['htmlhead'];
+			'$baseurl'         => $this->get_baseurl(),
+			'$local_user'      => local_user(),
+			'$generator'       => 'Friendica' . ' ' . FRIENDICA_VERSION,
+			'$delitem'         => t('Delete this item?'),
+			'$showmore'        => t('show more'),
+			'$showfewer'       => t('show fewer'),
+			'$update_interval' => $interval,
+			'$shortcut_icon'   => $shortcut_icon,
+			'$touch_icon'      => $touch_icon,
+			'$stylesheet'      => $stylesheet,
+			'$infinite_scroll' => $invinite_scroll,
+		]) . $this->page['htmlhead'];
 	}
 
-	function init_page_end() {
+	public function init_page_end()
+	{
 		if (!isset($this->page['end'])) {
 			$this->page['end'] = '';
 		}
 		$tpl = get_markup_template('end.tpl');
 		$this->page['end'] = replace_macros($tpl, [
-				'$baseurl' => $this->get_baseurl()
-			]) . $this->page['end'];
+			'$baseurl' => $this->get_baseurl()
+		]) . $this->page['end'];
 	}
 
-	function set_curl_code($code) {
+	public function set_curl_code($code)
+	{
 		$this->curl_code = $code;
 	}
 
-	function get_curl_code() {
+	public function get_curl_code()
+	{
 		return $this->curl_code;
 	}
 
-	function set_curl_content_type($content_type) {
+	public function set_curl_content_type($content_type)
+	{
 		$this->curl_content_type = $content_type;
 	}
 
-	function get_curl_content_type() {
+	public function get_curl_content_type()
+	{
 		return $this->curl_content_type;
 	}
 
-	function set_curl_headers($headers) {
+	public function set_curl_headers($headers)
+	{
 		$this->curl_headers = $headers;
 	}
 
-	function get_curl_headers() {
+	public function get_curl_headers()
+	{
 		return $this->curl_headers;
 	}
 
-	function get_cached_avatar_image($avatar_image) {
-		return $avatar_image;
-	}
-
 	/**
-	 * @brief Removes the baseurl from an url. This avoids some mixed content problems.
+	 * @brief Removes the base url from an url. This avoids some mixed content problems.
 	 *
 	 * @param string $orig_url
 	 *
 	 * @return string The cleaned url
 	 */
-	function remove_baseurl($orig_url) {
-
+	public function remove_baseurl($orig_url)
+	{
 		// Remove the hostname from the url if it is an internal link
 		$nurl = normalise_link($orig_url);
 		$base = normalise_link($this->get_baseurl());
@@ -592,23 +589,18 @@ class App {
 	/**
 	 * @brief Register template engine class
 	 *
-	 * If $name is '', is used class static property $class::$name
-	 *
 	 * @param string $class
-	 * @param string $name
 	 */
-	function register_template_engine($class, $name = '') {
-		/// @TODO Really === and not just == ?
-		if ($name === '') {
-			$v = get_class_vars($class);
-			if (x($v, 'name'))
-				$name = $v['name'];
-		}
-		if ($name === '') {
+	private function register_template_engine($class)
+	{
+		$v = get_class_vars($class);
+		if (x($v, 'name')) {
+			$name = $v['name'];
+			$this->template_engines[$name] = $class;
+		} else {
 			echo "template engine <tt>$class</tt> cannot be registered without a name.\n";
 			killme();
 		}
-		$this->template_engines[$name] = $class;
 	}
 
 	/**
@@ -617,18 +609,13 @@ class App {
 	 * If $name is not defined, return engine defined by theme,
 	 * or default
 	 *
-	 * @param string $name Template engine name
 	 * @return object Template Engine instance
 	 */
-	function template_engine($name = '') {
-		/// @TODO really type-check included?
-		if ($name !== '') {
-			$template_engine = $name;
-		} else {
-			$template_engine = 'smarty3';
-			if (x($this->theme, 'template_engine')) {
-				$template_engine = $this->theme['template_engine'];
-			}
+	public function template_engine()
+	{
+		$template_engine = 'smarty3';
+		if (x($this->theme, 'template_engine')) {
+			$template_engine = $this->theme['template_engine'];
 		}
 
 		if (isset($this->template_engines[$template_engine])) {
@@ -651,23 +638,28 @@ class App {
 	 *
 	 * @return string
 	 */
-	function get_template_engine() {
+	public function get_template_engine()
+	{
 		return $this->theme['template_engine'];
 	}
 
-	function set_template_engine($engine = 'smarty3') {
+	public function set_template_engine($engine = 'smarty3')
+	{
 		$this->theme['template_engine'] = $engine;
 	}
 
-	function get_template_ldelim($engine = 'smarty3') {
+	public function get_template_ldelim($engine = 'smarty3')
+	{
 		return $this->ldelim[$engine];
 	}
 
-	function get_template_rdelim($engine = 'smarty3') {
+	public function get_template_rdelim($engine = 'smarty3')
+	{
 		return $this->rdelim[$engine];
 	}
 
-	function save_timestamp($stamp, $value) {
+	public function save_timestamp($stamp, $value)
+	{
 		if (!isset($this->config['system']['profiler']) || !$this->config['system']['profiler']) {
 			return;
 		}
@@ -692,7 +684,8 @@ class App {
 		$this->callstack[$value][$callstack] += (float) $duration;
 	}
 
-	function get_useragent() {
+	public function get_useragent()
+	{
 		return
 			FRIENDICA_PLATFORM . " '" .
 			FRIENDICA_CODENAME . "' " .
@@ -701,7 +694,8 @@ class App {
 			$this->get_baseurl();
 	}
 
-	function is_friendica_app() {
+	public function is_friendica_app()
+	{
 		return $this->is_friendica_app;
 	}
 
@@ -713,27 +707,29 @@ class App {
 	 *
 	 * @return bool Is it a known backend?
 	 */
-	function is_backend() {
-		static $backends = [];
-		$backends[] = '_well_known';
-		$backends[] = 'api';
-		$backends[] = 'dfrn_notify';
-		$backends[] = 'fetch';
-		$backends[] = 'hcard';
-		$backends[] = 'hostxrd';
-		$backends[] = 'nodeinfo';
-		$backends[] = 'noscrape';
-		$backends[] = 'p';
-		$backends[] = 'poco';
-		$backends[] = 'post';
-		$backends[] = 'proxy';
-		$backends[] = 'pubsub';
-		$backends[] = 'pubsubhubbub';
-		$backends[] = 'receive';
-		$backends[] = 'rsd_xml';
-		$backends[] = 'salmon';
-		$backends[] = 'statistics_json';
-		$backends[] = 'xrd';
+	public function is_backend()
+	{
+		static $backends = [
+			'_well_known',
+			'api',
+			'dfrn_notify',
+			'fetch',
+			'hcard',
+			'hostxrd',
+			'nodeinfo',
+			'noscrape',
+			'p',
+			'poco',
+			'post',
+			'proxy',
+			'pubsub',
+			'pubsubhubbub',
+			'receive',
+			'rsd_xml',
+			'salmon',
+			'statistics_json',
+			'xrd',
+		];
 
 		// Check if current module is in backend or backend flag is set
 		return (in_array($this->module, $backends) || $this->backend);
@@ -744,10 +740,14 @@ class App {
 	 *
 	 * @return bool Is the limit reached?
 	 */
-	function max_processes_reached() {
+	public function max_processes_reached()
+	{
 		// Deactivated, needs more investigating if this check really makes sense
 		return false;
 
+		/*
+		 * Commented out to suppress static analyzer issues
+		 *
 		if ($this->is_backend()) {
 			$process = 'backend';
 			$max_processes = Config::get('system', 'max_processes_backend');
@@ -772,6 +772,7 @@ class App {
 			}
 		}
 		return false;
+		 */
 	}
 
 	/**
@@ -779,7 +780,8 @@ class App {
 	 *
 	 * @return bool Is the memory limit reached?
 	 */
-	public function min_memory_reached() {
+	public function min_memory_reached()
+	{
 		$min_memory = Config::get('system', 'min_memory', 0);
 		if ($min_memory == 0) {
 			return false;
@@ -798,7 +800,7 @@ class App {
 			$meminfo[$key] = (int) ($meminfo[$key] / 1024);
 		}
 
-		if (!isset($meminfo['MemAvailable']) || ! isset($meminfo['MemFree'])) {
+		if (!isset($meminfo['MemAvailable']) || !isset($meminfo['MemFree'])) {
 			return false;
 		}
 
@@ -818,8 +820,8 @@ class App {
 	 *
 	 * @return bool Is the load reached?
 	 */
-	function maxload_reached() {
-
+	public function maxload_reached()
+	{
 		if ($this->is_backend()) {
 			$process = 'backend';
 			$maxsysload = intval(Config::get('system', 'maxloadavg'));
@@ -844,8 +846,8 @@ class App {
 		return false;
 	}
 
-	function proc_run($args) {
-
+	public function proc_run($args)
+	{
 		if (!function_exists('proc_open')) {
 			return;
 		}
@@ -871,7 +873,7 @@ class App {
 			$args[$x] = escapeshellarg($args[$x]);
 		}
 
-		$cmdline = implode($args, ' ');
+		$cmdline = implode(' ', $args);
 
 		if ($this->min_memory_reached()) {
 			return;
@@ -896,8 +898,9 @@ class App {
 	 *
 	 * @return string system username
 	 */
-	static function systemuser() {
-		if (!function_exists('posix_getpwuid') || ! function_exists('posix_geteuid')) {
+	private static function systemuser()
+	{
+		if (!function_exists('posix_getpwuid') || !function_exists('posix_geteuid')) {
 			return '';
 		}
 
@@ -910,7 +913,8 @@ class App {
 	 *
 	 * @return boolean the directory is usable
 	 */
-	static function directory_usable($directory, $check_writable = true) {
+	public static function directory_usable($directory, $check_writable = true)
+	{
 		if ($directory == '') {
 			logger('Directory is empty. This shouldn\'t happen.', LOGGER_DEBUG);
 			return false;
@@ -920,18 +924,22 @@ class App {
 			logger('Path "' . $directory . '" does not exist for user ' . self::systemuser(), LOGGER_DEBUG);
 			return false;
 		}
+
 		if (is_file($directory)) {
 			logger('Path "' . $directory . '" is a file for user ' . self::systemuser(), LOGGER_DEBUG);
 			return false;
 		}
+
 		if (!is_dir($directory)) {
 			logger('Path "' . $directory . '" is not a directory for user ' . self::systemuser(), LOGGER_DEBUG);
 			return false;
 		}
+
 		if ($check_writable && !is_writable($directory)) {
 			logger('Path "' . $directory . '" is not writable for user ' . self::systemuser(), LOGGER_DEBUG);
 			return false;
 		}
+
 		return true;
 	}
 }

From f193f01a7d6ed4946b5ec4a4b92294bb75b294cb Mon Sep 17 00:00:00 2001
From: Hypolite Petovan <mrpetovan@gmail.com>
Date: Mon, 15 Jan 2018 19:27:48 -0500
Subject: [PATCH 4/5] Fix issues in changed files

- Initializes $return in Process::insert
- Update the type hint of dba::fetch
- Remove unused $a parameter in worker_init()
- Fix uses
---
 include/dba.php       | 2 +-
 mod/worker.php        | 5 ++---
 scripts/worker.php    | 2 +-
 src/Model/Process.php | 2 ++
 4 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/include/dba.php b/include/dba.php
index 6471d3645..b1b689fdf 100644
--- a/include/dba.php
+++ b/include/dba.php
@@ -659,7 +659,7 @@ class dba {
 	/**
 	 * @brief Fetch a single row
 	 *
-	 * @param PDOStatement|mysqli_result|mysqli_stmt $stmt statement object
+	 * @param mixed $stmt statement object
 	 * @return array current row
 	 */
 	public static function fetch($stmt) {
diff --git a/mod/worker.php b/mod/worker.php
index 056416e79..1bd588078 100644
--- a/mod/worker.php
+++ b/mod/worker.php
@@ -4,11 +4,10 @@
  * @brief Module for running the worker as frontend process
  */
 
-use Friendica\Core\Worker;
 use Friendica\Core\Config;
-use Friendica\Core\PConfig;
+use Friendica\Core\Worker;
 
-function worker_init($a){
+function worker_init(){
 
 	if (!Config::get("system", "frontend_worker")) {
 		return;
diff --git a/scripts/worker.php b/scripts/worker.php
index d69b089ae..ccf24b7b4 100755
--- a/scripts/worker.php
+++ b/scripts/worker.php
@@ -6,8 +6,8 @@
  */
 
 use Friendica\App;
-use Friendica\Core\Worker;
 use Friendica\Core\Config;
+use Friendica\Core\Worker;
 
 // Ensure that worker.php is executed from the base path of the installation
 if (!file_exists("boot.php") && (sizeof($_SERVER["argv"]) != 0)) {
diff --git a/src/Model/Process.php b/src/Model/Process.php
index 6b2d2fb64..fbc844c8d 100644
--- a/src/Model/Process.php
+++ b/src/Model/Process.php
@@ -23,6 +23,8 @@ class Process extends BaseObject
 	 */
 	public static function insert($command, $pid = null)
 	{
+		$return = true;
+
 		dba::transaction();
 
 		if (!dba::exists('process', ['pid' => getmypid()])) {

From c1c9a829faabb567a6de4d2ac1d4292eaca31b5f Mon Sep 17 00:00:00 2001
From: Hypolite Petovan <mrpetovan@gmail.com>
Date: Mon, 15 Jan 2018 20:54:06 -0500
Subject: [PATCH 5/5] Add missing include in Model\Process

---
 src/Model/Process.php | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/Model/Process.php b/src/Model/Process.php
index fbc844c8d..a96077712 100644
--- a/src/Model/Process.php
+++ b/src/Model/Process.php
@@ -8,6 +8,7 @@ use Friendica\BaseObject;
 use dba;
 
 require_once 'include/dba.php';
+require_once 'include/datetime.php';
 
 /**
  * @brief functions for interacting with a process