From 98365a88088388a1933109aa520c68a83b6db14d Mon Sep 17 00:00:00 2001
From: fabrixxm <fabrix.xm@gmail.com>
Date: Thu, 5 Apr 2018 22:21:59 +0200
Subject: [PATCH] Fix coding style

---
 catavatar/catavatar.php | 153 +++++++++++++++++++++-------------------
 1 file changed, 82 insertions(+), 71 deletions(-)

diff --git a/catavatar/catavatar.php b/catavatar/catavatar.php
index f8a6d2f1..8f812e9b 100644
--- a/catavatar/catavatar.php
+++ b/catavatar/catavatar.php
@@ -18,48 +18,58 @@ define("CATAVATAR_SIZE", 256);
 /**
  * Installs the addon hook
  */
-function catavatar_install() {
+function catavatar_install()
+{
 	Addon::registerHook('avatar_lookup', 'addon/catavatar/catavatar.php', 'catavatar_lookup');
 	Addon::registerHook('addon_settings', 'addon/catavatar/catavatar.php', 'catavatar_addon_settings');
 	Addon::registerHook('addon_settings_post', 'addon/catavatar/catavatar.php', 'catavatar_addon_settings_post');
 
-	logger("registered catavatar");
+	logger('registered catavatar');
 }
 
 /**
  * Removes the addon hook
  */
-function catavatar_uninstall() {
+function catavatar_uninstall()
+{
 	Addon::unregisterHook('avatar_lookup', 'addon/catavatar/catavatar.php', 'catavatar_lookup');
 	Addon::unregisterHook('addon_settings', 'addon/catavatar/catavatar.php', 'catavatar_addon_settings');
 	Addon::unregisterHook('addon_settings_post', 'addon/catavatar/catavatar.php', 'catavatar_addon_settings_post');
-	
-	logger("unregistered catavatar");
-}
 
+	logger('unregistered catavatar');
+}
 
-function catavatar_addon_settings(&$a, &$s) {
-	if(! local_user())
+/**
+ * Cat avatar user settings page
+ */
+function catavatar_addon_settings(&$a, &$s)
+{
+	if (!local_user()) {
 		return;
+	}
 
-	$t = get_markup_template("settings.tpl", "addon/catavatar/" );
+	$t = get_markup_template('settings.tpl', 'addon/catavatar/');
 	$s = replace_macros ($t, [
-		'$postpost' => x($_POST,"catavatar-morecat") || x($_POST,"catavatar-emailcat"),
+		'$postpost' => !empty($_POST['catavatar-morecat']) || !empty($_POST['catavatar-emailcat']),
 		'$uncache' => time(),
 		'$uid' => local_user(),
 		'$usecat' => L10n::t('Use Cat as Avatar'),
 		'$morecat' => L10n::t('More Random Cat!'),
 		'$emailcat' => L10n::t('Reset to email Cat'),
-		'$seed' => PConfig::get(local_user(), "catavatar", "seed", false),
-		'$header' => L10n::t('Cat Avatar').' '.L10n::t('Settings'),
+		'$seed' => PConfig::get(local_user(), 'catavatar', 'seed', false),
+		'$header' => L10n::t('Cat Avatar Settings'),
 	]);
-	return;
 }
 
-function catavatar_addon_settings_post(&$a, &$s) {
-	if(! local_user())
+/**
+ * Cat avatar user settings POST handle
+ */
+function catavatar_addon_settings_post(&$a, &$s)
+{
+	if (!local_user()) {
 		return;
-		
+	}
+
 	// delete the current cached cat avatar
 	$user = dba::selectFirst('user', ['email'],
 		[
@@ -69,44 +79,44 @@ function catavatar_addon_settings_post(&$a, &$s) {
 			'account_removed' => 0,
 		]
 	);
-	$seed = PConfig::get(local_user(), "catavatar", "seed", md5(trim(strtolower($user['email']))));
-	$imageurl = preg_replace('/[^A-Za-z0-9\._-]/', '', $seed); 
+	$seed = PConfig::get(local_user(), 'catavatar', 'seed', md5(trim(strtolower($user['email']))));
+	$imageurl = preg_replace('/[^A-Za-z0-9\._-]/', '', $seed);
 	$imageurl = substr($imageurl,0,35).'';
 	$cachefile = get_cachefile($imageurl);
 	if ($cachefile != "" && file_exists($cachefile)) {
 		unlink($cachefile);
 	}
-		
-		
-	if (x($_POST,"catavatar-usecat")) {
-		$url = $a->get_baseurl()."/catavatar/".local_user();
-		
+
+
+	if (!empty($_POST['catavatar-usecat'})) {
+		$url = $a->get_baseurl() . '/catavatar/' . local_user();
+
 		// set the catavatar url as avatar url in contact and default profile
 		// and set profile to 0 to current photo
 		// I'm not sure it's the correct way to do this...
-		$r = dba::update('contact', 
-			['photo'=>$url."/4", 'thumb'=>$url."/5", 'micro'=>$url."/6", 'avatar-date'=>DateTimeFormat::utcNow()], 
-			['uid'=>local_user(), 'self'=>1]
+		$r = dba::update('contact',
+			['photo' => $url . '/4', 'thumb' => $url . '/5', 'micro' => $url . '/6', 'avatar-date' => DateTimeFormat::utcNow()],
+			['uid' => local_user(), 'self' => 1]
 		);
 		if ($r===false) {
 			notice(L10n::t('There was an error, the cat ran away.'));
 			return;
 		}
 
-		$r = dba::update('profile', 
-			['photo'=>$url."/4", 'thumb'=>$url."/5"], 
-			['uid'=>local_user(), 'is-default'=>1]
+		$r = dba::update('profile',
+			['photo' => $url . '/4', 'thumb' => $url . '/5'],
+			['uid' => local_user(), 'is-default' => 1]
 		);
 		if ($r===false) {
 			notice(L10n::t('There was an error, the cat ran away.'));
 			return;
 		}
 
-		$r = dba::update('photo', 
-			['profile'=>0], 
-			['uid'=>local_user(), 'profile'=>1]
+		$r = dba::update('photo',
+			['profile' => 0],
+			['uid' => local_user(), 'profile' => 1]
 		);
-		if ($r===false) {
+		if ($r === false) {
 			notice(L10n::t('There was an error, the cat ran away.'));
 			return;
 		}
@@ -115,23 +125,23 @@ function catavatar_addon_settings_post(&$a, &$s) {
 		// Update global directory in background
 		$url = $a->get_baseurl() . '/profile/' . $a->user['nickname'];
 		if ($url && strlen(Config::get('system','directory'))) {
-			Worker::add(PRIORITY_LOW, "Directory", $url);
+			Worker::add(PRIORITY_LOW, 'Directory', $url);
 		}
 
 		Worker::add(PRIORITY_LOW, 'ProfileUpdate', local_user());
-		
-		info(L10n::t("Meow!"));
+
+		info(L10n::t('Meow!'));
 		return;
 	}
-	
 
 
-	if (x($_POST,"catavatar-morecat")) {
-		PConfig::set(local_user(), "catavatar", "seed", time());
+
+	if (!empty($_POST['catavatar-morecat'])) {
+		PConfig::set(local_user(), 'catavatar', 'seed', time());
 	}
 
-	if (x($_POST,"catavatar-emailcat")) {
-		PConfig::delete(local_user(), "catavatar", "seed");
+	if (!empty($_POST['catavatar-emailcat'])) {
+		PConfig::delete(local_user(), 'catavatar', 'seed');
 	}
 }
 
@@ -142,15 +152,15 @@ function catavatar_addon_settings_post(&$a, &$s) {
  * @param $a array
  * @param &$b array
  */
-function catavatar_lookup($a, &$b) {
-	$user = dba::selectFirst('user', ['uid'],['email'=>$b['email']]);
-	
-	$url = $a->get_baseurl().'/catavatar/'.$user['uid'];
+function catavatar_lookup($a, &$b)
+{
+	$user = dba::selectFirst('user', ['uid'], ['email' => $b['email']]);
+	$url = $a->get_baseurl() . '/catavatar/' . $user['uid'];
 
 	switch($b['size']) {
-		case 175: $url.="/4"; break;
-		case 80: $url.="/5"; break;
-		case 47: $url.="/6"; break;
+		case 175: $url .= "/4"; break;
+		case 80: $url .= "/5"; break;
+		case 47: $url .= "/6"; break;
 	}
 
 	$b['url'] = $url;
@@ -168,17 +178,19 @@ function catavatar_module(){}
  *
  * @TODO: support sizes
  */
-function catavatar_content($a) {
-	if ($a->argc < 2 || $a->argc > 3)
+function catavatar_content($a)
+{
+	if ($a->argc < 2 || $a->argc > 3) {
 		throw new NotFoundException(); // this should be catched on index and show default "not found" page.
+	}
 
 	$uid = intval($a->argv[1]);
-	
+
 	$size = 0;
 	if ($a->argc == 3) {
 		$size = intval($a->argv[2]);
 	}
-	
+
 	$user = dba::selectFirst('user', ['email'],
 		[
 			'uid' => $uid,
@@ -186,46 +198,45 @@ function catavatar_content($a) {
 			'account_expired' => 0,
 			'account_removed' => 0,
 		]
-	);	
-	
-	if ($user === False)
+	);
+
+	if ($user === false) {
 		throw new NotFoundException();
-	
+	}
+
 	$seed = PConfig::get(local_user(), "catavatar", "seed", md5(trim(strtolower($user['email']))));
-	//echo "<pre>"; var_dump($hash); killme();
 
-	
 	// from cat-avatar-generator.php
-
-	$imageurl = $seed."-".$size;
-	$imageurl = preg_replace('/[^A-Za-z0-9\._-]/', '', $imageurl); 
-	$imageurl = substr($imageurl,0,35).'';
+	$imageurl = $seed . "-" . $size;
+	$imageurl = preg_replace('/[^A-Za-z0-9\._-]/', '', $imageurl);
+	$imageurl = substr($imageurl,0,35) . '';
 	$cachefile = get_cachefile($imageurl);
 	$cachetime = 604800; # 1 week (1 day = 86400)
 
 	// Serve from the cache if it is younger than $cachetime
-	if ($cachefile != "" && file_exists($cachefile) && time() - $cachetime < filemtime($cachefile)) {
+	if ($cachefile != "" && file_exists($cachefile) && (time() - $cachetime) < filemtime($cachefile)) {
 		header('Pragma: public');
 		header('Cache-Control: max-age=86400');
 		header('Expires: '. gmdate('D, d M Y H:i:s \G\M\T', time() + 86400));
 		header('Content-Type: image/jpg');
 		readfile($cachefile);
-		exit;
+		killme();
 	}
 
 	// ...Or start generation
-	ob_start(); 
+	ob_start();
 
 	// render the picture:
 	build_cat($seed, $size);
 
 	// Save/cache the output to a file
-	if ($cachefile!=""){
+	if ($cachefile != "") {
 		$savedfile = fopen($cachefile, 'w+'); # w+ to be at start of the file, write mode, and attempt to create if not existing.
 		fwrite($savedfile, ob_get_contents());
 		fclose($savedfile);
 		chmod($cachefile, 0755);
 	}
+
 	ob_end_flush();
 
 	killme();
@@ -237,12 +248,12 @@ function catavatar_content($a) {
  * ====================
  * CAT-AVATAR-GENERATOR
  * ====================
- * 
+ *
  * @authors: Andreas Gohr, David Revoy
- * 
+ *
  * This PHP is licensed under the short and simple permissive:
  * [MIT License](https://en.wikipedia.org/wiki/MIT_License)
- * 
+ *
 **/
 
 function build_cat($seed='', $size=0){
@@ -283,7 +294,7 @@ function build_cat($seed='', $size=0){
 			case 5: $size = 80; break;
 			case 6: $size = 48; break;
 		}
-	
+
 		$dest = imagecreatetruecolor($size, $size);
 		imagealphablending($dest, false);
 		imagesavealpha($dest, true);
@@ -291,7 +302,7 @@ function build_cat($seed='', $size=0){
 		imagedestroy($cat);
 		$cat = $dest;
 	}
-	
+
 	// restore random seed
 	if($seed) srand();
 
-- 
2.39.5