Merge branch 'master', remote-tracking branch 'remotes/upstream/master'
* remotes/upstream/master: fix photos after permissions_sql change [privacy] rework latest fix remove stray debugging * master:
This commit is contained in:
commit
edfa7dbac2
3 changed files with 80 additions and 9 deletions
|
@ -213,3 +213,77 @@ function permissions_sql($owner_id,$remote_verified = false,$groups = null) {
|
||||||
}
|
}
|
||||||
return $sql;
|
return $sql;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
function item_permissions_sql($owner_id,$remote_verified = false,$groups = null) {
|
||||||
|
|
||||||
|
$local_user = local_user();
|
||||||
|
$remote_user = remote_user();
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Construct permissions
|
||||||
|
*
|
||||||
|
* default permissions - anonymous user
|
||||||
|
*/
|
||||||
|
|
||||||
|
$sql = " AND allow_cid = ''
|
||||||
|
AND allow_gid = ''
|
||||||
|
AND deny_cid = ''
|
||||||
|
AND deny_gid = ''
|
||||||
|
AND private = 0
|
||||||
|
";
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Profile owner - everything is visible
|
||||||
|
*/
|
||||||
|
|
||||||
|
if(($local_user) && ($local_user == $owner_id)) {
|
||||||
|
$sql = '';
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Authenticated visitor. Unless pre-verified,
|
||||||
|
* check that the contact belongs to this $owner_id
|
||||||
|
* and load the groups the visitor belongs to.
|
||||||
|
* If pre-verified, the caller is expected to have already
|
||||||
|
* done this and passed the groups into this function.
|
||||||
|
*/
|
||||||
|
|
||||||
|
elseif($remote_user) {
|
||||||
|
|
||||||
|
if(! $remote_verified) {
|
||||||
|
$r = q("SELECT id FROM contact WHERE id = %d AND uid = %d AND blocked = 0 LIMIT 1",
|
||||||
|
intval($remote_user),
|
||||||
|
intval($owner_id)
|
||||||
|
);
|
||||||
|
if(count($r)) {
|
||||||
|
$remote_verified = true;
|
||||||
|
$groups = init_groups_visitor($remote_user);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if($remote_verified) {
|
||||||
|
|
||||||
|
$gs = '<<>>'; // should be impossible to match
|
||||||
|
|
||||||
|
if(is_array($groups) && count($groups)) {
|
||||||
|
foreach($groups as $g)
|
||||||
|
$gs .= '|<' . intval($g) . '>';
|
||||||
|
}
|
||||||
|
|
||||||
|
$sql = sprintf(
|
||||||
|
" AND ( private = 0 OR (( allow_cid = '' OR allow_cid REGEXP '<%d>' )
|
||||||
|
AND ( deny_cid = '' OR NOT deny_cid REGEXP '<%d>' )
|
||||||
|
AND ( allow_gid = '' OR allow_gid REGEXP '%s' )
|
||||||
|
AND ( deny_gid = '' OR NOT deny_gid REGEXP '%s')))
|
||||||
|
",
|
||||||
|
intval($remote_user),
|
||||||
|
intval($remote_user),
|
||||||
|
dbesc($gs),
|
||||||
|
dbesc($gs)
|
||||||
|
);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return $sql;
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
|
|
@ -34,7 +34,7 @@ function display_content(&$a) {
|
||||||
|
|
||||||
$contact = null;
|
$contact = null;
|
||||||
$remote_contact = false;
|
$remote_contact = false;
|
||||||
dbg(1);
|
|
||||||
if(remote_user()) {
|
if(remote_user()) {
|
||||||
$contact_id = $_SESSION['visitor_id'];
|
$contact_id = $_SESSION['visitor_id'];
|
||||||
$groups = init_groups_visitor($contact_id);
|
$groups = init_groups_visitor($contact_id);
|
||||||
|
@ -85,10 +85,7 @@ dbg(1);
|
||||||
$o .= status_editor($a,$x,0,true);
|
$o .= status_editor($a,$x,0,true);
|
||||||
|
|
||||||
|
|
||||||
$sql_extra = permissions_sql($a->profile['uid'],$remote_contact,$groups);
|
$sql_extra = item_permissions_sql($a->profile['uid'],$remote_contact,$groups);
|
||||||
|
|
||||||
if(! local_user() && ! remote_user())
|
|
||||||
$sql_extra .= " and `item`.`private` = 0 ";
|
|
||||||
|
|
||||||
$r = q("SELECT `item`.*, `item`.`id` AS `item_id`,
|
$r = q("SELECT `item`.*, `item`.`id` AS `item_id`,
|
||||||
`contact`.`name`, `contact`.`photo`, `contact`.`url`, `contact`.`rel`,
|
`contact`.`name`, `contact`.`photo`, `contact`.`url`, `contact`.`rel`,
|
||||||
|
@ -138,7 +135,7 @@ dbg(1);
|
||||||
}
|
}
|
||||||
|
|
||||||
}
|
}
|
||||||
dbg(0);
|
|
||||||
return $o;
|
return $o;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -159,7 +159,7 @@ function profile_content(&$a, $update = 0) {
|
||||||
* Get permissions SQL - if $remote_contact is true, our remote user has been pre-verified and we already have fetched his/her groups
|
* Get permissions SQL - if $remote_contact is true, our remote user has been pre-verified and we already have fetched his/her groups
|
||||||
*/
|
*/
|
||||||
|
|
||||||
$sql_extra = permissions_sql($a->profile['profile_uid'],$remote_contact,$groups);
|
$sql_extra = item_permissions_sql($a->profile['profile_uid'],$remote_contact,$groups);
|
||||||
|
|
||||||
|
|
||||||
if($update) {
|
if($update) {
|
||||||
|
|
Loading…
Reference in a new issue