From 0b70a8ee10d3b4949ed3099221a2f0f94717165b Mon Sep 17 00:00:00 2001
From: Jason Streifling <jason@streifling.com>
Date: Sun, 20 Oct 2024 15:47:10 +0200
Subject: [PATCH] Turn updateDateTime into *Date.Update

---
 atom.go  | 10 ----------
 date.go  |  9 +++++++++
 entry.go | 20 ++++++++++----------
 feed.go  | 26 +++++++++++++-------------
 4 files changed, 32 insertions(+), 33 deletions(-)

diff --git a/atom.go b/atom.go
index cfa97ba..8158efb 100644
--- a/atom.go
+++ b/atom.go
@@ -7,7 +7,6 @@ import (
 	"mime"
 	"regexp"
 	"strings"
-	"time"
 
 	"github.com/google/uuid"
 	"golang.org/x/text/language"
@@ -110,15 +109,6 @@ func isValidAttribute(attribute string) bool {
 	return regex.MatchString(attribute)
 }
 
-// update updates the Date
-func updateDateTime(d *Date) {
-	if d == nil {
-		d = NewDate(time.Now())
-	} else {
-		d.DateTime = DateTime(time.Now())
-	}
-}
-
 // NewURN generates an new valid IRI based on a UUID. It returns an IRI.
 func NewURN() string {
 	return fmt.Sprint("urn:uuid:", uuid.New())
diff --git a/date.go b/date.go
index 809ad0b..603631d 100644
--- a/date.go
+++ b/date.go
@@ -24,6 +24,15 @@ func NewDate(t time.Time) *Date {
 	}
 }
 
+// Update updates the Date.
+func (d *Date) Update() {
+	if d == nil {
+		d = NewDate(time.Now())
+	} else {
+		d.DateTime = DateTime(time.Now())
+	}
+}
+
 // Check checks the Date for incompatibilities with RFC4287. It returns an
 // error.
 func (d *Date) Check() error {
diff --git a/entry.go b/entry.go
index e2e8880..9afbe71 100644
--- a/entry.go
+++ b/entry.go
@@ -69,7 +69,7 @@ func NewEntry(title string) *Entry {
 // AddAuthor adds the Person as an author to the Entry. It returns its index as
 // an int.
 func (e *Entry) AddAuthor(p *Person) int {
-	updateDateTime(e.Updated)
+	e.Updated.Update()
 	return addToSlice(&e.Authors, p)
 }
 
@@ -79,13 +79,13 @@ func (e *Entry) DeleteAuthor(index int) error {
 		return fmt.Errorf("error deleting author %v from entry %v: %v", index, e.ID.URI, err)
 	}
 
-	updateDateTime(e.Updated)
+	e.Updated.Update()
 	return nil
 }
 
 // AddCategory adds the Category to the Entry. It returns ts index as an int.
 func (e *Entry) AddCategory(c *Category) int {
-	updateDateTime(e.Updated)
+	e.Updated.Update()
 	return addToSlice(&e.Categories, c)
 }
 
@@ -96,14 +96,14 @@ func (e *Entry) DeleteCategory(index int) error {
 		return fmt.Errorf("error deleting category %v from entry %v: %v", index, e.ID.URI, err)
 	}
 
-	updateDateTime(e.Updated)
+	e.Updated.Update()
 	return nil
 }
 
 // AddContributor adds the Person as a contributor to the Entry. It returns its
 // index as an int.
 func (e *Entry) AddContributor(c *Person) int {
-	updateDateTime(e.Updated)
+	e.Updated.Update()
 	return addToSlice(&e.Contributors, c)
 }
 
@@ -114,13 +114,13 @@ func (e *Entry) DeleteContributor(index int) error {
 		return fmt.Errorf("error deleting contributor %v from entry %v: %v", index, e.ID.URI, err)
 	}
 
-	updateDateTime(e.Updated)
+	e.Updated.Update()
 	return nil
 }
 
 // AddLink adds the Link to the Entry. It returns its index as an int.
 func (e *Entry) AddLink(l *Link) int {
-	updateDateTime(e.Updated)
+	e.Updated.Update()
 	return addToSlice(&e.Links, l)
 }
 
@@ -130,14 +130,14 @@ func (e *Entry) DeleteLink(index int) error {
 		return fmt.Errorf("error deleting link %v from entry %v: %v", index, e.ID.URI, err)
 	}
 
-	updateDateTime(e.Updated)
+	e.Updated.Update()
 	return nil
 }
 
 // AddExtension adds the ExtensionElement to the Entry. It returns its index as
 // an int.
 func (e *Entry) AddExtension(x *ExtensionElement) int {
-	updateDateTime(e.Updated)
+	e.Updated.Update()
 	return addToSlice(&e.Extensions, x)
 }
 
@@ -148,7 +148,7 @@ func (e *Entry) DeleteExtension(index int) error {
 		return fmt.Errorf("error deleting extension %v from entry %v: %v", index, e.ID.URI, err)
 	}
 
-	updateDateTime(e.Updated)
+	e.Updated.Update()
 	return nil
 }
 
diff --git a/feed.go b/feed.go
index e62bbc6..c030320 100644
--- a/feed.go
+++ b/feed.go
@@ -38,7 +38,7 @@ func NewFeed(title string) *Feed {
 // AddAuthor adds the Person as an author to the Feed. It returns its index as
 // an int.
 func (f *Feed) AddAuthor(p *Person) int {
-	updateDateTime(f.Updated)
+	f.Updated.Update()
 	return addToSlice(&f.Authors, p)
 }
 
@@ -48,13 +48,13 @@ func (f *Feed) DeleteAuthor(index int) error {
 		return fmt.Errorf("error deleting author %v from entry %v: %v", index, f.ID.URI, err)
 	}
 
-	updateDateTime(f.Updated)
+	f.Updated.Update()
 	return nil
 }
 
 // AddCategory adds the Category to the Feed. It returns its index as an int.
 func (f *Feed) AddCategory(c *Category) int {
-	updateDateTime(f.Updated)
+	f.Updated.Update()
 	return addToSlice(&f.Categories, c)
 }
 
@@ -65,14 +65,14 @@ func (f *Feed) DeleteCategory(index int) error {
 		return fmt.Errorf("error deleting category %v from entry %v: %v", index, f.ID.URI, err)
 	}
 
-	updateDateTime(f.Updated)
+	f.Updated.Update()
 	return nil
 }
 
 // AddContributor adds the Person as a contributor to the Feed. It returns its
 // index as an int.
 func (f *Feed) AddContributor(c *Person) int {
-	updateDateTime(f.Updated)
+	f.Updated.Update()
 	return addToSlice(&f.Contributors, c)
 }
 
@@ -83,14 +83,14 @@ func (f *Feed) DeleteContributor(index int) error {
 		return fmt.Errorf("error deleting contributor %v from entry %v: %v", index, f.ID.URI, err)
 	}
 
-	updateDateTime(f.Updated)
+	f.Updated.Update()
 	return nil
 }
 
 // AddLink adds the Link to the Feed. There should be one Link with Rel "self".
 // It returns its index as an int.
 func (f *Feed) AddLink(l *Link) int {
-	updateDateTime(f.Updated)
+	f.Updated.Update()
 	return addToSlice(&f.Links, l)
 }
 
@@ -100,13 +100,13 @@ func (f *Feed) DeleteLink(index int) error {
 		return fmt.Errorf("error deleting link %v from entry %v: %v", index, f.ID.URI, err)
 	}
 
-	updateDateTime(f.Updated)
+	f.Updated.Update()
 	return nil
 }
 
 // AddExtension adds the Extension to the Feed. It returns its index as an int.
 func (f *Feed) AddExtension(e *ExtensionElement) int {
-	updateDateTime(f.Updated)
+	f.Updated.Update()
 	return addToSlice(&f.Extensions, e)
 }
 
@@ -117,13 +117,13 @@ func (f *Feed) DeleteExtension(index int) error {
 		return fmt.Errorf("error deleting extension %v from entry %v: %v", index, f.ID.URI, err)
 	}
 
-	updateDateTime(f.Updated)
+	f.Updated.Update()
 	return nil
 }
 
 // AddEntry adds the Entry to the Feed. It returns its index as an int.
 func (f *Feed) AddEntry(e *Entry) int {
-	updateDateTime(f.Updated)
+	f.Updated.Update()
 	return addToSlice(&f.Entries, e)
 }
 
@@ -133,7 +133,7 @@ func (f *Feed) DeleteEntry(index int) error {
 		return fmt.Errorf("error deleting entry %v from entry %v: %v", index, f.ID.URI, err)
 	}
 
-	updateDateTime(f.Updated)
+	f.Updated.Update()
 	return nil
 }
 
@@ -155,7 +155,7 @@ func (f *Feed) DeleteEntryByURI(uri string) error {
 	}
 
 	f.Entries = append(f.Entries[:index], f.Entries[index+1:]...)
-	updateDateTime(f.Updated)
+	f.Updated.Update()
 	return nil
 }