fix billing process

.github/user_bug_report.md

@@ -8,9 +8,11 @@ assignees: ''
 ---
 
 # Bug Description
+
 A clear and concise description of what the bug is.
 
 # How to Reproduce
+
 Steps to reproduce the behavior:
 
 1. Go to '...'
@@ -19,16 +21,20 @@ Steps to reproduce the behavior:
 4. See error
 
 # Expected Behavior
+
 A clear and concise description of what you expected to happen.
 
 # Screenshots
+
 If applicable, add screenshots to help explain your problem.
 
 # System Information
- - System: [e.g. PC or iPhone11, ...]
+
- - OS: [e.g. iOS]
+- System: [e.g. PC or iPhone11, ...]
- - Browser [e.g. chrome, safari]
+- OS: [e.g. iOS]
- - KarakaVersion [e.g. 22]
+- Browser [e.g. chrome, safari]
+- KarakaVersion [e.g. 22]
 
 # Additional Information
+
 Add any other context about the problem here.

Controller/ApiController.php

@@ -71,8 +71,8 @@ final class ApiController extends Controller
     private function validateTaskCreate(RequestAbstract $request) : array
     {
         $val = [];
-        if (($val['title'] = empty($request->getData('title')))
+        if (($val['title'] = !$request->hasData('title'))
-            || ($val['plain'] = empty($request->getData('plain')))
+            || ($val['plain'] = !$request->hasData('plain'))
         ) {
             return $val;
         }
@@ -294,7 +294,7 @@ final class ApiController extends Controller
         $task->setType(TaskType::SINGLE);
         $task->redirect = $request->getDataString('redirect') ?? '';
 
-        if (empty($request->getData('priority'))) {
+        if (!$request->hasData('priority')) {
             $task->due = $request->getDataDateTime('due');
         } else {
             $task->setPriority((int) $request->getData('priority'));
@@ -428,7 +428,7 @@ final class ApiController extends Controller
         if (($val['status'] = !TaskStatus::isValidValue((int) $request->getData('status')))
             || ($val['due'] = !((bool) \strtotime((string) $request->getData('due'))))
             || ($val['task'] = !(\is_numeric($request->getData('task'))))
-            || ($val['forward'] = !(\is_numeric(empty($request->getData('forward')) ? $request->header->account : $request->getData('forward'))))
+            || ($val['forward'] = !(\is_numeric(!$request->hasData('forward') ? $request->header->account : $request->getData('forward'))))
         ) {
             return $val;
         }
@@ -844,9 +844,9 @@ final class ApiController extends Controller
     private function validateTaskAttributeCreate(RequestAbstract $request) : array
     {
         $val = [];
-        if (($val['type'] = empty($request->getData('type')))
+        if (($val['type'] = !$request->hasData('type'))
-            || ($val['value'] = (empty($request->getData('value')) && empty($request->getData('custom'))))
+            || ($val['value'] = (!$request->hasData('value') && !$request->hasData('custom')))
-            || ($val['task'] = empty($request->getData('task')))
+            || ($val['task'] = !$request->hasData('task'))
         ) {
             return $val;
         }
@@ -922,8 +922,8 @@ final class ApiController extends Controller
     private function validateTaskAttributeTypeL11nCreate(RequestAbstract $request) : array
     {
         $val = [];
-        if (($val['title'] = empty($request->getData('title')))
+        if (($val['title'] = !$request->hasData('title'))
-            || ($val['type'] = empty($request->getData('type')))
+            || ($val['type'] = !$request->hasData('type'))
         ) {
             return $val;
         }
@@ -999,8 +999,8 @@ final class ApiController extends Controller
     private function validateTaskAttributeTypeCreate(RequestAbstract $request) : array
     {
         $val = [];
-        if (($val['title'] = empty($request->getData('title')))
+        if (($val['title'] = !$request->hasData('title'))
-            || ($val['name'] = empty($request->getData('name')))
+            || ($val['name'] = !$request->hasData('name'))
         ) {
             return $val;
         }
@@ -1091,8 +1091,8 @@ final class ApiController extends Controller
     private function validateTaskAttributeValueCreate(RequestAbstract $request) : array
     {
         $val = [];
-        if (($val['attributetype'] = empty($request->getData('attributetype')))
+        if (($val['attributetype'] = !$request->hasData('attributetype'))
-            || ($val['value'] = empty($request->getData('value')))
+            || ($val['value'] = !$request->hasData('value'))
         ) {
             return $val;
         }
@@ -1168,8 +1168,8 @@ final class ApiController extends Controller
     private function validateTaskAttributeValueL11nCreate(RequestAbstract $request) : array
     {
         $val = [];
-        if (($val['title'] = empty($request->getData('title')))
+        if (($val['title'] = !$request->hasData('title'))
-            || ($val['value'] = empty($request->getData('value')))
+            || ($val['value'] = !$request->hasData('value'))
         ) {
             return $val;
         }

Controller/BackendController.php

@@ -207,7 +207,7 @@ final class BackendController extends Controller implements DashboardElementInte
             || $task->isCCAccount($accountId)
             || $task->isToAccount($accountId))
             && !$this->app->accountManager->get($accountId)->hasPermission(
-                PermissionType::READ, $this->app->unitId, $this->app->appName, self::NAME, PermissionCategory::TASK, $task->getId())
+                PermissionType::READ, $this->app->unitId, $this->app->appId, self::NAME, PermissionCategory::TASK, $task->getId())
         ) {
             $view->setTemplate('/Web/Backend/Error/403_inline');
             $response->header->status = RequestStatusCode::R_403;

tests/Controller/ApiControllerTest.php

@@ -65,7 +65,7 @@ final class ControllerTest extends \PHPUnit\Framework\TestCase
 
         $permission = new AccountPermission();
         $permission->setUnit(1);
-        $permission->setApp('backend');
+        $permission->setApp(2);
         $permission->setPermission(
             PermissionType::READ
             | PermissionType::CREATE

tests/Controller/BackendControllerTest.php

@@ -59,7 +59,7 @@ final class BackendControllerTest extends \PHPUnit\Framework\TestCase
 
         $permission = new AccountPermission();
         $permission->setUnit(1);
-        $permission->setApp('backend');
+        $permission->setApp(2);
         $permission->setPermission(
             PermissionType::READ
             | PermissionType::CREATE