mexon/friendica
1
0
Fork 0
Code Pull requests Activity

Fixes:

Browse source 
- used empty() instead of x()
- removed superflous count()

Signed-off-by: Roland Häder <roland@mxchange.org>
This commit is contained in:
Roland Häder 2018-05-14 22:25:39 +02:00
parent d0c7ba5257
commit f34d72c143
No known key found for this signature in database
GPG key ID: B72F8185C6C7BD78
2 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
include/conversation.php
View file

@ -391,8 +391,8 @@ function visible_activity($item) {
} }
} }
/// @TODO such things are unneccessary code-complexibilities and should be avoided // @TODO below if() block can be rewritten to a single line: $isVisible = allConditionsHere;
if (activity_match($item['verb'], ACTIVITY_FOLLOW) && $item['object-type'] === ACTIVITY_OBJ_NOTE && !(x($item, 'self') && ($item['uid'] == local_user()))) { if (activity_match($item['verb'], ACTIVITY_FOLLOW) && $item['object-type'] === ACTIVITY_OBJ_NOTE && empty($item['self']) && $item['uid'] == local_user()) {
return false; return false;
} }

3
include/security.php
View file

@ -254,6 +254,7 @@ function can_write_wall($owner)
return false; return false;
} }
/// @TODO $groups should be array
function permissions_sql($owner_id, $remote_verified = false, $groups = null) function permissions_sql($owner_id, $remote_verified = false, $groups = null)
{ {
$local_user = local_user(); $local_user = local_user();
@ -305,7 +306,7 @@ function permissions_sql($owner_id, $remote_verified = false, $groups = null)
if ($remote_verified) { if ($remote_verified) {
$gs = '<<>>'; // should be impossible to match $gs = '<<>>'; // should be impossible to match
if (is_array($groups) && count($groups)) { if (is_array($groups)) {
foreach ($groups as $g) { foreach ($groups as $g) {
$gs .= '|<' . intval($g) . '>'; $gs .= '|<' . intval($g) . '>';
} }