DZ论坛会员组过期不自动退出的代码BUG和解决思路

看DZ方坛不少站反映们自己坛的收费员组或者某共用户组设置有效期，到期并不自动退出，就算是员本人登陆还是显示已期的那个用户组，不切换回普通员组。 

有这个现象应该是对会员组的判断逻辑有问题。DZ的用户组在usergroups数据表中，用id来区别。如果是公共用户组或者被禁止，则增加一个有效期时间记录。本来这个有效期应该是每天由程序检查一遍，过了有效期就删除时间，同时把会员组变为原来的普通会员组。但似乎默认功能中并不做这个日常检查工作，在代码中只有这几个功能： 

1、购买收费用户组，购买后积分扣除、增加有效时间、扩展用户组变为收费用户组，主组不变。 

             2、切换用户组，可以把主组与扩展组之间相互切换，手工操作。 

             

3、退出用户组，仅包括退出扩展用户组，手工操作。这个退出不管是不是收费组、是不是还没到期，只要会员点击“退出”按钮，扩展组就没了，变成只有主组了。 

至于另外两个功能就没看到有代码，应该就是没有这些功能，包括：购买收费组之后自动把主组变成收费组、主组或扩展组过期后自动退回为普通用户组。 

 

                   要修正这个漏洞是比较容易的，可以用两个方法办到： 

  1、添加一个计划任务，每天检查用户组的有效期，到期就升级一下用户组。这个适合于会员组变化很频繁、变化会员多的论坛，变化效果要第二天才能看到。 

2、不用计划任务，在member文件中添加一件判断代码，只要会员登陆就检查他的用户组情况，已过期的就升级一下。会员组变化不多的论坛可用这个，效果马上看到，但需要登陆。 

如果既不用登陆就自动升级会员组，而且想会员本人能马上看到正确的会员组显示，把计划任务与member文件两者都要就最好。 

            

查代码发现其实DZ默认程序每天有检查会员组是否过期的。只不过这个检查仅限于禁言会员和禁止访问这两个会员组，也就是groupid为4和5的会员，其它的都不检查，所以如果是VIP会员、收费会员这些，就都更新不了过期时间。 

如果要修改，只需要把默认的计划任务中的groupid这个限制删掉就可以了，不用另外写计划任务代码。所以更新一下源码文件。 



    

报名~ 

两个一起吧！

这个两天整一下代码

需 报个名两个都

再分析一下员组的据表记录，想技术的可以认了。 

会员组，DZ系统搞得很复杂，在pre_common_member_field_forum和pre_common_member两个表中都有记录，可以说是重复记录的。相关的记录就更多了，自己去找。 

         

           

其中pre_common_member这个表，有两个字段，一个是groupid，会员组ID，另一个是groupexpiry，有效期时间。每次在程序中判断会员组时，这两个字段是结合起来读的。也就是判断会员是哪个会员组，先读groupid再读groupexpiry，如果groupexpiry为0,groupid就正确了。如果groupexpiry不为0,就要判断这个时间记录是今天之前还是今天之后，如果是今天之前，算过期，要清除这个过期时间，并把会员组变为应该的组。如果是今天之后，就还有效，groupid继续着。 

按说pre_common_member这一个表的记录就已经够用了，但是DZ的会员组特别复杂，除了主组，还有扩展组、有效期、过期后要变为哪个组这些东东，所以在pre_common_member_field_forum表里有个更复杂的记录，用的是groupterms的字段。这个字段是一个格式化的数组记录，打开看时是这样的： 

          

1. a:2:{s:4:"main";a:3:{s:4:"time";i:1481644800;s:7:"adminid";i:-1;s:7:"groupid";s:2:"91";}s:3:"ext";a:2:{i:5;i:1481644800;i:88;i:1482163200;}}

复制代码

 

看不懂了吧！ 

 这个东东在程序中要做两步阅读，一是调出记录，二是用dunserialize()函数解析这个记录，变成一个数组。以上这条记录通过解析后变成的数组可能写成这样： 

$groupterms['groupterms']['main']['groupid']; 

   $groupterms['groupterms']['main']['time']; 

            $groupterms['groupterms']['main']['adminid']; 



$groupterms['groupterms']['ext'][$row['groupid']]; 



   

         这些具体的变量我们就能看明白了，无非就是记录的主组ID、有效时间、管理组、扩展组和有效期这些东东。其中扩展组会包括了现在的组，所以默认程序的会员组页面中的“切换”就是在读了这条记录之后，你可以在主组和多个扩展组中来回切－－实际上根本就是多余！ 

  而其中$groupterms['groupterms']['main']['time']跟会员表中的groupexpiry是一回事（默认程序肯定是一样，如果不一样，是自己哪里改坏了）。当升级会员组时，这两个记录是同时更新的。 

          

 

 

    

 会员组过期时，正确应该的进程是这样的： 

                                              1、已经读取到会员组的时间过期了，这个有个全局变量$_G['member']['groupexpiry']，一看就能判断。 

2、既然过期了，就读以上两个表，把相关字段全读出来。 

                     3、解析groupterms字段后得到$groupterms['groupterms']['main']['groupid']和$groupterms['groupterms']['main']['time']两个变量，这两个，一个是过期时间，一个是过期后应该变成的会员组，也就是后台所选择的这个： 

     

如果“过期后会员组变为”这里选择的跟“所属会员组”一样也是禁止发言，或者“会员组有效期”这里不填，提交后groupterms这个字段的main就会是空白内容，这是程序自动判断的，因为不需要记录过期时间和过期后的会员组。自己测试就知道了。 

4、通过判断知道会员组有效期已经是今天之前的，过期了，正确情况应该是把groupterms这个字段的main记录变空白，但DZ程序的BUG就在这里，缺少这个判断。我补充的代码就是把$groupterms['groupterms']['main']['groupid']和$groupterms['groupterms']['main']['time']两个变量清空，然后把groupterms字段重新格式化（因为很可能还有扩展组的记录存在），用的是serialize()这个函数，更新pre_common_member_field_forum表。同时把pre_common_member表的记录也更新为正确。 

        

  数据表记录正确之后，前台的会员组显示就是正确的了。 

你读懂了吗？ 

  

非常感谢！非常不错的干货。建议管员还是把支付宝即时到账申请下来 这样更佳的方便家。

6楼 lwq842635 2016-12-12 15:28 

非常感谢！非常不错的干货。建议管理员还是把支付宝即时到账申请下来 这样更佳的方便大家。 ...

了一蚊子肉买个射炮？

也是哦

看了楼的代码 但是还有个问题，就是pre_common_member_field_forum表中不存在这个UID用户，在pre_common_member_field_forum_archive存档表中，还有一种况就是pre_common_member_field_forum表和pre_common_member_field_forum_archive存档表都不存这个UID用的话都是法执行的

9楼 lwq842635 2016-12-12 22:26 

看了楼主的代码 但是还有个问题，就是pre_common_member_field_forum表中不存在这个UID用户，而在pre_commo ...

你说的这个，是自动冻结的会员，都已经冻结了就别搞什么升级了，先解冻吧 



          我很讨厌这个自动冻结功能，后台不要开启每日用户表优化的计划任务。 

 

看了一下关于冻结会员的代码，远不止你讲的这一个表，包括这些全动了手脚： 

   

1. if(DB::query($movesql, array('common_member_archive', 'common_member'), false, true)) { 

              

            

2.                                 DB::query($deletesql, array('common_member'), false, true); 

     

    

3.                         } 

      

4.                         if(DB::query($movesql, array('common_member_profile_archive', 'common_member_profile'), false, true)) { 

5.                                 DB::query($deletesql, array('common_member_profile'), false, true); 

6.                         } 

7.                         if(DB::query($movesql, array('common_member_field_forum_archive', 'common_member_field_forum'), false, true)) { 

8.                                 DB::query($deletesql, array('common_member_field_forum'), false, true); 

                     

9.                         } 

10.                         if(DB::query($movesql, array('common_member_field_home_archive', 'common_member_field_home'), false, true)) { 

    

      

11.                                 DB::query($deletesql, array('common_member_field_home'), false, true); 

                

    

12.                         } 

13.                         if(DB::query($movesql, array('common_member_status_archive', 'common_member_status'), false, true)) { 

14.                                 DB::query($deletesql, array('common_member_status'), false, true); 

15.                         } 

       

     

    

     

16.                         if(DB::query($movesql, array('common_member_count_archive', 'common_member_count'), false, true)) { 

    

       

17.                                 DB::query($deletesql, array('common_member_count'), false, true); 

             

18.                         }

复制代码

楼好强！ 

楼好强！

源码更新了一下，是查到DZ默认的计划任务中有每天检查禁言员的期，修改检查所有期员就行了。

厉啊

看看怎么用

研究下怎么使用

楼：了一蚊子肉买个射炮？ 

楼很幽默哦

楼很幽默哦楼很幽默哦

没有币买不了

到期用户还是法变换普通用户，登陆之卡死在到期页，法行操，怎么解？ 

现用户组是切换了，只是到期时间没有清0,导致VIP员到期切换到了应积分的用户组，但是依然卡死在到期页，编辑用户，现原因应该是VIP到期时间没有清空导致

19楼 wuxinzi 2018-1-15 20:17 



                  到期用户还是无法变换为普通用户，登陆之后卡死在到期页面，无法进行其他操作，怎么解？发现用户组是切换 ...

什么叫卡死？什么叫到期页？

20楼 magentoon 2018-1-16 13:00 

什么叫卡死？什么叫到期页面？

就是登陆以后只显示到期提醒页面，点其他都无权点不开了。 

应该是VIP到期，切换到普通员组了，可是，员到期时间那里没有清空，还存在着VIP的到期时间，因此，导致切换到普通员组，普通员组也没限制了，变到期了

21楼 wuxinzi 2018-1-18 19:21 

  

就是登陆以后只显示到期提醒页面，点其他都无权点不开了。应该是VIP到期后，切换到普通会员组了，可是，会 ...

到期没有自动更新数据。有两个可能：一是你的代码没改对，二是你的计划任务没执行。 

据更新了，是不再提醒到期，是直接显示常员组。

厉，需慢慢研究下

22楼 magentoon 2018-1-18 20:53 

到期没有自动更新数据。有两个可能：一是你的代码没改对，二是你的计划任务没执行。如果数据更新了，是不 ...

修改哪几个文？

24楼 wuxinzi 2018-1-21 10:54 

主要修改哪几个文件？

你到现在还不知道改哪几个文件？你这几天都在干嘛？ 



          

  你连文都没看的话，烦别再回这么多帖了。

我按你压缩里说修改的啊，可是问题一直存在，才问你还修改那些文啊? 

计划任务执行文，是不是：cron_cleanup_daily.php

26楼 wuxinzi 2018-2-13 07:24 



          我按你压缩包里面说明修改的啊，可是问题一直存在，才问你还要修改那些文件啊?计划任务执行文件，是不是： ...

你认为该改哪些文件，我会公开说一套、私下里再秘密有一套？ 

                    

如果你按说明修改的，那就可能是没改好，继续好好核对。 

且你一直说不行不行不行，我根本不道你哪里不行。一有效信息都没有，我话可答。

上两张图给老，帮我解决下问题 

         

 

                   1：前台变成图片那样了 

2.后台到期的VIP会员，居然变成了管理组 

 

3.到期时间依然存在，没有清0 

   

28楼 wuxinzi 2018-2-14 11:02 

上两张图片给老大，帮我解决下问题1：前台变成图片那样了2.后台到期的VIP会员，居然变成了管理组

你代码怎么改的？

按你压缩里说的修改了table_common_member.php 然覆盖了source/class/class_member.php文