]> git.mxchange.org Git - friendica.git/blobdiff - mod/videos.php
Remove unreliable ANY_VALUE from message list query
[friendica.git] / mod / videos.php
index 69adf1091f085d87a0ca9edb8159c0dfff68d845..9e64321f3ed52d2bc295c9cbcfa0b1af54405f32 100644 (file)
@@ -5,28 +5,28 @@
 
 use Friendica\App;
 use Friendica\Content\Nav;
+use Friendica\Content\Pager;
 use Friendica\Core\Config;
 use Friendica\Core\L10n;
+use Friendica\Core\Renderer;
 use Friendica\Core\System;
-use Friendica\Core\Worker;
-use Friendica\Database\DBM;
+use Friendica\Database\DBA;
 use Friendica\Model\Contact;
 use Friendica\Model\Group;
+use Friendica\Model\Item;
 use Friendica\Model\Profile;
 use Friendica\Protocol\DFRN;
-use Friendica\Util\DateTimeFormat;
+use Friendica\Util\Security;
 
 require_once 'include/items.php';
-require_once 'include/acl_selectors.php';
-require_once 'include/bbcode.php';
-require_once 'include/security.php';
 
-function videos_init(App $a) {
-
-       if($a->argc > 1)
+function videos_init(App $a)
+{
+       if ($a->argc > 1) {
                DFRN::autoRedir($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,14 +34,15 @@ function videos_init(App $a) {
 
        $o = '';
 
-       if($a->argc > 1) {
+       if ($a->argc > 1) {
                $nick = $a->argv[1];
                $user = q("SELECT * FROM `user` WHERE `nickname` = '%s' AND `blocked` = 0 LIMIT 1",
-                       dbesc($nick)
+                       DBA::escape($nick)
                );
 
-               if(! count($user))
+               if (!DBA::isResult($user)) {
                        return;
+               }
 
                $a->data['user'] = $user[0];
                $a->profile_uid = $user[0]['uid'];
@@ -50,9 +51,9 @@ function videos_init(App $a) {
 
                $account_type = Contact::getAccountType($profile);
 
-               $tpl = get_markup_template("vcard-widget.tpl");
+               $tpl = Renderer::getMarkupTemplate("vcard-widget.tpl");
 
-               $vcard_widget = replace_macros($tpl, [
+               $vcard_widget = Renderer::replaceMacros($tpl, [
                        '$name' => $profile['name'],
                        '$photo' => $profile['photo'],
                        '$addr' => defaults($profile, 'addr', ''),
@@ -60,8 +61,8 @@ function videos_init(App $a) {
                        '$pdesc' => defaults($profile, 'pdesc', ''),
                ]);
 
-
-               /*$sql_extra = permissions_sql($a->data['user']['uid']);
+               /// @TODO Old-lost code?
+               /*$sql_extra = Security::getPermissionsSQLByUserId($a->data['user']['uid']);
 
                $albums = q("SELECT distinct(`album`) AS `album` FROM `photo` WHERE `uid` = %d $sql_extra order by created desc",
                        intval($a->data['user']['uid'])
@@ -70,7 +71,7 @@ function videos_init(App $a) {
                if(count($albums)) {
                        $a->data['albums'] = $albums;
 
-                       $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);
 
                        if($albums_visible) {
                                $o .= '<div id="sidebar-photos-albums" class="widget">';
@@ -82,7 +83,7 @@ function videos_init(App $a) {
                                        // don't show contact photos. We once translated this name, but then you could still access it under
                                        // a different language setting. Now we store the name in English and check in English (and translated for legacy albums).
 
-                                       if((! strlen($album['album'])) || ($album['album'] === 'Contact Photos') || ($album['album'] === L10n::t('Contact Photos')))
+                                       if((!strlen($album['album'])) || ($album['album'] === 'Contact Photos') || ($album['album'] === L10n::t('Contact Photos')))
                                                continue;
                                        $o .= '<li>' . '<a href="photos/' . $a->argv[1] . '/album/' . bin2hex($album['album']) . '" >' . $album['album'] . '</a></li>';
                                }
@@ -95,51 +96,45 @@ function videos_init(App $a) {
                        $o .= '</div>';
                }*/
 
-               if(! x($a->page,'aside'))
+               // If not there, create 'aside' empty
+               if (!isset($a->page['aside'])) {
                        $a->page['aside'] = '';
-               $a->page['aside'] .= $vcard_widget;
-
+               }
 
-               $tpl = get_markup_template("videos_head.tpl");
-               $a->page['htmlhead'] .= replace_macros($tpl,[
-                       '$baseurl' => System::baseUrl(),
-               ]);
+               $a->page['aside'] .= $vcard_widget;
 
-               $tpl = get_markup_template("videos_end.tpl");
-               $a->page['end'] .= replace_macros($tpl,[
+               $tpl = Renderer::getMarkupTemplate("videos_head.tpl");
+               $a->page['htmlhead'] .= Renderer::replaceMacros($tpl,[
                        '$baseurl' => System::baseUrl(),
                ]);
-
        }
 
        return;
 }
 
-
-
-function videos_post(App $a) {
-
+function videos_post(App $a)
+{
        $owner_uid = $a->data['user']['uid'];
 
        if (local_user() != $owner_uid) {
-               goaway(System::baseUrl() . '/videos/' . $a->data['user']['nickname']);
+               $a->internalRedirect('videos/' . $a->data['user']['nickname']);
        }
 
-       if (($a->argc == 2) && x($_POST,'delete') && x($_POST, 'id')) {
-
+       if (($a->argc == 2) && !empty($_POST['delete']) && !empty($_POST['id'])) {
                // Check if we should do HTML-based delete confirmation
-               if (!x($_REQUEST,'confirm')) {
-                       if (x($_REQUEST,'canceled')) {
-                               goaway(System::baseUrl() . '/videos/' . $a->data['user']['nickname']);
+               if (empty($_REQUEST['confirm'])) {
+                       if (!empty($_REQUEST['canceled'])) {
+                               $a->internalRedirect('videos/' . $a->data['user']['nickname']);
                        }
 
                        $drop_url = $a->query_string;
-                       $a->page['content'] = replace_macros(get_markup_template('confirm.tpl'), [
+
+                       $a->page['content'] = Renderer::replaceMacros(Renderer::getMarkupTemplate('confirm.tpl'), [
                                '$method' => 'post',
                                '$message' => L10n::t('Do you really want to delete this video?'),
                                '$extra_inputs' => [
-                                       ['name'=>'id', 'value'=> $_POST['id']],
-                                       ['name'=>'delete', 'value'=>'x']
+                                       ['name' => 'id'    , 'value' => $_POST['id']],
+                                       ['name' => 'delete', 'value' => 'x']
                                ],
                                '$confirm' => L10n::t('Delete Video'),
                                '$confirm_url' => $drop_url,
@@ -147,7 +142,9 @@ function videos_post(App $a) {
                                '$cancel' => L10n::t('Cancel'),
 
                        ]);
+
                        $a->error = 1; // Set $a->error so the other module functions don't execute
+
                        return;
                }
 
@@ -155,50 +152,34 @@ function videos_post(App $a) {
 
                $r = q("SELECT `id`  FROM `attach` WHERE `uid` = %d AND `id` = '%s' LIMIT 1",
                        intval(local_user()),
-                       dbesc($video_id)
+                       DBA::escape($video_id)
                );
 
-               if (DBM::is_result($r)) {
+               if (DBA::isResult($r)) {
                        q("DELETE FROM `attach` WHERE `uid` = %d AND `id` = '%s'",
                                intval(local_user()),
-                               dbesc($video_id)
+                               DBA::escape($video_id)
                        );
-                       $i = q("SELECT * FROM `item` WHERE `attach` like '%%attach/%s%%' AND `uid` = %d LIMIT 1",
-                               dbesc($video_id),
+
+                       $i = q("SELECT `id` FROM `item` WHERE `attach` like '%%attach/%s%%' AND `uid` = %d LIMIT 1",
+                               DBA::escape($video_id),
                                intval(local_user())
                        );
-                       //echo "<pre>"; var_dump($i); killme();
-                       if (DBM::is_result($i)) {
-                               q("UPDATE `item` SET `deleted` = 1, `edited` = '%s', `changed` = '%s' WHERE `parent-uri` = '%s' AND `uid` = %d",
-                                       dbesc(DateTimeFormat::utcNow()),
-                                       dbesc(DateTimeFormat::utcNow()),
-                                       dbesc($i[0]['uri']),
-                                       intval(local_user())
-                               );
-                               create_tags_from_itemuri($i[0]['uri'], local_user());
-                               delete_thread_uri($i[0]['uri'], local_user());
-
-                               $url = System::baseUrl();
-                               $drop_id = intval($i[0]['id']);
-
-                               if ($i[0]['visible']) {
-                                       Worker::add(PRIORITY_HIGH, "Notifier", "drop", $drop_id);
-                               }
+
+                       if (DBA::isResult($i)) {
+                               Item::deleteForUser(['id' => $i[0]['id']], local_user());
                        }
                }
 
-               goaway(System::baseUrl() . '/videos/' . $a->data['user']['nickname']);
+               $a->internalRedirect('videos/' . $a->data['user']['nickname']);
                return; // NOTREACHED
        }
 
-       goaway(System::baseUrl() . '/videos/' . $a->data['user']['nickname']);
-
+       $a->internalRedirect('videos/' . $a->data['user']['nickname']);
 }
 
-
-
-function videos_content(App $a) {
-
+function videos_content(App $a)
+{
        // URLs (most aren't currently implemented):
        // videos/name
        // videos/name/upload
@@ -209,17 +190,14 @@ function videos_content(App $a) {
        // videos/name/video/xxxxx/edit
 
 
-       if((Config::get('system','block_public')) && (! local_user()) && (! remote_user())) {
+       if ((Config::get('system', 'block_public')) && (!local_user()) && (!remote_user())) {
                notice(L10n::t('Public access denied.') . EOL);
                return;
        }
 
+       require_once 'include/conversation.php';
 
-       require_once('include/bbcode.php');
-       require_once('include/security.php');
-       require_once('include/conversation.php');
-
-       if(! x($a->data,'user')) {
+       if (empty($a->data['user'])) {
                notice(L10n::t('No videos selected') . EOL );
                return;
        }
@@ -231,25 +209,24 @@ function videos_content(App $a) {
        //
        // Parse arguments
        //
-
-       if($a->argc > 3) {
+       if ($a->argc > 3) {
                $datatype = $a->argv[2];
                $datum = $a->argv[3];
-       }
-       elseif(($a->argc > 2) && ($a->argv[2] === 'upload'))
+       } elseif(($a->argc > 2) && ($a->argv[2] === 'upload')) {
                $datatype = 'upload';
-       else
+       } else {
                $datatype = 'summary';
+       }
 
-       if($a->argc > 4)
+       if ($a->argc > 4) {
                $cmd = $a->argv[4];
-       else
+       } else {
                $cmd = 'view';
+       }
 
        //
        // Setup permissions structures
        //
-
        $can_post       = false;
        $visitor        = 0;
        $contact        = null;
@@ -258,32 +235,31 @@ function videos_content(App $a) {
 
        $owner_uid = $a->data['user']['uid'];
 
-       $community_page = (($a->data['user']['page-flags'] == PAGE_COMMUNITY) ? true : false);
+       $community_page = (($a->data['user']['page-flags'] == Contact::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()) {
-                       if(is_array($_SESSION['remote'])) {
-                               foreach($_SESSION['remote'] as $v) {
-                                       if($v['uid'] == $owner_uid) {
-                                               $contact_id = $v['cid'];
-                                               break;
-                                       }
+       } elseif ($community_page && remote_user()) {
+               if (!empty($_SESSION['remote'])) {
+                       foreach ($_SESSION['remote'] as $v) {
+                               if ($v['uid'] == $owner_uid) {
+                                       $contact_id = $v['cid'];
+                                       break;
                                }
                        }
-                       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($owner_uid)
-                               );
-                               if (DBM::is_result($r)) {
-                                       $can_post = true;
-                                       $contact = $r[0];
-                                       $remote_contact = true;
-                                       $visitor = $contact_id;
-                               }
+               }
+
+               if ($contact_id > 0) {
+                       $r = q("SELECT `uid` FROM `contact` WHERE `blocked` = 0 AND `pending` = 0 AND `id` = %d AND `uid` = %d LIMIT 1",
+                               intval($contact_id),
+                               intval($owner_uid)
+                       );
+
+                       if (DBA::isResult($r)) {
+                               $can_post = true;
+                               $contact = $r[0];
+                               $remote_contact = true;
+                               $visitor = $contact_id;
                        }
                }
        }
@@ -291,9 +267,10 @@ function videos_content(App $a) {
        $groups = [];
 
        // perhaps they're visiting - but not a community page, so they wouldn't have write access
-       if(remote_user() && (! $visitor)) {
+       if (remote_user() && (!$visitor)) {
                $contact_id = 0;
-               if(is_array($_SESSION['remote'])) {
+
+               if (!empty($_SESSION['remote'])) {
                        foreach($_SESSION['remote'] as $v) {
                                if($v['uid'] == $owner_uid) {
                                        $contact_id = $v['cid'];
@@ -301,32 +278,32 @@ function videos_content(App $a) {
                                }
                        }
                }
-               if($contact_id) {
+
+               if ($contact_id > 0) {
                        $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 (DBA::isResult($r)) {
                                $contact = $r[0];
                                $remote_contact = true;
                        }
                }
        }
 
-       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(L10n::t('Access to this item is restricted.') . EOL);
                return;
        }
 
-       $sql_extra = permissions_sql($owner_uid, $remote_contact, $groups);
+       $sql_extra = Security::getPermissionsSQLByUserId($owner_uid, $remote_contact, $groups);
 
        $o = "";
 
@@ -337,24 +314,20 @@ function videos_content(App $a) {
        //
        // dispatch request
        //
-
-
-       if($datatype === 'upload') {
+       if ($datatype === 'upload') {
                return; // no uploading for now
 
                // DELETED -- look at mod/photos.php if you want to implement
        }
 
-       if($datatype === 'album') {
-
+       if ($datatype === 'album') {
                return; // no albums for now
 
                // DELETED -- look at mod/photos.php if you want to implement
        }
 
 
-       if($datatype === 'video') {
-
+       if ($datatype === 'video') {
                return; // no single video view for now
 
                // DELETED -- look at mod/photos.php if you want to implement
@@ -363,36 +336,39 @@ function videos_content(App $a) {
        // Default - show recent videos (no upload link for now)
        //$o = '';
 
+       $total = 0;
        $r = q("SELECT hash FROM `attach` WHERE `uid` = %d AND filetype LIKE '%%video%%'
                $sql_extra GROUP BY hash",
                intval($a->data['user']['uid'])
        );
-       if (DBM::is_result($r)) {
-               $a->set_pager_total(count($r));
-               $a->set_pager_itemspage(20);
+       if (DBA::isResult($r)) {
+               $total = count($r);
        }
 
+       $pager = new Pager($a->query_string, 20);
+
        $r = q("SELECT hash, ANY_VALUE(`id`) AS `id`, ANY_VALUE(`created`) AS `created`,
                ANY_VALUE(`filename`) AS `filename`, ANY_VALUE(`filetype`) as `filetype`
                FROM `attach`
                WHERE `uid` = %d AND filetype LIKE '%%video%%'
                $sql_extra GROUP BY hash ORDER BY `created` DESC LIMIT %d , %d",
                intval($a->data['user']['uid']),
-               intval($a->pager['start']),
-               intval($a->pager['itemspage'])
+               $pager->getStart(),
+               $pager->getItemsPerPage()
        );
 
-
-
        $videos = [];
-       if (DBM::is_result($r)) {
+
+       if (DBA::isResult($r)) {
                foreach ($r as $rr) {
                        $alt_e = $rr['filename'];
+                       /// @todo The album isn't part of the above query. This seems to be some unfinished code that needs to be reworked completely.
+                       $rr['album'] = '';
                        $name_e = $rr['album'];
 
                        $videos[] = [
                                'id'       => $rr['id'],
-                               'link'     => System::baseUrl() . '/videos/' . $a->data['user']['nickname'] . '/video/' . $rr['resource-id'],
+                               'link'     => System::baseUrl() . '/videos/' . $a->data['user']['nickname'] . '/video/' . $rr['hash'],
                                'title'    => L10n::t('View Video'),
                                'src'      => System::baseUrl() . '/attach/' . $rr['id'] . '?attachment=0',
                                'alt'      => $alt_e,
@@ -402,21 +378,20 @@ function videos_content(App $a) {
                                        'name'  => $name_e,
                                        'alt'   => L10n::t('View Album'),
                                ],
-
                        ];
                }
        }
 
-       $tpl = get_markup_template('videos_recent.tpl');
-       $o .= replace_macros($tpl, [
+       $tpl = Renderer::getMarkupTemplate('videos_recent.tpl');
+       $o .= Renderer::replaceMacros($tpl, [
                '$title'      => L10n::t('Recent Videos'),
                '$can_post'   => $can_post,
-               '$upload'     => [L10n::t('Upload New Videos'), System::baseUrl().'/videos/'.$a->data['user']['nickname'].'/upload'],
+               '$upload'     => [L10n::t('Upload New Videos'), System::baseUrl() . '/videos/' . $a->data['user']['nickname'] . '/upload'],
                '$videos'     => $videos,
-               '$delete_url' => (($can_post)?System::baseUrl().'/videos/'.$a->data['user']['nickname']:False)
+               '$delete_url' => (($can_post) ? System::baseUrl() . '/videos/' . $a->data['user']['nickname'] : false)
        ]);
 
+       $o .= $pager->renderFull($total);
 
-       $o .= paginate($a);
        return $o;
 }