X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=mod%2Fevents.php;h=ebb7d3f114649f045fd4ab3c737625644fd31345;hb=eb035771f118e1f289f2ebbbb8a474aa2765c81e;hp=69f6b6f326853cf2923379c94d982ce50008d57d;hpb=71820db023f769afa22a5f1362913b8e22199cf1;p=friendica.git diff --git a/mod/events.php b/mod/events.php index 69f6b6f326..ebb7d3f114 100644 --- a/mod/events.php +++ b/mod/events.php @@ -1,6 +1,6 @@ argc > 1 && $a->argv[1] === 'json') { + if (DI::args()->getArgc() > 1 && DI::args()->getArgv()[1] === 'json') { return; } @@ -56,7 +59,7 @@ function events_init(App $a) DI::page()['aside'] = ''; } - $cal_widget = CalendarExport::getHTML(); + $cal_widget = CalendarExport::getHTML(local_user()); DI::page()['aside'] .= $cal_widget; @@ -65,9 +68,7 @@ function events_init(App $a) function events_post(App $a) { - Logger::debug('post', ['request' => $_REQUEST]); - if (!local_user()) { return; } @@ -82,6 +83,8 @@ function events_post(App $a) $adjust = intval($_POST['adjust'] ?? 0); $nofinish = intval($_POST['nofinish'] ?? 0); + $share = intval($_POST['share'] ?? 0); + // The default setting for the `private` field in event_store() is false, so mirror that $private_event = false; @@ -129,7 +132,7 @@ function events_post(App $a) ]; $action = ($event_id == '') ? 'new' : 'event/' . $event_id; - $onerror_path = 'events/' . $action . '?' . http_build_query($params, null, null, PHP_QUERY_RFC3986); + $onerror_path = 'events/' . $action . '?' . http_build_query($params, '', '&', PHP_QUERY_RFC3986); if (strcmp($finish, $start) < 0 && !$nofinish) { notice(DI::l10n()->t('Event can not end before it has started.')); @@ -149,45 +152,42 @@ function events_post(App $a) DI::baseUrl()->redirect($onerror_path); } - $share = intval($_POST['share'] ?? 0); - - $c = q("SELECT `id` FROM `contact` WHERE `uid` = %d AND `self` LIMIT 1", - intval(local_user()) - ); - - if (DBA::isResult($c)) { - $self = $c[0]['id']; - } else { - $self = 0; - } + $self = \Friendica\Model\Contact::getPublicIdByUserId($uid); + $aclFormatter = DI::aclFormatter(); if ($share) { - - $aclFormatter = DI::aclFormatter(); - - $str_group_allow = $aclFormatter->toString($_POST['group_allow'] ?? ''); - $str_contact_allow = $aclFormatter->toString($_POST['contact_allow'] ?? ''); - $str_group_deny = $aclFormatter->toString($_POST['group_deny'] ?? ''); - $str_contact_deny = $aclFormatter->toString($_POST['contact_deny'] ?? ''); - - // Undo the pseudo-contact of self, since there are real contacts now - if (strpos($str_contact_allow, '<' . $self . '>') !== false) { - $str_contact_allow = str_replace('<' . $self . '>', '', $str_contact_allow); + $user = User::getById($uid, ['allow_cid', 'allow_gid', 'deny_cid', 'deny_gid']); + if (!DBA::isResult($user)) { + return; } - // Make sure to set the `private` field as true. This is necessary to - // have the posts show up correctly in Diaspora if an event is created - // as visible only to self at first, but then edited to display to others. - if (strlen($str_group_allow) || strlen($str_contact_allow) || strlen($str_group_deny) || strlen($str_contact_deny)) { - $private_event = true; + + $str_contact_allow = isset($_REQUEST['contact_allow']) ? $aclFormatter->toString($_REQUEST['contact_allow']) : $user['allow_cid'] ?? ''; + $str_group_allow = isset($_REQUEST['group_allow']) ? $aclFormatter->toString($_REQUEST['group_allow']) : $user['allow_gid'] ?? ''; + $str_contact_deny = isset($_REQUEST['contact_deny']) ? $aclFormatter->toString($_REQUEST['contact_deny']) : $user['deny_cid'] ?? ''; + $str_group_deny = isset($_REQUEST['group_deny']) ? $aclFormatter->toString($_REQUEST['group_deny']) : $user['deny_gid'] ?? ''; + + $visibility = $_REQUEST['visibility'] ?? ''; + if ($visibility === 'public') { + // The ACL selector introduced in version 2019.12 sends ACL input data even when the Public visibility is selected + $str_contact_allow = $str_group_allow = $str_contact_deny = $str_group_deny = ''; + } else if ($visibility === 'custom') { + // Since we know from the visibility parameter the item should be private, we have to prevent the empty ACL + // case that would make it public. So we always append the author's contact id to the allowed contacts. + // See https://github.com/friendica/friendica/issues/9672 + $str_contact_allow .= $aclFormatter->toString($self); } } else { - // Note: do not set `private` field for self-only events. It will - // keep even you from seeing them! - $str_contact_allow = '<' . $self . '>'; + $str_contact_allow = $aclFormatter->toString($self); $str_group_allow = $str_contact_deny = $str_group_deny = ''; } + // Make sure to set the `private` field as true. This is necessary to + // have the posts show up correctly in Diaspora if an event is created + // as visible only to self at first, but then edited to display to others. + if (strlen($str_group_allow) || strlen($str_contact_allow) || strlen($str_group_deny) || strlen($str_contact_deny)) { + $private_event = true; + } $datarray = []; $datarray['start'] = $start; @@ -213,10 +213,18 @@ function events_post(App $a) exit(); } - $item_id = Event::store($datarray); + $event_id = Event::store($datarray); - if (!$cid) { - Worker::add(PRIORITY_HIGH, "Notifier", Delivery::POST, $item_id); + $item = ['network' => Protocol::DFRN, 'protocol' => Conversation::PARCEL_DIRECT, 'direction' => Conversation::PUSH]; + $item = Event::getItemArrayForId($event_id, $item); + if (Item::insert($item)) { + $uri_id = $item['uri-id']; + } else { + $uri_id = 0; + } + + if (!$cid && $uri_id) { + Worker::add(PRIORITY_HIGH, "Notifier", Delivery::POST, (int)$uri_id, (int)$uid); } DI::baseUrl()->redirect('events'); @@ -229,25 +237,25 @@ function events_content(App $a) return Login::form(); } - if ($a->argc == 1) { + if (DI::args()->getArgc() == 1) { $_SESSION['return_path'] = DI::args()->getCommand(); } - if (($a->argc > 2) && ($a->argv[1] === 'ignore') && intval($a->argv[2])) { + if ((DI::args()->getArgc() > 2) && (DI::args()->getArgv()[1] === 'ignore') && intval(DI::args()->getArgv()[2])) { q("UPDATE `event` SET `ignore` = 1 WHERE `id` = %d AND `uid` = %d", - intval($a->argv[2]), + intval(DI::args()->getArgv()[2]), intval(local_user()) ); } - if (($a->argc > 2) && ($a->argv[1] === 'unignore') && intval($a->argv[2])) { + if ((DI::args()->getArgc() > 2) && (DI::args()->getArgv()[1] === 'unignore') && intval(DI::args()->getArgv()[2])) { q("UPDATE `event` SET `ignore` = 0 WHERE `id` = %d AND `uid` = %d", - intval($a->argv[2]), + intval(DI::args()->getArgv()[2]), intval(local_user()) ); } - if ($a->theme_events_in_profile) { + if ($a->getThemeInfoValue('events_in_profile')) { Nav::setSelected('home'); } else { Nav::setSelected('events'); @@ -256,6 +264,11 @@ function events_content(App $a) // get the translation strings for the callendar $i18n = Event::getStrings(); + DI::page()->registerStylesheet('view/asset/fullcalendar/dist/fullcalendar.min.css'); + DI::page()->registerStylesheet('view/asset/fullcalendar/dist/fullcalendar.print.min.css', 'print'); + DI::page()->registerFooterScript('view/asset/moment/min/moment-with-locales.min.js'); + DI::page()->registerFooterScript('view/asset/fullcalendar/dist/fullcalendar.min.js'); + $htpl = Renderer::getMarkupTemplate('event_head.tpl'); DI::page()['htmlhead'] .= Renderer::replaceMacros($htpl, [ '$module_url' => '/events', @@ -266,8 +279,8 @@ function events_content(App $a) $o = ''; $tabs = ''; // tabs - if ($a->theme_events_in_profile) { - $tabs = BaseProfile::getTabsHTML($a, 'events', true); + if ($a->getThemeInfoValue('events_in_profile')) { + $tabs = BaseProfile::getTabsHTML($a, 'events', true, $a->getLoggedInUserNickname(), false); } $mode = 'view'; @@ -275,27 +288,27 @@ function events_content(App $a) $m = 0; $ignored = !empty($_REQUEST['ignored']) ? intval($_REQUEST['ignored']) : 0; - if ($a->argc > 1) { - if ($a->argc > 2 && $a->argv[1] == 'event') { + if (DI::args()->getArgc() > 1) { + if (DI::args()->getArgc() > 2 && DI::args()->getArgv()[1] == 'event') { $mode = 'edit'; - $event_id = intval($a->argv[2]); + $event_id = intval(DI::args()->getArgv()[2]); } - if ($a->argc > 2 && $a->argv[1] == 'drop') { + if (DI::args()->getArgc() > 2 && DI::args()->getArgv()[1] == 'drop') { $mode = 'drop'; - $event_id = intval($a->argv[2]); + $event_id = intval(DI::args()->getArgv()[2]); } - if ($a->argc > 2 && $a->argv[1] == 'copy') { + if (DI::args()->getArgc() > 2 && DI::args()->getArgv()[1] == 'copy') { $mode = 'copy'; - $event_id = intval($a->argv[2]); + $event_id = intval(DI::args()->getArgv()[2]); } - if ($a->argv[1] === 'new') { + if (DI::args()->getArgv()[1] === 'new') { $mode = 'new'; $event_id = 0; } - if ($a->argc > 2 && intval($a->argv[1]) && intval($a->argv[2])) { + if (DI::args()->getArgc() > 2 && intval(DI::args()->getArgv()[1]) && intval(DI::args()->getArgv()[2])) { $mode = 'view'; - $y = intval($a->argv[1]); - $m = intval($a->argv[2]); + $y = intval(DI::args()->getArgv()[1]); + $m = intval(DI::args()->getArgv()[2]); } } @@ -324,7 +337,7 @@ function events_content(App $a) $start = sprintf('%d-%d-%d %d:%d:%d', $y, $m, 1, 0, 0, 0); $finish = sprintf('%d-%d-%d %d:%d:%d', $y, $m, $dim, 23, 59, 59); - if ($a->argc > 1 && $a->argv[1] === 'json') { + if (DI::args()->getArgc() > 1 && DI::args()->getArgv()[1] === 'json') { if (!empty($_GET['start'])) { $start = $_GET['start']; } @@ -376,7 +389,7 @@ function events_content(App $a) $events = Event::prepareListForTemplate($r); } - if ($a->argc > 1 && $a->argv[1] === 'json') { + if (DI::args()->getArgc() > 1 && DI::args()->getArgv()[1] === 'json') { header('Content-Type: application/json'); echo json_encode($events); exit(); @@ -469,19 +482,19 @@ function events_content(App $a) $t_orig = $orig_event['summary'] ?? ''; $d_orig = $orig_event['desc'] ?? ''; $l_orig = $orig_event['location'] ?? ''; - $eid = !empty($orig_event) ? $orig_event['id'] : 0; - $cid = !empty($orig_event) ? $orig_event['cid'] : 0; - $uri = !empty($orig_event) ? $orig_event['uri'] : ''; + $eid = $orig_event['id'] ?? 0; + $cid = $orig_event['cid'] ?? 0; + $uri = $orig_event['uri'] ?? ''; if ($cid || $mode === 'edit') { $share_disabled = 'disabled="disabled"'; } - $sdt = !empty($orig_event) ? $orig_event['start'] : 'now'; - $fdt = !empty($orig_event) ? $orig_event['finish'] : 'now'; + $sdt = $orig_event['start'] ?? 'now'; + $fdt = $orig_event['finish'] ?? 'now'; $tz = date_default_timezone_get(); - if (!empty($orig_event)) { + if (isset($orig_event['adjust'])) { $tz = ($orig_event['adjust'] ? date_default_timezone_get() : 'UTC'); } @@ -500,7 +513,7 @@ function events_content(App $a) $fminute = !empty($orig_event) ? DateTimeFormat::convert($fdt, $tz, 'UTC', 'i') : '00'; if (!$cid && in_array($mode, ['new', 'copy'])) { - $acl = ACL::getFullSelectorHTML(DI::page(), $a->user, false, ACL::getDefaultUserPermissions($orig_event)); + $acl = ACL::getFullSelectorHTML(DI::page(), $a->getLoggedInUserId(), false, ACL::getDefaultUserPermissions($orig_event)); } else { $acl = ''; } @@ -582,7 +595,7 @@ function events_content(App $a) Item::deleteForUser(['id' => $ev[0]['itemid']], local_user()); } - if (Item::exists(['id' => $ev[0]['itemid']])) { + if (Post::exists(['id' => $ev[0]['itemid']])) { notice(DI::l10n()->t('Failed to remove event')); }