added writeout function to requester interface to generalize file-bas…

…ed output

cmd/task.go

@@ -2,9 +2,6 @@ package cmd
 
 import (
 	"errors"
-	"fmt"
-	"log"
-	"os"
 	"strings"
 
 	"github.com/fantasticrabbit/ClickupCLI/internal"
@@ -31,17 +28,7 @@ var taskCmd = &cobra.Command{
 			TeamID:     viper.GetString("team_id"),
 			Subtasks:   subtasksFlag,
 		}
-		data := internal.GetJSON(t)
-
-		fileFlag, _ := cmd.Flags().GetBool("file")
-		if !fileFlag {
-			fmt.Println(string(data))
-		} else {
-			err := os.WriteFile("clickup_"+t.TaskID+".json", data, 0644)
-			if err != nil {
-				log.Fatalln("Error writing task JSON")
-			}
-		}
+		internal.GetJSON(t)
 	},
 }
 

internal/gettask.go

@@ -2,6 +2,8 @@ package internal
 
 import (
 	"fmt"
+	"log"
+	"os"
 )
 
 type TaskRequest struct {
@@ -21,3 +23,10 @@ func (t TaskRequest) BuildPath() string {
 			t.TaskID, t.CustomTask, t.TeamID, t.Subtasks)
 	}
 }
+
+func (t TaskRequest) WriteOut(payload []byte) {
+	err := os.WriteFile("clickup_"+t.TaskID+".json", payload, 0644)
+	if err != nil {
+		log.Fatalln("Error writing task JSON")
+	}
+}

internal/requester.go

@@ -1,6 +1,7 @@
 package internal
 
 import (
+	"fmt"
 	"io/ioutil"
 	"log"
 	"net/http"
@@ -11,10 +12,11 @@ import (
 //Requester interface needs an API path to request JSON data
 type Requester interface {
 	BuildPath() string
+	WriteOut([]byte)
 }
 
 //Gets JSON data for any struct that implements Requester interface
-func GetJSON(r Requester) []byte {
+func GetJSON(r Requester) {
 	apiPath := r.BuildPath()
 	token := viper.GetString("ctoken")
 	client := &http.Client{}
@@ -30,5 +32,10 @@ func GetJSON(r Requester) []byte {
 	defer resp.Body.Close()
 	resp_body, _ := ioutil.ReadAll(resp.Body)
 
-	return resp_body
+	fileFlag := viper.GetBool("file")
+	if !fileFlag {
+		fmt.Println(string(resp_body))
+	} else {
+		r.WriteOut(resp_body)
+	}
 }