STACKITGit/cli
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Restructured switch statement

Browse source 
As much as I like keeping statements as flat as possible, this
not-as-flat revision just seems easier to read.
This commit is contained in:
chemotaxis 2022-04-27 01:43:35 -04:00
parent 3723ab9230
commit 11fde2ee29
1 changed files with 18 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

33
pkg/cmd/issue/lock/lock.go
View file

@ -203,24 +203,27 @@ func padlock(state string, opts *LockOptions) error {
successMsg := fmt.Sprintf("%s %s\n",
cs.SuccessIconWithColor(cs.Green), status(state, issuePr, opts))
switch {
case state == Lock && !issuePr.Locked:
err = lockLockable(httpClient, baseRepo, issuePr, opts)
switch state {
case Lock:
if !issuePr.Locked {
err = lockLockable(httpClient, baseRepo, issuePr, opts)
} else {
var relocked bool
relocked, err = relockLockable(httpClient, baseRepo, issuePr, opts)
case state == Lock && issuePr.Locked:
relocked, errRelock := relockLockable(httpClient, baseRepo, issuePr, opts)
err = errRelock
if !relocked {
successMsg = fmt.Sprintf("%s #%d already locked%s. Nothing changed.\n",
parent.FullName, issuePr.Number, reason(issuePr.ActiveLockReason))
if !relocked {
successMsg = fmt.Sprintf("%s #%d already locked%s. Nothing changed.\n",
parent.FullName, issuePr.Number, reason(issuePr.ActiveLockReason))
}
}
case state == Unlock && issuePr.Locked:
err = unlockLockable(httpClient, baseRepo, issuePr, opts)
case state == Unlock && !issuePr.Locked:
successMsg = fmt.Sprintf("%s #%d already unlocked. Nothing changed.\n",
parent.FullName, issuePr.Number)
case Unlock:
if issuePr.Locked {
err = unlockLockable(httpClient, baseRepo, issuePr, opts)
} else {
successMsg = fmt.Sprintf("%s #%d already unlocked. Nothing changed.\n",
parent.FullName, issuePr.Number)
}
}
if err != nil {