]> git.mxchange.org Git - friendica.git/commitdiff
Merge pull request #4502 from annando/guid-from-mail
authorHypolite Petovan <mrpetovan@gmail.com>
Mon, 26 Feb 2018 14:36:56 +0000 (09:36 -0500)
committerGitHub <noreply@github.com>
Mon, 26 Feb 2018 14:36:56 +0000 (09:36 -0500)
guid creation: Take the hostname from mail addresses for mail posts

Vagrantfile
boot.php
database.sql
include/conversation.php
mod/network.php
src/Database/DBStructure.php
util/vagrant_provision.sh

index df019dd06fd25e196f63490b2c0e1405fccdeda8..e62efc618e40c07452a73c68a95d982884883ae4 100644 (file)
@@ -17,7 +17,7 @@ Vagrant.configure(2) do |config|
   # Create a hostname, don't forget to put it to the `hosts` file
   # This will point to the server's default virtual host
   # TO DO: Make this work with virtualhost along-side xip.io URL
-  config.vm.hostname = "friendica-xenial.dev"
+  config.vm.hostname = "friendica.dev"
 
   # Create a static IP
   config.vm.network :private_network, ip: server_ip
index 2a8bf1806a4489b9a55a75fcf8a494949f45cebf..472814a6db546d9a07e41732bc43533982249a69 100644 (file)
--- a/boot.php
+++ b/boot.php
@@ -39,7 +39,7 @@ define('FRIENDICA_PLATFORM',     'Friendica');
 define('FRIENDICA_CODENAME',     'Asparagus');
 define('FRIENDICA_VERSION',      '3.6-dev');
 define('DFRN_PROTOCOL_VERSION',  '2.23');
-define('DB_UPDATE_VERSION',      1254);
+define('DB_UPDATE_VERSION',      1255);
 define('NEW_UPDATE_ROUTINE_VERSION', 1170);
 
 /**
index a1c3e31e41eae77a39935d18d3a323e22717de63..4275c38315ccf26cd82c81b1df9dbcd3c93614de 100644 (file)
@@ -1,6 +1,6 @@
 -- ------------------------------------------
 -- Friendica 3.6-dev (Asparagus)
--- DB_UPDATE_VERSION 1254
+-- DB_UPDATE_VERSION 1255
 -- ------------------------------------------
 
 
@@ -512,6 +512,7 @@ CREATE TABLE IF NOT EXISTS `item` (
         INDEX `uid_id` (`uid`,`id`),
         INDEX `uid_contactid_id` (`uid`,`contact-id`,`id`),
         INDEX `uid_created` (`uid`,`created`),
+        INDEX `uid_commented` (`uid`,`commented`),
         INDEX `uid_unseen_contactid` (`uid`,`unseen`,`contact-id`),
         INDEX `uid_network_received` (`uid`,`network`,`received`),
         INDEX `uid_network_commented` (`uid`,`network`,`commented`),
index bc0a7586827bb2537a8e52cb5ed69927cb0aa6ad..b262ee4c828b498734a7e9b7e2e96c903674318b 100644 (file)
@@ -496,7 +496,7 @@ function item_condition() {
  * that are based on unique features of the calling module.
  *
  */
-function conversation(App $a, $items, $mode, $update, $preview = false) {
+function conversation(App $a, $items, $mode, $update, $preview = false, $order = 'commented') {
        require_once 'mod/proxy.php';
 
        $ssl_state = ((local_user()) ? true : false);
@@ -520,6 +520,7 @@ function conversation(App $a, $items, $mode, $update, $preview = false) {
        $previewing = (($preview) ? ' preview ' : '');
 
        if ($mode === 'network') {
+               $items = conversation_add_children($items, false, $order);
                $profile_owner = local_user();
                if (!$update) {
                        /*
@@ -580,7 +581,7 @@ function conversation(App $a, $items, $mode, $update, $preview = false) {
                                . " var profile_page = 1; </script>";
                }
        } elseif ($mode === 'community') {
-               $items = community_add_items($items);
+               $items = conversation_add_children($items, true, $order);
                $profile_owner = 0;
                if (!$update) {
                        $live_update_div = '<div id="live-community"></div>' . "\r\n"
@@ -899,15 +900,16 @@ function conversation(App $a, $items, $mode, $update, $preview = false) {
  *
  * @return array items with parents and comments
  */
-function community_add_items($parents) {
+function conversation_add_children($parents, $block_authors, $order) {
        $max_comments = Config::get("system", "max_comments", 100);
 
        $items = [];
 
+       $block_sql = $block_authors ? "AND NOT `author`.`hidden` AND NOT `author`.`blocked`" : "";
+
        foreach ($parents AS $parent) {
                $thread_items = dba::p(item_query()." AND `item`.`uid` = ?
-                       AND `item`.`parent-uri` = ?
-                       AND NOT `author`.`hidden` AND NOT `author`.`blocked`
+                       AND `item`.`parent-uri` = ? $block_sql
                        ORDER BY `item`.`commented` DESC LIMIT ".intval($max_comments + 1),
                        local_user(),
                        $parent['uri']
@@ -949,7 +951,7 @@ function community_add_items($parents) {
                }
        }
 
-       $items = conv_sort($items, "`commented`");
+       $items = conv_sort($items, $order);
 
        return $items;
 }
index 3013e94984101597289ee99d94b1f101e20493c2..fc3bdf0d689802a18df130f6eb59b5902df95df5 100644 (file)
@@ -340,12 +340,12 @@ function networkSetSeen($condition)
  * @param integer $update Used for the automatic reloading
  * @return string HTML of the conversation
  */
-function networkConversation($a, $items, $mode, $update)
+function networkConversation($a, $items, $mode, $update, $ordering = '')
 {
        // Set this so that the conversation function can find out contact info for our wall-wall items
        $a->page_contact = $a->contact;
 
-       $o = conversation($a, $items, $mode, $update);
+       $o = conversation($a, $items, $mode, $update, $ordering);
 
        if (!$update) {
                if (PConfig::get(local_user(), 'system', 'infinite_scroll')) {
@@ -764,9 +764,10 @@ function networkThreadedView(App $a, $update = 0)
                } else {
                        $sql_extra4 = '';
                }
-               $r = q("SELECT `item`.`parent` AS `item_id`, `item`.`network` AS `item_network`, `contact`.`uid` AS `contact_uid`, $sql_order AS `order_date`
-                       FROM $sql_table $sql_post_table INNER JOIN `contact` ON `contact`.`id` = `item`.`contact-id`
-                       AND (NOT `contact`.`blocked` OR `contact`.`pending`)
+               $r = q("SELECT `item`.`uri`, `item`.`parent` AS `item_id`, $sql_order AS `order_date`
+                       FROM `item` $sql_post_table
+                       STRAIGHT_JOIN `contact` ON `contact`.`id` = `item`.`contact-id`
+                               AND (NOT `contact`.`blocked` OR `contact`.`pending`)
                        WHERE `item`.`uid` = %d AND `item`.`visible` AND NOT `item`.`deleted` $sql_extra4
                        AND NOT `item`.`moderated` AND `item`.`unseen`
                        $sql_extra3 $sql_extra $sql_range $sql_nets
@@ -774,13 +775,15 @@ function networkThreadedView(App $a, $update = 0)
                        intval(local_user())
                );
        } else {
-               $r = q("SELECT `thread`.`iid` AS `item_id`, `thread`.`network` AS `item_network`, `contact`.`uid` AS `contact_uid`, $sql_order AS `order_date`
-                       FROM $sql_table $sql_post_table STRAIGHT_JOIN `contact` ON `contact`.`id` = `thread`.`contact-id`
-                       AND (NOT `contact`.`blocked` OR `contact`.`pending`)
+               $r = q("SELECT `item`.`uri`, `thread`.`iid` AS `item_id`, $sql_order AS `order_date`
+                       FROM `thread` $sql_post_table
+                       STRAIGHT_JOIN `contact` ON `contact`.`id` = `thread`.`contact-id`
+                               AND (NOT `contact`.`blocked` OR `contact`.`pending`)
+                       STRAIGHT_JOIN `item` ON `item`.`id` = `thread`.`iid`
                        WHERE `thread`.`uid` = %d AND `thread`.`visible` AND NOT `thread`.`deleted`
                        AND NOT `thread`.`moderated`
                        $sql_extra2 $sql_extra3 $sql_range $sql_extra $sql_nets
-                       ORDER BY $sql_order DESC $pager_sql",
+                       ORDER BY `order_date` DESC $pager_sql",
                        intval(local_user())
                );
        }
@@ -790,16 +793,24 @@ function networkThreadedView(App $a, $update = 0)
                if (DBM::is_result($r)) {
                        $top_limit = current($r)['order_date'];
                        $bottom_limit = end($r)['order_date'];
+                       if (empty($_SESSION['network_last_top_limit']) || ($_SESSION['network_last_top_limit'] < $top_limit)) {
+                               $_SESSION['network_last_top_limit'] = $top_limit;
+                       }
                } else {
-                       $top_limit = DateTimeFormat::utcNow();
-                       $bottom_limit = DateTimeFormat::utcNow();
+                       $top_limit = $bottom_limit = DateTimeFormat::utcNow();
                }
 
                // When checking for updates we need to fetch from the newest date to the newest date before
-               if ($update && !empty($_SESSION['network_last_date']) && ($bottom_limit > $_SESSION['network_last_date'])) {
+               // Only do this, when the last stored date isn't too long ago (10 times the update interval)
+               $browser_update = PConfig::get(local_user(), 'system', 'update_interval', 40000) / 1000;
+
+               if (($browser_update > 0) && $update && !empty($_SESSION['network_last_date']) &&
+                       (($bottom_limit < $_SESSION['network_last_date']) || ($top_limit == $bottom_limit)) &&
+                       ((time() - $_SESSION['network_last_date_timestamp']) < ($browser_update * 10))) {
                        $bottom_limit = $_SESSION['network_last_date'];
                }
-               $_SESSION['network_last_date'] = $top_limit;
+               $_SESSION['network_last_date'] = defaults($_SESSION, 'network_last_top_limit', $top_limit);
+               $_SESSION['network_last_date_timestamp'] = time();
 
                if ($last_date > $top_limit) {
                        $top_limit = $last_date;
@@ -808,58 +819,50 @@ function networkThreadedView(App $a, $update = 0)
                        $top_limit = DateTimeFormat::utcNow();
                }
 
-               $items = dba::p("SELECT `item`.`id` AS `item_id`, `item`.`network` AS `item_network`, `contact`.`uid` AS `contact_uid` FROM `item`
-                       INNER JOIN (SELECT `oid` FROM `term` WHERE `term` IN
+               $items = dba::p("SELECT `item`.`uri`, `item`.`id` AS `item_id`, `item`.$ordering AS `order_date` FROM `item`
+                       STRAIGHT_JOIN (SELECT `oid` FROM `term` WHERE `term` IN
                                (SELECT SUBSTR(`term`, 2) FROM `search` WHERE `uid` = ? AND `term` LIKE '#%') AND `otype` = ? AND `type` = ? AND `uid` = 0) AS `term`
                        ON `item`.`id` = `term`.`oid`
-                       INNER JOIN `contact` ON `contact`.`id` = `item`.`contact-id`
-                       INNER JOIN `contact` AS `author` ON `author`.`id`=`item`.`author-id`
+                       STRAIGHT_JOIN `contact` ON `contact`.`id` = `item`.`author-id`
                        WHERE `item`.`uid` = 0 AND `item`.$ordering < ? AND `item`.$ordering > ?
-                               AND NOT `author`.`hidden` AND NOT `author`.`blocked`" . $sql_tag_nets,
+                               AND NOT `contact`.`hidden` AND NOT `contact`.`blocked`" . $sql_tag_nets,
                        local_user(), TERM_OBJ_POST, TERM_HASHTAG, $top_limit, $bottom_limit);
+
                $data = dba::inArray($items);
 
                if (count($data) > 0) {
-                       logger('Tagged items: ' . count($data) . ' - ' . $bottom_limit . ' - ' . $top_limit . ' - ' . local_user()); //$last_date);
-                       $r = array_merge($data, $r);
+                       $tag_top_limit = current($data)['order_date'];
+                       if ($_SESSION['network_last_date'] < $tag_top_limit) {
+                               $_SESSION['network_last_date'] = $tag_top_limit;
+                       }
+
+                       logger('Tagged items: ' . count($data) . ' - ' . $bottom_limit . ' - ' . $top_limit . ' - ' . local_user().' - '.(int)$update);
+                       $s = [];
+                       foreach ($r as $item) {
+                               $s[$item['uri']] = $item;
+                       }
+                       foreach ($data as $item) {
+                               $s[$item['uri']] = $item;
+                       }
+                       $r = $s;
                }
        }
 
-       // Then fetch all the children of the parents that are on this page
-
-       $parents_arr = [];
        $parents_str = '';
        $date_offset = '';
 
-       $items = [];
-       if (DBM::is_result($r)) {
-               foreach ($r as $rr) {
-                       if (!in_array($rr['item_id'], $parents_arr)) {
-                               $parents_arr[] = $rr['item_id'];
-                       }
-               }
+       $items = $r;
 
-               $parents_str = implode(', ', $parents_arr);
-
-               // splitted into separate queries to avoid the problem with very long threads
-               // so always the last X comments are loaded
-               // This problem can occur expecially with imported facebook posts
-               $max_comments = Config::get('system', 'max_comments');
-               if ($max_comments == 0) {
-                       $max_comments = 100;
-               }
+       if (DBM::is_result($items)) {
+               $parents_arr = [];
 
-               foreach ($parents_arr AS $parents) {
-                       $thread_items = dba::p(item_query() . " AND `item`.`parent` = ?
-                               ORDER BY `item`.`commented` DESC LIMIT " . intval($max_comments + 1),
-                               $parents
-                       );
-
-                       if (DBM::is_result($thread_items)) {
-                               $items = array_merge($items, dba::inArray($thread_items));
+               foreach ($items as $item) {
+                       if (!in_array($item['item_id'], $parents_arr)) {
+                               $parents_arr[] = $item['item_id'];
                        }
                }
-               $items = conv_sort($items, $ordering);
+
+               $parents_str = implode(', ', $parents_arr);
        }
 
        if (x($_GET, 'offset')) {
@@ -886,7 +889,7 @@ function networkThreadedView(App $a, $update = 0)
 
 
        $mode = 'network';
-       $o .= networkConversation($a, $items, $mode, $update);
+       $o .= networkConversation($a, $items, $mode, $update, $ordering);
 
        return $o;
 }
index 0b5eca1ae05dda654fa7e590d348456721c81168..f4a88871d7a0c8032b5f874232c51f3ba28e637f 100644 (file)
@@ -1191,6 +1191,7 @@ class DBStructure
                                                "uid_id" => ["uid","id"],
                                                "uid_contactid_id" => ["uid","contact-id","id"],
                                                "uid_created" => ["uid","created"],
+                                               "uid_commented" => ["uid","commented"],
                                                "uid_unseen_contactid" => ["uid","unseen","contact-id"],
                                                "uid_network_received" => ["uid","network","received"],
                                                "uid_network_commented" => ["uid","network","commented"],
index 2029d97d58807cbfede8e29fb071cc11cff3569b..7ca5a1f4e3d33e7f1ec327249e66160ee01ae7df 100755 (executable)
@@ -92,5 +92,6 @@ echo "*/10 * * * * cd /vagrant; /usr/bin/php scripts/worker.php" >> friendicacro
 sudo crontab friendicacron
 sudo rm friendicacron
 
-#Optional: checkout addon repositroy
-#sudo git clone https://github.com/friendica/friendica-addons.git /vagrant/addon
+# friendica needs write access to /tmp
+sudo chmod 777 /tmp
+