Bugfix: probing failed when a profile was hidden

[friendica.git] / include / event.php

diff --git a/include/event.php b/include/event.php
index c826511f2307e0bac8b9d0bd6b78d561ece33877..a68e1c4626d37cf523711387fbd9539511317135 100644 (file)
--- a/include/event.php
+++ b/include/event.php
@@ -293,7 +293,7 @@ function event_store($arr) {
                        `location` = '%s',
                        `type` = '%s',
                        `adjust` = %d,
-                       `nofinish` = %d,
+                       `nofinish` = %d
                        WHERE `id` = %d AND `uid` = %d",
 
                        dbesc($arr['edited']),
@@ -673,7 +673,7 @@ function event_format_export ($events, $format = 'ical', $timezone) {
                                        $dtformat = "%Y%m%dT%H%M%S".$UTC;
                                        $o .= 'DTSTART:'.strftime($dtformat, $tmp).PHP_EOL;
                                }
-                               if ($event['finish']) {
+                               if (!$event['nofinish']) {
                                        $tmp = strtotime($event['finish']);
                                        $dtformat = "%Y%m%dT%H%M%S".$UTC;
                                        $o .= 'DTEND:'.strftime($dtformat, $tmp).PHP_EOL;
@@ -732,13 +732,13 @@ function events_by_uid($uid = 0, $sql_extra = '') {
        //  requested? then show all of your events, otherwise only those that 
        //  don't have limitations set in allow_cid and allow_gid
        if (local_user() == $uid) {
-               $r = q("SELECT `start`, `finish`, `adjust`, `summary`, `desc`, `location`
+               $r = q("SELECT `start`, `finish`, `adjust`, `summary`, `desc`, `location`, `nofinish`
                        FROM `event` WHERE `uid`= %d AND `cid` = 0 ",
                        intval($uid)
                );
        } else {
-               $r = q("SELECT `start`, `finish`, `adjust`, `summary`, `desc`, `location`FROM `event`
-                       WHERE  `uid` = %d AND `cid` = 0 $sql_extra ",
+               $r = q("SELECT `start`, `finish`, `adjust`, `summary`, `desc`, `location`, `nofinish`
+                       FROM `event` WHERE `uid`= %d AND `cid` = 0 $sql_extra ",
                        intval($uid)
                );
        }
@@ -818,7 +818,25 @@ function widget_events() {
        // of the profile page it should be the personal /events page. So we can use $a->user
        $user = ($a->data['user']['nickname'] ? $a->data['user']['nickname'] : $a->user['nickname']);
 
-       if( !(local_user()) && !(feature_enabled($owner_uid, "export_calendar")) )
+
+       // The permission testing is a little bit tricky because we have to respect many cases
+
+       // It's not the private events page (we don't get the $owner_uid for /events)
+       if(! local_user() && ! $owner_uid)
+               return;
+
+       // Cal logged in user (test permission at foreign profile page)
+       // If the $owner uid is available we know it is part of one of the profile pages (like /cal)
+       // So we have to test if if it's the own profile page of the logged in user 
+       // or a foreign one. For foreign profile pages we need to check if the feature
+       // for exporting the cal is enabled (otherwise the widget would appear for logged in users
+       // on foreigen profile pages even if the widget is disabled)
+       if(intval($owner_uid) && local_user() !== $owner_uid && ! feature_enabled($owner_uid, "export_calendar")) 
+               return;
+
+       // If it's a kind of profile page (intval($owner_uid)) return if the user not logged in and
+       // export feature isn't enabled
+       if(intval($owner_uid) && ! local_user() && ! feature_enabled($owner_uid, "export_calendar"))
                return;
 
        return replace_macros(get_markup_template("events_aside.tpl"), array(