X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=cf_custom_field_delete.php;h=c8a13369281167ab5f5cfec91e96de30c9825a39;hb=764eef2d0dd9c6561f88579dbb4e6323d1702d14;hp=86c10b1ff83288a953084b04bd63558fa8b4ff0e;hpb=a8a191541d6f05b08bd8331bdf409af5abfac896;p=timetracker.git diff --git a/cf_custom_field_delete.php b/cf_custom_field_delete.php index 86c10b1f..c8a13369 100644 --- a/cf_custom_field_delete.php +++ b/cf_custom_field_delete.php @@ -39,8 +39,13 @@ if (!$user->isPluginEnabled('cf')) { header('Location: feature_disabled.php'); exit(); } - -$id = $request->getParameter('id'); +$id = (int)$request->getParameter('id'); +$field = CustomFields::getField($id); +if (!$field) { + header('Location: access_denied.php'); + exit(); +} +// End of access checks. $form = new Form('fieldDeleteForm'); @@ -60,15 +65,9 @@ if ($request->isPost()) { exit(); } } else { - $field = CustomFields::getField($id); - if (false === $field) - $err->add($i18n->get('error.db')); - - if ($err->no()) { - $form->addInput(array('type'=>'hidden','name'=>'id','value'=>$id)); - $form->addInput(array('type'=>'submit','name'=>'btn_delete','value'=>$i18n->get('label.delete'))); - $form->addInput(array('type'=>'submit','name'=>'btn_cancel','value'=>$i18n->get('button.cancel'))); - } + $form->addInput(array('type'=>'hidden','name'=>'id','value'=>$id)); + $form->addInput(array('type'=>'submit','name'=>'btn_delete','value'=>$i18n->get('label.delete'))); + $form->addInput(array('type'=>'submit','name'=>'btn_cancel','value'=>$i18n->get('button.cancel'))); } $smarty->assign('field', $field['label']);