debug wxauthor

helper/login/ilogin.php

@@ -19,13 +19,14 @@ abstract class ILogin
 {
     const mobile_login = 1;
     const wxopen_login = 2;
-    const fields = 'member_id,member_mobile,member_wxunionid,member_wxopenid,member_name,member_truename,member_signname,' .
-    'member_nickname,member_avatar,member_sex,member_birthday,member_passwd,member_paypwd,member_email,member_mobile_bind,member_state';
+    const fields = '*';
+
+//    const fields = 'member_id,member_mobile,member_wxunionid,member_wxopenid,member_name,member_truename,member_signname,' .
+//    'member_nickname,member_avatar,member_sex,member_birthday,member_passwd,member_paypwd,member_email,member_mobile_bind,member_state';
 
     protected $mMemberId;
     protected $mMemberInfo;
-    public function __construct()
-    {
+    public function __construct() {
         $this->mMemberId = 0;
     }
     public function memberid() {
@@ -43,23 +44,20 @@ abstract class ILogin
         $this->mMemberInfo = $mod_member->getMemberInfo(['member_id' => $this->mMemberId],ILogin::fields,true);
         if(empty($this->mMemberInfo)) return false;
 
-        if(session_helper::logined())
-        {
+        if(session_helper::logined()) {
             $mod_member->createSession($this->mMemberInfo);
             session::instance()->set_cookie($_SESSION['MPHPSESSID']);
         }
-        else
-        {
+        else {
             $mod_member->createSession($this->mMemberInfo);
             Model('cart')->mergecart($this->mMemberInfo, $_SESSION['store_id']);
             Model('goods_browse')->mergebrowse($this->mMemberId, $_SESSION['store_id']);
-            account_helper::onLogin($this->mMemberId);
             session::instance()->set_cookie($_SESSION['MPHPSESSID']);
         }
+        account_helper::onLogin($this->mMemberId);
 
         return true;
     }
-
     public function bind($user_info,$fUseOpenid = true)
     {
         if($this->mMemberId <= 0) return false;

helper/session_helper.php

@@ -254,38 +254,39 @@ class session_helper
 
         if (util::from_wechat() == false) {
             return false;
-        } else {
-            if (array_key_exists('wx_author', $_SESSION)) {
-                if (array_key_exists('handled', $_SESSION['wx_author'])) {
+        }
+        else
+        {
+            if (array_key_exists('wx_author', $_SESSION))
+            {
+                if (array_key_exists('handled', $_SESSION['wx_author']))
+                {
                     $user_info = $_SESSION['wx_author']['user_info'];
-                    $loginer = new \login\unionid_log($user_info['unionid']);
-
-                    if (self::logined()) {
+                    $loginer = new login\unionid_log($user_info['unionid']);
+                    if ($loginer->ismember() == true) {
                         $loginer->bind($user_info);
                         $loginer->login();
                         unset($_SESSION['wx_author']);
-                    } else {
-                        if ($loginer->ismember() == true) {
-                            $loginer->bind($user_info);
-                            $loginer->login();
-                            unset($_SESSION['wx_author']);
-                        }
                     }
                 }
 
                 return false;
             }
-
-            $author_time = $_SESSION['wxauthor_time'];
-            $max_time = 2 * 86400;
-            if (time() - $author_time > $max_time) {
-                return true;
-            }
-
-            if (!empty($_SESSION['member_wxunionid']) && !empty($_SESSION['member_wxopenid'])) {
-                return false;
-            } else {
-                return true;
+            else
+            {
+                if (!empty($_SESSION['member_wxunionid']) && !empty($_SESSION['member_wxopenid']))
+                {
+                    $author_time = $_SESSION['wxauthor_time'];
+                    $max_time = 2 * 86400;
+                    if (time() - $author_time > $max_time) {
+                        return true;
+                    } else {
+                        return false;
+                    }
+                }
+                else {
+                    return true;
+                }
             }
         }
     }

helper/third_author/wxauthor.php

@@ -36,11 +36,11 @@ class wxauthor
     public function enter($origin_url)
     {
         $back_url = BASE_SITE_URL . '/mobile/wxauthor.php';
-        $params = ['appid' => self::appid,
-            'redirect_uri' => $back_url,
-            'response_type' => 'code',
-            'scope' => 'snsapi_userinfo',
-            'state' => $_SESSION['MPHPSESSID']];
+        $params = [ 'appid' => self::appid,
+                    'redirect_uri' => $back_url,
+                    'response_type' => 'code',
+                    'scope' => 'snsapi_userinfo',
+                    'state' => $_SESSION['MPHPSESSID']];
         $ref_url = util::http_add_params(self::authorize_url,$params);
         $ref_url .= '#wechat_redirect';
         $_SESSION['author_orgin_url'] = $origin_url;

mobile/control/bonusex.php

@@ -75,7 +75,7 @@ class bonusexControl extends mobileControl
         }
 
         $this->fileter_typeinfos($type_infos,$type_info,$binded_info,$avatars);
-        $type = \bonus\type::create_by_paramer($type_infos['type_info']);
+        $type = bonus\type::create_by_paramer($type_infos['type_info']);
 
         $isMineType = $this->isMineType($type);
         if($type->isStart() == false) {
@@ -86,7 +86,7 @@ class bonusexControl extends mobileControl
         $mine_bonus = bonus_helper::get_mine_by_typesn($type_sn);
         if($mine_bonus != false)
         {   //显示我抢到的红包，按照状态进行判断
-            $bonus = \bonus\user_bonus::create_by_param($mine_bonus);
+            $bonus = bonus\user_bonus::create_by_param($mine_bonus);
             $mine_bonus = bonus_helper::filter_bonus($mine_bonus);
 
             if($bonus->isBinded() == false && session_helper::isVerfiyMobile())

mobile/control/login.php

@@ -64,7 +64,7 @@ class loginControl extends mobileHomeControl
 
         if(session_helper::logined())
         {
-            $loginner = new \login\open_userinfo_log($user_info);
+            $loginner = new login\open_userinfo_log($user_info);
             if($loginner->ismember())
             {
                 if(session_helper::memberid() == $loginner->memberid())
@@ -89,19 +89,21 @@ class loginControl extends mobileHomeControl
         }
         else
         {
-            $loginner = new \login\open_userinfo_log($user_info);
+            $loginner = new login\open_userinfo_log($user_info);
             if($loginner->ismember()) {
                 $loginner->bind($user_info,false);
                 $loginner->login();
                 return self::outsuccess(['ismember' => true,'isauthor' => true,
-                                         'member_id' => $_SESSION['member_id'],'HPHPSESSID' => $_SESSION['MPHPSESSID'],'userinfo' => $this->userinfo()]);
+                                         'member_id' => $_SESSION['member_id'],
+                                         'HPHPSESSID' => $_SESSION['MPHPSESSID'],
+                                         'userinfo' => $this->userinfo()]);
             }
             else
             {
                 $user_info['openid'] = '';
                 $_SESSION['wx_author']['user_info'] = $user_info;
                 $_SESSION['wx_author']['handled'] = false;
-                return self::outsuccess(array('ismember' => false,'isauthor' => true));
+                return self::outsuccess(['ismember' => false,'isauthor' => true]);
             }
         }
     }