Verified Commit 39af230d authored by dockx thibault's avatar dockx thibault
Browse files

:sparkles: (templateDialog)

Fixes phpcs code style
Showing with 26 additions and 25 deletions
+26 -25
...@@ -21,14 +21,15 @@ ...@@ -21,14 +21,15 @@
/*! /*!
* \brief Template dialog handling * \brief Template dialog handling
*/ */
class templateDialog implements FusionDirectoryDialog class templateDialog implements FusionDirectoryDialog
{ {
protected $management; protected $management;
protected $type; protected $type;
protected $template = NULL; protected $template = NULL;
protected $templates; protected $templates;
protected $target = NULL; protected $target = NULL;
protected $closed = FALSE; protected $closed = FALSE;
protected $tabObject; protected $tabObject;
...@@ -37,14 +38,14 @@ class templateDialog implements FusionDirectoryDialog ...@@ -37,14 +38,14 @@ class templateDialog implements FusionDirectoryDialog
function __construct ($management, $type, $dn = NULL, $target = NULL) function __construct ($management, $type, $dn = NULL, $target = NULL)
{ {
$this->management = $management; $this->management = $management;
$this->type = $type; $this->type = $type;
$this->templates = objects::getTemplates($this->type); $this->templates = objects::getTemplates($this->type);
if ($dn !== NULL) { if ($dn !== NULL) {
if (isset($this->templates[$dn])) { if (isset($this->templates[$dn])) {
$this->template = new template($this->type, $dn); $this->template = new template($this->type, $dn);
} else { } else {
trigger_error('Unknown template "'.$dn.'"'); trigger_error('Unknown template "' . $dn . '"');
} }
} }
$this->target = $target; $this->target = $target;
...@@ -58,9 +59,9 @@ class templateDialog implements FusionDirectoryDialog ...@@ -58,9 +59,9 @@ class templateDialog implements FusionDirectoryDialog
} }
if (($this->target === NULL) && if (($this->target === NULL) &&
is_object($this->template) && is_object($this->template) &&
(isset($_POST[$this->post_finish]) || isset($_GET[$this->post_finish])) (isset($_POST[$this->post_finish]) || isset($_GET[$this->post_finish]))
) { ) {
$this->template->readPost(); $this->template->readPost();
$this->template->update(); $this->template->update();
$this->handleFinish(); $this->handleFinish();
...@@ -70,7 +71,7 @@ class templateDialog implements FusionDirectoryDialog ...@@ -70,7 +71,7 @@ class templateDialog implements FusionDirectoryDialog
if ( if (
isset($_POST['template']) && isset($_POST['template']) &&
isset($this->templates[$_POST['template']]) isset($this->templates[$_POST['template']])
) { ) {
if (is_object($this->template)) { if (is_object($this->template)) {
trigger_error('redefining template object'); trigger_error('redefining template object');
} }
...@@ -122,22 +123,22 @@ class templateDialog implements FusionDirectoryDialog ...@@ -122,22 +123,22 @@ class templateDialog implements FusionDirectoryDialog
return $display; return $display;
} }
/** /**
* @return void * @return void
* Note : The idea is to handle the save directly at the end of the validation of the template data. * Note : The idea is to handle the save directly at the end of the validation of the template data.
* Allowing direct saving in case of no error and opening the UI at the exact tab in case of error. * Allowing direct saving in case of no error and opening the UI at the exact tab in case of error.
*/ */
protected function handleFinish () protected function handleFinish ()
{ {
$this->management->closeDialogs(); $this->management->closeDialogs();
$tab = $this->template->apply(); $tab = $this->template->apply();
$errors = $tab->save(); $errors = $tab->save();
if (!empty($errors)) { if (!empty($errors)) {
$this->management->openTabObject($tab); $this->management->openTabObject($tab);
msg_dialog::displayChecks($errors); msg_dialog::displayChecks($errors);
}
} }
}
protected function handleCancel () protected function handleCancel ()
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment