bugfix(destroy): Do not check for container deleting errors

controller/create.go

@@ -26,7 +26,7 @@ func BoxCreate(name string, repoUrl string, pathspec string, stream *hub.Stream)
 		Pathspec:      pathspec,
 		CodeDirectory: os.Getenv("FARMER_BOX_DATA_LOCATION") + "/" + name,
 		CgroupParent:  "level1",
-		Status: 	   "creating",
+		Status:        "creating",
 	}
 
 	if err := db.DB.Save(&box).Error; err != nil {

farmer/box.go

@@ -2,7 +2,6 @@ package farmer
 
 import (
 	"io"
-	"github.com/fsouza/go-dockerclient"
 )
 
 type Box struct {
@@ -60,12 +59,7 @@ func (b *Box) Deploy() error {
 }
 
 func (b *Box) Destroy() error {
-	if err := dockerDeleteContainer(b); err != nil {
-		if _, ok := err.(*docker.NoSuchContainer); !ok {
-			return err
-		}
-	}
-
+	dockerDeleteContainer(b)
 	return b.removeCode()
 }
 

farmer/farmer.go

@@ -18,10 +18,7 @@ func findBoxBy(field string, value interface{}) (*Box, error) {
 		return box, err
 	}
 
-	if err = box.Inspect(); err != nil {
-		return box, err
-	}
-
+	box.Inspect()
 	if err = db.DB.Model(&box).Related(&box.Domains).Error; err != nil {
 		return box, err
 	}

farmer/parser.go

@@ -35,4 +35,4 @@ func setDefaultConfig(config *FarmerConfig) error {
 	}
 
 	return nil
-}
+}

reverse_proxy/domain.go

@@ -62,7 +62,7 @@ func checkDomain(url string) error {
 	err := db.DB.Where("url = ?", url).Find(domain).Error
 	if err != nil {
 		// Could not find a domain so domain is available and free to use.
-		if (err.Error() == "record not found") {
+		if err.Error() == "record not found" {
 			return nil
 		}
 		return err