From: Aleksey Filippov <Aleksey.Filippov@erp-flowers.ru>
Date: Mon, 2 Mar 2026 10:25:36 +0000 (+0300)
Subject: fix(ERP-251): replace file_put_contents with Yii::info() in BonusController
X-Git-Url: https://gitweb.erp-flowers.ru/?a=commitdiff_plain;h=6fcc11fb74e6975c7247ba3581abe422dc3d2141;p=erp24_rep%2Fyii-erp24%2F.git

fix(ERP-251): replace file_put_contents with Yii::info() in BonusController

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>
---

diff --git a/erp24/api2/controllers/BonusController.php b/erp24/api2/controllers/BonusController.php
index 22059dea..2fdbc87e 100644
--- a/erp24/api2/controllers/BonusController.php
+++ b/erp24/api2/controllers/BonusController.php
@@ -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;
 
 //        // продажа заносим в таблицу