]> git.mxchange.org Git - friendica.git/blobdiff - mod/photos.php
Remove unused parameter in can_write_wall()
[friendica.git] / mod / photos.php
index 4110e805778841af8ca284d808f0e37cf61b2f16..ab0ad75013c24f03678dc2e63730f2725b1cd48a 100644 (file)
@@ -1,13 +1,19 @@
 <?php
-
+/**
+ * @file mod/photos.php
+ */
 use Friendica\App;
+use Friendica\Content\Feature;
 use Friendica\Core\System;
 use Friendica\Core\Config;
 use Friendica\Core\Worker;
+use Friendica\Database\DBM;
+use Friendica\Model\Contact;
+use Friendica\Model\Group;
+use Friendica\Model\Photo;
 use Friendica\Network\Probe;
+use Friendica\Object\Image;
 
-require_once 'include/Photo.php';
-require_once 'include/photos.php';
 require_once 'include/items.php';
 require_once 'include/acl_selectors.php';
 require_once 'include/bbcode.php';
@@ -22,7 +28,7 @@ function photos_init(App $a) {
                auto_redir($a, $a->argv[1]);
        }
 
-       if ((Config::get('system', 'block_public')) && (! local_user()) && (! remote_user())) {
+       if (Config::get('system', 'block_public') && !local_user() && !remote_user()) {
                return;
        }
 
@@ -34,7 +40,7 @@ function photos_init(App $a) {
                        dbesc($nick)
                );
 
-               if (! dbm::is_result($user)) {
+               if (!DBM::is_result($user)) {
                        return;
                }
 
@@ -44,21 +50,21 @@ function photos_init(App $a) {
 
                $profile = get_profiledata_by_nick($nick, $a->profile_uid);
 
-               $account_type = account_type($profile);
+               $account_type = Contact::getAccountType($profile);
 
                $tpl = get_markup_template("vcard-widget.tpl");
 
-               $vcard_widget .= replace_macros($tpl, array(
+               $vcard_widget = replace_macros($tpl, array(
                        '$name' => $profile['name'],
                        '$photo' => $profile['photo'],
-                       '$addr' => (($profile['addr'] != "") ? $profile['addr'] : ""),
+                       '$addr' => defaults($profile, 'addr', ''),
                        '$account_type' => $account_type,
-                       '$pdesc' => (($profile['pdesc'] != "") ? $profile['pdesc'] : ""),
+                       '$pdesc' => defaults($profile, 'pdesc', ''),
                ));
 
-               $albums = photo_albums($a->data['user']['uid']);
+               $albums = Photo::getAlbums($a->data['user']['uid']);
 
-               $albums_visible = ((intval($a->data['user']['hidewall']) && (! local_user()) && (! remote_user())) ? false : true);
+               $albums_visible = ((intval($a->data['user']['hidewall']) && !local_user() && !remote_user()) ? false : true);
 
                // add various encodings to the array so we can just loop through and pick them out in a template
                $ret = array('success' => false);
@@ -72,7 +78,7 @@ function photos_init(App $a) {
                        $ret['albums'] = array();
                        foreach ($albums as $k => $album) {
                                //hide profile photos to others
-                               if ((! $is_owner) && (! remote_user()) && ($album['album'] == t('Profile Photos')))
+                               if (!$is_owner && !remote_user() && ($album['album'] == t('Profile Photos')))
                                        continue;
                                $entry = array(
                                        'text'      => $album['album'],
@@ -85,18 +91,16 @@ function photos_init(App $a) {
                        }
                }
 
-               $albums = $ret;
-
                if (local_user() && $a->data['user']['uid'] == local_user()) {
                        $can_post = true;
                }
 
-               if ($albums['success']) {
+               if ($ret['success']) {
                        $photo_albums_widget = replace_macros(get_markup_template('photo_albums.tpl'), array(
                                '$nick'     => $a->data['user']['nickname'],
                                '$title'    => t('Photo Albums'),
                                '$recent'   => t('Recent Photos'),
-                               '$albums'   => $albums['albums'],
+                               '$albums'   => $ret['albums'],
                                '$baseurl'  => System::baseUrl(),
                                '$upload'   => array(t('Upload New Photos'), 'photos/' . $a->data['user']['nickname'] . '/upload'),
                                '$can_post' => $can_post
@@ -104,47 +108,41 @@ function photos_init(App $a) {
                }
 
 
-               if (! x($a->page, 'aside')) {
+               if (!x($a->page, 'aside')) {
                        $a->page['aside'] = '';
                }
                $a->page['aside'] .= $vcard_widget;
                $a->page['aside'] .= $photo_albums_widget;
 
-
                $tpl = get_markup_template("photos_head.tpl");
                $a->page['htmlhead'] .= replace_macros($tpl,array(
                        '$ispublic' => t('everybody')
                ));
-
        }
 
        return;
 }
 
-
-
-function photos_post(App $a) {
-
+function photos_post(App $a)
+{
        logger('mod-photos: photos_post: begin' , LOGGER_DEBUG);
+       logger('mod_photos: REQUEST ' . print_r($_REQUEST, true), LOGGER_DATA);
+       logger('mod_photos: FILES '   . print_r($_FILES, true), LOGGER_DATA);
 
-
-       logger('mod_photos: REQUEST ' . print_r($_REQUEST,true), LOGGER_DATA);
-       logger('mod_photos: FILES '   . print_r($_FILES,true), LOGGER_DATA);
-
-       $phototypes = Photo::supportedTypes();
+       $phototypes = Image::supportedTypes();
 
        $can_post  = false;
        $visitor   = 0;
 
        $page_owner_uid = $a->data['user']['uid'];
-       $community_page = (($a->data['user']['page-flags'] == PAGE_COMMUNITY) ? true : false);
+       $community_page = $a->data['user']['page-flags'] == PAGE_COMMUNITY;
 
-       if ((local_user()) && (local_user() == $page_owner_uid)) {
+       if (local_user() && (local_user() == $page_owner_uid)) {
                $can_post = true;
        } else {
                if ($community_page && remote_user()) {
                        $contact_id = 0;
-                       if (is_array($_SESSION['remote'])) {
+                       if (x($_SESSION, 'remote') && is_array($_SESSION['remote'])) {
                                foreach ($_SESSION['remote'] as $v) {
                                        if ($v['uid'] == $page_owner_uid) {
                                                $contact_id = $v['cid'];
@@ -153,12 +151,11 @@ function photos_post(App $a) {
                                }
                        }
                        if ($contact_id) {
-
                                $r = q("SELECT `uid` FROM `contact` WHERE `blocked` = 0 AND `pending` = 0 AND `id` = %d AND `uid` = %d LIMIT 1",
                                        intval($contact_id),
                                        intval($page_owner_uid)
                                );
-                               if (dbm::is_result($r)) {
+                               if (DBM::is_result($r)) {
                                        $can_post = true;
                                        $visitor = $contact_id;
                                }
@@ -166,7 +163,7 @@ function photos_post(App $a) {
                }
        }
 
-       if (! $can_post) {
+       if (!$can_post) {
                notice( t('Permission denied.') . EOL );
                killme();
        }
@@ -176,7 +173,7 @@ function photos_post(App $a) {
                intval($page_owner_uid)
        );
 
-       if (! dbm::is_result($r)) {
+       if (!DBM::is_result($r)) {
                notice( t('Contact information unavailable') . EOL);
                logger('photos_post: unable to locate contact record for page owner. uid=' . $page_owner_uid);
                killme();
@@ -184,8 +181,7 @@ function photos_post(App $a) {
 
        $owner_record = $r[0];
 
-
-       if (($a->argc > 3) && ($a->argv[2] === 'album')) {
+       if ($a->argc > 3 && $a->argv[2] === 'album') {
                $album = hex2bin($a->argv[3]);
 
                if ($album === t('Profile Photos') || $album === 'Contact Photos' || $album === t('Contact Photos')) {
@@ -197,7 +193,7 @@ function photos_post(App $a) {
                        dbesc($album),
                        intval($page_owner_uid)
                );
-               if (!dbm::is_result($r)) {
+               if (!DBM::is_result($r)) {
                        notice( t('Album not found.') . EOL);
                        goaway($_SESSION['photo_return']);
                        return; // NOTREACHED
@@ -208,10 +204,7 @@ function photos_post(App $a) {
                        goaway($_SESSION['photo_return']);
                }
 
-               /*
-                * RENAME photo album
-                */
-
+               // RENAME photo album
                $newalbum = notags(trim($_POST['albumname']));
                if ($newalbum != $album) {
                        q("UPDATE `photo` SET `album` = '%s' WHERE `album` = '%s' AND `uid` = %d",
@@ -220,9 +213,9 @@ function photos_post(App $a) {
                                intval($page_owner_uid)
                        );
                        // Update the photo albums cache
-                       photo_albums($page_owner_uid, true);
+                       Photo::clearAlbumCache($page_owner_uid);
 
-                       $newurl = str_replace(bin2hex($album),bin2hex($newalbum),$_SESSION['photo_return']);
+                       $newurl = str_replace(bin2hex($album), bin2hex($newalbum), $_SESSION['photo_return']);
                        goaway($newurl);
                        return; // NOTREACHED
                }
@@ -232,9 +225,8 @@ function photos_post(App $a) {
                 */
 
                if ($_POST['dropalbum'] == t('Delete Album')) {
-
                        // Check if we should do HTML-based delete confirmation
-                       if ($_REQUEST['confirm']) {
+                       if (x($_REQUEST, 'confirm')) {
                                $drop_url = $a->query_string;
                                $extra_inputs = array(
                                        array('name' => 'albumname', 'value' => $_POST['albumname']),
@@ -268,7 +260,7 @@ function photos_post(App $a) {
                                        dbesc($album)
                                );
                        }
-                       if (dbm::is_result($r)) {
+                       if (DBM::is_result($r)) {
                                foreach ($r as $rr) {
                                        $res[] = "'" . dbesc($rr['rid']) . "'" ;
                                }
@@ -280,17 +272,15 @@ function photos_post(App $a) {
                        $str_res = implode(',', $res);
 
                        // remove the associated photos
-
                        q("DELETE FROM `photo` WHERE `resource-id` IN ( $str_res ) AND `uid` = %d",
                                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",
+                       $r = q("SELECT `id`, `parent-uri`, `visible` FROM `item` WHERE `resource-id` IN ( $str_res ) AND `uid` = %d",
                                intval($page_owner_uid)
                        );
-                       if (dbm::is_result($r)) {
+                       if (DBM::is_result($r)) {
                                foreach ($r as $rr) {
                                        q("UPDATE `item` SET `deleted` = 1, `changed` = '%s' WHERE `parent-uri` = '%s' AND `uid` = %d",
                                                dbesc(datetime_convert()),
@@ -303,15 +293,14 @@ function photos_post(App $a) {
                                        $drop_id = intval($rr['id']);
 
                                        // send the notification upstream/downstream as the case may be
-
                                        if ($rr['visible']) {
-                                               Worker::add(PRIORITY_HIGH, "notifier", "drop", $drop_id);
+                                               Worker::add(PRIORITY_HIGH, "Notifier", "drop", $drop_id);
                                        }
                                }
                        }
 
                        // Update the photo albums cache
-                       photo_albums($page_owner_uid, true);
+                       Photo::clearAlbumCache($page_owner_uid);
                }
 
                goaway('photos/' . $a->data['user']['nickname']);
@@ -320,16 +309,16 @@ function photos_post(App $a) {
 
 
        // Check if the user has responded to a delete confirmation query for a single photo
-       if (($a->argc > 2) && $_REQUEST['canceled']) {
+       if ($a->argc > 2 && x($_REQUEST, 'canceled')) {
                goaway($_SESSION['photo_return']);
        }
 
-       if (($a->argc > 2) && (x($_POST,'delete')) && ($_POST['delete'] == t('Delete Photo'))) {
+       if ($a->argc > 2 && defaults($_POST, 'delete', '') === t('Delete Photo')) {
 
                // same as above but remove single photo
 
                // Check if we should do HTML-based delete confirmation
-               if ($_REQUEST['confirm']) {
+               if (x($_REQUEST, 'confirm')) {
                        $drop_url = $a->query_string;
                        $a->page['content'] = replace_macros(get_markup_template('confirm.tpl'), array(
                                '$method' => 'post',
@@ -356,16 +345,16 @@ function photos_post(App $a) {
                                dbesc($a->argv[2])
                        );
                }
-               if (dbm::is_result($r)) {
+               if (DBM::is_result($r)) {
                        q("DELETE FROM `photo` WHERE `uid` = %d AND `resource-id` = '%s'",
                                intval($page_owner_uid),
                                dbesc($r[0]['resource-id'])
                        );
-                       $i = q("SELECT * FROM `item` WHERE `resource-id` = '%s' AND `uid` = %d LIMIT 1",
+                       $i = q("SELECT `id`, `uri`, `visible` FROM `item` WHERE `resource-id` = '%s' AND `uid` = %d LIMIT 1",
                                dbesc($r[0]['resource-id']),
                                intval($page_owner_uid)
                        );
-                       if (dbm::is_result($i)) {
+                       if (DBM::is_result($i)) {
                                q("UPDATE `item` SET `deleted` = 1, `edited` = '%s', `changed` = '%s' WHERE `parent-uri` = '%s' AND `uid` = %d",
                                        dbesc(datetime_convert()),
                                        dbesc(datetime_convert()),
@@ -379,10 +368,10 @@ function photos_post(App $a) {
                                $drop_id = intval($i[0]['id']);
 
                                // Update the photo albums cache
-                               photo_albums($page_owner_uid, true);
+                               Photo::clearAlbumCache($page_owner_uid);
 
                                if ($i[0]['visible']) {
-                                       Worker::add(PRIORITY_HIGH, "notifier", "drop", $drop_id);
+                                       Worker::add(PRIORITY_HIGH, "Notifier", "drop", $drop_id);
                                }
                        }
                }
@@ -391,13 +380,12 @@ function photos_post(App $a) {
                return; // NOTREACHED
        }
 
-       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'])) : '');
-               $origaname   = ((x($_POST,'origaname')) ? notags(trim($_POST['origaname'])) : '');
+       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']))   : '';
+               $origaname   = x($_POST, 'origaname') ? notags(trim($_POST['origaname'])) : '';
                $str_group_allow   = perms2str($_POST['group_allow']);
                $str_contact_allow = perms2str($_POST['contact_allow']);
                $str_group_deny    = perms2str($_POST['group_deny']);
@@ -405,30 +393,29 @@ function photos_post(App $a) {
 
                $resource_id = $a->argv[2];
 
-               if (! strlen($albname)) {
+               if (!strlen($albname)) {
                        $albname = datetime_convert('UTC',date_default_timezone_get(),'now', 'Y');
                }
 
-
-               if ((x($_POST,'rotate') !== false) &&
-                  ( (intval($_POST['rotate']) == 1) || (intval($_POST['rotate']) == 2) )) {
+               if (x($_POST,'rotate') !== false &&
+                  (intval($_POST['rotate']) == 1 || intval($_POST['rotate']) == 2)) {
                        logger('rotate');
 
                        $r = q("SELECT * FROM `photo` WHERE `resource-id` = '%s' AND `uid` = %d AND `scale` = 0 LIMIT 1",
                                dbesc($resource_id),
                                intval($page_owner_uid)
                        );
-                       if (dbm::is_result($r)) {
-                               $ph = new Photo($r[0]['data'], $r[0]['type']);
-                               if ($ph->is_valid()) {
+                       if (DBM::is_result($r)) {
+                               $Image = new Image($r[0]['data'], $r[0]['type']);
+                               if ($Image->isValid()) {
                                        $rotate_deg = ( (intval($_POST['rotate']) == 1) ? 270 : 90 );
-                                       $ph->rotate($rotate_deg);
+                                       $Image->rotate($rotate_deg);
 
-                                       $width  = $ph->getWidth();
-                                       $height = $ph->getHeight();
+                                       $width  = $Image->getWidth();
+                                       $height = $Image->getHeight();
 
                                        $x = q("UPDATE `photo` SET `data` = '%s', `height` = %d, `width` = %d WHERE `resource-id` = '%s' AND `uid` = %d AND `scale` = 0",
-                                               dbesc($ph->imageString()),
+                                               dbesc($Image->asString()),
                                                intval($height),
                                                intval($width),
                                                dbesc($resource_id),
@@ -436,12 +423,12 @@ function photos_post(App $a) {
                                        );
 
                                        if ($width > 640 || $height > 640) {
-                                               $ph->scaleImage(640);
-                                               $width  = $ph->getWidth();
-                                               $height = $ph->getHeight();
+                                               $Image->scaleDown(640);
+                                               $width  = $Image->getWidth();
+                                               $height = $Image->getHeight();
 
                                                $x = q("UPDATE `photo` SET `data` = '%s', `height` = %d, `width` = %d WHERE `resource-id` = '%s' AND `uid` = %d AND `scale` = 1",
-                                                       dbesc($ph->imageString()),
+                                                       dbesc($Image->asString()),
                                                        intval($height),
                                                        intval($width),
                                                        dbesc($resource_id),
@@ -450,12 +437,12 @@ function photos_post(App $a) {
                                        }
 
                                        if ($width > 320 || $height > 320) {
-                                               $ph->scaleImage(320);
-                                               $width  = $ph->getWidth();
-                                               $height = $ph->getHeight();
+                                               $Image->scaleDown(320);
+                                               $width  = $Image->getWidth();
+                                               $height = $Image->getHeight();
 
                                                $x = q("UPDATE `photo` SET `data` = '%s', `height` = %d, `width` = %d WHERE `resource-id` = '%s' AND `uid` = %d AND `scale` = 2",
-                                                       dbesc($ph->imageString()),
+                                                       dbesc($Image->asString()),
                                                        intval($height),
                                                        intval($width),
                                                        dbesc($resource_id),
@@ -470,7 +457,7 @@ function photos_post(App $a) {
                        dbesc($resource_id),
                        intval($page_owner_uid)
                );
-               if (dbm::is_result($p)) {
+               if (DBM::is_result($p)) {
                        $ext = $phototypes[$p[0]['type']];
                        $r = q("UPDATE `photo` SET `desc` = '%s', `album` = '%s', `allow_cid` = '%s', `allow_gid` = '%s', `deny_cid` = '%s', `deny_gid` = '%s' WHERE `resource-id` = '%s' AND `uid` = %d",
                                dbesc($desc),
@@ -485,7 +472,7 @@ function photos_post(App $a) {
 
                        // Update the photo albums cache if album name was changed
                        if ($albname !== $origaname) {
-                               photo_albums($page_owner_uid, true);
+                               Photo::clearAlbumCache($page_owner_uid);
                        }
                }
 
@@ -496,10 +483,8 @@ function photos_post(App $a) {
                        $visibility = 1;
                }
 
-               if (! $item_id) {
-
+               if (!$item_id) {
                        // Create item container
-
                        $title = '';
                        $uri = item_new_uri($a->get_hostname(),$page_owner_uid);
 
@@ -532,7 +517,6 @@ function photos_post(App $a) {
                                                . '[/url]';
 
                        $item_id = item_store($arr);
-
                }
 
                if ($item_id) {
@@ -541,19 +525,17 @@ function photos_post(App $a) {
                                intval($page_owner_uid)
                        );
                }
-               if (dbm::is_result($r)) {
+               if (DBM::is_result($r)) {
                        $old_tag    = $r[0]['tag'];
                        $old_inform = $r[0]['inform'];
                }
 
                if (strlen($rawtags)) {
-
                        $str_tags = '';
                        $inform   = '';
 
                        // if the new tag doesn't have a namespace specifier (@foo or #foo) give it a hashtag
-
-                       $x = substr($rawtags,0,1);
+                       $x = substr($rawtags, 0, 1);
                        if ($x !== '@' && $x !== '#') {
                                $rawtags = '#' . $rawtags;
                        }
@@ -563,10 +545,8 @@ function photos_post(App $a) {
 
                        if (count($tags)) {
                                foreach ($tags as $tag) {
-                                       if (isset($profile)) {
-                                               unset($profile);
-                                       }
                                        if (strpos($tag, '@') === 0) {
+                                               $profile = '';
                                                $name = substr($tag,1);
                                                if ((strpos($name, '@')) || (strpos($name, 'http://'))) {
                                                        $newname = $name;
@@ -608,7 +588,7 @@ function photos_post(App $a) {
                                                                                intval($page_owner_uid)
                                                                );
 
-                                                               if (! dbm::is_result($r)) {
+                                                               if (!DBM::is_result($r)) {
                                                                        //select someone by attag or nick and the name passed in
                                                                        $r = q("SELECT * FROM `contact` WHERE `attag` = '%s' OR `nick` = '%s' AND `uid` = %d ORDER BY `attag` DESC LIMIT 1",
                                                                                        dbesc($name),
@@ -618,7 +598,7 @@ function photos_post(App $a) {
                                                                }
                                                        }
 
-                                                       if (dbm::is_result($r)) {
+                                                       if (DBM::is_result($r)) {
                                                                $newname = $r[0]['name'];
                                                                $profile = $r[0]['url'];
                                                                $notify = 'cid:' . $r[0]['id'];
@@ -684,8 +664,7 @@ function photos_post(App $a) {
 
                        if (count($taginfo)) {
                                foreach ($taginfo as $tagged) {
-
-                                       $uri = item_new_uri($a->get_hostname(),$page_owner_uid);
+                                       $uri = item_new_uri($a->get_hostname(), $page_owner_uid);
 
                                        $arr = array();
                                        $arr['guid']          = get_guid(32);
@@ -730,7 +709,7 @@ function photos_post(App $a) {
 
                                        $item_id = item_store($arr);
                                        if ($item_id) {
-                                               Worker::add(PRIORITY_HIGH, "notifier", "tag", $item_id);
+                                               Worker::add(PRIORITY_HIGH, "Notifier", "tag", $item_id);
                                        }
                                }
                        }
@@ -740,22 +719,16 @@ function photos_post(App $a) {
        }
 
 
-       /**
-        * default post action - upload a photo
-        */
-
+       // default post action - upload a photo
        call_hooks('photo_post_init', $_POST);
 
-       /**
-        * Determine the album to use
-        */
-
-       $album    = notags(trim($_REQUEST['album']));
-       $newalbum = notags(trim($_REQUEST['newalbum']));
+       // Determine the album to use
+       $album    = x($_REQUEST, 'album') ? notags(trim($_REQUEST['album'])) : '';
+       $newalbum = x($_REQUEST, 'newalbum') ? notags(trim($_REQUEST['newalbum'])) : '';
 
        logger('mod/photos.php: photos_post(): album= ' . $album . ' newalbum= ' . $newalbum , LOGGER_DEBUG);
 
-       if (! strlen($album)) {
+       if (!strlen($album)) {
                if (strlen($newalbum)) {
                        $album = $newalbum;
                } else {
@@ -775,57 +748,91 @@ function photos_post(App $a) {
                dbesc($album),
                intval($page_owner_uid)
        );
-       if ((! dbm::is_result($r)) || ($album == t('Profile Photos'))) {
+       if (!DBM::is_result($r) || ($album == t('Profile Photos'))) {
                $visible = 1;
        } else {
                $visible = 0;
        }
 
-       if (intval($_REQUEST['not_visible']) || $_REQUEST['not_visible'] === 'true') {
+       if (x($_REQUEST, 'not_visible') && $_REQUEST['not_visible'] !== 'false') {
                $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'])));
+       $group_allow   = defaults($_REQUEST, 'group_allow'  , []);
+       $contact_allow = defaults($_REQUEST, 'contact_allow', []);
+       $group_deny    = defaults($_REQUEST, 'group_deny'   , []);
+       $contact_deny  = defaults($_REQUEST, 'contact_deny' , []);
+
+       $str_group_allow   = perms2str(is_array($group_allow)   ? $group_allow   : explode(',', $group_allow));
+       $str_contact_allow = perms2str(is_array($contact_allow) ? $contact_allow : explode(',', $contact_allow));
+       $str_group_deny    = perms2str(is_array($group_deny)    ? $group_deny    : explode(',', $group_deny));
+       $str_contact_deny  = perms2str(is_array($contact_deny)  ? $contact_deny  : explode(',', $contact_deny));
 
        $ret = array('src' => '', 'filename' => '', 'filesize' => 0, 'type' => '');
 
-       call_hooks('photo_post_file',$ret);
+       call_hooks('photo_post_file', $ret);
 
-       if (x($ret,'src') && x($ret,'filesize')) {
+       if (x($ret, 'src') && x($ret, 'filesize')) {
                $src      = $ret['src'];
                $filename = $ret['filename'];
                $filesize = $ret['filesize'];
                $type     = $ret['type'];
+               $error    = UPLOAD_ERR_OK;
        } else {
-               $src        = $_FILES['userfile']['tmp_name'];
-               $filename   = basename($_FILES['userfile']['name']);
-               $filesize   = intval($_FILES['userfile']['size']);
-               $type       = $_FILES['userfile']['type'];
+               $src      = $_FILES['userfile']['tmp_name'];
+               $filename = basename($_FILES['userfile']['name']);
+               $filesize = intval($_FILES['userfile']['size']);
+               $type     = $_FILES['userfile']['type'];
+               $error    = $_FILES['userfile']['error'];
        }
+
+       if ($error !== UPLOAD_ERR_OK) {
+               switch ($error) {
+                       case UPLOAD_ERR_INI_SIZE:
+                               notice(t('Image exceeds size limit of %s', ini_get('upload_max_filesize')) . EOL);
+                               break;
+                       case UPLOAD_ERR_FORM_SIZE:
+                               notice(t('Image exceeds size limit of %s', formatBytes(defaults($_REQUEST, 'MAX_FILE_SIZE', 0))) . EOL);
+                               break;
+                       case UPLOAD_ERR_PARTIAL:
+                               notice(t('Image upload didn\'t complete, please try again') . EOL);
+                               break;
+                       case UPLOAD_ERR_NO_FILE:
+                               notice(t('Image file is missing') . EOL);
+                               break;
+                       case UPLOAD_ERR_NO_TMP_DIR:
+                       case UPLOAD_ERR_CANT_WRITE:
+                       case UPLOAD_ERR_EXTENSION:
+                               notice(t('Server can\'t accept new file upload at this time, please contact your administrator') . EOL);
+                               break;
+               }
+               @unlink($src);
+               $foo = 0;
+               call_hooks('photo_post_end', $foo);
+               return;
+       }
+
        if ($type == "") {
-               $type = guess_image_type($filename);
+               $type = Image::guessType($filename);
        }
 
        logger('photos: upload: received file: ' . $filename . ' as ' . $src . ' ('. $type . ') ' . $filesize . ' bytes', LOGGER_DEBUG);
 
-       $maximagesize = Config::get('system','maximagesize');
+       $maximagesize = Config::get('system', 'maximagesize');
 
-       if (($maximagesize) && ($filesize > $maximagesize)) {
-               notice( sprintf(t('Image exceeds size limit of %s'), formatBytes($maximagesize)) . EOL);
+       if ($maximagesize && ($filesize > $maximagesize)) {
+               notice(t('Image exceeds size limit of %s', formatBytes($maximagesize)) . EOL);
                @unlink($src);
                $foo = 0;
-               call_hooks('photo_post_end',$foo);
+               call_hooks('photo_post_end', $foo);
                return;
        }
 
-       if (! $filesize) {
-               notice( t('Image file is empty.') . EOL);
+       if (!$filesize) {
+               notice(t('Image file is empty.') . EOL);
                @unlink($src);
                $foo = 0;
-               call_hooks('photo_post_end',$foo);
+               call_hooks('photo_post_end', $foo);
                return;
        }
 
@@ -833,72 +840,65 @@ function photos_post(App $a) {
 
        $imagedata = @file_get_contents($src);
 
-       $ph = new Photo($imagedata, $type);
+       $Image = new Image($imagedata, $type);
 
-       if (! $ph->is_valid()) {
+       if (!$Image->isValid()) {
                logger('mod/photos.php: photos_post(): unable to process image' , LOGGER_DEBUG);
-               notice( t('Unable to process image.') . EOL );
+               notice(t('Unable to process image.') . EOL);
                @unlink($src);
                $foo = 0;
                call_hooks('photo_post_end',$foo);
                killme();
        }
 
-       $exif = $ph->orient($src);
+       $exif = $Image->orient($src);
        @unlink($src);
 
        $max_length = Config::get('system', 'max_image_length');
-       if (! $max_length) {
+       if (!$max_length) {
                $max_length = MAX_IMAGE_LENGTH;
        }
        if ($max_length > 0) {
-               $ph->scaleImage($max_length);
+               $Image->scaleDown($max_length);
        }
 
-       $width  = $ph->getWidth();
-       $height = $ph->getHeight();
+       $width  = $Image->getWidth();
+       $height = $Image->getHeight();
 
        $smallest = 0;
 
        $photo_hash = photo_new_resource();
 
-       $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);
+       $r = Photo::store($Image, $page_owner_uid, $visitor, $photo_hash, $filename, $album, 0 , 0, $str_contact_allow, $str_group_allow, $str_contact_deny, $str_group_deny);
 
-       if (! $r) {
+       if (!$r) {
                logger('mod/photos.php: photos_post(): image store failed' , LOGGER_DEBUG);
-               notice( t('Image upload failed.') . EOL );
+               notice(t('Image upload failed.') . EOL);
                killme();
        }
 
        if ($width > 640 || $height > 640) {
-               $ph->scaleImage(640);
-               $ph->store($page_owner_uid, $visitor, $photo_hash, $filename, $album, 1, 0, $str_contact_allow, $str_group_allow, $str_contact_deny, $str_group_deny);
+               $Image->scaleDown(640);
+               Photo::store($Image, $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($page_owner_uid, $visitor, $photo_hash, $filename, $album, 2, 0, $str_contact_allow, $str_group_allow, $str_contact_deny, $str_group_deny);
+               $Image->scaleDown(320);
+               Photo::store($Image, $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(), $page_owner_uid);
 
        // Create item container
-
        $lat = $lon = null;
-
-       /// @TODO merge these 2 if() into one?
-       if ($exif && $exif['GPS']) {
-               if (feature_enabled($channel_id,'photo_location')) {
-                       $lat = getGps($exif['GPS']['GPSLatitude'], $exif['GPS']['GPSLatitudeRef']);
-                       $lon = getGps($exif['GPS']['GPSLongitude'], $exif['GPS']['GPSLongitudeRef']);
-               }
+       if ($exif && $exif['GPS'] && Feature::isEnabled($channel_id, 'photo_location')) {
+               $lat = Photo::getGps($exif['GPS']['GPSLatitude'], $exif['GPS']['GPSLatitudeRef']);
+               $lon = Photo::getGps($exif['GPS']['GPSLongitude'], $exif['GPS']['GPSLongitudeRef']);
        }
 
        $arr = array();
-
        if ($lat && $lon) {
                $arr['coord'] = $lat . ' ' . $lon;
        }
@@ -927,30 +927,28 @@ function photos_post(App $a) {
        $arr['origin']        = 1;
 
        $arr['body']          = '[url=' . System::baseUrl() . '/photos/' . $owner_record['nickname'] . '/image/' . $photo_hash . ']'
-                               . '[img]' . System::baseUrl() . "/photo/{$photo_hash}-{$smallest}.".$ph->getExt() . '[/img]'
+                               . '[img]' . System::baseUrl() . "/photo/{$photo_hash}-{$smallest}.".$Image->getExt() . '[/img]'
                                . '[/url]';
 
        $item_id = item_store($arr);
        // Update the photo albums cache
-       photo_albums($page_owner_uid, true);
+       Photo::clearAlbumCache($page_owner_uid);
 
        if ($visible) {
-               Worker::add(PRIORITY_HIGH, "notifier", 'wall-new', $item_id);
+               Worker::add(PRIORITY_HIGH, "Notifier", 'wall-new', $item_id);
        }
 
-       call_hooks('photo_post_end',intval($item_id));
+       call_hooks('photo_post_end', intval($item_id));
 
-       /*
-        * addon uploaders should call "killme()" [e.g. exit] within the photo_post_end hook
-        * if they do not wish to be redirected
-        */
+       // addon uploaders should call "killme()" [e.g. exit] within the photo_post_end hook
+       // if they do not wish to be redirected
 
        goaway($_SESSION['photo_return']);
        // NOTREACHED
 }
 
-function photos_content(App $a) {
-
+function photos_content(App $a)
+{
        // URLs:
        // photos/name
        // photos/name/upload
@@ -960,8 +958,7 @@ function photos_content(App $a) {
        // photos/name/image/xxxxx
        // photos/name/image/xxxxx/edit
 
-
-       if ((Config::get('system', 'block_public')) && (! local_user()) && (! remote_user())) {
+       if (Config::get('system', 'block_public') && !local_user() && !remote_user()) {
                notice( t('Public access denied.') . EOL);
                return;
        }
@@ -970,19 +967,17 @@ function photos_content(App $a) {
        require_once 'include/security.php';
        require_once 'include/conversation.php';
 
-       if (! x($a->data,'user')) {
+       if (!x($a->data,'user')) {
                notice( t('No photos selected') . EOL );
                return;
        }
 
-       $phototypes = Photo::supportedTypes();
+       $phototypes = Image::supportedTypes();
 
        $_SESSION['photo_return'] = $a->cmd;
 
-       //
        // Parse arguments
-       //
-
+       $datum = null;
        if ($a->argc > 3) {
                $datatype = $a->argv[2];
                $datum = $a->argv[3];
@@ -998,10 +993,7 @@ function photos_content(App $a) {
                $cmd = 'view';
        }
 
-       //
        // Setup permissions structures
-       //
-
        $can_post       = false;
        $visitor        = 0;
        $contact        = null;
@@ -1012,7 +1004,7 @@ function photos_content(App $a) {
 
        $community_page = (($a->data['user']['page-flags'] == PAGE_COMMUNITY) ? true : false);
 
-       if ((local_user()) && (local_user() == $owner_uid)) {
+       if (local_user() && (local_user() == $owner_uid)) {
                $can_post = true;
        } else {
                if ($community_page && remote_user()) {
@@ -1030,7 +1022,7 @@ function photos_content(App $a) {
                                        intval($contact_id),
                                        intval($owner_uid)
                                );
-                               if (dbm::is_result($r)) {
+                               if (DBM::is_result($r)) {
                                        $can_post = true;
                                        $contact = $r[0];
                                        $remote_contact = true;
@@ -1040,9 +1032,10 @@ function photos_content(App $a) {
                }
        }
 
-       // perhaps they're visiting - but not a community page, so they wouldn't have write access
+       $groups = [];
 
-       if (remote_user() && (! $visitor)) {
+       // perhaps they're visiting - but not a community page, so they wouldn't have write access
+       if (remote_user() && !$visitor) {
                $contact_id = 0;
                if (is_array($_SESSION['remote'])) {
                        foreach ($_SESSION['remote'] as $v) {
@@ -1053,32 +1046,29 @@ function photos_content(App $a) {
                        }
                }
                if ($contact_id) {
-                       $groups = init_groups_visitor($contact_id);
+                       $groups = Group::getIdsByContactId($contact_id);
                        $r = q("SELECT * FROM `contact` WHERE `blocked` = 0 AND `pending` = 0 AND `id` = %d AND `uid` = %d LIMIT 1",
                                intval($contact_id),
                                intval($owner_uid)
                        );
-                       if (dbm::is_result($r)) {
+                       if (DBM::is_result($r)) {
                                $contact = $r[0];
                                $remote_contact = true;
                        }
                }
        }
 
-       /// @TODO merge these 2 if() into one?
-       if (! $remote_contact) {
-               if (local_user()) {
-                       $contact_id = $_SESSION['cid'];
-                       $contact = $a->contact;
-               }
+       if (!$remote_contact && local_user()) {
+               $contact_id = $_SESSION['cid'];
+               $contact = $a->contact;
        }
 
-       if ($a->data['user']['hidewall'] && (local_user() != $owner_uid) && (! $remote_contact)) {
+       if ($a->data['user']['hidewall'] && (local_user() != $owner_uid) && !$remote_contact) {
                notice( t('Access to this item is restricted.') . EOL);
                return;
        }
 
-       $sql_extra = permissions_sql($owner_uid,$remote_contact,$groups);
+       $sql_extra = permissions_sql($owner_uid, $remote_contact, $groups);
 
        $o = "";
 
@@ -1086,24 +1076,18 @@ function photos_content(App $a) {
        $is_owner = (local_user() && (local_user() == $owner_uid));
        $o .= profile_tabs($a, $is_owner, $a->data['user']['nickname']);
 
-       /**
-        * Display upload form
-        */
-
+       // Display upload form
        if ($datatype === 'upload') {
-               if (! ($can_post)) {
+               if (!$can_post) {
                        notice(t('Permission denied.'));
                        return;
                }
 
-
-               $selname = (($datum) ? hex2bin($datum) : '');
-
+               $selname = $datum ? hex2bin($datum) : '';
 
                $albumselect = '';
 
-
-               $albumselect .= '<option value="" ' . ((! $selname) ? ' selected="selected" ' : '') . '>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</option>';
+               $albumselect .= '<option value="" ' . (!$selname ? ' selected="selected" ' : '') . '>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</option>';
                if (count($a->data['albums'])) {
                        foreach ($a->data['albums'] as $album) {
                                if (($album['album'] === '') || ($album['album'] === 'Contact Photos') || ($album['album'] === t('Contact Photos'))) {
@@ -1129,37 +1113,9 @@ function photos_content(App $a) {
 
                $usage_message = '';
 
-               // Private/public post links for the non-JS ACL form
-               $private_post = 1;
-               if ($_REQUEST['public']) {
-                       $private_post = 0;
-               }
-
-               $query_str = $a->query_string;
-               if (strpos($query_str, 'public=1') !== false) {
-                       $query_str = str_replace(array('?public=1', '&public=1'), array('', ''), $query_str);
-               }
-
-               /*
-                * I think $a->query_string may never have ? in it, but I could be wrong
-                * It looks like it's from the index.php?q=[etc] rewrite that the web
-                * server does, which converts any ? to &, e.g. suggest&ignore=61 for suggest?ignore=61
-                */
-               if (strpos($query_str, '?') === false) {
-                       $public_post_link = '?public=1';
-               } else {
-                       $public_post_link = '&public=1';
-               }
-
                $tpl = get_markup_template('photos_upload.tpl');
 
-               if ($a->theme['template_engine'] === 'internal') {
-                       $albumselect_e = template_escape($albumselect);
-                       $aclselect_e = (($visitor) ? '' : template_escape(populate_acl($a->user)));
-               } else {
-                       $albumselect_e = $albumselect;
-                       $aclselect_e = (($visitor) ? '' : populate_acl($a->user));
-               }
+               $aclselect_e = ($visitor ? '' : populate_acl($a->user));
 
                $o .= replace_macros($tpl,array(
                        '$pagename' => t('Upload Photos'),
@@ -1169,35 +1125,25 @@ function photos_content(App $a) {
                        '$newalbum' => t('New album name: '),
                        '$existalbumtext' => t('or existing album name: '),
                        '$nosharetext' => t('Do not show a status post for this upload'),
-                       '$albumselect' => $albumselect_e,
+                       '$albumselect' => $albumselect,
                        '$permissions' => t('Permissions'),
                        '$aclselect' => $aclselect_e,
                        '$alt_uploader' => $ret['addon_text'],
-                       '$default_upload_box' => (($ret['default_upload']) ? $default_upload_box : ''),
-                       '$default_upload_submit' => (($ret['default_upload']) ? $default_upload_submit : ''),
+                       '$default_upload_box' => ($ret['default_upload'] ? $default_upload_box : ''),
+                       '$default_upload_submit' => ($ret['default_upload'] ? $default_upload_submit : ''),
                        '$uploadurl' => $ret['post_url'],
 
                        // ACL permissions box
-                       '$acl_data' => construct_acl_data($a, $a->user), // For non-Javascript ACL selector
                        '$group_perms' => t('Show to Groups'),
                        '$contact_perms' => t('Show to Contacts'),
-                       '$private' => t('Private Photo'),
-                       '$public' => t('Public Photo'),
-                       '$is_private' => $private_post,
-                       '$return_path' => $query_str,
-                       '$public_link' => $public_post_link,
-
+                       '$return_path' => $a->query_string,
                ));
 
                return $o;
        }
 
-       /*
-        * Display a single photo album
-        */
-
+       // Display a single photo album
        if ($datatype === 'album') {
-
                $album = hex2bin($datum);
 
                $r = q("SELECT `resource-id`, max(`scale`) AS `scale` FROM `photo` WHERE `uid` = %d AND `album` = '%s'
@@ -1205,13 +1151,14 @@ function photos_content(App $a) {
                        intval($owner_uid),
                        dbesc($album)
                );
-               if (dbm::is_result($r)) {
+               if (DBM::is_result($r)) {
                        $a->set_pager_total(count($r));
                        $a->set_pager_itemspage(20);
                }
 
                /// @TODO I have seen this many times, maybe generalize it script-wide and encapsulate it?
-               if ($_GET['order'] === 'posted') {
+               $order_field = defaults($_GET, 'order', '');
+               if ($order_field === 'posted') {
                        $order = 'ASC';
                } else {
                        $order = 'DESC';
@@ -1234,11 +1181,7 @@ function photos_content(App $a) {
                                if ($can_post) {
                                        $edit_tpl = get_markup_template('album_edit.tpl');
 
-                                       if ($a->theme['template_engine'] === 'internal') {
-                                               $album_e = template_escape($album);
-                                       } else {
-                                               $album_e = $album;
-                                       }
+                                       $album_e = $album;
 
                                        $o .= replace_macros($edit_tpl,array(
                                                '$nametext' => t('New album name: '),
@@ -1256,7 +1199,7 @@ function photos_content(App $a) {
                        }
                }
 
-               if ($_GET['order'] === 'posted') {
+               if ($order_field === 'posted') {
                        $order =  array(t('Show Newest First'), 'photos/' . $a->data['user']['nickname'] . '/album/' . bin2hex($album));
                } else {
                        $order = array(t('Show Oldest First'), 'photos/' . $a->data['user']['nickname'] . '/album/' . bin2hex($album) . '?f=&order=posted');
@@ -1264,37 +1207,30 @@ function photos_content(App $a) {
 
                $photos = array();
 
-               if (dbm::is_result($r))
-                       $twist = 'rotright';
+               if (DBM::is_result($r)) {
+                       // "Twist" is only used for the duepunto theme with style "slackr"
+                       $twist = false;
                        foreach ($r as $rr) {
-                               if ($twist == 'rotright') {
-                                       $twist = 'rotleft';
-                               } else {
-                                       $twist = 'rotright';
-                               }
+                               $twist = !$twist;
 
                                $ext = $phototypes[$rr['type']];
 
-                               if ($a->theme['template_engine'] === 'internal') {
-                                       $imgalt_e = template_escape($rr['filename']);
-                                       $desc_e = template_escape($rr['desc']);
-                               } else {
-                                       $imgalt_e = $rr['filename'];
-                                       $desc_e = $rr['desc'];
-                               }
+                               $imgalt_e = $rr['filename'];
+                               $desc_e = $rr['desc'];
 
                                $photos[] = array(
                                        'id' => $rr['id'],
-                                       'twist' => ' ' . $twist . rand(2,4),
+                                       'twist' => ' ' . ($twist ? 'rotleft' : 'rotright') . rand(2,4),
                                        'link' => 'photos/' . $a->data['user']['nickname'] . '/image/' . $rr['resource-id']
-                                               . (($_GET['order'] === 'posted') ? '?f=&order=posted' : ''),
+                                               . ($order_field === 'posted' ? '?f=&order=posted' : ''),
                                        'title' => t('View Photo'),
                                        'src' => 'photo/' . $rr['resource-id'] . '-' . $rr['scale'] . '.' .$ext,
                                        'alt' => $imgalt_e,
                                        'desc'=> $desc_e,
                                        'ext' => $ext,
-                                       'hash'=> $rr['resource_id'],
+                                       'hash'=> $rr['resource-id'],
                                );
+                       }
                }
 
                $tpl = get_markup_template('photo_album.tpl');
@@ -1312,27 +1248,22 @@ function photos_content(App $a) {
 
        }
 
-       /*
-        * Display one photo
-        */
+       // Display one photo
        if ($datatype === 'image') {
-
-               //$o = '';
                // fetch image, item containing image, then comments
-
                $ph = q("SELECT * FROM `photo` WHERE `uid` = %d AND `resource-id` = '%s'
                        $sql_extra ORDER BY `scale` ASC ",
                        intval($owner_uid),
                        dbesc($datum)
                );
 
-               if (! dbm::is_result($ph)) {
+               if (!DBM::is_result($ph)) {
                        $ph = q("SELECT `id` FROM `photo` WHERE `uid` = %d AND `resource-id` = '%s'
                                LIMIT 1",
                                intval($owner_uid),
                                dbesc($datum)
                        );
-                       if (dbm::is_result($ph)) {
+                       if (DBM::is_result($ph)) {
                                notice(t('Permission denied. Access to this item may be restricted.'));
                        } else {
                                notice(t('Photo not available') . EOL );
@@ -1347,10 +1278,12 @@ function photos_content(App $a) {
                // The query leads to a really intense used index.
                // By now we hide it if someone wants to.
                if (!Config::get('system', 'no_count', false)) {
-                       if ($_GET['order'] === 'posted')
+                       $order_field = defaults($_GET, 'order', '');
+                       if ($order_field === 'posted') {
                                $order = 'ASC';
-                       else
+                       } else {
                                $order = 'DESC';
+                       }
 
                        $prvnxt = q("SELECT `resource-id` FROM `photo` WHERE `album` = '%s' AND `uid` = %d AND `scale` = 0
                                $sql_extra ORDER BY `created` $order ",
@@ -1358,7 +1291,7 @@ function photos_content(App $a) {
                                intval($owner_uid)
                        );
 
-                       if (dbm::is_result($prvnxt)) {
+                       if (DBM::is_result($prvnxt)) {
                                foreach ($prvnxt as $z => $entry) {
                                        if ($entry['resource-id'] == $ph[0]['resource-id']) {
                                                $prv = $z - 1;
@@ -1372,9 +1305,9 @@ function photos_content(App $a) {
                                                break;
                                        }
                                }
-                               $edit_suffix = ((($cmd === 'edit') && ($can_post)) ? '/edit' : '');
-                               $prevlink = 'photos/' . $a->data['user']['nickname'] . '/image/' . $prvnxt[$prv]['resource-id'] . $edit_suffix . (($_GET['order'] === 'posted') ? '?f=&order=posted' : '');
-                               $nextlink = 'photos/' . $a->data['user']['nickname'] . '/image/' . $prvnxt[$nxt]['resource-id'] . $edit_suffix . (($_GET['order'] === 'posted') ? '?f=&order=posted' : '');
+                               $edit_suffix = ((($cmd === 'edit') && $can_post) ? '/edit' : '');
+                               $prevlink = 'photos/' . $a->data['user']['nickname'] . '/image/' . $prvnxt[$prv]['resource-id'] . $edit_suffix . ($order_field === 'posted' ? '?f=&order=posted' : '');
+                               $nextlink = 'photos/' . $a->data['user']['nickname'] . '/image/' . $prvnxt[$nxt]['resource-id'] . $edit_suffix . ($order_field === 'posted' ? '?f=&order=posted' : '');
                        }
                }
 
@@ -1448,8 +1381,9 @@ function photos_content(App $a) {
                );
 
                $map = null;
+               $link_item = [];
 
-               if (dbm::is_result($linked_items)) {
+               if (DBM::is_result($linked_items)) {
                        $link_item = $linked_items[0];
 
                        $r = q("SELECT COUNT(*) AS `total`
@@ -1464,7 +1398,7 @@ function photos_content(App $a) {
 
                        );
 
-                       if (dbm::is_result($r)) {
+                       if (DBM::is_result($r)) {
                                $a->set_pager_total($r[0]['total']);
                        }
 
@@ -1487,7 +1421,7 @@ function photos_content(App $a) {
 
                        );
 
-                       if ((local_user()) && (local_user() == $link_item['uid'])) {
+                       if (local_user() && (local_user() == $link_item['uid'])) {
                                q("UPDATE `item` SET `unseen` = 0 WHERE `parent` = %d and `uid` = %d",
                                        intval($link_item['parent']),
                                        intval(local_user())
@@ -1496,7 +1430,7 @@ function photos_content(App $a) {
                        }
 
                        if ($link_item['coord']) {
-                               $map = generate_map($link_item['coord']);
+                               $map = Map::byCoordinates($link_item['coord']);
                        }
                }
 
@@ -1521,40 +1455,12 @@ function photos_content(App $a) {
 
 
                $edit = Null;
-               if (($cmd === 'edit') && ($can_post)) {
+               if ($cmd === 'edit' && $can_post) {
                        $edit_tpl = get_markup_template('photo_edit.tpl');
 
-                       // Private/public post links for the non-JS ACL form
-                       $private_post = 1;
-                       if ($_REQUEST['public']) {
-                               $private_post = 0;
-                       }
-
-                       $query_str = $a->query_string;
-                       if (strpos($query_str, 'public=1') !== false) {
-                               $query_str = str_replace(array('?public=1', '&public=1'), array('', ''), $query_str);
-                       }
-
-                       /*
-                        * I think $a->query_string may never have ? in it, but I could be wrong
-                        * It looks like it's from the index.php?q=[etc] rewrite that the web
-                        * server does, which converts any ? to &, e.g. suggest&ignore=61 for suggest?ignore=61
-                        */
-                       if (strpos($query_str, '?') === false) {
-                               $public_post_link = '?public=1';
-                       } else {
-                               $public_post_link = '&public=1';
-                       }
-
-                       if ($a->theme['template_engine'] === 'internal') {
-                               $album_e = template_escape($ph[0]['album']);
-                               $caption_e = template_escape($ph[0]['desc']);
-                               $aclselect_e = template_escape(populate_acl($ph[0]));
-                       } else {
-                               $album_e = $ph[0]['album'];
-                               $caption_e = $ph[0]['desc'];
-                               $aclselect_e = populate_acl($ph[0]);
-                       }
+                       $album_e = $ph[0]['album'];
+                       $caption_e = $ph[0]['desc'];
+                       $aclselect_e = populate_acl($ph[0]);
 
                        $edit = replace_macros($edit_tpl, array(
                                '$id' => $ph[0]['id'],
@@ -1570,45 +1476,42 @@ function photos_content(App $a) {
                                '$permissions' => t('Permissions'),
                                '$aclselect' => $aclselect_e,
 
-                               '$item_id' => ((count($linked_items)) ? $link_item['id'] : 0),
+                               '$item_id' => defaults($link_item, 'id', 0),
                                '$submit' => t('Submit'),
                                '$delete' => t('Delete Photo'),
 
                                // ACL permissions box
-                               '$acl_data' => construct_acl_data($a, $ph[0]), // For non-Javascript ACL selector
                                '$group_perms' => t('Show to Groups'),
                                '$contact_perms' => t('Show to Contacts'),
-                               '$private' => t('Private photo'),
-                               '$public' => t('Public photo'),
-                               '$is_private' => $private_post,
-                               '$return_path' => $query_str,
-                               '$public_link' => $public_post_link,
+                               '$return_path' => $a->query_string,
                        ));
                }
 
-               if (count($linked_items)) {
+               $like = '';
+               $dislike = '';
+               $likebuttons = '';
+               $comments = '';
+               $paginate = '';
+               $responses = '';
 
+               if (count($linked_items)) {
                        $cmnt_tpl = get_markup_template('comment_item.tpl');
                        $tpl = get_markup_template('photo_item.tpl');
                        $return_url = $a->cmd;
 
-                       $like_tpl = get_markup_template('like_noshare.tpl');
-
-                       $likebuttons = '';
-
-                       if ($can_post || can_write_wall($a, $owner_uid)) {
+                       if ($can_post || can_write_wall($owner_uid)) {
+                               $like_tpl = get_markup_template('like_noshare.tpl');
                                $likebuttons = replace_macros($like_tpl, array(
                                        '$id' => $link_item['id'],
                                        '$likethis' => t("I like this \x28toggle\x29"),
-                                       '$nolike' => (feature_enabled(local_user(), 'dislike') ? t("I don't like this \x28toggle\x29") : ''),
+                                       '$nolike' => (Feature::isEnabled(local_user(), 'dislike') ? t("I don't like this \x28toggle\x29") : ''),
                                        '$wait' => t('Please wait'),
                                        '$return_path' => $a->query_string,
                                ));
                        }
 
-                       $comments = '';
-                       if (! dbm::is_result($r)) {
-                               if (($can_post || can_write_wall($a, $owner_uid)) && $link_item['last-child']) {
+                       if (!DBM::is_result($r)) {
+                               if (($can_post || can_write_wall($owner_uid)) && $link_item['last-child']) {
                                        $comments .= replace_macros($cmnt_tpl, array(
                                                '$return_path' => '',
                                                '$jsreload' => $return_url,
@@ -1629,28 +1532,25 @@ function photos_content(App $a) {
                                }
                        }
 
-                       $alike = array();
-                       $dlike = array();
-
-                       $like = '';
-                       $dislike = '';
-
                        $conv_responses = array(
                                'like' => array('title' => t('Likes','title')),'dislike' => array('title' => t('Dislikes','title')),
                                'attendyes' => array('title' => t('Attending','title')), 'attendno' => array('title' => t('Not attending','title')), 'attendmaybe' => array('title' => t('Might attend','title'))
                        );
 
                        // display comments
-                       if (dbm::is_result($r)) {
-
+                       if (DBM::is_result($r)) {
                                foreach ($r as $item) {
                                        builtin_activity_puller($item, $conv_responses);
                                }
 
-                               $like    = ((x($conv_responses['like'], $link_item['uri'])) ? format_like($conv_responses['like'][$link_item['uri']], $conv_responses['like'][$link_item['uri'] . '-l'], 'like',$link_item['id']) : '');
-                               $dislike = ((x($conv_responses['dislike'], $link_item['uri'])) ? format_like($conv_responses['dislike'][$link_item['uri']], $conv_responses['dislike'][$link_item['uri'] . '-l'], 'dislike',$link_item['id']) : '');
+                               if (x($conv_responses['like'], $link_item['uri'])) {
+                                       $like = format_like($conv_responses['like'][$link_item['uri']], $conv_responses['like'][$link_item['uri'] . '-l'], 'like', $link_item['id']);
+                               }
+                               if (x($conv_responses['dislike'], $link_item['uri'])) {
+                                       $dislike = format_like($conv_responses['dislike'][$link_item['uri']], $conv_responses['dislike'][$link_item['uri'] . '-l'], 'dislike', $link_item['id']);
+                               }
 
-                               if (($can_post || can_write_wall($a, $owner_uid)) && $link_item['last-child']) {
+                               if (($can_post || can_write_wall($owner_uid)) && $link_item['last-child']) {
                                        $comments .= replace_macros($cmnt_tpl,array(
                                                '$return_path' => '',
                                                '$jsreload' => $return_url,
@@ -1670,20 +1570,19 @@ function photos_content(App $a) {
                                        ));
                                }
 
-
                                foreach ($r as $item) {
                                        $comment = '';
                                        $template = $tpl;
                                        $sparkle = '';
 
-                                       if (((activity_match($item['verb'],ACTIVITY_LIKE)) || (activity_match($item['verb'],ACTIVITY_DISLIKE))) && ($item['id'] != $item['parent']))
+                                       if ((activity_match($item['verb'], ACTIVITY_LIKE) || activity_match($item['verb'], ACTIVITY_DISLIKE)) && ($item['id'] != $item['parent'])) {
                                                continue;
+                                       }
 
                                        $redirect_url = 'redir/' . $item['cid'];
 
-
                                        if (local_user() && ($item['contact-uid'] == local_user())
-                                               && ($item['network'] == NETWORK_DFRN) && (! $item['self'] )) {
+                                               && ($item['network'] == NETWORK_DFRN) && !$item['self']) {
                                                $profile_url = $redirect_url;
                                                $sparkle = ' sparkle';
                                        } else {
@@ -1693,8 +1592,8 @@ function photos_content(App $a) {
 
                                        $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_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;
 
@@ -1706,15 +1605,9 @@ function photos_content(App $a) {
                                                'delete' => t('Delete'),
                                        );
 
-                                       if ($a->theme['template_engine'] === 'internal') {
-                                               $name_e = template_escape($profile_name);
-                                               $title_e = template_escape($item['title']);
-                                               $body_e = template_escape(bbcode($item['body']));
-                                       } else {
-                                               $name_e = $profile_name;
-                                               $title_e = $item['title'];
-                                               $body_e = bbcode($item['body']);
-                                       }
+                                       $name_e = $profile_name;
+                                       $title_e = $item['title'];
+                                       $body_e = bbcode($item['body']);
 
                                        $comments .= replace_macros($template,array(
                                                '$id' => $item['item_id'],
@@ -1730,7 +1623,7 @@ function photos_content(App $a) {
                                                '$comment' => $comment
                                        ));
 
-                                       if (($can_post || can_write_wall($a, $owner_uid)) && $item['last-child']) {
+                                       if (($can_post || can_write_wall($owner_uid)) && $item['last-child']) {
                                                $comments .= replace_macros($cmnt_tpl, array(
                                                        '$return_path' => '',
                                                        '$jsreload' => $return_url,
@@ -1751,47 +1644,32 @@ function photos_content(App $a) {
                                        }
                                }
                        }
+                       $response_verbs = array('like');
+                       if (Feature::isEnabled($owner_uid, 'dislike')) {
+                               $response_verbs[] = 'dislike';
+                       }
+                       $responses = get_responses($conv_responses, $response_verbs, '', $link_item);
 
                        $paginate = paginate($a);
                }
 
-
-               $response_verbs = array('like');
-               if (feature_enabled($owner_uid, 'dislike')) {
-                       $response_verbs[] = 'dislike';
-               }
-               $responses = get_responses($conv_responses,$response_verbs, '', $link_item);
-
                $photo_tpl = get_markup_template('photo_view.tpl');
-
-               if ($a->theme['template_engine'] === 'internal') {
-                       $album_e = array($album_link,template_escape($ph[0]['album']));
-                       $tags_e = template_escape($tags);
-                       $like_e = template_escape($like);
-                       $dislike_e = template_escape($dislike);
-               } else {
-                       $album_e = array($album_link, $ph[0]['album']);
-                       $tags_e = $tags;
-                       $like_e = $like;
-                       $dislike_e = $dislike;
-               }
-
                $o .= replace_macros($photo_tpl, array(
                        '$id' => $ph[0]['id'],
-                       '$album' => $album_e,
+                       '$album' => [$album_link, $ph[0]['album']],
                        '$tools' => $tools,
                        '$lock' => $lock,
                        '$photo' => $photo,
                        '$prevlink' => $prevlink,
                        '$nextlink' => $nextlink,
                        '$desc' => $ph[0]['desc'],
-                       '$tags' => $tags_e,
+                       '$tags' => $tags,
                        '$edit' => $edit,
                        '$map' => $map,
                        '$map_text' => t('Map'),
                        '$likebuttons' => $likebuttons,
-                       '$like' => $like_e,
-                       '$dislike' => $dikslike_e,
+                       '$like' => $like,
+                       '$dislike' => $dislike,
                        'responses' => $responses,
                        '$comments' => $comments,
                        '$paginate' => $paginate,
@@ -1815,7 +1693,7 @@ function photos_content(App $a) {
                dbesc('Contact Photos'),
                dbesc( t('Contact Photos'))
        );
-       if (dbm::is_result($r)) {
+       if (DBM::is_result($r)) {
                $a->set_pager_total(count($r));
                $a->set_pager_itemspage(20);
        }
@@ -1833,31 +1711,24 @@ function photos_content(App $a) {
        );
 
        $photos = array();
-       if (dbm::is_result($r)) {
-               $twist = 'rotright';
+       if (DBM::is_result($r)) {
+               // "Twist" is only used for the duepunto theme with style "slackr"
+               $twist = false;
                foreach ($r as $rr) {
                        //hide profile photos to others
-                       if ((! $is_owner) && (! remote_user()) && ($rr['album'] == t('Profile Photos')))
-                                       continue;
+                       if (!$is_owner && !remote_user() && ($rr['album'] == t('Profile Photos')))
+                               continue;
 
-                       if ($twist == 'rotright')
-                               $twist = 'rotleft';
-                       else
-                               $twist = 'rotright';
+                       $twist = !$twist;
 
                        $ext = $phototypes[$rr['type']];
 
-                       if ($a->theme['template_engine'] === 'internal') {
-                               $alt_e = template_escape($rr['filename']);
-                               $name_e = template_escape($rr['album']);
-                       } else {
-                               $alt_e = $rr['filename'];
-                               $name_e = $rr['album'];
-                       }
+                       $alt_e = $rr['filename'];
+                       $name_e = $rr['album'];
 
                        $photos[] = array(
                                'id'            => $rr['id'],
-                               'twist'         => ' ' . $twist . rand(2,4),
+                               'twist'         => ' ' . ($twist ? 'rotleft' : 'rotright') . rand(2,4),
                                'link'          => 'photos/' . $a->data['user']['nickname'] . '/image/' . $rr['resource-id'],
                                'title'         => t('View Photo'),
                                'src'           => 'photo/' . $rr['resource-id'] . '-' . ((($rr['scale']) == 6) ? 4 : $rr['scale']) . '.' . $ext,