Fixes/rewrites on videos.php (#5489)

* Fixes/rewrites:
- use empty()/isset() instead of deprecated x()
- merged 2 nested if() blocks into one
- avoided nested if() block inside else block by rewriting it to elseif()
- $contact_id is an integer, let's test on > 0 here
- added a lot spaces and some empty lines for better readability

* CR request:
- removed stray spaces
- added some where they improve readability

mod/videos.php

@@ -18,10 +18,11 @@ use Friendica\Protocol\DFRN;
 require_once 'include/items.php';
 require_once 'include/security.php';
 
-function videos_init(App $a) {
+function videos_init(App $a)
-
+{
-	if($a->argc > 1)
+	if ($a->argc > 1) {
 		DFRN::autoRedir($a, $a->argv[1]);
+	}
 
 	if ((Config::get('system', 'block_public')) && (!local_user()) && (!remote_user())) {
 		return;
@@ -37,8 +38,9 @@ function videos_init(App $a) {
 			DBA::escape($nick)
 		);
 
-		if(! count($user))
+		if (!DBA::isResult($user)) {
 			return;
+		}
 
 		$a->data['user'] = $user[0];
 		$a->profile_uid = $user[0]['uid'];
@@ -57,7 +59,7 @@ function videos_init(App $a) {
 			'$pdesc' => defaults($profile, 'pdesc', ''),
 		]);
 
-
+		/// @TODO Old-lost code?
 		/*$sql_extra = permissions_sql($a->data['user']['uid']);
 
 		$albums = q("SELECT distinct(`album`) AS `album` FROM `photo` WHERE `uid` = %d $sql_extra order by created desc",
@@ -92,10 +94,12 @@ function videos_init(App $a) {
 			$o .= '</div>';
 		}*/
 
-		if(! x($a->page,'aside'))
+		// If not there, create 'aside' empty
+		if (!isset($a->page['aside'])) {
 			$a->page['aside'] = '';
-		$a->page['aside'] .= $vcard_widget;
+		}
 
+		$a->page['aside'] .= $vcard_widget;
 
 		$tpl = get_markup_template("videos_head.tpl");
 		$a->page['htmlhead'] .= replace_macros($tpl,[
@@ -112,25 +116,23 @@ function videos_init(App $a) {
 	return;
 }
 
-
+function videos_post(App $a)
-
+{
-function videos_post(App $a) {
-
 	$owner_uid = $a->data['user']['uid'];
 
 	if (local_user() != $owner_uid) {
 		goaway(System::baseUrl() . '/videos/' . $a->data['user']['nickname']);
 	}
 
-	if (($a->argc == 2) && x($_POST,'delete') && x($_POST, 'id')) {
+	if (($a->argc == 2) && !empty($_POST['delete']) && !empty($_POST['id'])) {
-
 		// Check if we should do HTML-based delete confirmation
-		if (!x($_REQUEST,'confirm')) {
+		if (empty($_REQUEST['confirm'])) {
-			if (x($_REQUEST,'canceled')) {
+			if (!empty($_REQUEST['canceled'])) {
 				goaway(System::baseUrl() . '/videos/' . $a->data['user']['nickname']);
 			}
 
 			$drop_url = $a->query_string;
+
 			$a->page['content'] = replace_macros(get_markup_template('confirm.tpl'), [
 				'$method' => 'post',
 				'$message' => L10n::t('Do you really want to delete this video?'),
@@ -144,7 +146,9 @@ function videos_post(App $a) {
 				'$cancel' => L10n::t('Cancel'),
 
 			]);
+
 			$a->error = 1; // Set $a->error so the other module functions don't execute
+
 			return;
 		}
 
@@ -160,6 +164,7 @@ function videos_post(App $a) {
 				intval(local_user()),
 				DBA::escape($video_id)
 			);
+
 			$i = q("SELECT `id` FROM `item` WHERE `attach` like '%%attach/%s%%' AND `uid` = %d LIMIT 1",
 				DBA::escape($video_id),
 				intval(local_user())
@@ -175,13 +180,10 @@ function videos_post(App $a) {
 	}
 
 	goaway(System::baseUrl() . '/videos/' . $a->data['user']['nickname']);
-
 }
 
-
+function videos_content(App $a)
-
+{
-function videos_content(App $a) {
-
 	// URLs (most aren't currently implemented):
 	// videos/name
 	// videos/name/upload
@@ -197,10 +199,10 @@ function videos_content(App $a) {
 		return;
 	}
 
-	require_once('include/security.php');
+	require_once 'include/security.php';
-	require_once('include/conversation.php');
+	require_once 'include/conversation.php';
 
-	if(! x($a->data,'user')) {
+	if (empty($a->data['user'])) {
 		notice(L10n::t('No videos selected') . EOL );
 		return;
 	}
@@ -212,25 +214,24 @@ function videos_content(App $a) {
 	//
 	// Parse arguments
 	//
-
 	if ($a->argc > 3) {
 		$datatype = $a->argv[2];
 		$datum = $a->argv[3];
-	}
+	} elseif(($a->argc > 2) && ($a->argv[2] === 'upload')) {
-	elseif(($a->argc > 2) && ($a->argv[2] === 'upload'))
 		$datatype = 'upload';
-	else
+	} else {
 		$datatype = 'summary';
+	}
 
-	if($a->argc > 4)
+	if ($a->argc > 4) {
 		$cmd = $a->argv[4];
-	else
+	} else {
 		$cmd = 'view';
+	}
 
 	//
 	// Setup permissions structures
 	//
-
 	$can_post       = false;
 	$visitor        = 0;
 	$contact        = null;
@@ -241,11 +242,10 @@ function videos_content(App $a) {
 
 	$community_page = (($a->data['user']['page-flags'] == PAGE_COMMUNITY) ? true : false);
 
-	if((local_user()) && (local_user() == $owner_uid))
+	if ((local_user()) && (local_user() == $owner_uid)) {
 		$can_post = true;
-	else {
+	} elseif ($community_page && remote_user()) {
-		if($community_page && remote_user()) {
+		if (!empty($_SESSION['remote'])) {
-			if(is_array($_SESSION['remote'])) {
 			foreach ($_SESSION['remote'] as $v) {
 				if ($v['uid'] == $owner_uid) {
 					$contact_id = $v['cid'];
@@ -253,12 +253,13 @@ function videos_content(App $a) {
 				}
 			}
 		}
-			if($contact_id) {
 
+		if ($contact_id > 0) {
 			$r = q("SELECT `uid` FROM `contact` WHERE `blocked` = 0 AND `pending` = 0 AND `id` = %d AND `uid` = %d LIMIT 1",
 				intval($contact_id),
 				intval($owner_uid)
 			);
+
 			if (DBA::isResult($r)) {
 				$can_post = true;
 				$contact = $r[0];
@@ -267,14 +268,13 @@ function videos_content(App $a) {
 			}
 		}
 	}
-	}
 
 	$groups = [];
 
 	// perhaps they're visiting - but not a community page, so they wouldn't have write access
 	if (remote_user() && (!$visitor)) {
 		$contact_id = 0;
-		if(is_array($_SESSION['remote'])) {
+		if (!empty($_SESSION['remote'])) {
 			foreach($_SESSION['remote'] as $v) {
 				if($v['uid'] == $owner_uid) {
 					$contact_id = $v['cid'];
@@ -282,12 +282,14 @@ function videos_content(App $a) {
 				}
 			}
 		}
-		if($contact_id) {
+
+		if ($contact_id > 0) {
 			$groups = Group::getIdsByContactId($contact_id);
 			$r = q("SELECT * FROM `contact` WHERE `blocked` = 0 AND `pending` = 0 AND `id` = %d AND `uid` = %d LIMIT 1",
 				intval($contact_id),
 				intval($owner_uid)
 			);
+
 			if (DBA::isResult($r)) {
 				$contact = $r[0];
 				$remote_contact = true;
@@ -295,12 +297,10 @@ function videos_content(App $a) {
 		}
 	}
 
-	if(! $remote_contact) {
+	if (!$remote_contact && local_user()) {
-		if(local_user()) {
 		$contact_id = $_SESSION['cid'];
 		$contact = $a->contact;
 	}
-	}
 
 	if ($a->data['user']['hidewall'] && (local_user() != $owner_uid) && (!$remote_contact)) {
 		notice(L10n::t('Access to this item is restricted.') . EOL);
@@ -318,8 +318,6 @@ function videos_content(App $a) {
 	//
 	// dispatch request
 	//
-
-
 	if ($datatype === 'upload') {
 		return; // no uploading for now
 
@@ -327,7 +325,6 @@ function videos_content(App $a) {
 	}
 
 	if ($datatype === 'album') {
-
 		return; // no albums for now
 
 		// DELETED -- look at mod/photos.php if you want to implement
@@ -335,7 +332,6 @@ function videos_content(App $a) {
 
 
 	if ($datatype === 'video') {
-
 		return; // no single video view for now
 
 		// DELETED -- look at mod/photos.php if you want to implement
@@ -348,6 +344,7 @@ function videos_content(App $a) {
 		$sql_extra GROUP BY hash",
 		intval($a->data['user']['uid'])
 	);
+
 	if (DBA::isResult($r)) {
 		$a->set_pager_total(count($r));
 		$a->set_pager_itemspage(20);
@@ -363,9 +360,8 @@ function videos_content(App $a) {
 		intval($a->pager['itemspage'])
 	);
 
-
-
 	$videos = [];
+
 	if (DBA::isResult($r)) {
 		foreach ($r as $rr) {
 			$alt_e = $rr['filename'];
@@ -383,7 +379,6 @@ function videos_content(App $a) {
 					'name'  => $name_e,
 					'alt'   => L10n::t('View Album'),
 				],
-
 			];
 		}
 	}
@@ -394,10 +389,10 @@ function videos_content(App $a) {
 		'$can_post'   => $can_post,
 		'$upload'     => [L10n::t('Upload New Videos'), System::baseUrl() . '/videos/' . $a->data['user']['nickname'] . '/upload'],
 		'$videos'     => $videos,
-		'$delete_url' => (($can_post)?System::baseUrl().'/videos/'.$a->data['user']['nickname']:False)
+		'$delete_url' => (($can_post) ? System::baseUrl() . '/videos/' . $a->data['user']['nickname'] : false)
 	]);
 
-
 	$o .= paginate($a);
+
 	return $o;
 }