fix error, PR suggestion

checkmarble · May 24, 2024 · 5541dc0 · 5541dc0
1 parent 3dfba82
commit 5541dc0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/api/handle_ingestion.go b/api/handle_ingestion.go
@@ -6,9 +6,11 @@ import (
 	"io"
 	"net/http"
 
+	"github.com/cockroachdb/errors"
 	"github.com/gin-gonic/gin"
 
 	"github.com/checkmarble/marble-backend/dto"
+	"github.com/checkmarble/marble-backend/models"
 	"github.com/checkmarble/marble-backend/pure_utils"
 	"github.com/checkmarble/marble-backend/utils"
 )
@@ -21,7 +23,8 @@ func (api *API) handleIngestion(c *gin.Context) {
 
 	objectType := c.Param("object_type")
 	objectBody, err := io.ReadAll(c.Request.Body)
-	if presentError(c, err) {
+	if err != nil {
+		presentError(c, errors.Wrap(models.BadParameterError, err.Error()))
 		return
 	}
 

diff --git a/usecases/ingestion_usecase.go b/usecases/ingestion_usecase.go
@@ -95,7 +95,6 @@ func (usecase *IngestionUseCase) IngestObjects(
 	var nb int
 	ingestClosure := func() error {
 		return usecase.transactionFactory.TransactionInOrgSchema(ctx, organizationId, func(tx repositories.Executor) error {
-			var err error
 			nb, err = usecase.ingestionRepository.IngestObjects(ctx, tx, []models.ClientObject{payload}, table)
 			return err
 		})