From f989c331a3fcf7fa34f7111f4ffa2939e04b4600 Mon Sep 17 00:00:00 2001
From: Hypolite Petovan <hypolite@mrpetovan.com>
Date: Tue, 3 Dec 2019 06:59:51 -0500
Subject: [PATCH] Fix warnings about unexpected parameter array values type in
 ACL::getFullSelectorHTML

- Addresses https://github.com/friendica/friendica/issues/7676#issuecomment-560503744
---
 mod/events.php   |  4 +---
 mod/photos.php   |  2 +-
 src/Core/ACL.php | 19 ++++++++++---------
 3 files changed, 12 insertions(+), 13 deletions(-)

diff --git a/mod/events.php b/mod/events.php
index a642f16651..5a5a22d318 100644
--- a/mod/events.php
+++ b/mod/events.php
@@ -490,10 +490,8 @@ function events_content(App $a)
 		$fhour   = !empty($orig_event) ? DateTimeFormat::convert($fdt, $tz, 'UTC', 'H') : '00';
 		$fminute = !empty($orig_event) ? DateTimeFormat::convert($fdt, $tz, 'UTC', 'i') : '00';
 
-		$perms = ACL::getDefaultUserPermissions($orig_event);
-
 		if (!$cid && in_array($mode, ['new', 'copy'])) {
-			$acl = ACL::getFullSelectorHTML($a->page, $a->user, false, $perms);
+			$acl = ACL::getFullSelectorHTML($a->page, $a->user, false, ACL::getDefaultUserPermissions($orig_event));
 		} else {
 			$acl = '';
 		}
diff --git a/mod/photos.php b/mod/photos.php
index 684e525d43..2f3e7d3117 100644
--- a/mod/photos.php
+++ b/mod/photos.php
@@ -1332,7 +1332,7 @@ function photos_content(App $a)
 
 			$album_e = $ph[0]['album'];
 			$caption_e = $ph[0]['desc'];
-			$aclselect_e = ACL::getFullSelectorHTML($a->page, $a->user, false, $ph[0]);
+			$aclselect_e = ACL::getFullSelectorHTML($a->page, $a->user, false, ACL::getDefaultUserPermissions($ph[0]));
 
 			$edit = Renderer::replaceMacros($edit_tpl, [
 				'$id' => $ph[0]['id'],
diff --git a/src/Core/ACL.php b/src/Core/ACL.php
index 53e6c19691..e6dd1a8f32 100644
--- a/src/Core/ACL.php
+++ b/src/Core/ACL.php
@@ -333,15 +333,16 @@ class ACL extends BaseObject
 		// Defaults user permissions
 		if (empty($default_permissions)) {
 			$default_permissions = self::getDefaultUserPermissions($user);
-		} else {
-			$default_permissions = [
-				'allow_cid' => $default_permissions['allow_cid'] ?? [],
-				'allow_gid' => $default_permissions['allow_gid'] ?? [],
-				'deny_cid'  => $default_permissions['deny_cid']  ?? [],
-				'deny_gid'  => $default_permissions['deny_gid']  ?? [],
-			];
 		}
 
+		$default_permissions = [
+			'allow_cid' => $default_permissions['allow_cid'] ?? [],
+			'allow_gid' => $default_permissions['allow_gid'] ?? [],
+			'deny_cid'  => $default_permissions['deny_cid']  ?? [],
+			'deny_gid'  => $default_permissions['deny_gid']  ?? [],
+			'hidewall'  => $default_permissions['hidewall']  ?? false,
+		];
+
 		if (count($default_permissions['allow_cid'])
 			+ count($default_permissions['allow_gid'])
 			+ count($default_permissions['deny_cid'])
@@ -366,7 +367,7 @@ class ACL extends BaseObject
 				}
 			}
 
-			if (empty($default_permissions['hidewall'])) {
+			if ($default_permissions['hidewall']) {
 				if ($mail_enabled) {
 					$jotnets_fields[] = [
 						'type' => 'checkbox',
@@ -410,7 +411,7 @@ class ACL extends BaseObject
 			'$group_deny'     => implode(',', $default_permissions['deny_gid']),
 			'$for_federation' => $for_federation,
 			'$jotnets_fields' => $jotnets_fields,
-			'$user_hidewall'  => $default_permissions['hidewall'] ?? false,
+			'$user_hidewall'  => $default_permissions['hidewall'],
 		]);
 
 		return $o;