use "optional" correctly

Author: wxiaoguang

models/issues/issue_search.go

@@ -353,7 +353,8 @@ func issuePullAccessibleRepoCond(repoIDstr string, userID int64, org *organizati
 }
 
 func applyAssigneeCondition(sess *xorm.Session, assigneeID optional.Option[int64]) {
-	if assigneeID == nil || assigneeID.Value() == 0 {
+	// old logic: 0 is also treated as "not filtering assignee", because the "assignee" was read as FormInt64
+	if !assigneeID.Has() || assigneeID.Value() == 0 {
 		return
 	}
 	if assigneeID.Value() == db.NoConditionID {
@@ -365,7 +366,7 @@ func applyAssigneeCondition(sess *xorm.Session, assigneeID optional.Option[int64
 }
 
 func applyPosterCondition(sess *xorm.Session, posterID optional.Option[int64]) {
-	if posterID == nil {
+	if !posterID.Has() {
 		return
 	}
 	// poster doesn't need to support db.NoConditionID(-1), so just use the value as-is

routers/web/org/projects.go

@@ -339,7 +339,7 @@ func ViewProject(ctx *context.Context) {
 	if ctx.Written() {
 		return
 	}
-	assigneeID := ctx.FormInt64("assignee")
+	assigneeID := ctx.FormInt64("assignee") // TODO: use "optional" but not 0 in the future
 
 	issuesMap, err := issues_model.LoadIssuesFromColumnList(ctx, columns, &issues_model.IssuesOptions{
 		LabelIDs:   labelIDs,

routers/web/repo/issue_list.go

@@ -488,7 +488,7 @@ func issues(ctx *context.Context, milestoneID, projectID int64, isPullOption opt
 		viewType = "all"
 	}
 
-	assigneeID := ctx.FormInt64("assignee")
+	assigneeID := ctx.FormInt64("assignee") // TODO: use "optional" but not 0 in the future
 	posterUsername := ctx.FormString("poster")
 	posterUserID := shared_user.GetFilterUserIDByName(ctx, posterUsername)
 	var mentionedID, reviewRequestedID, reviewedID int64

routers/web/repo/projects.go

@@ -310,7 +310,7 @@ func ViewProject(ctx *context.Context) {
 
 	labelIDs := issue.PrepareFilterIssueLabels(ctx, ctx.Repo.Repository.ID, ctx.Repo.Owner)
 
-	assigneeID := ctx.FormInt64("assignee")
+	assigneeID := ctx.FormInt64("assignee") // TODO: use "optional" but not 0 in the future
 
 	issuesMap, err := issues_model.LoadIssuesFromColumnList(ctx, columns, &issues_model.IssuesOptions{
 		LabelIDs:   labelIDs,

routers/web/shared/user/helper.go

@@ -39,7 +39,7 @@ func MakeSelfOnTop(doer *user.User, users []*user.User) []*user.User {
 // * some(NonExistingID): match no issue (due to the user doesn't exist)
 func GetFilterUserIDByName(ctx context.Context, name string) optional.Option[int64] {
 	if name == "" {
-		return nil
+		return optional.None[int64]()
 	}
 	u, err := user.GetUserByName(ctx, name)
 	if err != nil {