diff --git a/Controller/ApiController.php b/Controller/ApiController.php index a7bbd01..e088431 100755 --- a/Controller/ApiController.php +++ b/Controller/ApiController.php @@ -511,7 +511,7 @@ final class ApiController extends Controller $newRequest->header->account = $request->header->account; $newRequest->setData('name', 'pos:' . \strtr(\strtolower($position->name), ' ', '_')); $newRequest->setData('status', GroupStatus::ACTIVE); - $this->app->moduleManager->get('Admin')->apiGroupCreate($newRequest, $response, $data); + $this->app->moduleManager->get('Admin')->apiGroupCreate($newRequest, clone $response, $data); } $this->createStandardCreateResponse($request, $response, $position); @@ -686,7 +686,7 @@ final class ApiController extends Controller $newRequest->header->account = $request->header->account; $newRequest->setData('name', 'dep:' . \strtolower($department->name)); $newRequest->setData('status', GroupStatus::ACTIVE); - $this->app->moduleManager->get('Admin')->apiGroupCreate($newRequest, $response, $data); + $this->app->moduleManager->get('Admin')->apiGroupCreate($newRequest, clone $response, $data); } $this->createStandardCreateResponse($request, $response, $department); diff --git a/Controller/BackendController.php b/Controller/BackendController.php index f91072e..a9c6c2a 100755 --- a/Controller/BackendController.php +++ b/Controller/BackendController.php @@ -299,8 +299,9 @@ final class BackendController extends Controller { $view = new View($this->app->l11nManager, $request, $response); - $view->setTemplate('/Modules/Organization/Theme/Backend/department-create'); + $view->setTemplate('/Modules/Organization/Theme/Backend/department-view'); $view->data['nav'] = $this->app->moduleManager->get('Navigation')->createNavigationMid(1004704001, $request, $response); + $view->data['unit'] = $this->app->unitId; $selectorView = new \Modules\Organization\Theme\Backend\Components\DepartmentTagSelector\DepartmentTagSelectorView($this->app->l11nManager, $request, $response); $view->data['department-selector'] = $selectorView; @@ -396,7 +397,7 @@ final class BackendController extends Controller { $view = new View($this->app->l11nManager, $request, $response); - $view->setTemplate('/Modules/Organization/Theme/Backend/position-create'); + $view->setTemplate('/Modules/Organization/Theme/Backend/position-view'); $view->data['nav'] = $this->app->moduleManager->get('Navigation')->createNavigationMid(1004705001, $request, $response); $selectorView = new \Modules\Organization\Theme\Backend\Components\PositionTagSelector\PositionTagSelectorView($this->app->l11nManager, $request, $response); diff --git a/Theme/Backend/department-create.tpl.php b/Theme/Backend/department-create.tpl.php deleted file mode 100755 index 78cde78..0000000 --- a/Theme/Backend/department-create.tpl.php +++ /dev/null @@ -1,59 +0,0 @@ -data['nav']->render(); ?> - -
-
-
-
-
getHtml('Department'); ?>
-
-
- - -
- -
- - getData('department-selector')->render('iParent', 'parent', false); ?> -
- -
- - getData('unit-selector')->render('iUnit', 'unit', false); ?> -
- -
- - getData('editor')->render('department-editor'); ?> -
- - getData('editor')->getData('text')->render('department-editor', 'description', 'fDepartmentCreate'); ?> -
-
- -
-
-
-
-
- -getData('department-selector')->getData('department-selector-popup')->render(); ?> diff --git a/Theme/Backend/department-view.tpl.php b/Theme/Backend/department-view.tpl.php index e2f3968..2911bc1 100644 --- a/Theme/Backend/department-view.tpl.php +++ b/Theme/Backend/department-view.tpl.php @@ -12,20 +12,24 @@ */ declare(strict_types=1); +use Modules\Organization\Models\NullDepartment; use phpOMS\Uri\UriFactory; /** * @var \phpOMS\Views\View $this * @var \Mouldes\Organization\Models $department; */ -$department = $this->data['department']; +$department = $this->data['department'] ?? new NullDepartment(); +$isNew = $department->id === 0; echo $this->data['nav']->render(); ?> -
-
+ >
getHtml('Department'); ?>
@@ -64,7 +68,12 @@ echo $this->data['nav']->render(); ?> ); ?>
- + + + + + +
diff --git a/Theme/Backend/position-create.tpl.php b/Theme/Backend/position-create.tpl.php deleted file mode 100755 index ba0c627..0000000 --- a/Theme/Backend/position-create.tpl.php +++ /dev/null @@ -1,67 +0,0 @@ -data['nav']->render(); ?> - -
-
-
-
-
getHtml('Position'); ?>
-
-
- - -
- -
- - getData('position-selector')->render('iParent', 'parent', false); ?> -
- -
- - getData('department-selector')->render('iDepartment', 'department', false); ?> -
- -
- - -
- -
- getData('editor')->render('position-editor'); ?> -
- - getData('editor')->getData('text')->render('position-editor', 'description', 'fPositionCreate'); ?> -
-
- -
-
-
-
-
- -getData('position-selector')->getData('position-selector-popup')->render(); ?> diff --git a/Theme/Backend/position-view.tpl.php b/Theme/Backend/position-view.tpl.php index 38a6ccf..d716cc3 100644 --- a/Theme/Backend/position-view.tpl.php +++ b/Theme/Backend/position-view.tpl.php @@ -12,20 +12,26 @@ */ declare(strict_types=1); +use Modules\Organization\Models\Status; +use Modules\Organization\Models\NullPosition; use phpOMS\Uri\UriFactory; /** * @var \phpOMS\Views\View $this * @var \Modules\Organization\Models\Position; */ -$position = $this->data['position']; +$position = $this->data['position'] ?? new NullPosition(); + +$isNew = $position->id === 0; echo $this->data['nav']->render(); ?> -
-
+ >
getHtml('Position'); ?>
@@ -46,8 +52,8 @@ echo $this->data['nav']->render(); ?>
@@ -64,7 +70,11 @@ echo $this->data['nav']->render(); ?> ); ?>
- + + + + +
diff --git a/Theme/Backend/unit-create.tpl.php b/Theme/Backend/unit-create.tpl.php index e903182..86e1464 100755 --- a/Theme/Backend/unit-create.tpl.php +++ b/Theme/Backend/unit-create.tpl.php @@ -23,7 +23,10 @@ $countries = ISO3166NameEnum::getConstants(); /** @var \phpOMS\Views\View $this */ echo $this->data['nav']->render(); ?> -
+