From 3981677d98f94070762fea02803ff8e91044bbee Mon Sep 17 00:00:00 2001 From: Robin Olsen Date: Thu, 5 Feb 2026 13:55:20 +0100 Subject: [PATCH] chack for nil pointer in helper use helper for converting to issue --- internal/models/beads.go | 6 ++++-- internal/service/beads.go | 7 +------ 2 files changed, 5 insertions(+), 8 deletions(-) diff --git a/internal/models/beads.go b/internal/models/beads.go index 83ec37b..0e9c663 100644 --- a/internal/models/beads.go +++ b/internal/models/beads.go @@ -81,8 +81,10 @@ const ( func IssuesPtrToIssues(issuePtr []*Issue) []Issue { issues := make([]Issue, 0, len(issuePtr)) - for _, issue := range issuePtr { - issues = append(issues, *issue) + for _, issuePtr := range issuePtr { + if issuePtr != nil { + issues = append(issues, *issuePtr) + } } return issues } diff --git a/internal/service/beads.go b/internal/service/beads.go index e9a191d..06c8373 100644 --- a/internal/service/beads.go +++ b/internal/service/beads.go @@ -37,12 +37,7 @@ func (s *BeadsService) AllIssues(ctx context.Context) ([]models.Issue, error) { return []models.Issue{}, nil } - issues := make([]models.Issue, 0, len(issuesPtr)) - for _, issuePtr := range issuesPtr { - if issuePtr != nil { - issues = append(issues, *issuePtr) - } - } + issues := models.IssuesPtrToIssues(issuesPtr) return issues, nil }