fixing duplicate facts bug

cmd/gather.go

@@ -69,14 +69,12 @@ var gatherCmd = &cobra.Command{
 						log.Println(err.Error())
 						continue
 					}
-					if verbose := viper.Get("verbose"); verbose == true {
-						for _, f := range gatheredFacts {
-							if f.Value != "" {
-								log.Printf("Registering %s", f.Name)
-								facts = append(facts, gatheredFacts...)
-							}
+					for _, f := range gatheredFacts {
+						if verbose := viper.Get("verbose"); verbose == true {
+							log.Printf("Registering %s", f.Name)
 						}
 					}
+					facts = append(facts, gatheredFacts...)
 				}
 			}
 		}

gatherers/DrushPMLGatherer.go

@@ -40,13 +40,15 @@ func (p *drushPmlGatherer) AppliesToEnvironment() bool {
 	}
 
 	for key, element := range result {
-		p.GatheredFacts = append(p.GatheredFacts, GatheredFact{
-			Name:        key,
-			Value:       fmt.Sprintf("%v", element.Version),
-			Source:      "drush_pml",
-			Description: "Drupal " + element.Type + " status: " + element.Status,
-			Category:    Drupal,
-		})
+		if element.Version != nil {
+			p.GatheredFacts = append(p.GatheredFacts, GatheredFact{
+				Name:        key,
+				Value:       fmt.Sprintf("%v", element.Version),
+				Source:      "drush_pml",
+				Description: "Status: " + element.Status,
+				Category:    Drupal,
+			})
+		}
 	}
 
 	return true

gatherers/defs.go

@@ -7,7 +7,7 @@ type GatheredFact struct {
 	Source      string       `json:"source"`
 	Environment int          `json:"environment"`
 	Description string       `json:"description"`
-	Category    FactCategory `json:"-"` //`json:"category,omitempty"`
+	Category    FactCategory `json:"-"` // `json:"category,omitempty"`
 }
 
 const (
@@ -17,7 +17,7 @@ const (
 
 type Gatherer interface {
 	GetGathererCmdType() string
-	AppliesToEnvironment() bool //Whether this gatherer can run in the local environment
+	AppliesToEnvironment() bool // Whether this gatherer can run in the local environment
 	GatherFacts() ([]GatheredFact, error)
 }
 

gatherers/graphql.go

@@ -46,8 +46,10 @@ func Writefacts(projectName string, environmentName string, facts []GatheredFact
 
 	client, err := getGraphqlClient()
 	if err != nil {
+		log.Println(err)
 		return err
 	}
+
 	var addFactMutation struct {
 		AddFacts []struct {
 			Id int