diff --git a/calendar/inc/class.uical.inc.php b/calendar/inc/class.uical.inc.php index 8151e74ab2..a4c41274c9 100644 --- a/calendar/inc/class.uical.inc.php +++ b/calendar/inc/class.uical.inc.php @@ -148,6 +148,28 @@ class uical // calendar does not work with hidden sidebox atm. unset($GLOBALS['egw_info']['user']['preferences']['common']['auto_hide_sidebox']); } + + /** + * Checks and terminates with a message if $this->owner include a user/resource we have no read-access to + */ + function check_owners_access() + { + $no_access = array(); + foreach(explode(',',$this->owner) as $owner) + { + if (!$this->bo->check_perms(EGW_ACL_READ,0,$owner)) + { + $no_access[$owner] = $this->bo->participant_name($owner); + } + } + if (count($no_access)) + { + $GLOBALS['egw']->common->egw_header(); + echo '
'.lang('Access denied to the calendar of %1 !!!',implode(', ',$no_access))."
\n"; + $GLOBALS['egw']->common->egw_footer(); + $GLOBALS['egw']->common->egw_exit(); + } + } /** * Manages the states of certain controls in the UI: date shown, category selected, ... @@ -213,12 +235,22 @@ class uical $states['view'] = $this->view = 'planner'; } // set the actual view as return_to - list($app,$class,$func) = explode('.',$_GET['menuaction']); - if (($class == 'uiviews' || $class == 'uilist') && $func) + if ($_GET['menuaction']) + { + list($app,$class,$func) = explode('.',$_GET['menuaction']); + } + else // eg. calendar/index.php + { + $func = $this->view; + $class = $this->view == 'listview' ? 'uilist' : 'uiviews'; + } + if ($class == 'uiviews' || $class == 'uilist') { // if planner_start_with_group is set in the users prefs: switch owner for planner to planner_start_with_group and back if ($this->cal_prefs['planner_start_with_group']) { + if (!$states_session && !$_GET['menuaction']) $this->view = ''; // first call to calendar + if ($func == 'planner' && $this->view != 'planner' && $this->owner == $this->user) { //echo "switched for planner to {$this->cal_prefs['planner_start_with_group']}, view was $this->view, func=$func, owner was $this->owner
\n"; @@ -238,7 +270,7 @@ class uical $states['multiple'] = $this->multiple = $_GET['multiple'] || count(explode(',',$this->owner)) > 1; - if ($this->debug > 0 || $this->debug == 'menage_states') $this->bo->debug_message('uical::manage_states(%1) session was %2, states now %3, is_group=%4, g_owner=%5',True,$set_states,$states_session,$states,$this->is_group,$this->g_owner); + if ($this->debug > 0 || $this->debug == 'menage_states') $this->bo->debug_message('uical::manage_states(%1) session was %2, states now %3',True,$set_states,$states_session,$states); // save the states in the session $GLOBALS['egw']->session->appsession('session_data','calendar',$states); } @@ -407,47 +439,42 @@ class uical // special views and view-options menu $options = ''; foreach(array( - array( - 'text' => lang('select one'), - 'value' => '', - 'selected' => False, - ), array( 'text' => lang('dayview'), 'value' => 'menuaction=calendar.uiviews.day', - 'selected' => $_GET['menuaction'] == 'calendar.uiviews.day', + 'selected' => $this->view == 'day', ), array( 'text' => lang('weekview with weekend'), 'value' => 'menuaction=calendar.uiviews.week&days=7', - 'selected' => $_GET['menuaction'] == 'calendar.uiviews.week' && $this->cal_prefs['days_in_weekview'] != 5, + 'selected' => $this->view == 'week' && $this->cal_prefs['days_in_weekview'] != 5, ), array( 'text' => lang('weekview without weekend'), 'value' => 'menuaction=calendar.uiviews.week&days=5', - 'selected' => $_GET['menuaction'] == 'calendar.uiviews.week' && $this->cal_prefs['days_in_weekview'] == 5, + 'selected' => $this->view == 'week' && $this->cal_prefs['days_in_weekview'] == 5, ), array( 'text' => lang('monthview'), 'value' => 'menuaction=calendar.uiviews.month', - 'selected' => $_GET['menuaction'] == 'calendar.uiviews.month', + 'selected' => $this->view == 'month', ), array( 'text' => lang('planner by category'), 'value' => 'menuaction=calendar.uiviews.planner&sortby=category'. ($planner_days_for_view !== false ? '&planner_days='.$planner_days_for_view : ''), - 'selected' => $_GET['menuaction'] == 'calendar.uiviews.planner' && $this->sortby != 'user', + 'selected' => $this->view == 'planner' && $this->sortby != 'user', ), array( 'text' => lang('planner by user'), 'value' => 'menuaction=calendar.uiviews.planner&sortby=user'. ($planner_days_for_view !== false ? '&planner_days='.$planner_days_for_view : ''), - 'selected' => $_GET['menuaction'] == 'calendar.uiviews.planner' && $this->sortby == 'user', + 'selected' => $this->view == 'planner' && $this->sortby == 'user', ), array( 'text' => lang('listview'), 'value' => 'menuaction=calendar.uilist.listview', - 'selected' => $_GET['menuaction'] == 'calendar.uilist.listview', + 'selected' => $this->view == 'listview', ), ) as $data) { diff --git a/calendar/inc/class.uilist.inc.php b/calendar/inc/class.uilist.inc.php index 45f17e3af2..b863856ed7 100644 --- a/calendar/inc/class.uilist.inc.php +++ b/calendar/inc/class.uilist.inc.php @@ -50,13 +50,17 @@ class uilist extends uical { $this->uical(true,$set_states); // call the parent's constructor - $GLOBALS['egw_info']['flags']['app_header'] = $GLOBALS['egw_info']['apps']['calendar']['title'].' - '.lang('Listview'); + $GLOBALS['egw_info']['flags']['app_header'] = $GLOBALS['egw_info']['apps']['calendar']['title'].' - '.lang('Listview'). + // for a single owner we add it's name to the app-header + (count(explode(',',$this->owner)) == 1 ? ': '.$this->bo->participant_name($this->owner) : ''); $this->date_filters = array( 'after' => lang('After current date'), 'before' => lang('Before current date'), 'all' => lang('All events'), ); + + $this->check_owners_access(); } /** @@ -193,7 +197,7 @@ class uilist extends uical } elseif(empty($params['search'])) // active search displays entries from all users { - $search_params['users'] = $this->is_group ? $this->g_owner : explode(',',$this->owner); + $search_params['users'] = explode(',',$this->owner); } $rows = array(); foreach((array) $this->bo->search($search_params) as $event) diff --git a/calendar/inc/class.uiviews.inc.php b/calendar/inc/class.uiviews.inc.php index f107878b17..54a6dcb26b 100644 --- a/calendar/inc/class.uiviews.inc.php +++ b/calendar/inc/class.uiviews.inc.php @@ -73,23 +73,27 @@ class uiviews extends uical $GLOBALS['egw_info']['flags']['nonavbar'] = False; $app_header = array( - 'calendar.uiviews.day' => lang('Dayview'), - 'calendar.uiviews.week' => lang('Weekview'), - 'calendar.uiviews.month' => lang('Monthview'), - 'calendar.uiviews.planner' => lang('Group planner'), + 'day' => lang('Dayview'), + 'week' => lang('Weekview'), + 'month' => lang('Monthview'), + 'planner' => lang('Group planner'), ); $GLOBALS['egw_info']['flags']['app_header'] = $GLOBALS['egw_info']['apps']['calendar']['title']. - (isset($app_header[$_GET['menuaction']]) ? ' - '.$app_header[$_GET['menuaction']] : ''); + (isset($app_header[$this->view]) ? ' - '.$app_header[$this->view] : ''). + // for a single owner we add it's name to the app-header + (count(explode(',',$this->owner)) == 1 ? ': '.$this->bo->participant_name($this->owner) : ''); // standard params for calling bocal::search for all views $this->search_params = array( 'start' => $this->date, 'cat_id' => $this->cat_id, - 'users' => $this->is_group ? $this->g_owner : explode(',',$this->owner), + 'users' => explode(',',$this->owner), 'filter' => $this->filter, 'daywise' => True, ); $this->holidays = $this->bo->read_holidays($this->year); + + $this->check_owners_access(); } /**