diff --git a/webapp/models/BcOrder.php b/webapp/models/BcOrder.php
index 36d93cff..1fa5ae77 100644
--- a/webapp/models/BcOrder.php
+++ b/webapp/models/BcOrder.php
@@ -23,17 +23,17 @@ class BcOrder {
         return $row;
     }
 
-    public static function upsert($OrderId, $fieldsKv){
+    public static function upsert($orderId, $fieldsKv){
         SqlHelper::upsert
             (myself()->_getSelfMysql(),
              't_bc_order',
              array(
-                 'order_id' => $OrderId
+                 'order_id' => $orderId
              ),
              array(
              ),
              array(
-                 'order_id' => $OrderId,
+                 'order_id' => $orderId,
                  'order_type' => getXVal($fieldsKv,'order_type',0),
                  'account_id' => myself()->_getAccountId(),
                  'address' => myself()->_getAddress(),
@@ -48,12 +48,12 @@ class BcOrder {
             );
     }
 
-    public static function update($OrderId, $fieldsKv){
-        SqlHelper::upsert
+    public static function update($orderId, $fieldsKv){
+        SqlHelper::update
             (myself()->_getMysql(''),
              't_bc_order',
              array(
-                 'order_id' => $OrderId
+                 'order_id' => $orderId
              ),
              $fieldsKv
             );
diff --git a/webapp/services/callback/GameItemMallBuyOk.php b/webapp/services/callback/GameItemMallBuyOk.php
index 6e408af8..27b1aa60 100644
--- a/webapp/services/callback/GameItemMallBuyOk.php
+++ b/webapp/services/callback/GameItemMallBuyOk.php
@@ -47,7 +47,7 @@ class GameItemMallBuyOk {
                             'status' => 1
                         ));
         if ($orderDb['order_type'] == BcOrder::SPEC_ORDER_TYPE) {
-            $extData = json_decode($order['ext_data'], true);
+            $extData = json_decode($orderDb['ext_data'], true);
             switch ($extData['mode']) {
             case BcOrder::SHOP_BUY_MODE_NORMAL:
             {
@@ -94,7 +94,7 @@ class GameItemMallBuyOk {
         if ($itemId == V_ITEM_DIAMOND) {
             $event = [
                 'name' => LogService::CEBG_TO_DIAMOND,
-                'val' => $item_num
+                'val' => $itemNum
             ];
             LogService::productDiamondCallback(
                 ['account_id' => $accountId],