header('Location: access_denied.php');
exit();
}
+$group_id = (int)$request->getParameter('id');
+if ($group_id && !$user->isGroupValid($group_id)) {
+ header('Location: access_denied.php');
+ exit();
+}
// End of access checks.
-$config = new ttConfigHelper($user->config);
+if (!$group_id) $group_id = $user->getActiveGroup();
+$group = ttGroupHelper::getGroupAttrs($group_id);
+$config = new ttConfigHelper($group['config']);
$advanced_settings = $user->can('manage_advanced_settings');
if (!defined('CURRENCY_DEFAULT')) define('CURRENCY_DEFAULT', '$');
if ($request->isPost()) {
$cl_group = trim($request->getParameter('group_name'));
+ $cl_description = trim($request->getParameter('description'));
$cl_currency = trim($request->getParameter('currency'));
if (!$cl_currency) $cl_currency = CURRENCY_DEFAULT;
$cl_lang = $request->getParameter('lang');
$cl_uncompleted_indicators = $request->getParameter('uncompleted_indicators');
$cl_bcc_email = trim($request->getParameter('bcc_email'));
$cl_allow_ip = trim($request->getParameter('allow_ip'));
-
- // Plugin checkboxes.
- $cl_charts = $request->getParameter('charts');
- $cl_clients = $request->getParameter('clients');
- $cl_client_required = $request->getParameter('client_required');
- $cl_invoices = $request->getParameter('invoices');
- $cl_paid_status = $request->getParameter('paid_status');
- $cl_custom_fields = $request->getParameter('custom_fields');
- $cl_expenses = $request->getParameter('expenses');
- $cl_tax_expenses = $request->getParameter('tax_expenses');
- $cl_notifications = $request->getParameter('notifications');
- $cl_locking = $request->getParameter('locking');
- $cl_quotas = $request->getParameter('quotas');
- $cl_week_view = $request->getParameter('week_view');
- $cl_work_units = $request->getParameter('work_units');
} else {
- $cl_group = $user->group;
- $cl_currency = ($user->currency == ''? CURRENCY_DEFAULT : $user->currency);
- $cl_lang = $user->lang;
- $cl_decimal_mark = $user->decimal_mark;
- $cl_date_format = $user->date_format;
- $cl_time_format = $user->time_format;
- $cl_start_week = $user->week_start;
- $cl_show_holidays = $user->show_holidays;
- $cl_tracking_mode = $user->tracking_mode;
- $cl_project_required = $user->project_required;
- $cl_task_required = $user->task_required;
- $cl_record_type = $user->record_type;
- $cl_punch_mode = $user->punch_mode;
- $cl_allow_overlap = $user->allow_overlap;
- $cl_future_entries = $user->future_entries;
- $cl_uncompleted_indicators = $user->uncompleted_indicators;
- $cl_bcc_email = $user->bcc_email;
- $cl_allow_ip = $user->allow_ip;
-
- // Which plugins do we have enabled?
- $plugins = explode(',', $user->plugins);
- $cl_charts = in_array('ch', $plugins);
- $cl_clients = in_array('cl', $plugins);
- $cl_client_required = in_array('cm', $plugins);
- $cl_invoices = in_array('iv', $plugins);
- $cl_paid_status = in_array('ps', $plugins);
- $cl_custom_fields = in_array('cf', $plugins);
- $cl_expenses = in_array('ex', $plugins);
- $cl_tax_expenses = in_array('et', $plugins);
- $cl_notifications = in_array('no', $plugins);
- $cl_locking = in_array('lk', $plugins);
- $cl_quotas = in_array('mq', $plugins);
- $cl_week_view = in_array('wv', $plugins);
- $cl_work_units = in_array('wu', $plugins);
+ $cl_group = $group['name'];
+ $cl_description = $group['description'];
+ $cl_currency = ($group['currency'] == ''? CURRENCY_DEFAULT : $group['currency']);
+ $cl_lang = $group['lang'];
+ $cl_decimal_mark = $group['decimal_mark'];
+ $cl_date_format = $group['date_format'];
+ $cl_time_format = $group['time_format'];
+ $cl_start_week = $group['week_start'];
+ $cl_show_holidays = $config->getDefinedValue('show_holidays');
+ $cl_tracking_mode = $group['tracking_mode'];
+ $cl_project_required = $group['project_required'];
+ $cl_task_required = $group['task_required'];
+ $cl_record_type = $group['record_type'];
+ $cl_punch_mode = $config->getDefinedValue('punch_mode');
+ $cl_allow_overlap = $config->getDefinedValue('allow_overlap');
+ $cl_future_entries = $config->getDefinedValue('future_entries');
+ $cl_uncompleted_indicators = $config->getDefinedValue('uncompleted_indicators');
+ $cl_bcc_email = $group['bcc_email'];
+ $cl_allow_ip = $group['allow_ip'];
}
$form = new Form('groupForm');
+$form->addInput(array('type'=>'hidden','name'=>'id','value'=>$group_id));
$form->addInput(array('type'=>'text','maxlength'=>'200','name'=>'group_name','value'=>$cl_group,'enable'=>$advanced_settings));
+$form->addInput(array('type'=>'textarea','name'=>'description','style'=>'width: 250px; height: 40px;','value'=>$cl_description));
$form->addInput(array('type'=>'text','maxlength'=>'7','name'=>'currency','value'=>$cl_currency));
// Prepare an array of available languages.
$form->addInput(array('type'=>'text','maxlength'=>'100','name'=>'allow_ip','value'=>$cl_allow_ip));
}
-// Plugin checkboxes.
-$form->addInput(array('type'=>'checkbox','name'=>'charts','value'=>$cl_charts));
-$form->addInput(array('type'=>'checkbox','name'=>'clients','value'=>$cl_clients,'onchange'=>'handlePluginCheckboxes()'));
-$form->addInput(array('type'=>'checkbox','name'=>'client_required','value'=>$cl_client_required));
-$form->addInput(array('type'=>'checkbox','name'=>'invoices','value'=>$cl_invoices));
-$form->addInput(array('type'=>'checkbox','name'=>'paid_status','value'=>$cl_paid_status));
-$form->addInput(array('type'=>'checkbox','name'=>'custom_fields','value'=>$cl_custom_fields,'onchange'=>'handlePluginCheckboxes()'));
-$form->addInput(array('type'=>'checkbox','name'=>'expenses','value'=>$cl_expenses,'onchange'=>'handlePluginCheckboxes()'));
-$form->addInput(array('type'=>'checkbox','name'=>'tax_expenses','value'=>$cl_tax_expenses));
-$form->addInput(array('type'=>'checkbox','name'=>'notifications','value'=>$cl_notifications,'onchange'=>'handlePluginCheckboxes()'));
-$form->addInput(array('type'=>'checkbox','name'=>'locking','value'=>$cl_locking,'onchange'=>'handlePluginCheckboxes()'));
-$form->addInput(array('type'=>'checkbox','name'=>'quotas','value'=>$cl_quotas,'onchange'=>'handlePluginCheckboxes()'));
-$form->addInput(array('type'=>'checkbox','name'=>'week_view','value'=>$cl_week_view,'onchange'=>'handlePluginCheckboxes()'));
-$form->addInput(array('type'=>'checkbox','name'=>'work_units','value'=>$cl_work_units,'onchange'=>'handlePluginCheckboxes()'));
-
$form->addInput(array('type'=>'submit','name'=>'btn_save','value'=>$i18n->get('button.save')));
if ($user->can('delete_group')) $form->addInput(array('type'=>'submit','name'=>'btn_delete','value'=>$i18n->get('button.delete')));
if ($request->getParameter('btn_delete')) {
// Delete button pressed, redirect.
- header('Location: group_delete.php?id='.$user->group_id);
+ header('Location: group_delete.php?id='.$group_id);
exit();
}
// Validate user input.
- if (!ttValidString($cl_group, true)) $err->add($i18n->get('error.field'), $i18n->get('label.group_name'));
+ if (!ttValidString($cl_group)) $err->add($i18n->get('error.field'), $i18n->get('label.group_name'));
+ if (!ttValidString($cl_description, true)) $err->add($i18n->get('error.field'), $i18n->get('label.description'));
if (!ttValidString($cl_currency, true)) $err->add($i18n->get('error.field'), $i18n->get('label.currency'));
if ($advanced_settings) {
if (!ttValidEmail($cl_bcc_email, true)) $err->add($i18n->get('error.field'), $i18n->get('label.bcc'));
// Finished validating user input.
if ($err->no()) {
- // Prepare plugins string.
- if ($cl_charts)
- $plugins .= ',ch';
- if ($cl_clients)
- $plugins .= ',cl';
- if ($cl_client_required)
- $plugins .= ',cm';
- if ($cl_invoices)
- $plugins .= ',iv';
- if ($cl_paid_status)
- $plugins .= ',ps';
- if ($cl_custom_fields)
- $plugins .= ',cf';
- if ($cl_expenses)
- $plugins .= ',ex';
- if ($cl_tax_expenses)
- $plugins .= ',et';
- if ($cl_notifications)
- $plugins .= ',no';
- if ($cl_locking)
- $plugins .= ',lk';
- if ($cl_quotas)
- $plugins .= ',mq';
- if ($cl_week_view)
- $plugins .= ',wv';
- if ($cl_work_units)
- $plugins .= ',wu';
-
- // Recycle week view plugin options as they are not configured on this page.
- $existing_plugins = explode(',', $user->plugins);
- if (in_array('wvn', $existing_plugins))
- $plugins .= ',wvn';
- if (in_array('wvl', $existing_plugins))
- $plugins .= ',wvl';
- if (in_array('wvns', $existing_plugins))
- $plugins .= ',wvns';
-
- $plugins = trim($plugins, ',');
-
// Update config.
$config->setDefinedValue('show_holidays', $cl_show_holidays);
$config->setDefinedValue('punch_mode', $cl_punch_mode);
$config->setDefinedValue('uncompleted_indicators', $cl_uncompleted_indicators);
if ($user->updateGroup(array(
+ 'group_id' => $group_id,
'name' => $cl_group,
+ 'description' => $cl_description,
'currency' => $cl_currency,
'lang' => $cl_lang,
'decimal_mark' => $cl_decimal_mark,
'uncompleted_indicators' => $cl_uncompleted_indicators,
'bcc_email' => $cl_bcc_email,
'allow_ip' => $cl_allow_ip,
- 'plugins' => $plugins,
'config' => $config->getConfig()))) {
- header('Location: time.php');
+ header('Location: success.php');
exit();
} else
$err->add($i18n->get('error.db'));
$smarty->assign('auth_external', $auth->isPasswordExternal());
$smarty->assign('forms', array($form->getName()=>$form->toArray()));
$smarty->assign('onload', 'onLoad="handleTaskRequiredCheckbox(); handlePluginCheckboxes();"');
-$smarty->assign('title', $i18n->get('title.group'));
+$smarty->assign('title', $i18n->get('title.edit_group'));
$smarty->assign('content_page_name', 'group_edit.tpl');
$smarty->display('index.tpl');