bugfix：email公开问题

polaris1119 · polaris1119 · commit 2179b8612b7c · 2013-03-16T23:32:38.000+08:00
diff --git a/websites/code/studygolang/src/controller/account.go b/websites/code/studygolang/src/controller/account.go
@@ -45,10 +45,10 @@ func RegisterHandler(rw http.ResponseWriter, req *http.Request) {
 }
 
 func sendWelcomeMail(email []string) {
-	content := `Welcome to Study Golang.
-欢迎您，成功注册成为 Golang中文社区 | Go语言学习园地 会员
-
-Study Golang 是一个Go语言技术社区，完全用Go语言开发。我们为gopher们提供一个好的学习交流场所。加入到社区中来，参与分享，学习，不断提高吧。前往 <a href="http://studygolang.com">Golang中文社区 | Go语言学习园地</a>`
+	content := `Welcome to Study Golang.<br><br>
+欢迎您，成功注册成为 Golang中文社区 | Go语言学习园地 会员<br><br>
+Golang中文社区是一个Go语言技术社区，完全用Go语言开发。我们为gopher们提供一个好的学习交流场所。加入到社区中来，参与分享，学习，不断提高吧。前往 <a href="http://studygolang.com">Golang中文社区 | Go语言学习园地</a><br>
+<div style="text-align:right;">&copy;2013 studygolang.com Golang中文社区 | Go语言学习园地</div>`
 	service.SendMail("Golang中文社区 | Go语言学习园地 注册成功通知", content, email)
 }
 
@@ -80,7 +80,6 @@ func AccountEditHandler(rw http.ResponseWriter, req *http.Request) {
 	vars := mux.Vars(req)
 	username := req.FormValue("username")
 	curUser, _ := filter.CurrentUser(req)
-	// 请求注册页面
 	if username == "" || req.Method != "POST" || vars["json"] == "" {
 		// 获取用户信息
 		user := service.FindUserByUsername(curUser["username"].(string))
@@ -96,6 +95,12 @@ func AccountEditHandler(rw http.ResponseWriter, req *http.Request) {
 		return
 	}
 
+	// open传递过来的是“on”或没传递
+	if req.FormValue("open") == "on" {
+		req.Form.Set("open", "1")
+	} else {
+		req.Form.Set("open", "0")
+	}
 	// 更新个人信息
 	errMsg, err := service.UpdateUser(req.Form)
 	if err != nil {
@@ -235,7 +240,7 @@ func sendResetpwdMail(email, uuid string) {
 
 如果您有任何疑问，可以回复这封邮件向我们提问。谢谢！<br/><br/>
 
-<div style="text-align:right;">@2013 studygolang.com  Golang中文社区 | Go语言学习园地</div>`
+<div style="text-align:right;">&copy;2013 studygolang.com  Golang中文社区 | Go语言学习园地</div>`
 	service.SendMail("【Golang中文社区】重设密码 ", content, []string{email})
 }
 
diff --git a/websites/code/studygolang/src/model/dao.go b/websites/code/studygolang/src/model/dao.go
@@ -63,6 +63,11 @@ func (this *Dao) Insert() (sql.Result, error) {
 // Update 更新数据
 func (this *Dao) Update() error {
 	strSql := util.UpdateSql(this)
+	if strSql == "" {
+		// 没有字段需要更新，当作更新成功
+		logger.Errorln("no field need update")
+		return nil
+	}
 	logger.Debugln("Update sql:", strSql)
 	err := this.Open()
 	if err != nil {
diff --git a/websites/code/studygolang/src/util/db.go b/websites/code/studygolang/src/util/db.go
@@ -30,6 +30,9 @@ func InsertSql(sqler Sqler) string {
 
 func UpdateSql(sqler Sqler) string {
 	columnStr := strings.Join(sqler.Columns(), ",")
+	if columnStr == "" {
+		return ""
+	}
 	where := sqler.GetWhere()
 	if where != "" {
 		where = "WHERE " + where
diff --git a/websites/code/studygolang/template/user/edit.html b/websites/code/studygolang/template/user/edit.html
@@ -17,7 +17,7 @@
           <div class="control-group email required"><label class="email required control-label" for="user_email"><abbr title="required">*</abbr> Email</label><div class="controls"><input class="string email required span4" disabled="disabled" id="user_email" name="email" size="50" type="email" value="{{.user.Email}}"></div></div>
           <div class="control-group">
             <div class="controls">
-              <label for="user_email_public" class="checkbox"><input {{if .user.Open}}checked="checked"{{end}} id="user_email_public" name="open" type="checkbox" value="{{.user.Open}}"> 公开 Email</label>
+              <label for="user_email_public" class="checkbox"><input {{if .user.Open}}checked="checked"{{end}} id="user_email_public" name="open" type="checkbox"> 公开 Email</label>
             </div>
           </div>
           <div class="control-group string optional"><label class="string optional control-label" for="user_city">城市</label><div class="controls"><input class="string optional span3" id="user_city" name="city" size="50" type="text" value="{{.user.City}}"/><p class="help-block">请用中文填写所在城市</p></div></div>
@@ -96,7 +96,7 @@ <h2>绑定其他帐号用于登陆</h2>
                 alert(data.error);
             } else {
                 alert(data.msg);
-                loaction.reload();
+                location.reload();
             }
         });
         return false;
diff --git a/websites/code/studygolang/template/user/profile.html b/websites/code/studygolang/template/user/profile.html
@@ -25,10 +25,12 @@
             <label>ID:</label>
             <strong class="fn">{{.user.Username}}</strong>
           </li>
+          {{if .user.Open}}
           <li>
             <label>Email:</label>
             <span><a href="mailto:{{.user.Email}}">{{.user.Email}}</a></span>
           </li>
+          {{end}}
           <li>
             <label>注册时间:</label>
             <span>{{.user.Ctime}}</span>
