From 2a83fe8bd5b10cb1b6e1b9b4f8fc76096e185456 Mon Sep 17 00:00:00 2001
From: piaip <piaip@63ad8ddf-47c3-0310-b6dd-a9e9d9715204>
Date: Thu, 17 Sep 2009 04:07:47 +0000
Subject:  * refine badpost code again  * remove more deprecated stuff

git-svn-id: http://opensvn.csie.org/pttbbs/trunk/pttbbs@4847 63ad8ddf-47c3-0310-b6dd-a9e9d9715204
---
 mbbsd/passwd.c |  6 +++++-
 mbbsd/user.c   | 20 ++++++++------------
 2 files changed, 13 insertions(+), 13 deletions(-)

(limited to 'mbbsd')

diff --git a/mbbsd/passwd.c b/mbbsd/passwd.c
index 6a6ed455..060f0937 100644
--- a/mbbsd/passwd.c
+++ b/mbbsd/passwd.c
@@ -143,9 +143,13 @@ pwcuCancelBadpost()
     int day;
     PWCU_START();
 
+    // no matter what, reload the timebomb
+    cuser.badpost = u.badpost;
+    cuser.timeremovebadpost = u.timeremovebadpost;
+
     // check timebomb again
     day = (now - u.timeremovebadpost ) / DAY_SECONDS;
-    if (day <= BADPOST_CLEAR_DURATION)
+    if (day < BADPOST_CLEAR_DURATION)
 	return -1;
     if (u.badpost < 1)
 	return -1;
diff --git a/mbbsd/user.c b/mbbsd/user.c
index 67f5c163..d6897099 100644
--- a/mbbsd/user.c
+++ b/mbbsd/user.c
@@ -76,24 +76,26 @@ int u_cancelbadpost(void)
    int day, prev = cuser.badpost;
 
    // early check.
-   if(cuser.badpost==0) {
+   if(cuser.badpost == 0) {
        vmsg("你並沒有劣文."); 
        return 0;
    }
         
    // early check for race condition
+   // XXX 由於帳號API已同步化 (pwcuAPI*) 所以這個 check 可有可無
    if(search_ulistn(usernum,2)) {
        vmsg("請登出其他視窗, 否則不受理."); 
        return 0;
    }
 
-   // XXX reload account here?
+   // XXX reload account here? (also simply optional)
    pwcuReload();
 
    // early check for time (must do again later)
-   day = BADPOST_CLEAR_DURATION - (now - cuser.timeremovebadpost ) / DAY_SECONDS;
-   if(day>0 && day<=BADPOST_CLEAR_DURATION) {
-       vmsgf("每 %d 天才能申請一次, 還剩 %d 天.", BADPOST_CLEAR_DURATION, day);
+   day = (now - cuser.timeremovebadpost ) / DAY_SECONDS;
+   if (day < BADPOST_CLEAR_DURATION) {
+       vmsgf("每 %d 天才能申請一次, 還剩 %d 天.", 
+	       BADPOST_CLEAR_DURATION, BADPOST_CLEAR_DURATION-day);
        return 0;
    }
 
@@ -106,12 +108,6 @@ int u_cancelbadpost(void)
        return 0;
    }
 
-   // check again for race condition
-   if(search_ulistn(usernum,2)) {
-       vmsg("請登出其他視窗, 否則不受理."); 
-       return 0;
-   }
-
    if (pwcuCancelBadpost() != 0) {
        vmsg("刪除失敗，請洽站務人員。"); 
        return 0;
@@ -857,7 +853,7 @@ uinfo_query(const char *orig_uid, int adminmode, int unum)
 	    do {
 		int max_days = (x.lastlogin - x.firstlogin) / DAY_SECONDS;
 		snprintf(genbuf, sizeof(genbuf), "%d", x.numlogindays);
-		if (getdata_str(y++, 0, STR_LOGINDAYS, buf, 10, DOECHO, genbuf))
+		if (getdata_str(y++, 0, STR_LOGINDAYS "：", buf, 10, DOECHO, genbuf))
 		    if ((tmp = atoi(buf)) >= 0)
 			x.numlogindays = tmp;
 		if (x.numlogindays > max_days)
-- 
cgit v1.2.3