X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=mod%2Fphotos.php;h=d4835ba5ea65edb515a35087c08483f8ac3419e9;hb=3ad7c395fb9a33319531e04673563e7c9983d8f0;hp=f3eccd12799e6bc9f51e7eaf26039a71e28d2d7e;hpb=b94cb8d234d70bc534ad1cef767b9f62d10eef26;p=friendica.git

diff --git a/mod/photos.php b/mod/photos.php
index f3eccd1279..d4835ba5ea 100644
--- a/mod/photos.php
+++ b/mod/photos.php
@@ -1,8 +1,7 @@
 <?php
-
-require_once('Photo.php');
+require_once('include/Photo.php');
 require_once('include/items.php');
-require_once('view/acl_selectors.php');
+require_once('include/acl_selectors.php');
 require_once('include/bbcode.php');
 
 function photos_init(&$a) {
@@ -11,7 +10,7 @@ function photos_init(&$a) {
 
 	if($a->argc > 1) {
 		$nick = $a->argv[1];
-		$r = q("SELECT * FROM `user` WHERE `nickname` = '%s' LIMIT 1",
+		$r = q("SELECT * FROM `user` WHERE `nickname` = '%s' AND `blocked` = 0 LIMIT 1",
 			dbesc($nick)
 		);
 
@@ -43,36 +42,66 @@ function photos_init(&$a) {
 			$a->page['aside'] = '';
 		$a->page['aside'] .= $o;
 	}
+
 	return;
 }
 
 
 
-
 function photos_post(&$a) {
 
+logger('mod/photos.php: photos_post(): begin' , 'LOGGER_DEBUG');
+
+foreach($_REQUEST AS $key => $val) {
+	logger('mod/photos.php: photos_post(): $_REQUEST key: ' . $key . ' val: ' . $val , 'LOGGER_DEBUG');
+}
+
+foreach($_FILES AS $key => $val) {
+	logger('mod/photos.php: photos_post(): $_FILES key: ' . $key . ' val: ' . $val , 'LOGGER_DEBUG');
+}
+
+	$can_post  = false;
+	$visitor   = 0;
+
+	$page_owner_uid = $a->data['user']['uid'];
+	$community_page = (($a->data['user']['page-flags'] == PAGE_COMMUNITY) ? true : false);
+
+	if((local_user()) && (local_user() == $page_owner_uid))
+		$can_post = true;
+	else {
+		if($community_page && remote_user()) {
+			$r = q("SELECT `uid` FROM `contact` WHERE `blocked` = 0 AND `pending` = 0 AND `id` = %d AND `uid` = %d LIMIT 1",
+				intval(remote_user()),
+				intval($page_owner_uid)
+			);
+			if(count($r)) {
+				$can_post = true;
+				$visitor = remote_user();
+			}
+		}
+	}
 
-	if(! local_user()) {
+	if(! $can_post) {
 		notice( t('Permission denied.') . EOL );
 		killme();
 	}
 
 	$r = q("SELECT `contact`.*, `user`.`nickname` FROM `contact` LEFT JOIN `user` ON `user`.`uid` = `contact`.`uid` 
 		WHERE `user`.`uid` = %d AND `self` = 1 LIMIT 1",
-		intval(local_user())
+		intval($page_owner_uid)
 	);
 
 	if(! count($r)) {
 		notice( t('Contact information unavailable') . EOL);
-		logger('photos_post: unable to locate contact record for logged in user. uid=' . local_user());
+		logger('photos_post: unable to locate contact record for page owner. uid=' . $page_owner_uid);
 		killme();
 	}
 
-	$contact_record = $r[0];	
+	$owner_record = $r[0];	
 
 
-	if(($a->argc > 2) && ($a->argv[1] === 'album')) {
-		$album = hex2bin($a->argv[2]);
+	if(($a->argc > 3) && ($a->argv[2] === 'album')) {
+		$album = hex2bin($a->argv[3]);
 
 		if($album == t('Profile Photos') || $album == t('Contact Photos')) {
 			goaway($a->get_baseurl() . '/' . $_SESSION['photo_return']);
@@ -81,7 +110,7 @@ function photos_post(&$a) {
 
 		$r = q("SELECT count(*) FROM `photo` WHERE `album` = '%s' AND `uid` = %d",
 			dbesc($album),
-			intval(local_user())
+			intval($page_owner_uid)
 		);
 		if(! count($r)) {
 			notice( t('Album not found.') . EOL);
@@ -94,20 +123,33 @@ function photos_post(&$a) {
 			q("UPDATE `photo` SET `album` = '%s' WHERE `album` = '%s' AND `uid` = %d",
 				dbesc($newalbum),
 				dbesc($album),
-				intval(local_user())
+				intval($page_owner_uid)
 			);
 			$newurl = str_replace(bin2hex($album),bin2hex($newalbum),$_SESSION['photo_return']);
 			goaway($a->get_baseurl() . '/' . $newurl);
 			return; // NOTREACHED
 		}
 
+
 		if($_POST['dropalbum'] == t('Delete Album')) {
 
 			$res = array();
-			$r = q("SELECT distinct(`resource-id`) as `rid` FROM `photo` WHERE `uid` = %d AND `album` = '%s'",
-				intval(local_user()),
-				dbesc($album)
-			);
+
+			// get the list of photos we are about to delete
+
+			if($visitor) {
+				$r = q("SELECT distinct(`resource-id`) as `rid` FROM `photo` WHERE `contact-id` = %d AND `uid` = %d AND `album` = '%s'",
+					intval($visitor),
+					intval($page_owner_uid),
+					dbesc($album)
+				);
+			}
+			else {
+				$r = q("SELECT distinct(`resource-id`) as `rid` FROM `photo` WHERE `uid` = %d AND `album` = '%s'",
+					intval(local_user()),
+					dbesc($album)
+				);
+			}
 			if(count($r)) {
 				foreach($r as $rr) {
 					$res[] = "'" . dbesc($rr['rid']) . "'" ;
@@ -117,20 +159,26 @@ function photos_post(&$a) {
 				goaway($a->get_baseurl() . '/' . $_SESSION['photo_return']);
 				return; // NOTREACHED
 			}
+
 			$str_res = implode(',', $res);
 
+			// remove the associated photos
+
 			q("DELETE FROM `photo` WHERE `resource-id` IN ( $str_res ) AND `uid` = %d",
-				intval(local_user())
+				intval($page_owner_uid)
 			);
+
+			// find and delete the corresponding item with all the comments and likes/dislikes
+
 			$r = q("SELECT `parent-uri` FROM `item` WHERE `resource-id` IN ( $str_res ) AND `uid` = %d",
-				intval(local_user())
+				intval($page_owner_uid)
 			);
 			if(count($r)) {
 				foreach($r as $rr) {
 					q("UPDATE `item` SET `deleted` = 1, `changed` = '%s' WHERE `parent-uri` = '%s' AND `uid` = %d",
 						dbesc(datetime_convert()),
 						dbesc($rr['parent-uri']),
-						intval(local_user())
+						intval($page_owner_uid)
 					);
 
 					$drop_id = intval($rr['id']);
@@ -139,9 +187,8 @@ function photos_post(&$a) {
 					// send the notification upstream/downstream as the case may be
 
 					if($rr['visible'])
-						proc_close(proc_open("\"$php_path\" \"include/notifier.php\" \"drop\" \"$drop_id\" & ",
-							array(),$foo));
-
+						//proc_close(proc_open("\"$php_path\" \"include/notifier.php\" \"drop\" \"$drop_id\" & ",array(),$foo));
+						proc_run($php_path,"include/notifier.php","drop","$drop_id");
 				}
 			}
 		}
@@ -149,26 +196,38 @@ function photos_post(&$a) {
 		return; // NOTREACHED
 	}
 
-	if(($a->argc > 1) && (x($_POST,'delete')) && ($_POST['delete'] == t('Delete Photo'))) {
-		$r = q("SELECT `id` FROM `photo` WHERE `uid` = %d AND `resource-id` = '%s' LIMIT 1",
-			intval(local_user()),
-			dbesc($a->argv[1])
-		);
+	if(($a->argc > 2) && (x($_POST,'delete')) && ($_POST['delete'] == t('Delete Photo'))) {
+
+		// same as above but remove single photo
+
+		if($visitor) {
+			$r = q("SELECT `id`, `resource-id` FROM `photo` WHERE `contact-id` = %d AND `uid` = %d AND `resource-id` = '%s' LIMIT 1",
+				intval($visitor),
+				intval($page_owner_uid),
+				dbesc($a->argv[2])
+			);
+		}
+		else {
+			$r = q("SELECT `id`, `resource-id` FROM `photo` WHERE `uid` = %d AND `resource-id` = '%s' LIMIT 1",
+				intval(local_user()),
+				dbesc($a->argv[2])
+			);
+		}
 		if(count($r)) {
 			q("DELETE FROM `photo` WHERE `uid` = %d AND `resource-id` = '%s'",
-				intval(local_user()),
+				intval($page_owner_uid),
 				dbesc($r[0]['resource-id'])
 			);
 			$i = q("SELECT * FROM `item` WHERE `resource-id` = '%s' AND `uid` = %d LIMIT 1",
 				dbesc($r[0]['resource-id']),
-				intval(local_user())
+				intval($page_owner_uid)
 			);
 			if(count($i)) {
 				q("UPDATE `item` SET `deleted` = 1, `edited` = '%s', `changed` = '%s' WHERE `parent-uri` = '%s' AND `uid` = %d",
 					dbesc(datetime_convert()),
 					dbesc(datetime_convert()),
 					dbesc($i[0]['uri']),
-					intval(local_user())
+					intval($page_owner_uid)
 				);
 
 				$url = $a->get_baseurl();
@@ -176,8 +235,8 @@ function photos_post(&$a) {
 				$php_path = ((strlen($a->config['php_path'])) ? $a->config['php_path'] : 'php');
 
 				if($i[0]['visible'])
-					proc_close(proc_open("\"$php_path\" \"include/notifier.php\" \"drop\" \"$drop_id\" & ",
-						array(),$foo));
+					//proc_close(proc_open("\"$php_path\" \"include/notifier.php\" \"drop\" \"$drop_id\" & ", 	array(),$foo));
+					proc_run($php_path,"include/notifier.php","drop","$drop_id");
 			}
 		}
 
@@ -185,53 +244,67 @@ function photos_post(&$a) {
 		return; // NOTREACHED
 	}
 
-	if(($a->argc > 1) && ((x($_POST,'desc') !== false) || (x($_POST,'newtag') !== false))) {
+	if(($a->argc > 2) && ((x($_POST,'desc') !== false) || (x($_POST,'newtag') !== false)) || (x($_POST,'albname') !== false)) {
+
+		$desc        = ((x($_POST,'desc'))    ? notags(trim($_POST['desc']))    : '');
+		$rawtags     = ((x($_POST,'newtag'))  ? notags(trim($_POST['newtag']))  : '');
+		$item_id     = ((x($_POST,'item_id')) ? intval($_POST['item_id'])       : 0);
+		$albname     = ((x($_POST,'albname')) ? notags(trim($_POST['albname'])) : '');
+		$resource_id = $a->argv[2];
 
-		$desc        = ((x($_POST,'desc'))    ? notags(trim($_POST['desc']))   : '');
-		$rawtags     = ((x($_POST,'newtag'))  ? notags(trim($_POST['newtag'])) : '');
-		$item_id     = ((x($_POST,'item_id')) ? intval($_POST['item_id'])      : 0);
-		$resource_id = $a->argv[1];
+		if(! strlen($albname))
+			$albname = datetime_convert('UTC',date_default_timezone_get(),'now', 'Y');
+		
 
 		$p = q("SELECT * FROM `photo` WHERE `resource-id` = '%s' AND `uid` = %d ORDER BY `scale` DESC",
 			dbesc($resource_id),
-			intval(local_user())
+			intval($page_owner_uid)
 		);
-		if((count($p)) && ($p[0]['desc'] !== $desc)) {
-			$r = q("UPDATE `photo` SET `desc` = '%s' WHERE `resource-id` = '%s' AND `uid` = %d",
+		if((count($p)) && (($p[0]['desc'] !== $desc) || ($p[0]['album'] !== $albname))) {
+			$r = q("UPDATE `photo` SET `desc` = '%s', `album` = '%s' WHERE `resource-id` = '%s' AND `uid` = %d",
 				dbesc($desc),
+				dbesc($albname),
 				dbesc($resource_id),
-				intval(local_user())
+				intval($page_owner_uid)
 			);
 		}
+
+		/* Don't make the item visible if the only change was the album name */
+
+		$visibility = 0;
+		if($p[0]['desc'] !== $desc || strlen($rawtags))
+			$visibility = 1;
+		
 		if(! $item_id) {
 
 			// Create item container
 
 			$title = '';
 			$basename = basename($filename);
-			$uri = item_new_uri($a->get_hostname(),local_user());
+			$uri = item_new_uri($a->get_hostname(),$page_owner_uid);
 
 			$arr = array();
 
-			$arr['uid']           = local_user();
+			$arr['uid']           = $page_owner_uid;
 			$arr['uri']           = $uri;
 			$arr['parent-uri']    = $uri; 
 			$arr['type']          = 'photo';
 			$arr['wall']          = 1;
 			$arr['resource-id']   = $p[0]['resource-id'];
-			$arr['contact-id']    = $contact_record['id'];
-			$arr['owner-name']    = $contact_record['name'];
-			$arr['owner-link']    = $contact_record['url'];
-			$arr['owner-avatar']  = $contact_record['thumb'];
-			$arr['author-name']   = $contact_record['name'];
-			$arr['author-link']   = $contact_record['url'];
-			$arr['author-avatar'] = $contact_record['thumb'];
+			$arr['contact-id']    = $owner_record['id'];
+			$arr['owner-name']    = $owner_record['name'];
+			$arr['owner-link']    = $owner_record['url'];
+			$arr['owner-avatar']  = $owner_record['thumb'];
+			$arr['author-name']   = $owner_record['name'];
+			$arr['author-link']   = $owner_record['url'];
+			$arr['author-avatar'] = $owner_record['thumb'];
 			$arr['title']         = $title;
 			$arr['allow_cid']     = $p[0]['allow_cid'];
 			$arr['allow_gid']     = $p[0]['allow_gid'];
 			$arr['deny_cid']      = $p[0]['deny_cid'];
 			$arr['deny_gid']      = $p[0]['deny_gid'];
 			$arr['last-child']    = 1;
+			$arr['visible']       = $visibility;
 			$arr['body']          = '[url=' . $a->get_baseurl() . '/photos/' . $a->data['user']['nickname'] . '/image/' . $p[0]['resource-id'] . ']' 
 						. '[img]' . $a->get_baseurl() . '/photo/' . $p[0]['resource-id'] . '-' . $p[0]['scale'] . '.jpg' . '[/img]' 
 						. '[/url]';
@@ -243,7 +316,7 @@ function photos_post(&$a) {
 		if($item_id) {
 			$r = q("SELECT * FROM `item` WHERE `id` = %d AND `uid` = %d LIMIT 1",
 				intval($item_id),
-				intval(local_user())
+				intval($page_owner_uid)
 			);
 		}
 		if(count($r)) {
@@ -292,13 +365,13 @@ function photos_post(&$a) {
 								$newname = str_replace('_',' ',$name);
 								$r = q("SELECT * FROM `contact` WHERE `name` = '%s' AND `uid` = %d LIMIT 1",
 									dbesc($newname),
-									intval(local_user())
+									intval($page_owner_uid)
 								);
 							}
 							else {
 								$r = q("SELECT * FROM `contact` WHERE `nick` = '%s' AND `uid` = %d LIMIT 1",
 									dbesc($name),
-									intval(local_user())
+									intval($page_owner_uid)
 								);
 							}
 							if(count($r)) {
@@ -340,7 +413,7 @@ function photos_post(&$a) {
 				dbesc(datetime_convert()),
 				dbesc(datetime_convert()),
 				intval($item_id),
-				intval(local_user())
+				intval($page_owner_uid)
 			);
 
 			$best = 0;
@@ -358,22 +431,22 @@ function photos_post(&$a) {
 			if(count($taginfo)) {
 				foreach($taginfo as $tagged) {
 		
-					$uri = item_new_uri($a->get_hostname(),local_user());
+					$uri = item_new_uri($a->get_hostname(),$page_owner_uid);
 
 					$arr = array();
 
-					$arr['uid']           = local_user();
+					$arr['uid']           = $page_owner_uid;
 					$arr['uri']           = $uri;
 					$arr['parent-uri']    = $uri;
 					$arr['type']          = 'activity';
 					$arr['wall']          = 1;
-					$arr['contact-id']    = $contact_record['id'];
-					$arr['owner-name']    = $contact_record['name'];
-					$arr['owner-link']    = $contact_record['url'];
-					$arr['owner-avatar']  = $contact_record['thumb'];
-					$arr['author-name']   = $contact_record['name'];
-					$arr['author-link']   = $contact_record['url'];
-					$arr['author-avatar'] = $contact_record['thumb'];
+					$arr['contact-id']    = $owner_record['id'];
+					$arr['owner-name']    = $owner_record['name'];
+					$arr['owner-link']    = $owner_record['url'];
+					$arr['owner-avatar']  = $owner_record['thumb'];
+					$arr['author-name']   = $owner_record['name'];
+					$arr['author-link']   = $owner_record['url'];
+					$arr['author-avatar'] = $owner_record['thumb'];
 					$arr['title']         = '';
 					$arr['allow_cid']     = $p[0]['allow_cid'];
 					$arr['allow_gid']     = $p[0]['allow_gid'];
@@ -387,8 +460,8 @@ function photos_post(&$a) {
 					$arr['tag']           = $tagged[4];
 					$arr['inform']        = $tagged[2];
 
-					$arr['body']          = '[url=' . $tagged[1] . ']' . $tagged[0] . '[/url]' . ' ' . t('was tagged in a') . ' ' . '[url=' . $a->get_baseurl() . '/photos/' . $contact_record['nickname'] . '/image/' . $p[0]['resource-id'] . ']' . t('photo') . '[/url]' . ' ' . t('by') . ' ' . '[url=' . $contact_record['url'] . ']' . $contact_record['name'] . '[/url]' ;
-					$arr['body'] .= "\n\n" . '[url=' . $a->get_baseurl() . '/photos/' . $contact_record['nickname'] . '/image/' . $p[0]['resource-id'] . ']' . '[img]' . $a->get_baseurl() . "/photo/" . $p[0]['resource-id'] . '-' . $best . '.jpg' . '[/img][/url]' . "\n" ;
+					$arr['body']          = '[url=' . $tagged[1] . ']' . $tagged[0] . '[/url]' . ' ' . t('was tagged in a') . ' ' . '[url=' . $a->get_baseurl() . '/photos/' . $owner_record['nickname'] . '/image/' . $p[0]['resource-id'] . ']' . t('photo') . '[/url]' . ' ' . t('by') . ' ' . '[url=' . $owner_record['url'] . ']' . $owner_record['name'] . '[/url]' ;
+					$arr['body'] .= "\n\n" . '[url=' . $a->get_baseurl() . '/photos/' . $owner_record['nickname'] . '/image/' . $p[0]['resource-id'] . ']' . '[img]' . $a->get_baseurl() . "/photo/" . $p[0]['resource-id'] . '-' . $best . '.jpg' . '[/img][/url]' . "\n" ;
 
 					$arr['object'] = '<object><type>' . ACTIVITY_OBJ_PERSON . '</type><title>' . $tagged[0] . '</title><id>' . $tagged[1] . '/' . $tagged[0] . '</id>';
 					$arr['object'] .= '<link>' . xmlify('<link rel="alternate" type="text/html" href="' . $tagged[1] . '" />' . "\n");
@@ -397,13 +470,13 @@ function photos_post(&$a) {
 					$arr['object'] .= '</link></object>' . "\n";
 
 					$arr['target'] = '<target><type>' . ACTIVITY_OBJ_PHOTO . '</type><title>' . $p[0]['desc'] . '</title><id>'
-						. $a->get_baseurl() . '/photos/' . $contact_record['nickname'] . '/image/' . $p[0]['resource-id'] . '</id>';
-					$arr['target'] .= '<link>' . xmlify('<link rel="alternate" type="text/html" href="' . $a->get_baseurl() . '/photos/' . $contact_record['nickname'] . '/image/' . $p[0]['resource-id'] . '" />' . "\n" . '<link rel="preview" type="image/jpeg" href="' . $a->get_baseurl() . "/photo/" . $p[0]['resource-id'] . '-' . $best . '.jpg' . '" />') . '</link></target>';
+						. $a->get_baseurl() . '/photos/' . $owner_record['nickname'] . '/image/' . $p[0]['resource-id'] . '</id>';
+					$arr['target'] .= '<link>' . xmlify('<link rel="alternate" type="text/html" href="' . $a->get_baseurl() . '/photos/' . $owner_record['nickname'] . '/image/' . $p[0]['resource-id'] . '" />' . "\n" . '<link rel="preview" type="image/jpeg" href="' . $a->get_baseurl() . "/photo/" . $p[0]['resource-id'] . '-' . $best . '.jpg' . '" />') . '</link></target>';
 
 					$item_id = item_store($arr);
 					$php_path = ((strlen($a->config['php_path'])) ? $a->config['php_path'] : 'php');
-					proc_close(proc_open("\"$php_path\" \"include/notifier.php\" \"tag\" \"$item_id\" & ",
-						array(),$foo));
+					//proc_close(proc_open("\"$php_path\" \"include/notifier.php\" \"tag\" \"$item_id\" & ",array(),$foo));
+					proc_run($php_path,"include/notifier.php","tag","$item_id");
 				}
 
 			}
@@ -414,18 +487,20 @@ function photos_post(&$a) {
 	}
 
 
-	// default post action - upload a photo
+	/**
+	 * default post action - upload a photo
+	 */
 
-	if(! x($_FILES,'userfile'))
-		killme();
+	call_hooks('photo_post_init', $_POST);
 
-	if($_POST['partitionCount'])
-		$java_upload = true;
-	else
-		$java_upload = false;
+	/**
+	 * Determine the album to use
+	 */
 
-	$album =  notags(trim($_POST['album']));
-	$newalbum = notags(trim($_POST['newalbum']));
+	$album    = notags(trim($_REQUEST['album']));
+	$newalbum = notags(trim($_REQUEST['newalbum']));
+
+	logger('mod/photos.php: photos_post(): album= ' . $album . ' newalbum= ' . $newalbum , 'LOGGER_DEBUG');
 
 	if(! strlen($album)) {
 		if(strlen($newalbum))
@@ -434,29 +509,60 @@ function photos_post(&$a) {
 			$album = datetime_convert('UTC',date_default_timezone_get(),'now', 'Y');
 	}
 
+	/**
+	 *
+	 * We create a wall item for every photo, but we don't want to
+	 * overwhelm the data stream with a hundred newly uploaded photos.
+	 * So we will make one photo (the first one uploaded to this album)
+	 * visible by default, the rest will become visible over time when and if
+	 * they acquire comments, likes, dislikes, and/or tags 
+	 *
+	 */
+
 	$r = q("SELECT * FROM `photo` WHERE `album` = '%s' AND `uid` = %d",
 		dbesc($album),
-		intval(local_user())
+		intval($page_owner_uid)
 	);
 	if((! count($r)) || ($album == t('Profile Photos')))
 		$visible = 1;
 	else
 		$visible = 0;
 
+	$str_group_allow   = perms2str(((is_array($_REQUEST['group_allow']))   ? $_REQUEST['group_allow']   : explode(',',$_REQUEST['group_allow'])));
+	$str_contact_allow = perms2str(((is_array($_REQUEST['contact_allow'])) ? $_REQUEST['contact_allow'] : explode(',',$_REQUEST['contact_allow'])));
+	$str_group_deny    = perms2str(((is_array($_REQUEST['group_deny']))    ? $_REQUEST['group_deny']    : explode(',',$_REQUEST['group_deny'])));
+	$str_contact_deny  = perms2str(((is_array($_REQUEST['contact_deny']))  ? $_REQUEST['contact_deny']  : explode(',',$_REQUEST['contact_deny'])));
+
+	$ret = array('src' => '', 'filename' => '', 'filesize' => 0);
+
+	call_hooks('photo_post_file',$ret);
+
+	if(x($ret,'src') && x($ret,'filesize')) {
+		$src      = $ret['src'];
+		$filename = $ret['filename'];
+		$filesize = $ret['filesize'];
+	}
+	else {
+		$src        = $_FILES['userfile']['tmp_name'];
+		$filename   = basename($_FILES['userfile']['name']);
+		$filesize   = intval($_FILES['userfile']['size']);
+	}
+
+	$maximagesize = get_config('system','maximagesize');
 
-	$str_group_allow   = perms2str($_POST['group_allow']);
-	$str_contact_allow = perms2str($_POST['contact_allow']);
-	$str_group_deny    = perms2str($_POST['group_deny']);
-	$str_contact_deny  = perms2str($_POST['contact_deny']);
+	if(($maximagesize) && ($filesize > $maximagesize)) {
+		notice( t('Image exceeds size limit of ') . $maximagesize . EOL);
+		@unlink($src);
+		return;
+	}
 
-	$src               = $_FILES['userfile']['tmp_name'];
-	$filename          = basename($_FILES['userfile']['name']);
-	$filesize          = intval($_FILES['userfile']['size']);
+	logger('mod/photos.php: photos_post(): loading the contents of ' . $src , 'LOGGER_DEBUG');
 
 	$imagedata = @file_get_contents($src);
 	$ph = new Photo($imagedata);
 
 	if(! $ph->is_valid()) {
+		logger('mod/photos.php: photos_post(): unable to process image' , 'LOGGER_DEBUG');
 		notice( t('Unable to process image.') . EOL );
 		@unlink($src);
 		killme();
@@ -464,53 +570,53 @@ function photos_post(&$a) {
 
 	@unlink($src);
 
-	$width = $ph->getWidth();
+	$width  = $ph->getWidth();
 	$height = $ph->getHeight();
 
 	$smallest = 0;
 
 	$photo_hash = photo_new_resource();
 
-	$r = $ph->store(local_user(), 0, $photo_hash, $filename, $album, 0 , 0, $str_contact_allow, $str_group_allow, $str_contact_deny, $str_group_deny);
+	$r = $ph->store($page_owner_uid, $visitor, $photo_hash, $filename, $album, 0 , 0, $str_contact_allow, $str_group_allow, $str_contact_deny, $str_group_deny);
 
 	if(! $r) {
+		logger('mod/photos.php: photos_post(): image store failed' , 'LOGGER_DEBUG');
 		notice( t('Image upload failed.') . EOL );
 		killme();
 	}
 
 	if($width > 640 || $height > 640) {
 		$ph->scaleImage(640);
-		$ph->store(local_user(), 0, $photo_hash, $filename, $album, 1, 0, $str_contact_allow, $str_group_allow, $str_contact_deny, $str_group_deny);
+		$ph->store($page_owner_uid, $visitor, $photo_hash, $filename, $album, 1, 0, $str_contact_allow, $str_group_allow, $str_contact_deny, $str_group_deny);
 		$smallest = 1;
 	}
 
 	if($width > 320 || $height > 320) {
 		$ph->scaleImage(320);
-		$ph->store(local_user(), 0, $photo_hash, $filename, $album, 2, 0, $str_contact_allow, $str_group_allow, $str_contact_deny, $str_group_deny);
+		$ph->store($page_owner_uid, $visitor, $photo_hash, $filename, $album, 2, 0, $str_contact_allow, $str_group_allow, $str_contact_deny, $str_group_deny);
 		$smallest = 2;
 	}
 	
 	$basename = basename($filename);
-	$uri = item_new_uri($a->get_hostname(), local_user());
+	$uri = item_new_uri($a->get_hostname(), $page_owner_uid);
 
 	// Create item container
 
-
 	$arr = array();
 
-	$arr['uid']           = local_user();
+	$arr['uid']           = $page_owner_uid;
 	$arr['uri']           = $uri;
 	$arr['parent-uri']    = $uri;
 	$arr['type']          = 'photo';
 	$arr['wall']          = 1;
 	$arr['resource-id']   = $photo_hash;
-	$arr['contact-id']    = $contact_record['id'];
-	$arr['owner-name']    = $contact_record['name'];
-	$arr['owner-link']    = $contact_record['url'];
-	$arr['owner-avatar']  = $contact_record['thumb'];
-	$arr['author-name']   = $contact_record['name'];
-	$arr['author-link']   = $contact_record['url'];
-	$arr['author-avatar'] = $contact_record['thumb'];
+	$arr['contact-id']    = $owner_record['id'];
+	$arr['owner-name']    = $owner_record['name'];
+	$arr['owner-link']    = $owner_record['url'];
+	$arr['owner-avatar']  = $owner_record['thumb'];
+	$arr['author-name']   = $owner_record['name'];
+	$arr['author-link']   = $owner_record['url'];
+	$arr['author-avatar'] = $owner_record['thumb'];
 	$arr['title']         = '';
 	$arr['allow_cid']     = $str_contact_allow;
 	$arr['allow_gid']     = $str_group_allow;
@@ -518,20 +624,19 @@ function photos_post(&$a) {
 	$arr['deny_gid']      = $str_group_deny;
 	$arr['last-child']    = 1;
 	$arr['visible']       = $visible;
-	$arr['body']          = '[url=' . $a->get_baseurl() . '/photos/' . $contact_record['nickname'] . '/image/' . $photo_hash . ']' 
+	$arr['body']          = '[url=' . $a->get_baseurl() . '/photos/' . $owner_record['nickname'] . '/image/' . $photo_hash . ']' 
 				. '[img]' . $a->get_baseurl() . "/photo/{$photo_hash}-{$smallest}.jpg" . '[/img]' 
 				. '[/url]';
 
 	$item_id = item_store($arr);
 
-	if(! $java_upload) {
-		goaway($a->get_baseurl() . '/' . $_SESSION['photo_return']);
-		return; // NOTREACHED
-	}
+	call_hooks('photo_post_end',intval($item_id));
 
-	killme();
-	return; // NOTREACHED
+	// addon uploaders should call "killme()" [e.g. exit] within the photo_post_end hook
+	// if they do not wish to be redirected
 
+	goaway($a->get_baseurl() . '/' . $_SESSION['photo_return']);
+	// NOTREACHED
 }
 
 
@@ -576,18 +681,39 @@ function photos_content(&$a) {
 	// Setup permissions structures
 	//
 
+	$can_post       = false;
+	$visitor        = 0;
+	$contact        = null;
+	$remote_contact = false;
+
 	$owner_uid = $a->data['user']['uid'];
 
+	$community_page = (($a->data['user']['page-flags'] == PAGE_COMMUNITY) ? true : false);
 
+	if((local_user()) && (local_user() == $owner_uid))
+		$can_post = true;
+	else {
+		if($community_page && remote_user()) {
+			$r = q("SELECT `uid` FROM `contact` WHERE `blocked` = 0 AND `pending` = 0 AND `id` = %d AND `uid` = %d LIMIT 1",
+				intval(remote_user()),
+				intval($owner_uid)
+			);
+			if(count($r)) {
+				$can_post = true;
+				$contact  = $r[0];
+				$remote_contact = true;
+				$visitor = remote_user();
+			}
+		}
+	}
 
-	$contact = null;
-	$remote_contact = false;
+	// perhaps they're visiting - but not a community page, so they wouldn't have write access
 
-	if(remote_user()) {
+	if(remote_user() && (! $visitor)) {
 		$contact_id = $_SESSION['visitor_id'];
 		$groups = init_groups_visitor($contact_id);
-		$r = q("SELECT * FROM `contact` WHERE `id` = %d AND `uid` = %d LIMIT 1",
-			intval($contact_id),
+		$r = q("SELECT * FROM `contact` WHERE `blocked` = 0 AND `pending` = 0 AND `id` = %d AND `uid` = %d LIMIT 1",
+			intval(remote_user()),
 			intval($owner_uid)
 		);
 		if(count($r)) {
@@ -603,7 +729,6 @@ function photos_content(&$a) {
 		}
 	}
 
-
 	// default permissions - anonymous user
 
 	$sql_extra = " AND `allow_cid` = '' AND `allow_gid` = '' AND `deny_cid` = '' AND `deny_gid` = '' ";
@@ -626,8 +751,8 @@ function photos_content(&$a) {
 			  AND ( `allow_gid` = '' OR `allow_gid` REGEXP '%s' )
 			  AND ( `deny_gid`  = '' OR NOT `deny_gid` REGEXP '%s') ",
 
-			intval($_SESSION['visitor_id']),
-			intval($_SESSION['visitor_id']),
+			intval(remote_user()),
+			intval(remote_user()),
 			dbesc($gs),
 			dbesc($gs)
 		);
@@ -639,7 +764,7 @@ function photos_content(&$a) {
 
 
 	if($datatype === 'upload') {
-		if( ! (local_user() && (local_user() == $a->data['user']['uid']))) {
+		if(! ($can_post)) {
 			notice( t('Permission denied.'));
 			return;
 		}
@@ -657,24 +782,39 @@ function photos_content(&$a) {
 		$celeb = ((($a->user['page-flags'] == PAGE_SOAPBOX) || ($a->user['page-flags'] == PAGE_COMMUNITY)) ? true : false);
 
 		$albumselect .= '</select>';
+
+		$uploader = '';
+
+		$ret = array('post_url' => $a->get_baseurl() . '/photos/' . $a->data['user']['nickname'],
+				'addon_text' => $uploader,
+				'default_upload' => true);
+
+
+		call_hooks('photo_upload_form',$ret);
+
+		$default_upload = '<input type="file" name="userfile" /> 	<div class="photos-upload-submit-wrapper" >
+		<input type="submit" name="submit" value="' . t('Submit') . '" id="photos-upload-submit" /> </div>';
+
+
+ 
+
 		$tpl = load_view_file('view/photos_upload.tpl');
 		$o .= replace_macros($tpl,array(
 			'$pagename' => t('Upload Photos'),
 			'$sessid' => session_id(),
+			'$nickname' => $a->data['user']['nickname'],
 			'$newalbum' => t('New album name: '),
 			'$existalbumtext' => t('or existing album name: '),
-			'$filestext' => t('Select files to upload: '),
 			'$albumselect' => $albumselect,
 			'$permissions' => t('Permissions'),
-			'$aclselect' => populate_acl($a->user, $celeb),
-			'$archive' => $a->get_baseurl() . '/jumploader_z.jar',
-			'$nojava' => t('Use the following controls only if the Java uploader [above] fails to launch.'),
-			'$uploadurl' => $a->get_baseurl() . '/photos',
-			'$submit' => t('Submit')
+			'$aclselect' => (($visitor) ? '' : populate_acl($a->user, $celeb)),
+			'$uploader' => $ret['addon_text'],
+			'$default' => (($ret['default_upload']) ? $default_upload : ''),
+			'$uploadurl' => $ret['post_url']
+
 		));
 
 		return $o; 
-
 	}
 
 	if($datatype === 'album') {
@@ -683,16 +823,17 @@ function photos_content(&$a) {
 
 		$r = q("SELECT `resource-id`, max(`scale`) AS `scale` FROM `photo` WHERE `uid` = %d AND `album` = '%s' 
 			$sql_extra GROUP BY `resource-id`",
-			intval($a->data['user']['uid']),
+			intval($owner_uid),
 			dbesc($album)
 		);
-		if(count($r))
+		if(count($r)) {
 			$a->set_pager_total(count($r));
+			$a->set_pager_itemspage(20);
+		}
 
-
-		$r = q("SELECT `resource-id`, `id`, `filename`, max(`scale`) AS `scale` FROM `photo` WHERE `uid` = %d AND `album` = '%s' 
+		$r = q("SELECT `resource-id`, `id`, `filename`, max(`scale`) AS `scale`, `desc` FROM `photo` WHERE `uid` = %d AND `album` = '%s' 
 			$sql_extra GROUP BY `resource-id` ORDER BY `created` DESC LIMIT %d , %d",
-			intval($a->data['user']['uid']),
+			intval($owner_uid),
 			dbesc($album),
 			intval($a->pager['start']),
 			intval($a->pager['itemspage'])
@@ -702,10 +843,11 @@ function photos_content(&$a) {
 		
 		if($cmd === 'edit') {		
 			if(($album != t('Profile Photos')) && ($album != t('Contact Photos'))) {
-				if(local_user() && (local_user() == $a->data['user']['uid'])) {
+				if($can_post) {
 					$edit_tpl = load_view_file('view/album_edit.tpl');
 					$o .= replace_macros($edit_tpl,array(
 						'$nametext' => t('New album name: '),
+						'$nickname' => $a->data['user']['nickname'],
 						'$album' => $album,
 						'$hexalbum' => bin2hex($album),
 						'$submit' => t('Submit'),
@@ -716,7 +858,7 @@ function photos_content(&$a) {
 		}
 		else {
 			if(($album != t('Profile Photos')) && ($album != t('Contact Photos'))) {
-				if(local_user() && (local_user() == $a->data['user']['uid'])) {
+				if($can_post) {
 					$o .= '<div id="album-edit-link"><a href="'. $a->get_baseurl() . '/photos/' 
 						. $a->data['user']['nickname'] . '/album/' . bin2hex($album) . '/edit' . '">' 
 						. t('Edit Album') . '</a></div>';
@@ -731,11 +873,14 @@ function photos_content(&$a) {
 					'$photolink' => $a->get_baseurl() . '/photos/' . $a->data['user']['nickname'] . '/image/' . $rr['resource-id'],
 					'$phototitle' => t('View Photo'),
 					'$imgsrc' => $a->get_baseurl() . '/photo/' . $rr['resource-id'] . '-' . $rr['scale'] . '.jpg',
-					'$imgalt' => $rr['filename']
+					'$imgalt' => $rr['filename'],
+					'$desc'=> $rr['desc']
 				));
 
 		}
 		$o .= '<div id="photo-album-end"></div>';
+		$o .= paginate($a);
+
 		return $o;
 
 	}	
@@ -751,7 +896,7 @@ function photos_content(&$a) {
 
 		$ph = q("SELECT * FROM `photo` WHERE `uid` = %d AND `resource-id` = '%s' 
 			$sql_extra ORDER BY `scale` ASC ",
-			intval($a->data['user']['uid']),
+			intval($owner_uid),
 			dbesc($datum)
 		);
 
@@ -776,8 +921,17 @@ function photos_content(&$a) {
 		
 		$o .= '<h3>' . '<a href="' . $a->get_baseurl() . '/photos/' . $a->data['user']['nickname'] . '/album/' . bin2hex($ph[0]['album']) . '">' . $ph[0]['album'] . '</a></h3>';
  
-		if(local_user() && ($ph[0]['uid'] == local_user())) {
-			$o .= '<div id="photo-edit-link-wrap" ><a id="photo-edit-link" href="' . $a->get_baseurl() . '/photos/' . $a->data['user']['nickname'] . '/image/' . $datum . '/edit' . '">' . t('Edit photo') . '</a></div>';
+		if($can_post && ($ph[0]['uid'] == $owner_uid)) {
+			$o .= '<div id="photo-edit-link-wrap" ><a id="photo-edit-link" href="' . $a->get_baseurl() . '/photos/' . $a->data['user']['nickname'] . '/image/' . $datum . '/edit' . '">' . t('Edit photo') . '</a>';
+			
+			$o .= ' - <a id="photo-toprofile-link" href="' . $a->get_baseurl() . '/profile_photo/use/'.$ph[0]['resource-id'].'">'.t('Use as profile photo').'</a>';
+			// lock
+			$o .= ( ( ($ph[0]['uid'] == local_user()) && (strlen($ph[0]['allow_cid']) || strlen($ph[0]['allow_gid']) 
+					|| strlen($ph[0]['deny_cid']) || strlen($ph[0]['deny_gid'])) ) 
+					? ' - <img src="images/lock_icon.gif" class="lockview" alt="' . t('Private Message') . '" onclick="lockview(event,\'photo/' . $ph[0]['id'] . '\');" />'
+					: '');
+	  		
+			$o .= '</div>';
 		}
 
 
@@ -850,13 +1004,18 @@ function photos_content(&$a) {
 				$tag_str .= bbcode($t);
 			} 
 			$o .= $tag_str . '</div>';
+			if($cmd === 'edit')
+				$o .= '<div id="tag-remove"><a href="' . $a->get_baseurl() . '/tagrm/' . $link_item['id'] . '">' . t('[Remove any tag]') . '</a></div>';
 		}
 
 
-		if($cmd === 'edit') {
+		if(($cmd === 'edit') && ($can_post)) {
 			$edit_tpl = load_view_file('view/photo_edit.tpl');
 			$o .= replace_macros($edit_tpl, array(
 				'$id' => $ph[0]['id'],
+				'$album' => $ph[0]['album'],
+				'$newalbum' => t('New album name'), 
+				'$nickname' => $a->data['user']['nickname'],
 				'$resource_id' => $ph[0]['resource-id'],
 				'$capt_label' => t('Caption'),
 				'$caption' => $ph[0]['desc'],
@@ -866,7 +1025,6 @@ function photos_content(&$a) {
 				'$item_id' => ((count($linked_items)) ? $link_item['id'] : 0),
 				'$submit' => t('Submit'),
 				'$delete' => t('Delete Photo')
-
 			));
 		}
 
@@ -880,7 +1038,7 @@ function photos_content(&$a) {
 
 			$likebuttons = '';
 
-			if(can_write_wall($a,$a->data['user']['uid']))
+			if($can_post || can_write_wall($a,$owner_uid))
 				$likebuttons = replace_macros($like_tpl,array('$id' => $link_item['id']));
 
 			if(! count($r)) {
@@ -888,14 +1046,15 @@ function photos_content(&$a) {
 				$o .= $likebuttons;
 				$o .= '</div>';
 
-				if(can_write_wall($a,$a->data['user']['uid'])) {
+				if($can_post || can_write_wall($a,$owner_uid)) {
 					if($link_item['last-child']) {
 						$o .= replace_macros($cmnt_tpl,array(
-							'$return_path' => $return_url,
+							'$return_path' => '', 
+							'$jsreload' => $return_url,
 							'$type' => 'wall-comment',
 							'$id' => $link_item['id'],
 							'$parent' => $link_item['id'],
-							'$profile_uid' =>  $a->data['user']['uid'],
+							'$profile_uid' =>  $owner_uid,
 							'$mylink' => $contact['url'],
 							'$mytitle' => t('This is you'),
 							'$myphoto' => $contact['thumb'],
@@ -916,7 +1075,7 @@ function photos_content(&$a) {
 					like_puller($a,$item,$dlike,'dislike');
 				}
 
-	            $like    = ((isset($alike[$link_item['id']])) ? format_like($alike[$link_item['id']],$alike[$link_item['id'] . '-l'],'like',$link_item['id']) : '');
+				$like    = ((isset($alike[$link_item['id']])) ? format_like($alike[$link_item['id']],$alike[$link_item['id'] . '-l'],'like',$link_item['id']) : '');
 				$dislike = ((isset($dlike[$link_item['id']])) ? format_like($dlike[$link_item['id']],$dlike[$link_item['id'] . '-l'],'dislike',$link_item['id']) : '');
 
 				$o .= '<div id="photo-like-div">';
@@ -927,14 +1086,15 @@ function photos_content(&$a) {
 
 
 
-				if(can_write_wall($a,$a->data['user']['uid'])) {
+				if($can_post || can_write_wall($a,$owner_uid)) {
 					if($link_item['last-child']) {
 						$o .= replace_macros($cmnt_tpl,array(
-							'$return_path' => $return_url,
+							'$return_path' => '',
+							'$jsreload' => $return_url,
 							'$type' => 'wall-comment',
 							'$id' => $link_item['id'],
 							'$parent' => $link_item['id'],
-							'$profile_uid' =>  $a->data['user']['uid'],
+							'$profile_uid' =>  $owner_uid,
 							'$mylink' => $contact['url'],
 							'$mytitle' => t('This is you'),
 							'$myphoto' => $contact['thumb'],
@@ -954,15 +1114,16 @@ function photos_content(&$a) {
 
 					$redirect_url = $a->get_baseurl() . '/redir/' . $item['cid'] ;
 			
-					if(can_write_wall($a,$a->data['user']['uid'])) {
+					if($can_post || can_write_wall($a,$owner_uid)) {
 
 						if($item['last-child']) {
 							$comment = replace_macros($cmnt_tpl,array(
-								'$return_path' => $return_url,
+								'$return_path' => '',
+								'$jsreload' => $return_url,
 								'$type' => 'wall-comment',
 								'$id' => $item['item_id'],
 								'$parent' => $item['parent'],
-								'$profile_uid' =>  $a->data['user']['uid'],
+								'$profile_uid' =>  $owner_uid,
 								'$mylink' => $contact['url'],
 								'$mytitle' => t('This is you'),
 								'$myphoto' => $contact['thumb'],
@@ -982,13 +1143,16 @@ function photos_content(&$a) {
 						$sparkle = '';
 					}
  
-					$profile_name = ((strlen($item['author-name'])) ? $item['author-name'] : $item['name']);
-					$profile_avatar = ((strlen($item['author-avatar'])) ? $item['author-avatar'] : $item['thumb']);
+					$diff_author = (($item['url'] !== $item['author-link']) ? true : false);
+
+					$profile_name   = (((strlen($item['author-name']))   && $diff_author) ? $item['author-name']   : $item['name']);
+					$profile_avatar = (((strlen($item['author-avatar'])) && $diff_author) ? $item['author-avatar'] : $item['thumb']);
+
 					$profile_link = $profile_url;
 
 					$drop = '';
 
-					if(($item['contact-id'] == $_SESSION['visitor_id']) || ($item['uid'] == local_user()))
+					if(($item['contact-id'] == remote_user()) || ($item['uid'] == local_user()))
 						$drop = replace_macros(load_view_file('view/wall_item_drop.tpl'), array('$id' => $item['id']));
 
 
@@ -1021,11 +1185,12 @@ function photos_content(&$a) {
 		intval($a->data['user']['uid']),
 		dbesc( t('Contact Photos'))
 	);
-	if(count($r))
+	if(count($r)) {
 		$a->set_pager_total(count($r));
+		$a->set_pager_itemspage(20);
+	}
 
-
-	$r = q("SELECT `resource-id`, `id`, `filename`, `album`, max(`scale`) AS `scale` FROM `photo` 
+	$r = q("SELECT `resource-id`, `id`, `filename`, `album`, max(`scale`) AS `scale` FROM `photo`
 		WHERE `uid` = %d AND `album` != '%s' 
 		$sql_extra GROUP BY `resource-id` ORDER BY `created` DESC LIMIT %d , %d",
 		intval($a->data['user']['uid']),
@@ -1036,7 +1201,7 @@ function photos_content(&$a) {
 
 	$o .= '<h3>' . t('Recent Photos') . '</h3>';
 
-	if( local_user() && (local_user() == $a->data['user']['uid'])) {
+	if($can_post) {
 		$o .= '<div id="photo-top-links"><a id="photo-top-upload-link" href="'. $a->get_baseurl() . '/photos/' 
 			. $a->data['user']['nickname'] . '/upload' . '">' . t('Upload New Photos') . '</a></div>';
 	}
@@ -1050,7 +1215,7 @@ function photos_content(&$a) {
 					. '/image/' . $rr['resource-id'],
 				'$phototitle' => t('View Photo'),
 				'$imgsrc' => $a->get_baseurl() . '/photo/' 
-					. $rr['resource-id'] . '-' . $rr['scale'] . '.jpg',
+					. $rr['resource-id'] . '-' . ((($rr['scale']) == 6) ? 4 : $rr['scale']) . '.jpg',
 				'$albumlink' => $a->get_baseurl() . '/photos/' 
 					. $a->data['user']['nickname'] . '/album/' . bin2hex($rr['album']),
 				'$albumname' => $rr['album'],
@@ -1061,5 +1226,7 @@ function photos_content(&$a) {
 		}
 		$o .= '<div id="photo-top-end"></div>';
 	}
+	$o .= paginate($a);
 	return $o;
 }
+