othman.suseno/geek-life
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix golangci-lint warning (#33)

Browse Source
This commit is contained in:
Kuprijanov Roman
2021-12-02 20:38:45 +02:00
committed by GitHub
parent 21defdafdc
commit 9233c89ab6
3 changed files with 20 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/task_detail.go
View File

@@ -90,7 +90,7 @@ func (td *TaskDetailPane) Export() {
} }
content.WriteString("\n" + td.task.Details + " \n") content.WriteString("\n" + td.task.Details + " \n")
clipboard.WriteAll(content.String()) _ = clipboard.WriteAll(content.String())
app.SetFocus(td) app.SetFocus(td)
statusBar.showForSeconds("Task copyed. Try Pasting anywhere.", 5) statusBar.showForSeconds("Task copyed. Try Pasting anywhere.", 5)
} }

8
app/tasks.go
View File

@@ -148,21 +148,21 @@ func (pane *TaskPane) LoadDynamicList(logic string) {
switch logic { switch logic {
case "today": case "today":
tasks, err = pane.taskRepo.GetAllByDateRange(zeroTime, today) tasks, err = pane.taskRepo.GetAllByDateRange(zeroTime, today)
rangeDesc = fmt.Sprintf("Today (and overdue)") rangeDesc = "Today (and overdue)"
case "tomorrow": case "tomorrow":
tomorrow := today.AddDate(0, 0, 1) tomorrow := today.AddDate(0, 0, 1)
tasks, err = pane.taskRepo.GetAllByDate(tomorrow) tasks, err = pane.taskRepo.GetAllByDate(tomorrow)
rangeDesc = fmt.Sprintf("Tomorrow") rangeDesc = "Tomorrow"
case "upcoming": case "upcoming":
week := today.Add(7 * 24 * time.Hour) week := today.Add(7 * 24 * time.Hour)
tasks, err = pane.taskRepo.GetAllByDateRange(today, week) tasks, err = pane.taskRepo.GetAllByDateRange(today, week)
rangeDesc = fmt.Sprintf("Upcoming (next 7 days)") rangeDesc = "Upcoming (next 7 days)"
case "unscheduled": case "unscheduled":
tasks, err = pane.taskRepo.GetAllByDate(zeroTime) tasks, err = pane.taskRepo.GetAllByDate(zeroTime)
rangeDesc = fmt.Sprintf("Unscheduled (task with no due date) ") rangeDesc = "Unscheduled (task with no due date) "
} }
projectPane.activeProject = nil projectPane.activeProject = nil

29
app/util.go
View File

@@ -67,10 +67,11 @@ func ignoreKeyEvt() bool {
} }
// yetToImplement - to use as callback for unimplemented features // yetToImplement - to use as callback for unimplemented features
func yetToImplement(feature string) func() { // `yetToImplement` is unused (deadcode)
message := fmt.Sprintf("[yellow]%s is yet to implement. Please Check in next version.", feature) // func yetToImplement(feature string) func() {
return func() { statusBar.showForSeconds(message, 5) } // message := fmt.Sprintf("[yellow]%s is yet to implement. Please Check in next version.", feature)
} // return func() { statusBar.showForSeconds(message, 5) }
// }
func removeThirdCol() { func removeThirdCol() {
contents.RemoveItem(taskDetailPane) contents.RemoveItem(taskDetailPane)
@@ -83,8 +84,7 @@ func getTaskTitleColor(task model.Task) string {
if task.Completed { if task.Completed {
colorName = "green" colorName = "green"
} else if task.DueDate != 0 { } else if task.DueDate != 0 {
dayDiff := int(time.Unix(task.DueDate, 0).Sub(time.Now()).Hours() / 24) dayDiff := int(time.Until(time.Unix(task.DueDate, 0)).Hours() / 24)
if dayDiff == 0 { if dayDiff == 0 {
colorName = "orange" colorName = "orange"
} else if dayDiff < 0 { } else if dayDiff < 0 {
@@ -111,12 +111,13 @@ func makeTaskListingTitle(task model.Task) string {
return fmt.Sprintf("[%s]%s %s%s", getTaskTitleColor(task), checkbox, prefix, task.Title) return fmt.Sprintf("[%s]%s %s%s", getTaskTitleColor(task), checkbox, prefix, task.Title)
} }
func findProjectByID(id int64) *model.Project { // `findProjectByID` is unused (deadcode)
for i := range projectPane.projects { // func findProjectByID(id int64) *model.Project {
if projectPane.projects[i].ID == id { // for i := range projectPane.projects {
return &projectPane.projects[i] // if projectPane.projects[i].ID == id {
} // return &projectPane.projects[i]
} // }
// }
return nil // return nil
} // }