some bug fixes

viewmodels/issue_view.go

@@ -53,19 +53,13 @@ func NewSingleIssueView(y string, no string, lib *xmlprovider.Library) (*IssueVM
 }
 
 func PiecesForIsssue(lib *xmlprovider.Library, issue xmlprovider.Issue) (*PiecesByPage, *PiecesByPage, error) {
-	date := issue.Datum.Date()
-	if date == nil {
-		return nil, nil, fmt.Errorf("Issue has no date")
-	}
-
-	year := date.Year()
+	year := issue.Datum.When.Year
 
 	ppi := PiecesByPage{Items: make(map[int][]PieceListitemVM)}
 	ppa := PiecesByPage{Items: make(map[int][]PieceListitemVM)}
 
 	slog.Debug(fmt.Sprintf("Checking piece for year %v, number %v", year, issue.Number.No))
 	for _, piece := range lib.Pieces.Array {
-
 		if d, ok := piece.ReferencesIssue(year, issue.Number.No); ok {
 			slog.Debug(fmt.Sprintf("Found piece %v in issue %v-%v", piece, year, issue.Number.No))
 			p := PieceListitemVM{Piece: piece, Reference: *d}

viewmodels/issuelistitem_view.go

@@ -1,8 +1,6 @@
 package viewmodels
 
 import (
-	"time"
-
 	"github.com/Theodor-Springmann-Stiftung/kgpz_web/providers/xmlprovider"
 )
 
@@ -17,16 +15,11 @@ type IssueListitemVM struct {
 }
 
 func ListitemFromIssue(i xmlprovider.Issue) (*IssueListitemVM, error) {
-	t, err := time.Parse(TLAYOUT, i.Datum.When)
-	if err != nil {
-		return nil, err
-	}
-
 	return &IssueListitemVM{
 		No:    i.Number.No,
 		Issue: i,
-		Day:   t.Day(),
-		Month: int(t.Month()),
-		Year:  t.Year(),
+		Day:   i.Datum.When.Day,
+		Month: i.Datum.When.Month,
+		Year:  i.Datum.When.Year,
 	}, nil
 }

viewmodels/year_view.go

@@ -31,12 +31,11 @@ func YearView(year int, lib *xmlprovider.Library) (*YearVM, error) {
 
 	lib.Issues.Lock()
 	for _, issue := range lib.Issues.Array {
-		if y, err := issue.Year(); err == nil {
-			years[y] = true
-			if y == year {
-				if issuevm, err := ListitemFromIssue(issue); err == nil {
-					issues[issuevm.Month] = append(issues[issuevm.Month], *issuevm)
-				}
+		y := issue.Datum.When.Year
+		years[y] = true
+		if y == year {
+			if issuevm, err := ListitemFromIssue(issue); err == nil {
+				issues[issuevm.Month] = append(issues[issuevm.Month], *issuevm)
 			}
 		}
 	}