Skip to content

Commit 3217417

Browse files
authored
Merge pull request #1757 from NextDom/bugfix/save_view_no_zone
Change the condition to permit the deletion of all view zone
2 parents e9f4fcf + 58d2924 commit 3217417

File tree

1 file changed

+17
-15
lines changed

1 file changed

+17
-15
lines changed

src/Ajax/ViewAjax.php

+17-15
Original file line numberDiff line numberDiff line change
@@ -94,22 +94,24 @@ public function save()
9494
$view_ajax = json_decode(Utils::init('view'), true);
9595
Utils::a2o($view, $view_ajax);
9696
$view->save();
97-
if (isset($view_ajax['zones']) && count($view_ajax['zones']) > 0) {
97+
if (isset($view_ajax['zones'])) {
9898
$view->removeviewZone();
99-
foreach ($view_ajax['zones'] as $viewZone_info) {
100-
$viewZone = new ViewZone();
101-
$viewZone->setView_id($view->getId());
102-
Utils::a2o($viewZone, $viewZone_info);
103-
$viewZone->save();
104-
if (isset($viewZone_info['viewData'])) {
105-
$order = 0;
106-
foreach ($viewZone_info['viewData'] as $viewData_info) {
107-
$viewData = new ViewData();
108-
$viewData->setviewZone_id($viewZone->getId());
109-
$viewData->setOrder($order);
110-
Utils::a2o($viewData, NextDomHelper::fromHumanReadable($viewData_info));
111-
$viewData->save();
112-
$order++;
99+
if(count($view_ajax['zones']) > 0){
100+
foreach ($view_ajax['zones'] as $viewZone_info) {
101+
$viewZone = new ViewZone();
102+
$viewZone->setView_id($view->getId());
103+
Utils::a2o($viewZone, $viewZone_info);
104+
$viewZone->save();
105+
if (isset($viewZone_info['viewData'])) {
106+
$order = 0;
107+
foreach ($viewZone_info['viewData'] as $viewData_info) {
108+
$viewData = new ViewData();
109+
$viewData->setviewZone_id($viewZone->getId());
110+
$viewData->setOrder($order);
111+
Utils::a2o($viewData, NextDomHelper::fromHumanReadable($viewData_info));
112+
$viewData->save();
113+
$order++;
114+
}
113115
}
114116
}
115117
}

0 commit comments

Comments
 (0)