Refaator service from *Addon to Addon

addonService.go

@@ -3,7 +3,7 @@ package main
 import "fmt"
 
 type AddonService struct {
-	Addons map[string]*Addon
+	Addons map[string]Addon
 }
 
 //var addons = map[string]*Addon{
@@ -12,16 +12,13 @@ type AddonService struct {
 //	"Dechickenator": NewAddon("Dechickenator", "https://git.site.quack-lab.dev/dave/wow_dechickenator"),
 //}
 
-func NewAddonService() *AddonService {
-	return &AddonService{}
-}
-
-func (a *AddonService) GetAddon(name string) (*Addon, error) {
-	addon, ok := a.Addons[name]
+func (a *AddonService) GetAddon(name string) (addon Addon, err error) {
+	var ok bool
+	addon, ok = a.Addons[name]
 	if !ok {
-		return nil, fmt.Errorf("addon with name %s not found", name)
+		return addon, fmt.Errorf("addon with name %s not found", name)
 	}
-	return addon, nil
+	return
 }
 
 func (a *AddonService) GetRemoteVersion(name string) (string, error) {
@@ -39,18 +36,18 @@ func (a *AddonService) GetLocalVersion(name string) (string, error) {
 	return addon.GetLocalVersion()
 }
 
-func (a *AddonService) UpdateAddon(name string) (*Addon, error) {
-	addon, err := a.GetAddon(name)
+func (a *AddonService) UpdateAddon(name string) (addon Addon, err error) {
+	addon, err = a.GetAddon(name)
 	if err != nil {
-		return nil, err
+		return
 	}
 	release, err := addon.GetRemoteRelease()
 	if err != nil {
-		return nil, err
+		return
 	}
 	err = addon.Update(release)
 	if err != nil {
-		return nil, err
+		return
 	}
 	return addon, nil
 }