much better solution to privacy on ajax wall photos
This commit is contained in:
parent
f057cc3a70
commit
15c81716e8
2 changed files with 29 additions and 30 deletions
17
mod/item.php
17
mod/item.php
|
@ -131,19 +131,18 @@ function item_post(&$a) {
|
|||
|
||||
/**
|
||||
*
|
||||
* If a photo was uploaded into the message using the ajax uploader,
|
||||
* it can be seen by anybody. Set the permissions to match the message.
|
||||
* Ideally this should be done when the photo was uploaded, but the permissions
|
||||
* may not have been set at that time, and passing the permission arrays via
|
||||
* javascript to the ajax upload is going to be a challenge.
|
||||
* This is a compromise. Granted there is a window of time when the photo
|
||||
* is public. You are welcome to suggest other ways to fix this.
|
||||
* When a photo was uploaded into the message using the (profile wall) ajax
|
||||
* uploader, The permissions are initially set to disallow anybody but the
|
||||
* owner from seeing it. This is because the permissions may not yet have been
|
||||
* set for the post. If it's private, the photo permissions should be set
|
||||
* appropriately. But we didn't know the final permissions on the post until
|
||||
* now. So now we'll look for links of uploaded messages that are in the
|
||||
* post and set them to the same permissions as the post itself.
|
||||
*
|
||||
*/
|
||||
|
||||
$match = null;
|
||||
|
||||
if($private) {
|
||||
if(preg_match_all("/\[img\](.+?)\[\/img\]/",$body,$match)) {
|
||||
$images = $match[1];
|
||||
if(count($images)) {
|
||||
|
@ -165,7 +164,7 @@ function item_post(&$a) {
|
|||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
|
||||
/**
|
||||
|
|
|
@ -76,7 +76,7 @@ function wall_upload_post(&$a) {
|
|||
|
||||
$smallest = 0;
|
||||
|
||||
$r = $ph->store($page_owner_uid, $visitor, $hash, $filename, t('Wall Photos'), 0 );
|
||||
$r = $ph->store($page_owner_uid, $visitor, $hash, $filename, t('Wall Photos'), 0, 0, '<0>');
|
||||
|
||||
if(! $r) {
|
||||
echo ( t('Image upload failed.') . EOL);
|
||||
|
@ -85,14 +85,14 @@ function wall_upload_post(&$a) {
|
|||
|
||||
if($width > 640 || $height > 640) {
|
||||
$ph->scaleImage(640);
|
||||
$r = $ph->store($page_owner_uid, $visitor, $hash, $filename, t('Wall Photos'), 1 );
|
||||
$r = $ph->store($page_owner_uid, $visitor, $hash, $filename, t('Wall Photos'), 1, 0, '<0>');
|
||||
if($r)
|
||||
$smallest = 1;
|
||||
}
|
||||
|
||||
if($width > 320 || $height > 320) {
|
||||
$ph->scaleImage(320);
|
||||
$r = $ph->store($page_owner_uid, $visitor, $hash, $filename, t('Wall Photos'), 2 );
|
||||
$r = $ph->store($page_owner_uid, $visitor, $hash, $filename, t('Wall Photos'), 2, 0, '<0>' );
|
||||
if($r)
|
||||
$smallest = 2;
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue