mirror of
https://codeberg.org/streams/streams.git
synced 2024-09-19 23:55:19 +00:00
begin the process of using the relevant attach directory/path for photo albums instead of an album basename which may not be unique. Created an 'ellipsify()' function to shorten long names and keep the beginning and end intact
This commit is contained in:
zotlabs
Mario Vavti
parent
542fa4a08c
commit
b03cd330e5
4 changed files with 62 additions and 48 deletions
|
|
@ -92,13 +92,9 @@ class Photos extends \Zotlabs\Web\Controller {
|
|||
|
||||
if((argc() > 3) && (argv(2) === 'album')) {
|
||||
|
||||
$album = hex2bin(argv(3));
|
||||
|
||||
if($album === t('Profile Photos')) {
|
||||
// not allowed
|
||||
goaway(z_root() . '/' . $_SESSION['photo_return']);
|
||||
}
|
||||
|
||||
$album = argv(3);
|
||||
|
||||
|
||||
if(! photos_album_exists($page_owner_uid,$album)) {
|
||||
notice( t('Album not found.') . EOL);
|
||||
goaway(z_root() . '/' . $_SESSION['photo_return']);
|
||||
|
|
@ -680,29 +676,16 @@ class Photos extends \Zotlabs\Web\Controller {
|
|||
|
||||
if($datatype === 'album') {
|
||||
|
||||
if(strlen($datum)) {
|
||||
if((strlen($datum) & 1) || (! ctype_xdigit($datum))) {
|
||||
notice( t('Album name could not be decoded') . EOL);
|
||||
logger('mod_photos: illegal album encoding: ' . $datum);
|
||||
$datum = '';
|
||||
goaway(z_root() . '/photos/' . \App::$data['channel']['channel_address']);
|
||||
}
|
||||
}
|
||||
|
||||
$album = (($datum) ? hex2bin($datum) : '');
|
||||
$album = $datum;
|
||||
|
||||
\App::$page['htmlhead'] .= "\r\n" . '<link rel="alternate" type="application/json+oembed" href="' . z_root() . '/oep?f=&url=' . urlencode(z_root() . '/' . \App::$cmd) . '" title="oembed" />' . "\r\n";
|
||||
|
||||
//check if the album exists and if we have perms
|
||||
$r = q("SELECT album FROM photo WHERE uid = %d AND album = '%s' and is_nsfw = %d $sql_extra LIMIT 1",
|
||||
intval($owner_uid),
|
||||
dbesc($album),
|
||||
intval($unsafe)
|
||||
);
|
||||
|
||||
if($r) {
|
||||
if($x = photos_album_exists($owner_uid,$datum)) {
|
||||
\App::set_pager_itemspage(60);
|
||||
} else {
|
||||
$album = $x['display_path'];
|
||||
}
|
||||
else {
|
||||
goaway(z_root() . '/photos/' . \App::$data['channel']['channel_address']);
|
||||
}
|
||||
|
||||
|
|
@ -712,19 +695,19 @@ class Photos extends \Zotlabs\Web\Controller {
|
|||
$order = 'DESC';
|
||||
|
||||
$r = q("SELECT p.resource_id, p.id, p.filename, p.mimetype, p.imgscale, p.description, p.created FROM photo p INNER JOIN
|
||||
(SELECT resource_id, max(imgscale) imgscale FROM photo WHERE uid = %d AND album = '%s' AND imgscale <= 4 AND photo_usage IN ( %d, %d ) and is_nsfw = %d $sql_extra GROUP BY resource_id) ph
|
||||
(SELECT resource_id, max(imgscale) imgscale FROM photo left join attach on folder = '%s' and photo.resource_id = attach.hash WHERE attach.uid = %d AND imgscale <= 4 AND photo_usage IN ( %d, %d ) and is_nsfw = %d $sql_extra GROUP BY resource_id) ph
|
||||
ON (p.resource_id = ph.resource_id AND p.imgscale = ph.imgscale)
|
||||
ORDER BY created $order LIMIT %d OFFSET %d",
|
||||
dbesc($datum),
|
||||
intval($owner_uid),
|
||||
dbesc($album),
|
||||
intval(PHOTO_NORMAL),
|
||||
intval(PHOTO_PROFILE),
|
||||
intval($unsafe),
|
||||
intval(\App::$pager['itemspage']),
|
||||
intval(\App::$pager['start'])
|
||||
);
|
||||
|
||||
//edit album name
|
||||
|
||||
// edit album name
|
||||
$album_edit = null;
|
||||
if(($album !== t('Profile Photos')) && ($album !== 'Profile Photos') && ($album !== 'Contact Photos') && ($album !== t('Contact Photos'))) {
|
||||
if($can_post) {
|
||||
|
|
@ -749,9 +732,9 @@ class Photos extends \Zotlabs\Web\Controller {
|
|||
}
|
||||
|
||||
if($_GET['order'] === 'posted')
|
||||
$order = array(t('Show Newest First'), z_root() . '/photos/' . \App::$data['channel']['channel_address'] . '/album/' . bin2hex($album));
|
||||
$order = array(t('Show Newest First'), z_root() . '/photos/' . \App::$data['channel']['channel_address'] . '/album/' . $datum);
|
||||
else
|
||||
$order = array(t('Show Oldest First'), z_root() . '/photos/' . \App::$data['channel']['channel_address'] . '/album/' . bin2hex($album) . '?f=&order=posted');
|
||||
$order = array(t('Show Oldest First'), z_root() . '/photos/' . \App::$data['channel']['channel_address'] . '/album/' . $datum . '?f=&order=posted');
|
||||
|
||||
$photos = array();
|
||||
if(count($r)) {
|
||||
|
|
|
|||
|
|
@ -446,7 +446,7 @@ function photo_upload($channel, $observer, $args) {
|
|||
* * \e boolean \b success
|
||||
* * \e array \b albums
|
||||
*/
|
||||
function photos_albums_list($channel, $observer, $sort_key = 'album', $direction = 'asc') {
|
||||
function photos_albums_list($channel, $observer, $sort_key = 'display_path', $direction = 'asc') {
|
||||
|
||||
$channel_id = $channel['channel_id'];
|
||||
$observer_xchan = (($observer) ? $observer['xchan_hash'] : '');
|
||||
|
|
@ -459,16 +459,31 @@ function photos_albums_list($channel, $observer, $sort_key = 'album', $direction
|
|||
$sort_key = dbesc($sort_key);
|
||||
$direction = dbesc($direction);
|
||||
|
||||
//$albums = q("SELECT count( distinct resource_id ) as total, album from photo where uid = %d and photo_usage IN ( %d, %d ) $sql_extra group by album order by $sort_key $direction",
|
||||
// intval($channel_id),
|
||||
// intval(PHOTO_NORMAL),
|
||||
// intval(PHOTO_PROFILE)
|
||||
//);
|
||||
|
||||
// this query provides the same results but might perform better
|
||||
$albums = q("SELECT count( distinct resource_id ) as total, album from photo where uid = %d and os_storage = 1 $sql_extra group by album order by $sort_key $direction",
|
||||
$r = q("select display_path, hash from attach where is_dir = 1 and uid = %d order by $sort_key $direction",
|
||||
intval($channel_id)
|
||||
);
|
||||
array_unshift($r,[ 'display_path' => '/', 'hash' => '' ]);
|
||||
$str = ids_to_querystr($r,'hash',true);
|
||||
|
||||
$albums = [];
|
||||
|
||||
if($str) {
|
||||
$x = q("select count( distinct hash ) as total, folder from attach where is_photo = 1 and uid = %d and folder in ( $str ) group by folder ",
|
||||
intval($channel_id)
|
||||
);
|
||||
if($x) {
|
||||
foreach($r as $rv) {
|
||||
foreach($x as $xv) {
|
||||
if($xv['folder'] === $rv['hash']) {
|
||||
if($xv['total'] != 0) {
|
||||
$albums[] = [ 'album' => $rv['display_path'], 'folder' => $xv['folder'], 'total' => $xv['total'] ];
|
||||
}
|
||||
continue;
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
// add various encodings to the array so we can just loop through and pick them out in a template
|
||||
|
||||
|
|
@ -480,11 +495,12 @@ function photos_albums_list($channel, $observer, $sort_key = 'album', $direction
|
|||
foreach($albums as $k => $album) {
|
||||
$entry = array(
|
||||
'text' => (($album['album']) ? $album['album'] : '/'),
|
||||
'shorttext' => (($album['album']) ? ellipsify($album['album'],28) : '/'),
|
||||
'jstext' => (($album['album']) ? addslashes($album['album']) : '/'),
|
||||
'total' => $album['total'],
|
||||
'url' => z_root() . '/photos/' . $channel['channel_address'] . '/album/' . bin2hex($album['album']),
|
||||
'url' => z_root() . '/photos/' . $channel['channel_address'] . '/album/' . $album['folder'],
|
||||
'urlencode' => urlencode($album['album']),
|
||||
'bin2hex' => bin2hex($album['album'])
|
||||
'bin2hex' => $album['folder']
|
||||
);
|
||||
$ret['albums'][] = $entry;
|
||||
}
|
||||
|
|
@ -495,7 +511,7 @@ function photos_albums_list($channel, $observer, $sort_key = 'album', $direction
|
|||
return $ret;
|
||||
}
|
||||
|
||||
function photos_album_widget($channelx,$observer,$sortkey = 'album',$direction = 'asc') {
|
||||
function photos_album_widget($channelx,$observer,$sortkey = 'display_path',$direction = 'asc') {
|
||||
|
||||
$o = '';
|
||||
|
||||
|
|
@ -508,6 +524,7 @@ function photos_album_widget($channelx,$observer,$sortkey = 'album',$direction =
|
|||
$o = replace_macros(get_markup_template('photo_albums.tpl'),array(
|
||||
'$nick' => $channelx['channel_address'],
|
||||
'$title' => t('Photo Albums'),
|
||||
'$recent' => t('Recent Photos'),
|
||||
'$albums' => $albums['albums'],
|
||||
'$baseurl' => z_root(),
|
||||
'$upload' => ((perm_is_allowed($channelx['channel_id'],(($observer) ? $observer['xchan_hash'] : ''),'write_storage'))
|
||||
|
|
@ -566,12 +583,15 @@ function photos_list_photos($channel, $observer, $album = '') {
|
|||
* @return boolean
|
||||
*/
|
||||
function photos_album_exists($channel_id, $album) {
|
||||
$r = q("SELECT id FROM photo WHERE album = '%s' AND uid = %d limit 1",
|
||||
|
||||
$sql_extra = permissions_sql($channel_id);
|
||||
|
||||
$r = q("SELECT folder, hash, is_dir, filename, os_path, display_path FROM attach WHERE hash = '%s' AND is_dir = 1 AND uid = %d $sql_extra limit 1",
|
||||
dbesc($album),
|
||||
intval($channel_id)
|
||||
);
|
||||
|
||||
return (($r) ? true : false);
|
||||
return (($r) ? $r[0] : false);
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
|
|||
|
|
@ -3130,3 +3130,14 @@ function array_escape_tags(&$v,$k) {
|
|||
$v = escape_tags($v);
|
||||
}
|
||||
|
||||
function ellipsify($s,$maxlen) {
|
||||
if($maxlen & 1)
|
||||
$maxlen --;
|
||||
if($maxlen < 4)
|
||||
$maxlen = 4;
|
||||
|
||||
if(mb_strlen($s) < $maxlen)
|
||||
return $s;
|
||||
|
||||
return mb_substr($s,0,$maxlen / 2) . '...' . mb_substr($s,mb_strlen($s) - ($maxlen / 2));
|
||||
}
|
||||
|
|
@ -1,11 +1,11 @@
|
|||
<div id="side-bar-photos-albums" class="widget">
|
||||
<h3>{{$title}}</h3>
|
||||
<ul class="nav nav-pills flex-column">
|
||||
<li class="nav-item"><a class="nav-link"href="{{$baseurl}}/photos/{{$nick}}" title="{{$title}}" >Recent Photos</a></li>
|
||||
<li class="nav-item"><a class="nav-link" href="{{$baseurl}}/photos/{{$nick}}" title="{{$title}}" >{{$recent_photos}}</a></li>
|
||||
{{if $albums}}
|
||||
{{foreach $albums as $al}}
|
||||
{{if $al.text}}
|
||||
<li class="nav-item"><a class="nav-link" href="{{$baseurl}}/photos/{{$nick}}/album/{{$al.bin2hex}}"><span class="badge badge-default float-right">{{$al.total}}</span>{{$al.text}}</a></li>
|
||||
{{if $al.shorttext}}
|
||||
<li class="nav-item"><a class="nav-link" href="{{$baseurl}}/photos/{{$nick}}/album/{{$al.bin2hex}}"><span class="badge badge-default float-right">{{$al.total}}</span>{{$al.shorttext}}</a></li>
|
||||
{{/if}}
|
||||
{{/foreach}}
|
||||
{{/if}}
|
||||
|
|
|
|||
Loading…
Reference in a new issue