]> gitweb.erp-flowers.ru Git - erp24_rep/yii-erp24/.git/commitdiff
fix(ERP-251): replace file_put_contents with Yii::info() in BonusController origin/feature_filippov_ERP-251_fix_bonus_log_permission
authorAleksey Filippov <Aleksey.Filippov@erp-flowers.ru>
Mon, 2 Mar 2026 10:25:36 +0000 (13:25 +0300)
committerAleksey Filippov <Aleksey.Filippov@erp-flowers.ru>
Mon, 2 Mar 2026 10:25:36 +0000 (13:25 +0300)
Replace USERS_AUTH_CALL_LOG2 file writes that caused permission denied
errors on production with standard Yii::info() logging under
'bonus.auth' category. Remove unused $LOG and $USERS_AUTH_CALL_LOG2
static properties.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
erp24/api2/controllers/BonusController.php

index 22059dea14cbfebc56da8235b1b6539d143a86d8..2fdbc87eaa63ae3e02ddc836045fe1b73884c440 100644 (file)
@@ -31,11 +31,7 @@ use yii_app\services\SiteService;
 
 class BonusController extends BaseController
 {
-    private static $LOG = "/var/www/erp24/api2/log.txt";
-    private static $USERS_AUTH_CALL_LOG2 =
-//        '/tmp/users_auth_call_log2.txt';
-//        "/var/www/erp24/api2/users_auth_call_log2.txt";
-        __DIR__ . '/../users_auth_call_log2.txt';
+    private const LOG_CATEGORY_BONUS = 'bonus.auth';
 
     private static $YEAR_PERIOD = 366;
     private static $FIRST_SALE_PROCENT = 0.1;
@@ -361,10 +357,7 @@ class BonusController extends BaseController
             return $this->asJson(["error_id" => 4.15, "error" => $userAuthCallLog->getErrors()]);
         }
 
-        file_put_contents(self::$USERS_AUTH_CALL_LOG2,
-            "" . date("d.m.Y H:i:s", time()) . " keykod=" . $user->keycode
-            . " store_id=" . $result['store_id'] . " seller_id=" . $result['seller_id'] . " phone=" . $result['phone'] . " $name\n",
-            FILE_APPEND | LOCK_EX);
+        Yii::info("keykod={$user->keycode} store_id={$result['store_id']} seller_id={$result['seller_id']} phone={$result['phone']} $name", self::LOG_CATEGORY_BONUS);
 
         $mess["timeout"] = 15;
 
@@ -419,16 +412,6 @@ class BonusController extends BaseController
 //            return $this->asJson(["error_id" => 4, "error" => $userAuthCallLog->getErrors()]);
 //        }
 //
-//        file_put_contents(self::$USERS_AUTH_CALL_LOG2,
-//            "" . date("d.m.Y H:i:s", time()) . " keykod=" . $user->keycode
-//            . " store_id=" . $result['store_id'] . " seller_id=" . $result['seller_id'] . " phone=" . $result['phone'] . " $name\n",
-//            FILE_APPEND | LOCK_EX);
-//
-//        $mess["timeout"] = 15;
-//
-//        LogService::apiLogs(1, json_encode($mess, JSON_UNESCAPED_UNICODE));
-//
-//        return $this->asJson($mess);
 
     public function actionSaveClientInfo()
     {
@@ -903,9 +886,7 @@ class BonusController extends BaseController
             }
 
             file_put_contents(self::OUT_DIR . '/sale_bonuses_' . $fl . '.json', PHP_EOL . '--' . __LINE__, FILE_APPEND);
-            file_put_contents(self::$USERS_AUTH_CALL_LOG2,
-                "" . date("d.m.Y H:i:s", time()) . " MINUS write_off_bonuses=" . $write_off_bonuses . "\n",
-                FILE_APPEND | LOCK_EX);
+            Yii::info("MINUS write_off_bonuses={$write_off_bonuses}", self::LOG_CATEGORY_BONUS);
         }
         //начисляем кэшбек клиенту 10% от покупки - с базы за вычитом бонусов которые он списывает
 
@@ -975,11 +956,7 @@ class BonusController extends BaseController
             }
 
             file_put_contents(self::OUT_DIR . '/sale_bonuses_' . $fl . '.json', PHP_EOL . '--' . __LINE__, FILE_APPEND);
-            file_put_contents(
-                self::$USERS_AUTH_CALL_LOG2,
-                "" . date("d.m.Y H:i:s", time()) . " PLUS bonus=" . $back . "\n",
-                FILE_APPEND | LOCK_EX
-            );
+            Yii::info("PLUS bonus={$back}", self::LOG_CATEGORY_BONUS);
 
             file_put_contents(self::OUT_DIR . '/sale_bonuses_' . $fl . '.json', PHP_EOL . '--' . __LINE__, FILE_APPEND);
             if ($credit_procent_index) {
@@ -1164,17 +1141,13 @@ class BonusController extends BaseController
             file_put_contents(self::OUT_DIR . '/sale_bonuses_' . $fl . '.json', PHP_EOL . '--' . __LINE__, FILE_APPEND);
             LogService::apiErrorLog(json_encode(["error_id" => 6, "error" => $user->getErrors()], JSON_UNESCAPED_UNICODE));
 
-            file_put_contents(self::$USERS_AUTH_CALL_LOG2,
-                "" . date("d.m.Y H:i:s", time()) . " BEFORE BEFORE END " . json_encode($user->getErrors(), JSON_UNESCAPED_UNICODE). "\n",
-                FILE_APPEND | LOCK_EX);
+            Yii::info("BEFORE END errors=" . json_encode($user->getErrors(), JSON_UNESCAPED_UNICODE), self::LOG_CATEGORY_BONUS);
 
             return $this->asJson(["error_id" => 6, "error" => $user->getErrors()]);
         } else {
             $this->updateUserBonusLevel($user, $sale_price, $check_id, $check_name);
         }
-        file_put_contents(self::$USERS_AUTH_CALL_LOG2,
-            "" . date("d.m.Y H:i:s", time()) . " BEFORE END \n",
-            FILE_APPEND | LOCK_EX);
+        Yii::info("BEFORE END", self::LOG_CATEGORY_BONUS);
 //        $itogo -= $write_off_bonuses;
 
 //        // продажа заносим в таблицу