rename Iif to Ternary

TheFox0x7 · TheFox0x7 · commit 40ae095269c2 · 2025-03-16T22:45:12.000+01:00
diff --git a/cmd/dump.go b/cmd/dump.go
@@ -132,7 +132,7 @@ func runDump(ctx *cli.Context) error {
 		defer outFile.Close()
 	}
 
-	setupConsoleLogger(util.Iif(quite, log.WARN, log.INFO), log.CanColorStderr, os.Stderr)
+	setupConsoleLogger(util.Ternary(quite, log.WARN, log.INFO), log.CanColorStderr, os.Stderr)
 
 	setting.DisableLoggerInit()
 	setting.LoadSettings() // cannot access session settings otherwise
diff --git a/cmd/web.go b/cmd/web.go
@@ -220,7 +220,7 @@ func serveInstalled(ctx *cli.Context) error {
 		}
 	}
 
-	gtprof.EnableBuiltinTracer(util.Iif(setting.IsProd, 2000*time.Millisecond, 100*time.Millisecond))
+	gtprof.EnableBuiltinTracer(util.Ternary(setting.IsProd, 2000*time.Millisecond, 100*time.Millisecond))
 
 	// Set up Chi routes
 	webRoutes := routers.NormalRoutes()
diff --git a/models/asymkey/ssh_key_fingerprint.go b/models/asymkey/ssh_key_fingerprint.go
@@ -77,13 +77,13 @@ func calcFingerprintNative(publicKeyContent string) (string, error) {
 func CalcFingerprint(publicKeyContent string) (string, error) {
 	// Call the method based on configuration
 	useNative := setting.SSH.KeygenPath == ""
-	calcFn := util.Iif(useNative, calcFingerprintNative, calcFingerprintSSHKeygen)
+	calcFn := util.Ternary(useNative, calcFingerprintNative, calcFingerprintSSHKeygen)
 	fp, err := calcFn(publicKeyContent)
 	if err != nil {
 		if IsErrKeyUnableVerify(err) {
 			return "", err
 		}
-		return "", fmt.Errorf("CalcFingerprint(%s): %w", util.Iif(useNative, "native", "ssh-keygen"), err)
+		return "", fmt.Errorf("CalcFingerprint(%s): %w", util.Ternary(useNative, "native", "ssh-keygen"), err)
 	}
 	return fp, nil
 }
diff --git a/models/issues/issue_project.go b/models/issues/issue_project.go
@@ -155,7 +155,7 @@ func IssueAssignOrRemoveProject(ctx context.Context, issue *Issue, doer *user_mo
 			Get(&res); err != nil {
 			return err
 		}
-		newSorting := util.Iif(res.IssueCount > 0, res.MaxSorting+1, 0)
+		newSorting := util.Ternary(res.IssueCount > 0, res.MaxSorting+1, 0)
 		return db.Insert(ctx, &project_model.ProjectIssue{
 			IssueID:         issue.ID,
 			ProjectID:       newProjectID,
diff --git a/models/issues/issue_update.go b/models/issues/issue_update.go
@@ -47,7 +47,7 @@ func IsErrIssueIsClosed(err error) bool {
 }
 
 func (err ErrIssueIsClosed) Error() string {
-	return fmt.Sprintf("%s [id: %d, repo_id: %d, index: %d] is already closed", util.Iif(err.IsPull, "Pull Request", "Issue"), err.ID, err.RepoID, err.Index)
+	return fmt.Sprintf("%s [id: %d, repo_id: %d, index: %d] is already closed", util.Ternary(err.IsPull, "Pull Request", "Issue"), err.ID, err.RepoID, err.Index)
 }
 
 func SetIssueAsClosed(ctx context.Context, issue *Issue, doer *user_model.User, isMergePull bool) (*Comment, error) {
@@ -84,7 +84,7 @@ func SetIssueAsClosed(ctx context.Context, issue *Issue, doer *user_model.User,
 		return nil, ErrIssueAlreadyChanged
 	}
 
-	return updateIssueNumbers(ctx, issue, doer, util.Iif(isMergePull, CommentTypeMergePull, CommentTypeClose))
+	return updateIssueNumbers(ctx, issue, doer, util.Ternary(isMergePull, CommentTypeMergePull, CommentTypeClose))
 }
 
 // ErrIssueIsOpen is used when reopen an opened issue
@@ -102,7 +102,7 @@ func IsErrIssueIsOpen(err error) bool {
 }
 
 func (err ErrIssueIsOpen) Error() string {
-	return fmt.Sprintf("%s [id: %d, repo_id: %d, index: %d] is already open", util.Iif(err.IsPull, "Pull Request", "Issue"), err.ID, err.RepoID, err.Index)
+	return fmt.Sprintf("%s [id: %d, repo_id: %d, index: %d] is already open", util.Ternary(err.IsPull, "Pull Request", "Issue"), err.ID, err.RepoID, err.Index)
 }
 
 func setIssueAsReopen(ctx context.Context, issue *Issue, doer *user_model.User) (*Comment, error) {
diff --git a/models/perm/access/repo_permission.go b/models/perm/access/repo_permission.go
@@ -77,12 +77,12 @@ func (p *Permission) GetFirstUnitRepoID() int64 {
 func (p *Permission) UnitAccessMode(unitType unit.Type) perm_model.AccessMode {
 	// if the units map contains the access mode, use it, but admin/owner mode could override it
 	if m, ok := p.unitsMode[unitType]; ok {
-		return util.Iif(p.AccessMode >= perm_model.AccessModeAdmin, p.AccessMode, m)
+		return util.Ternary(p.AccessMode >= perm_model.AccessModeAdmin, p.AccessMode, m)
 	}
 	// if the units map does not contain the access mode, return the default access mode if the unit exists
 	unitDefaultAccessMode := max(p.AccessMode, p.everyoneAccessMode[unitType])
 	hasUnit := slices.ContainsFunc(p.units, func(u *repo_model.RepoUnit) bool { return u.Type == unitType })
-	return util.Iif(hasUnit, unitDefaultAccessMode, perm_model.AccessModeNone)
+	return util.Ternary(hasUnit, unitDefaultAccessMode, perm_model.AccessModeNone)
 }
 
 func (p *Permission) SetUnitsWithDefaultAccessMode(units []*repo_model.RepoUnit, mode perm_model.AccessMode) {
diff --git a/models/perm/access_mode.go b/models/perm/access_mode.go
@@ -58,7 +58,7 @@ func ParseAccessMode(permission string, allowed ...AccessMode) AccessMode {
 	if len(allowed) == 0 {
 		return m
 	}
-	return util.Iif(slices.Contains(allowed, m), m, AccessModeNone)
+	return util.Ternary(slices.Contains(allowed, m), m, AccessModeNone)
 }
 
 // ErrInvalidAccessMode is returned when an invalid access mode is used
diff --git a/models/project/column.go b/models/project/column.go
@@ -149,7 +149,7 @@ func NewColumn(ctx context.Context, column *Column) error {
 	if res.ColumnCount >= maxProjectColumns {
 		return fmt.Errorf("NewBoard: maximum number of columns reached")
 	}
-	column.Sorting = int8(util.Iif(res.ColumnCount > 0, res.MaxSorting+1, 0))
+	column.Sorting = int8(util.Ternary(res.ColumnCount > 0, res.MaxSorting+1, 0))
 	_, err := db.GetEngine(ctx).Insert(column)
 	return err
 }
diff --git a/models/project/issue.go b/models/project/issue.go
@@ -62,7 +62,7 @@ func (c *Column) moveIssuesToAnotherColumn(ctx context.Context, newColumn *Colum
 		return nil
 	}
 
-	nextSorting := util.Iif(res.IssueCount > 0, res.MaxSorting+1, 0)
+	nextSorting := util.Ternary(res.IssueCount > 0, res.MaxSorting+1, 0)
 	return db.WithTx(ctx, func(ctx context.Context) error {
 		for i, issue := range issues {
 			issue.ProjectColumnID = newColumn.ID
diff --git a/modules/httpcache/httpcache.go b/modules/httpcache/httpcache.go
@@ -26,7 +26,7 @@ func SetCacheControlInHeader(h http.Header, opts *CacheControlOptions) {
 
 	// "max-age=0 + must-revalidate" (aka "no-cache") is preferred instead of "no-store"
 	// because browsers may restore some input fields after navigate-back / reload a page.
-	publicPrivate := util.Iif(opts.IsPublic, "public", "private")
+	publicPrivate := util.Ternary(opts.IsPublic, "public", "private")
 	if setting.IsProd {
 		if opts.MaxAge == 0 {
 			directives = append(directives, "max-age=0", "private", "must-revalidate")
diff --git a/modules/httplib/url.go b/modules/httplib/url.go
@@ -45,10 +45,10 @@ func getRequestScheme(req *http.Request) string {
 		return s
 	}
 	if s := req.Header.Get("Front-End-Https"); s != "" {
-		return util.Iif(s == "on", "https", "http")
+		return util.Ternary(s == "on", "https", "http")
 	}
 	if s := req.Header.Get("X-Forwarded-Ssl"); s != "" {
-		return util.Iif(s == "on", "https", "http")
+		return util.Ternary(s == "on", "https", "http")
 	}
 	return ""
 }
@@ -119,7 +119,7 @@ func detectURLRoutePath(ctx context.Context, s string) (routePath string, ut url
 	cleanedPath := ""
 	if u.Path != "" {
 		cleanedPath = util.PathJoinRelX(u.Path)
-		cleanedPath = util.Iif(cleanedPath == ".", "", "/"+cleanedPath)
+		cleanedPath = util.Ternary(cleanedPath == ".", "", "/"+cleanedPath)
 	}
 	if urlIsRelative(s, u) {
 		if u.Path == "" {
diff --git a/modules/markup/html_internal_test.go b/modules/markup/html_internal_test.go
@@ -33,7 +33,7 @@ func numericIssueLink(baseURL, class string, index int, marker string) string {
 
 // link an HTML link
 func link(href, class, contents string) string {
-	extra := util.Iif(class != "", ` class="`+class+`"`, "")
+	extra := util.Ternary(class != "", ` class="`+class+`"`, "")
 	return fmt.Sprintf(`<a href="%s"%s>%s</a>`, href, extra, contents)
 }
 
diff --git a/modules/markup/html_issue.go b/modules/markup/html_issue.go
@@ -159,9 +159,9 @@ func issueIndexPatternProcessor(ctx *RenderContext, node *html.Node) {
 			// Path determines the type of link that will be rendered. It's unknown at this point whether
 			// the linked item is actually a PR or an issue. Luckily it's of no real consequence because
 			// Gitea will redirect on click as appropriate.
-			issueOwner := util.Iif(ref.Owner == "", ctx.RenderOptions.Metas["user"], ref.Owner)
-			issueRepo := util.Iif(ref.Owner == "", ctx.RenderOptions.Metas["repo"], ref.Name)
-			issuePath := util.Iif(ref.IsPull, "pulls", "issues")
+			issueOwner := util.Ternary(ref.Owner == "", ctx.RenderOptions.Metas["user"], ref.Owner)
+			issueRepo := util.Ternary(ref.Owner == "", ctx.RenderOptions.Metas["repo"], ref.Name)
+			issuePath := util.Ternary(ref.IsPull, "pulls", "issues")
 			linkHref := ctx.RenderHelper.ResolveLink(util.URLJoin(issueOwner, issueRepo, issuePath, ref.Issue), LinkTypeApp)
 
 			// at the moment, only render the issue index in a full line (or simple line) as icon+title
diff --git a/modules/markup/markdown/math/block_parser.go b/modules/markup/markdown/math/block_parser.go
@@ -55,7 +55,7 @@ func (b *blockParser) Open(parent ast.Node, reader text.Reader, pc parser.Contex
 	node := NewBlock(dollars, pos)
 
 	// Now we need to check if the ending block is on the segment...
-	endBytes := giteaUtil.Iif(dollars, b.endBytesDollars, b.endBytesSquare)
+	endBytes := giteaUtil.Ternary(dollars, b.endBytesDollars, b.endBytesSquare)
 	idx := bytes.Index(line[pos+2:], endBytes)
 	if idx >= 0 {
 		// for case: "$$ ... $$ any other text" (this case will be handled by the inline parser)
@@ -94,16 +94,16 @@ func (b *blockParser) Continue(node ast.Node, reader text.Reader, pc parser.Cont
 	line, segment := reader.PeekLine()
 	w, pos := util.IndentWidth(line, reader.LineOffset())
 	if w < 4 {
-		endBytes := giteaUtil.Iif(block.Dollars, b.endBytesDollars, b.endBytesSquare)
+		endBytes := giteaUtil.Ternary(block.Dollars, b.endBytesDollars, b.endBytesSquare)
 		if bytes.HasPrefix(line[pos:], endBytes) && util.IsBlank(line[pos+len(endBytes):]) {
 			if util.IsBlank(line[pos+len(endBytes):]) {
-				newline := giteaUtil.Iif(line[len(line)-1] != '\n', 0, 1)
+				newline := giteaUtil.Ternary(line[len(line)-1] != '\n', 0, 1)
 				reader.Advance(segment.Stop - segment.Start - newline + segment.Padding)
 				return parser.Close
 			}
 		}
 	}
-	start := segment.Start + giteaUtil.Iif(pos > block.Indent, block.Indent, pos)
+	start := segment.Start + giteaUtil.Ternary(pos > block.Indent, block.Indent, pos)
 	seg := text.NewSegmentPadding(start, segment.Stop, segment.Padding)
 	node.Lines().Append(seg)
 	return parser.Continue | parser.NoChildren
diff --git a/modules/markup/markdown/math/block_renderer.go b/modules/markup/markdown/math/block_renderer.go
@@ -51,11 +51,11 @@ func (r *BlockRenderer) writeLines(w util.BufWriter, source []byte, n gast.Node)
 func (r *BlockRenderer) renderBlock(w util.BufWriter, source []byte, node gast.Node, entering bool) (gast.WalkStatus, error) {
 	n := node.(*Block)
 	if entering {
-		code := giteaUtil.Iif(n.Inline, "", `<pre class="code-block is-loading">`) + `<code class="language-math display">`
+		code := giteaUtil.Ternary(n.Inline, "", `<pre class="code-block is-loading">`) + `<code class="language-math display">`
 		_ = r.renderInternal.FormatWithSafeAttrs(w, template.HTML(code))
 		r.writeLines(w, source, n)
 	} else {
-		_, _ = w.WriteString(`</code>` + giteaUtil.Iif(n.Inline, "", `</pre>`) + "\n")
+		_, _ = w.WriteString(`</code>` + giteaUtil.Ternary(n.Inline, "", `</pre>`) + "\n")
 	}
 	return gast.WalkContinue, nil
 }
diff --git a/modules/templates/helper_test.go b/modules/templates/helper_test.go
@@ -77,7 +77,7 @@ func TestTemplateIif(t *testing.T) {
 		w.Reset()
 		assert.NoError(t, tmpl.Execute(w, struct{ Value any }{v}), "case %d (%T) %#v fails", i, v, v)
 		out := w.String()
-		truthyCount += util.Iif(out == "true:true", 1, 0)
+		truthyCount += util.Ternary(out == "true:true", 1, 0)
 		truthyMatches := out == "true:true" || out == "false:false"
 		assert.True(t, truthyMatches, "case %d (%T) %#v fail: %s", i, v, v, out)
 	}
diff --git a/modules/templates/util_render.go b/modules/templates/util_render.go
@@ -221,7 +221,7 @@ func (ut *RenderUtils) MarkdownToHtml(input string) template.HTML { //nolint:rev
 
 func (ut *RenderUtils) RenderLabels(labels []*issues_model.Label, repoLink string, issue *issues_model.Issue) template.HTML {
 	isPullRequest := issue != nil && issue.IsPull
-	baseLink := fmt.Sprintf("%s/%s", repoLink, util.Iif(isPullRequest, "pulls", "issues"))
+	baseLink := fmt.Sprintf("%s/%s", repoLink, util.Ternary(isPullRequest, "pulls", "issues"))
 	htmlCode := `<span class="labels-list">`
 	for _, label := range labels {
 		// Protect against nil value in labels - shouldn't happen but would cause a panic if so
diff --git a/modules/util/util.go b/modules/util/util.go
@@ -213,8 +213,8 @@ func ToPointer[T any](val T) *T {
 	return &val
 }
 
-// Iif is an "inline-if", it returns "trueVal" if "condition" is true, otherwise "falseVal"
-func Iif[T any](condition bool, trueVal, falseVal T) T {
+// Ternary is an "inline-if", it returns "trueVal" if "condition" is true, otherwise "falseVal"
+func Ternary[T any](condition bool, trueVal, falseVal T) T {
 	if condition {
 		return trueVal
 	}
diff --git a/modules/web/router_test.go b/modules/web/router_test.go
@@ -27,7 +27,7 @@ func chiURLParamsToMap(chiCtx *chi.Context) map[string]string {
 		}
 		m[key] = pathParams.Values[i]
 	}
-	return util.Iif(len(m) == 0, nil, m)
+	return util.Ternary(len(m) == 0, nil, m)
 }
 
 func TestPathProcessor(t *testing.T) {
diff --git a/routers/api/packages/rubygems/rubygems.go b/routers/api/packages/rubygems/rubygems.go
@@ -347,7 +347,7 @@ func GetAllPackagesVersions(ctx *context.Context) {
 		// format: RUBYGEM [-]VERSION_PLATFORM[,VERSION_PLATFORM],...] MD5
 		_, _ = fmt.Fprintf(out, "%s ", pkg.Name)
 		for i, v := range versions {
-			sep := util.Iif(i == len(versions)-1, "", ",")
+			sep := util.Ternary(i == len(versions)-1, "", ",")
 			_, _ = fmt.Fprintf(out, "%s%s", v.Version, sep)
 		}
 		_, _ = fmt.Fprintf(out, " %x\n", md5.Sum([]byte(info)))
@@ -362,7 +362,7 @@ func writePackageVersionRequirements(prefix string, reqs []rubygems_module.Versi
 		reqs = []rubygems_module.VersionRequirement{{Restriction: ">=", Version: "0"}}
 	}
 	for i, req := range reqs {
-		sep := util.Iif(i == 0, "", "&")
+		sep := util.Ternary(i == 0, "", "&")
 		_, _ = fmt.Fprintf(out, "%s%s %s", sep, req.Restriction, req.Version)
 	}
 }
@@ -391,7 +391,7 @@ func makePackageVersionDependency(ctx *context.Context, version *packages_model.
 	buf.WriteString(version.Version)
 	buf.WriteByte(' ')
 	for i, dep := range metadata.RuntimeDependencies {
-		sep := util.Iif(i == 0, "", ",")
+		sep := util.Ternary(i == 0, "", ",")
 		writePackageVersionRequirements(fmt.Sprintf("%s%s:", sep, dep.Name), dep.Version, buf)
 	}
 	_, _ = fmt.Fprintf(buf, "|checksum:%s", blob.HashSHA256)
diff --git a/routers/api/v1/misc/markup.go b/routers/api/v1/misc/markup.go
@@ -42,7 +42,7 @@ func Markup(ctx *context.APIContext) {
 		return
 	}
 
-	mode := util.Iif(form.Wiki, "wiki", form.Mode) //nolint:staticcheck
+	mode := util.Ternary(form.Wiki, "wiki", form.Mode) //nolint:staticcheck
 	common.RenderMarkup(ctx.Base, ctx.Repo, mode, form.Text, form.Context, form.FilePath)
 }
 
@@ -73,7 +73,7 @@ func Markdown(ctx *context.APIContext) {
 		return
 	}
 
-	mode := util.Iif(form.Wiki, "wiki", form.Mode) //nolint:staticcheck
+	mode := util.Ternary(form.Wiki, "wiki", form.Mode) //nolint:staticcheck
 	common.RenderMarkup(ctx.Base, ctx.Repo, mode, form.Text, form.Context, "")
 }
 
diff --git a/routers/init.go b/routers/init.go
@@ -116,7 +116,7 @@ func InitWebInstalled(ctx context.Context) {
 	mustInitCtx(ctx, git.InitFull)
 	log.Info("Git version: %s (home: %s)", git.DefaultFeatures().VersionInfo(), git.HomeDir())
 	if !git.DefaultFeatures().SupportHashSha256 {
-		log.Warn("sha256 hash support is disabled - requires Git >= 2.42." + util.Iif(git.DefaultFeatures().UsingGogit, " Gogit is currently unsupported.", ""))
+		log.Warn("sha256 hash support is disabled - requires Git >= 2.42." + util.Ternary(git.DefaultFeatures().UsingGogit, " Gogit is currently unsupported.", ""))
 	}
 
 	// Setup i18n
diff --git a/routers/web/devtest/mock_actions.go b/routers/web/devtest/mock_actions.go
@@ -31,7 +31,7 @@ func generateMockStepsLog(logCur actions.LogCursor) (stepsLog []*actions.ViewSte
 		"##[endgroup]",
 	}
 	cur := logCur.Cursor // usually the cursor is the "file offset", but here we abuse it as "line number" to make the mock easier, intentionally
-	mockCount := util.Iif(logCur.Step == 0, 3, 1)
+	mockCount := util.Ternary(logCur.Step == 0, 3, 1)
 	if logCur.Step == 1 && logCur.Cursor == 0 {
 		mockCount = 30 // for the first batch, return as many as possible to test the auto-expand and auto-scroll
 	}
diff --git a/routers/web/explore/org.go b/routers/web/explore/org.go
@@ -40,7 +40,7 @@ func Organizations(ctx *context.Context) {
 	)
 	sortOrder := ctx.FormString("sort")
 	if sortOrder == "" {
-		sortOrder = util.Iif(supportedSortOrders.Contains(setting.UI.ExploreDefaultSort), setting.UI.ExploreDefaultSort, "newest")
+		sortOrder = util.Ternary(supportedSortOrders.Contains(setting.UI.ExploreDefaultSort), setting.UI.ExploreDefaultSort, "newest")
 		ctx.SetFormString("sort", sortOrder)
 	}
 
diff --git a/routers/web/explore/user.go b/routers/web/explore/user.go
@@ -147,7 +147,7 @@ func Users(ctx *context.Context) {
 	)
 	sortOrder := ctx.FormString("sort")
 	if sortOrder == "" {
-		sortOrder = util.Iif(supportedSortOrders.Contains(setting.UI.ExploreDefaultSort), setting.UI.ExploreDefaultSort, "newest")
+		sortOrder = util.Ternary(supportedSortOrders.Contains(setting.UI.ExploreDefaultSort), setting.UI.ExploreDefaultSort, "newest")
 		ctx.SetFormString("sort", sortOrder)
 	}
 
diff --git a/routers/web/misc/markup.go b/routers/web/misc/markup.go
@@ -15,6 +15,6 @@ import (
 // Markup render markup document to HTML
 func Markup(ctx *context.Context) {
 	form := web.GetForm(ctx).(*api.MarkupOption)
-	mode := util.Iif(form.Wiki, "wiki", form.Mode) //nolint:staticcheck
+	mode := util.Ternary(form.Wiki, "wiki", form.Mode) //nolint:staticcheck
 	common.RenderMarkup(ctx.Base, ctx.Repo, mode, form.Text, form.Context, form.FilePath)
 }
diff --git a/routers/web/org/home.go b/routers/web/org/home.go
@@ -155,7 +155,7 @@ func home(ctx *context.Context, viewRepositories bool) {
 }
 
 func prepareOrgProfileReadme(ctx *context.Context, prepareResult *shared_user.PrepareOrgHeaderResult) bool {
-	viewAs := ctx.FormString("view_as", util.Iif(ctx.Org.IsMember, "member", "public"))
+	viewAs := ctx.FormString("view_as", util.Ternary(ctx.Org.IsMember, "member", "public"))
 	viewAsMember := viewAs == "member"
 
 	var profileRepo *repo_model.Repository
diff --git a/routers/web/repo/actions/view.go b/routers/web/repo/actions/view.go
@@ -169,7 +169,7 @@ func getActionsViewArtifacts(ctx context.Context, repoID, runIndex int64) (artif
 		artifactsViewItems = append(artifactsViewItems, &ArtifactsViewItem{
 			Name:   art.ArtifactName,
 			Size:   art.FileSize,
-			Status: util.Iif(art.Status == actions_model.ArtifactStatusExpired, "expired", "completed"),
+			Status: util.Ternary(art.Status == actions_model.ArtifactStatusExpired, "expired", "completed"),
 		})
 	}
 	return artifactsViewItems, nil
diff --git a/routers/web/repo/editor.go b/routers/web/repo/editor.go
@@ -190,7 +190,7 @@ func editFile(ctx *context.Context, isNewFile bool) {
 	ctx.Data["TreePaths"] = treePaths
 	ctx.Data["commit_summary"] = ""
 	ctx.Data["commit_message"] = ""
-	ctx.Data["commit_choice"] = util.Iif(canCommit, frmCommitChoiceDirect, frmCommitChoiceNewBranch)
+	ctx.Data["commit_choice"] = util.Ternary(canCommit, frmCommitChoiceDirect, frmCommitChoiceNewBranch)
 	ctx.Data["new_branch_name"] = GetUniquePatchBranchName(ctx)
 	ctx.Data["last_commit"] = ctx.Repo.CommitID
 
diff --git a/routers/web/repo/release.go b/routers/web/repo/release.go
@@ -465,7 +465,7 @@ func NewReleasePost(ctx *context.Context) {
 		return
 	}
 
-	attachmentUUIDs := util.Iif(setting.Attachment.Enabled, form.Files, nil)
+	attachmentUUIDs := util.Ternary(setting.Attachment.Enabled, form.Files, nil)
 
 	// no existing release, create a new release
 	if rel == nil {
diff --git a/services/convert/issue.go b/services/convert/issue.go
@@ -58,7 +58,7 @@ func toIssue(ctx context.Context, doer *user_model.User, issue *issues_model.Iss
 		Comments:    issue.NumComments,
 		Created:     issue.CreatedUnix.AsTime(),
 		Updated:     issue.UpdatedUnix.AsTime(),
-		PinOrder:    util.Iif(issue.PinOrder == -1, 0, issue.PinOrder), // -1 means loaded with no pin order
+		PinOrder:    util.Ternary(issue.PinOrder == -1, 0, issue.PinOrder), // -1 means loaded with no pin order
 	}
 
 	if issue.Repo != nil {
diff --git a/services/convert/pull.go b/services/convert/pull.go
@@ -95,7 +95,7 @@ func ToAPIPullRequest(ctx context.Context, pr *issues_model.PullRequest, doer *u
 		Deadline:       apiIssue.Deadline,
 		Created:        pr.Issue.CreatedUnix.AsTimePtr(),
 		Updated:        pr.Issue.UpdatedUnix.AsTimePtr(),
-		PinOrder:       util.Iif(apiIssue.PinOrder == -1, 0, apiIssue.PinOrder),
+		PinOrder:       util.Ternary(apiIssue.PinOrder == -1, 0, apiIssue.PinOrder),
 
 		// output "[]" rather than null to align to github outputs
 		RequestedReviewers:      []*api.User{},
@@ -377,7 +377,7 @@ func ToAPIPullRequests(ctx context.Context, baseRepo *repo_model.Repository, prs
 			Deadline:       apiIssue.Deadline,
 			Created:        pr.Issue.CreatedUnix.AsTimePtr(),
 			Updated:        pr.Issue.UpdatedUnix.AsTimePtr(),
-			PinOrder:       util.Iif(apiIssue.PinOrder == -1, 0, apiIssue.PinOrder),
+			PinOrder:       util.Ternary(apiIssue.PinOrder == -1, 0, apiIssue.PinOrder),
 
 			AllowMaintainerEdit: pr.AllowMaintainerEdit,
 
diff --git a/services/gitdiff/gitdiff.go b/services/gitdiff/gitdiff.go
diff --git a/services/repository/files/tree.go b/services/repository/files/tree.go
diff --git a/tests/integration/auth_ldap_test.go b/tests/integration/auth_ldap_test.go

Original file line number	Diff line number	Diff line change
`@@ -132,7 +132,7 @@ func runDump(ctx *cli.Context) error {`
`132`	`132`	`defer outFile.Close()`
`133`	`133`	`}`
`134`	`134`
`135`		`- setupConsoleLogger(util.Iif(quite, log.WARN, log.INFO), log.CanColorStderr, os.Stderr)`
	`135`	`+ setupConsoleLogger(util.Ternary(quite, log.WARN, log.INFO), log.CanColorStderr, os.Stderr)`
`136`	`136`
`137`	`137`	`setting.DisableLoggerInit()`
`138`	`138`	`setting.LoadSettings() // cannot access session settings otherwise`
Original file line number	Diff line number	Diff line change
`@@ -220,7 +220,7 @@ func serveInstalled(ctx *cli.Context) error {`
`220`	`220`	`}`
`221`	`221`	`}`
`222`	`222`
`223`		`- gtprof.EnableBuiltinTracer(util.Iif(setting.IsProd, 2000time.Millisecond, 100time.Millisecond))`
	`223`	`+ gtprof.EnableBuiltinTracer(util.Ternary(setting.IsProd, 2000time.Millisecond, 100time.Millisecond))`
`224`	`224`
`225`	`225`	`// Set up Chi routes`
`226`	`226`	`webRoutes := routers.NormalRoutes()`
Original file line number	Diff line number	Diff line change
`@@ -77,13 +77,13 @@ func calcFingerprintNative(publicKeyContent string) (string, error) {`
`77`	`77`	`func CalcFingerprint(publicKeyContent string) (string, error) {`
`78`	`78`	`// Call the method based on configuration`
`79`	`79`	`useNative := setting.SSH.KeygenPath == ""`
`80`		`- calcFn := util.Iif(useNative, calcFingerprintNative, calcFingerprintSSHKeygen)`
	`80`	`+ calcFn := util.Ternary(useNative, calcFingerprintNative, calcFingerprintSSHKeygen)`
`81`	`81`	`fp, err := calcFn(publicKeyContent)`
`82`	`82`	`if err != nil {`
`83`	`83`	`if IsErrKeyUnableVerify(err) {`
`84`	`84`	`return "", err`
`85`	`85`	`}`
`86`		`- return "", fmt.Errorf("CalcFingerprint(%s): %w", util.Iif(useNative, "native", "ssh-keygen"), err)`
	`86`	`+ return "", fmt.Errorf("CalcFingerprint(%s): %w", util.Ternary(useNative, "native", "ssh-keygen"), err)`
`87`	`87`	`}`
`88`	`88`	`return fp, nil`
`89`	`89`	`}`
Original file line number	Diff line number	Diff line change
`@@ -155,7 +155,7 @@ func IssueAssignOrRemoveProject(ctx context.Context, issue Issue, doer user_mo`
`155`	`155`	`Get(&res); err != nil {`
`156`	`156`	`return err`
`157`	`157`	`}`
`158`		`- newSorting := util.Iif(res.IssueCount > 0, res.MaxSorting+1, 0)`
	`158`	`+ newSorting := util.Ternary(res.IssueCount > 0, res.MaxSorting+1, 0)`
`159`	`159`	`return db.Insert(ctx, &project_model.ProjectIssue{`
`160`	`160`	`IssueID: issue.ID,`
`161`	`161`	`ProjectID: newProjectID,`
Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ func IsErrIssueIsClosed(err error) bool {`
`47`	`47`	`}`
`48`	`48`
`49`	`49`	`func (err ErrIssueIsClosed) Error() string {`
`50`		`- return fmt.Sprintf("%s [id: %d, repo_id: %d, index: %d] is already closed", util.Iif(err.IsPull, "Pull Request", "Issue"), err.ID, err.RepoID, err.Index)`
	`50`	`+ return fmt.Sprintf("%s [id: %d, repo_id: %d, index: %d] is already closed", util.Ternary(err.IsPull, "Pull Request", "Issue"), err.ID, err.RepoID, err.Index)`
`51`	`51`	`}`
`52`	`52`
`53`	`53`	`func SetIssueAsClosed(ctx context.Context, issue Issue, doer user_model.User, isMergePull bool) (*Comment, error) {`
`@@ -84,7 +84,7 @@ func SetIssueAsClosed(ctx context.Context, issue Issue, doer user_model.User,`
`84`	`84`	`return nil, ErrIssueAlreadyChanged`
`85`	`85`	`}`
`86`	`86`
`87`		`- return updateIssueNumbers(ctx, issue, doer, util.Iif(isMergePull, CommentTypeMergePull, CommentTypeClose))`
	`87`	`+ return updateIssueNumbers(ctx, issue, doer, util.Ternary(isMergePull, CommentTypeMergePull, CommentTypeClose))`
`88`	`88`	`}`
`89`	`89`
`90`	`90`	`// ErrIssueIsOpen is used when reopen an opened issue`
`@@ -102,7 +102,7 @@ func IsErrIssueIsOpen(err error) bool {`
`102`	`102`	`}`
`103`	`103`
`104`	`104`	`func (err ErrIssueIsOpen) Error() string {`
`105`		`- return fmt.Sprintf("%s [id: %d, repo_id: %d, index: %d] is already open", util.Iif(err.IsPull, "Pull Request", "Issue"), err.ID, err.RepoID, err.Index)`
	`105`	`+ return fmt.Sprintf("%s [id: %d, repo_id: %d, index: %d] is already open", util.Ternary(err.IsPull, "Pull Request", "Issue"), err.ID, err.RepoID, err.Index)`
`106`	`106`	`}`
`107`	`107`
`108`	`108`	`func setIssueAsReopen(ctx context.Context, issue Issue, doer user_model.User) (*Comment, error) {`
Original file line number	Diff line number	Diff line change
`@@ -77,12 +77,12 @@ func (p *Permission) GetFirstUnitRepoID() int64 {`
`77`	`77`	`func (p *Permission) UnitAccessMode(unitType unit.Type) perm_model.AccessMode {`
`78`	`78`	`// if the units map contains the access mode, use it, but admin/owner mode could override it`
`79`	`79`	`if m, ok := p.unitsMode[unitType]; ok {`
`80`		`- return util.Iif(p.AccessMode >= perm_model.AccessModeAdmin, p.AccessMode, m)`
	`80`	`+ return util.Ternary(p.AccessMode >= perm_model.AccessModeAdmin, p.AccessMode, m)`
`81`	`81`	`}`
`82`	`82`	`// if the units map does not contain the access mode, return the default access mode if the unit exists`
`83`	`83`	`unitDefaultAccessMode := max(p.AccessMode, p.everyoneAccessMode[unitType])`
`84`	`84`	`hasUnit := slices.ContainsFunc(p.units, func(u *repo_model.RepoUnit) bool { return u.Type == unitType })`
`85`		`- return util.Iif(hasUnit, unitDefaultAccessMode, perm_model.AccessModeNone)`
	`85`	`+ return util.Ternary(hasUnit, unitDefaultAccessMode, perm_model.AccessModeNone)`
`86`	`86`	`}`
`87`	`87`
`88`	`88`	`func (p Permission) SetUnitsWithDefaultAccessMode(units []repo_model.RepoUnit, mode perm_model.AccessMode) {`
Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,7 @@ func ParseAccessMode(permission string, allowed ...AccessMode) AccessMode {`
`58`	`58`	`if len(allowed) == 0 {`
`59`	`59`	`return m`
`60`	`60`	`}`
`61`		`- return util.Iif(slices.Contains(allowed, m), m, AccessModeNone)`
	`61`	`+ return util.Ternary(slices.Contains(allowed, m), m, AccessModeNone)`
`62`	`62`	`}`
`63`	`63`
`64`	`64`	`// ErrInvalidAccessMode is returned when an invalid access mode is used`
Original file line number	Diff line number	Diff line change
`@@ -149,7 +149,7 @@ func NewColumn(ctx context.Context, column *Column) error {`
`149`	`149`	`if res.ColumnCount >= maxProjectColumns {`
`150`	`150`	`return fmt.Errorf("NewBoard: maximum number of columns reached")`
`151`	`151`	`}`
`152`		`- column.Sorting = int8(util.Iif(res.ColumnCount > 0, res.MaxSorting+1, 0))`
	`152`	`+ column.Sorting = int8(util.Ternary(res.ColumnCount > 0, res.MaxSorting+1, 0))`
`153`	`153`	`_, err := db.GetEngine(ctx).Insert(column)`
`154`	`154`	`return err`
`155`	`155`	`}`
Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,7 @@ func (c Column) moveIssuesToAnotherColumn(ctx context.Context, newColumn Colum`
`62`	`62`	`return nil`
`63`	`63`	`}`
`64`	`64`
`65`		`- nextSorting := util.Iif(res.IssueCount > 0, res.MaxSorting+1, 0)`
	`65`	`+ nextSorting := util.Ternary(res.IssueCount > 0, res.MaxSorting+1, 0)`
`66`	`66`	`return db.WithTx(ctx, func(ctx context.Context) error {`
`67`	`67`	`for i, issue := range issues {`
`68`	`68`	`issue.ProjectColumnID = newColumn.ID`