]> git.mxchange.org Git - friendica.git/blobdiff - src/Model/Event.php
Fixes issue 5322 - events now again are having a plink
[friendica.git] / src / Model / Event.php
index 9c35e34e45ae00856e2d5d38ebd762165d13bad9..49d759585a20bc513d865014dd21e383638f42eb 100644 (file)
@@ -29,7 +29,7 @@ class Event extends BaseObject
 
        public static function getHTML(array $event, $simple = false)
        {
-               if (!is_array($event) || !!count($event)) {
+               if (empty($event)) {
                        return '';
                }
 
@@ -105,7 +105,7 @@ class Event extends BaseObject
         * @param array $event Array which contains the event data.
         * @return string The event as a bbcode formatted string.
         */
-       public static function getBBCode(array $event)
+       private static function getBBCode(array $event)
        {
                $o = '';
 
@@ -223,17 +223,18 @@ class Event extends BaseObject
         *
         * Store the event in the event table and create an event item in the item table.
         *
-        * @param array $event Array with event data.
-        * @return int The event id.
+        * @param array $arr Array with event data.
+        * @return int The new event id.
         */
        public static function store($arr)
        {
                $a = self::getApp();
 
-               $event['uri']       =        defaults($arr, 'uri'      , item_new_uri($a->get_hostname(), $event['uid']));
+               $event = [];
                $event['id']        = intval(defaults($arr, 'id'       , 0));
                $event['uid']       = intval(defaults($arr, 'uid'      , 0));
                $event['cid']       = intval(defaults($arr, 'cid'      , 0));
+               $event['uri']       =        defaults($arr, 'uri'      , Item::newURI($event['uid']));
                $event['type']      =        defaults($arr, 'type'     , 'event');
                $event['summary']   =        defaults($arr, 'summary'  , '');
                $event['desc']      =        defaults($arr, 'desc'     , '');
@@ -242,7 +243,6 @@ class Event extends BaseObject
                $event['allow_gid'] =        defaults($arr, 'allow_gid', '');
                $event['deny_cid']  =        defaults($arr, 'deny_cid' , '');
                $event['deny_gid']  =        defaults($arr, 'deny_gid' , '');
-               $event['private']   = intval(defaults($arr, 'private'  , 0));
                $event['adjust']    = intval(defaults($arr, 'adjust'   , 0));
                $event['nofinish']  = intval(defaults($arr, 'nofinish' , !empty($event['start']) && empty($event['finish'])));
 
@@ -253,21 +253,24 @@ class Event extends BaseObject
                if ($event['finish'] < NULL_DATE) {
                        $event['finish'] = NULL_DATE;
                }
+               $private = intval(defaults($arr, 'private', 0));
 
-               $condition = ['uid' => $event['uid']];
+               $conditions = ['uid' => $event['uid']];
                if ($event['cid']) {
-                       $condition['id'] = $event['cid'];
+                       $conditions['id'] = $event['cid'];
+               } else {
+                       $conditions['self'] = true;
                }
 
-               $contact = dba::selectFirst('contact', [], ['id' => $event['cid'], 'uid' => $event['uid']]);
+               $contact = dba::selectFirst('contact', [], $conditions);
 
                // Existing event being modified.
                if ($event['id']) {
                        // has the event actually changed?
                        $existing_event = dba::selectFirst('event', ['edited'], ['id' => $event['id'], 'uid' => $event['uid']]);
-                       if ((! DBM::is_result($existing_event)) || ($existing_event['edited'] === $event['edited'])) {
+                       if (!DBM::is_result($existing_event) || ($existing_event['edited'] === $event['edited'])) {
 
-                               $item = dba::selectFirst('item', [], ['event-id' => $event['id'], 'uid' => $event['uid']]);
+                               $item = Item::selectFirst(['id'], ['event-id' => $event['id'], 'uid' => $event['uid']]);
 
                                return DBM::is_result($item) ? $item['id'] : 0;
                        }
@@ -284,15 +287,15 @@ class Event extends BaseObject
                                'nofinish' => $event['nofinish'],
                        ];
 
-                       dba::update('event', $updated_fields, ['id' => $event['cid'], 'uid' => $event['uid']]);
+                       dba::update('event', $updated_fields, ['id' => $event['id'], 'uid' => $event['uid']]);
 
-                       $item = dba::selectFirst('item', ['id'], ['event-id' => $event['id'], 'uid' => $event['uid']]);
+                       $item = Item::selectFirst(['id'], ['event-id' => $event['id'], 'uid' => $event['uid']]);
                        if (DBM::is_result($item)) {
                                $object = '<object><type>' . xmlify(ACTIVITY_OBJ_EVENT) . '</type><title></title><id>' . xmlify($event['uri']) . '</id>';
-                               $object .= '<content>' . xmlify(format_event_bbcode($event)) . '</content>';
+                               $object .= '<content>' . xmlify(self::getBBCode($event)) . '</content>';
                                $object .= '</object>' . "\n";
 
-                               $fields = ['body' => format_event_bbcode($event), 'object' => $object, 'edited' => $event['edited']];
+                               $fields = ['body' => self::getBBCode($event), 'object' => $object, 'edited' => $event['edited']];
                                Item::update($fields, ['id' => $item['id']]);
 
                                $item_id = $item['id'];
@@ -330,16 +333,16 @@ class Event extends BaseObject
                        $item_arr['allow_gid']     = $event['allow_gid'];
                        $item_arr['deny_cid']      = $event['deny_cid'];
                        $item_arr['deny_gid']      = $event['deny_gid'];
-                       $item_arr['private']       = $event['private'];
+                       $item_arr['private']       = $private;
                        $item_arr['visible']       = 1;
                        $item_arr['verb']          = ACTIVITY_POST;
                        $item_arr['object-type']   = ACTIVITY_OBJ_EVENT;
                        $item_arr['origin']        = $event['cid'] === 0 ? 1 : 0;
-                       $item_arr['body']          = format_event_bbcode($event);
+                       $item_arr['body']          = self::getBBCode($event);
                        $item_arr['event-id']      = $event['id'];
 
                        $item_arr['object']  = '<object><type>' . xmlify(ACTIVITY_OBJ_EVENT) . '</type><title></title><id>' . xmlify($event['uri']) . '</id>';
-                       $item_arr['object'] .= '<content>' . xmlify(format_event_bbcode($event)) . '</content>';
+                       $item_arr['object'] .= '<content>' . xmlify(self::getBBCode($event)) . '</content>';
                        $item_arr['object'] .= '</object>' . "\n";
 
                        $item_id = Item::insert($item_arr);
@@ -429,7 +432,7 @@ class Event extends BaseObject
         *
         * @todo We should replace this with a separate update function if there is some time left.
         */
-       public static function removeDuplicates(array $dates)
+       private static function removeDuplicates(array $dates)
        {
                $dates2 = [];
 
@@ -461,8 +464,7 @@ class Event extends BaseObject
                }
 
                // Query for the event by event id
-               $r = q("SELECT `event`.*, `item`.`id` AS `itemid`,`item`.`plink`,
-                               `item`.`author-name`, `item`.`author-avatar`, `item`.`author-link` FROM `event`
+               $r = q("SELECT `event`.*, `item`.`id` AS `itemid` FROM `event`
                        LEFT JOIN `item` ON `item`.`event-id` = `event`.`id` AND `item`.`uid` = `event`.`uid`
                        WHERE `event`.`uid` = %d AND `event`.`id` = %d $sql_extra",
                        intval($owner_uid),
@@ -470,7 +472,7 @@ class Event extends BaseObject
                );
 
                if (DBM::is_result($r)) {
-                       $return = event_remove_duplicates($r);
+                       $return = self::removeDuplicates($r);
                }
 
                return $return;
@@ -501,8 +503,8 @@ class Event extends BaseObject
                }
 
                // Query for the event by date.
-               $r = q("SELECT `event`.*, `item`.`id` AS `itemid`,`item`.`plink`,
-                                       `item`.`author-name`, `item`.`author-avatar`, `item`.`author-link` FROM `event`
+               // @todo Slow query (518 seconds to run), to be optimzed
+               $r = q("SELECT `event`.*, `item`.`id` AS `itemid` FROM `event`
                                LEFT JOIN `item` ON `item`.`event-id` = `event`.`id` AND `item`.`uid` = `event`.`uid`
                                WHERE `event`.`uid` = %d AND event.ignore = %d
                                AND ((`adjust` = 0 AND (`finish` >= '%s' OR (nofinish AND start >= '%s')) AND `start` <= '%s')
@@ -519,7 +521,7 @@ class Event extends BaseObject
                );
 
                if (DBM::is_result($r)) {
-                       $return = event_remove_duplicates($r);
+                       $return = self::removeDuplicates($r);
                }
 
                return $return;
@@ -538,6 +540,11 @@ class Event extends BaseObject
                $last_date = '';
                $fmt = L10n::t('l, F j');
                foreach ($event_result as $event) {
+                       $item = Item::selectFirst(['plink', 'author-name', 'author-avatar', 'author-link'], ['id' => $event['itemid']]);
+                       if (DBM::is_result($item)) {
+                               $event = array_merge($event, $item);
+                       }
+
                        $start = $event['adjust'] ? DateTimeFormat::local($event['start'], 'c')  : DateTimeFormat::utc($event['start'], 'c');
                        $j     = $event['adjust'] ? DateTimeFormat::local($event['start'], 'j')  : DateTimeFormat::utc($event['start'], 'j');
                        $day   = $event['adjust'] ? DateTimeFormat::local($event['start'], $fmt) : DateTimeFormat::utc($event['start'], $fmt);
@@ -570,7 +577,7 @@ class Event extends BaseObject
                                $title = strip_tags(html_entity_decode($title, ENT_QUOTES, 'UTF-8'));
                        }
 
-                       $html = format_event_html($event);
+                       $html = self::getHTML($event);
                        $event['desc']     = BBCode::convert($event['desc']);
                        $event['location'] = BBCode::convert($event['location']);
                        $event_list[] = [
@@ -605,7 +612,7 @@ class Event extends BaseObject
         *
         * @todo Implement timezone support
         */
-       public static function formatListForExport(array $events, $format, $timezone)
+       private static function formatListForExport(array $events, $format, $timezone)
        {
                if (!count($events)) {
                        return '';
@@ -712,7 +719,7 @@ class Event extends BaseObject
         *
         * @return array Query results.
         */
-       public static function getListByUserId($uid = 0)
+       private static function getListByUserId($uid = 0)
        {
                $return = [];
 
@@ -733,7 +740,7 @@ class Event extends BaseObject
 
                $events = dba::select('event', $fields, $conditions);
                if (DBM::is_result($events)) {
-                       $return = $events;
+                       $return = dba::inArray($events);
                }
 
                return $return;
@@ -761,11 +768,11 @@ class Event extends BaseObject
                }
 
                // Get all events which are owned by a uid (respects permissions).
-               $events = events_by_uid($uid);
+               $events = self::getListByUserId($uid);
 
                // We have the events that are available for the requestor.
                // Now format the output according to the requested format.
-               $res = event_format_export($events, $format, $timezone);
+               $res = self::formatListForExport($events, $format, $timezone);
 
                // If there are results the precess was successfull.
                if (!empty($res)) {
@@ -799,10 +806,10 @@ class Event extends BaseObject
        /**
         * @brief Format an item array with event data to HTML.
         *
-        * @param arr $item Array with item and event data.
+        * @param array $item Array with item and event data.
         * @return string HTML output.
         */
-       public static function getItemHTML($item) {
+       public static function getItemHTML(array $item) {
                $same_date = false;
                $finish    = false;
 
@@ -861,15 +868,10 @@ class Event extends BaseObject
                }
 
                // Format the event location.
-               $location = event_location2array($item['event-location']);
+               $location = self::locationToArray($item['event-location']);
 
                // Construct the profile link (magic-auth).
-               $sp = false;
-               $profile_link = best_link_url($item, $sp);
-
-               if (!$sp) {
-                       $profile_link = Profile::zrl($profile_link);
-               }
+               $profile_link = Contact::magicLinkById($item['author-id']);
 
                $tpl = get_markup_template('event_stream_item.tpl');
                $return = replace_macros($tpl, [
@@ -916,7 +918,7 @@ class Event extends BaseObject
         * 'address' => The address of the location,<br>
         * 'coordinates' => Latitude‎ and longitude‎ (e.g. '48.864716,2.349014').<br>
         */
-       public static function locationToArray($s = '') {
+       private static function locationToArray($s = '') {
                if ($s == '') {
                        return [];
                }