Fix update view

Author: gotokatsuya

core/models/item_impression_entity_repository.go

@@ -44,14 +44,14 @@ func (r *ItemImpressionRepository) UpdateByID(ent *ItemImpression) error {
 	return nil
 }
 
-func (r *ItemImpressionRepository) UpdateCountByID(itemID uint64, view int) error {
+func (r *ItemImpressionRepository) UpdateCountByID(itemID uint64, view uint) error {
 	if err := r.Orm.Model(&ItemImpression{}).Where("item_id = ?", itemID).Update("view", view).Error; err != nil {
 		return err
 	}
 	return nil
 }
 
-func (r *ItemImpressionRepository) UpdateStarByID(itemID uint64, star int) error {
+func (r *ItemImpressionRepository) UpdateStarByID(itemID uint64, star uint) error {
 	if err := r.Orm.Model(&ItemImpression{}).Where("item_id = ?", itemID).Update("star", star).Error; err != nil {
 		return err
 	}

internal/controllers/api/api_controllers.go

@@ -79,6 +79,32 @@ func (c *APIMeController) LoginMe() {
 }
 
 func (c *APIItemController) GetItem(id string) {
+	//// Apply Request Parameters
+	param := parameters.NewGetItemRequest()
+	if ok := c.API().Preprocess(&param); !ok {
+		return
+	}
+
+	itemFacade := facades.NewItemFacade(c.GetContext())
+	// Parse string id to uint64
+	uint64ID := c.ParseUint64(id)
+	if uint64ID <= 0 {
+		return
+	}
+
+	if err := itemFacade.UpdateItemImpressionView(uint64ID); err != nil {
+		c.API().InternalServerError(map[string]interface{}{
+			"status":  "NG",
+			"func":    "UpdateItemImpressionCount::ItemFacade",
+			"message": err,
+		})
+		return
+	}
+
+	// Render result
+	c.API().OK(map[string]interface{}{
+		"status": "OK",
+	})
 }
 
 func (c *APIItemController) ConvertItems(items []models.Item) ([]responses.ItemResponse, error) {

internal/facades/item_facade.go

@@ -129,3 +129,12 @@ func (f ItemFacade) FindItemTagIDsMap(itemIDs []uint64) (map[uint64][]uint64, er
 	}
 	return res, nil
 }
+
+func (f ItemFacade) UpdateItemImpressionView(itemID uint64) error {
+	itemImpressionService := services.NewItemImpressionService(f.ctx)
+	itemImp, err := itemImpressionService.GetItemImpressionByID(itemID)
+	if err != nil {
+		return err
+	}
+	return itemImpressionService.UpdateItemImpressionViewByID(itemID, uint(itemImp.View+1))
+}

internal/services/item_impression_service.go

@@ -32,7 +32,7 @@ func (s ItemImpressionService) CreateItemImpression(itemID uint64, star uint) (*
 	return itemImpRepository.Create(itemImp)
 }
 
-func (s ItemImpressionService) UpdateItemImpressionCountByID(itemID uint64, count int) error {
+func (s ItemImpressionService) UpdateItemImpressionViewByID(itemID uint64, view uint) error {
 	itemImpRepository := models.NewItemImpressionRepository(s.ctx)
-	return itemImpRepository.UpdateCountByID(itemID, count)
+	return itemImpRepository.UpdateCountByID(itemID, view)
 }