[Feat] : remove downloading image from url, instead save the url to t…

…he db (#350)

* Feat : remove downloading image from url, and save the url to the database instead

* Fix: update marketevent test for the updated model

* fix: remove comments

* feat: update frontend submodule

models/MarketEvent.go

@@ -1,12 +1,7 @@
 package models
 
 import (
-	"errors"
 	"fmt"
-	"io"
-	"net/http"
-	"os"
-	"strings"
 
 	models_pb "github.com/delta/dalal-street-server/proto_build/models"
 	"github.com/delta/dalal-street-server/utils"
@@ -88,51 +83,14 @@ func AddMarketEvent(stockId uint32, headline, text string, isGlobal bool, imageU
 		"param_imageURL": imageURL,
 	})
 
-	l.Infof("Attempting")
-
-	// Try downloading image first
-	response, err := http.Get(imageURL)
-	if err != nil || response.StatusCode != http.StatusOK {
-		l.Errorf("Error : %v, StatusCode : %d", err, response.StatusCode)
-		if err != nil {
-			return err
-		}
-		return errors.New("NOT OK status code")
-	}
-	defer response.Body.Close()
-
-	// Extract filename
-	var basename = imageURL[strings.LastIndex(imageURL, "/")+1:]
-	var getParamStartIndex = strings.Index(basename, "?")
-	if getParamStartIndex != -1 {
-		basename = basename[:getParamStartIndex]
-	}
-	l.Debugf("ImageURL : %s Basename : %s", imageURL, basename)
-
-	// open file for saving image
-	file, err := os.Create(utils.GetImageBasePath() + basename)
-
-	if err != nil {
-		l.Error(err)
-		return err
-	}
-	defer file.Close()
-
-	// copy image to file
-	_, err = io.Copy(file, response.Body)
-	if err != nil {
-		l.Error(err)
-		return err
-	}
-
 	db := getDB()
 
 	me := &MarketEvent{
 		StockId:   stockId,
 		Headline:  headline,
 		Text:      text,
 		IsGlobal:  isGlobal,
-		ImagePath: basename,
+		ImagePath: imageURL,
 		CreatedAt: utils.GetCurrentTimeISO8601(),
 	}
 
@@ -143,7 +101,7 @@ func AddMarketEvent(stockId uint32, headline, text string, isGlobal bool, imageU
 		ImageUrl: imageURL,
 	})
 
-	if err = db.Save(me).Error; err != nil {
+	if err := db.Save(me).Error; err != nil {
 		l.Error(err)
 		return err
 	}

models/MarketEvent_test.go

@@ -3,7 +3,7 @@ package models
 import (
 	"testing"
 
-	"github.com/delta/dalal-street-server/utils/test"
+	testutils "github.com/delta/dalal-street-server/utils/test"
 )
 
 func TestMarketEventToProto(t *testing.T) {
@@ -14,7 +14,7 @@ func TestMarketEventToProto(t *testing.T) {
 		Text:         "Hello World",
 		IsGlobal:     true,
 		EmotionScore: -54,
-		ImagePath:    "bitcoin_1516197589.jpg",
+		ImagePath:    "http://www.valuewalk.com/wp-content/uploads/2018/01/bitcoin_1516197589.jpg",
 		CreatedAt:    "2017-02-09T00:00:00",
 	}
 
@@ -33,7 +33,7 @@ func Test_GetMarketEvents(t *testing.T) {
 		Text:         "Hello World",
 		IsGlobal:     true,
 		EmotionScore: -54,
-		ImagePath:    "bitcoin_1516197589.jpg",
+		ImagePath:    "http://www.valuewalk.com/wp-content/uploads/2018/01/bitcoin_1516197589.jpg",
 		CreatedAt:    "2017-02-09T00:00:00",
 	}
 	db := getDB()
@@ -78,7 +78,7 @@ func Test_AddMarketEvent(t *testing.T) {
 		Headline:  "Hello",
 		Text:      "Hello World",
 		IsGlobal:  true,
-		ImagePath: "bitcoin_1516197589.jpg",
+		ImagePath: "http://www.valuewalk.com/wp-content/uploads/2018/01/bitcoin_1516197589.jpg",
 	}
 	db := getDB()
 	defer func() {