fix linter

cs3org · Feb 8, 2023 · e3d00d6 · e3d00d6
1 parent 174c247
commit e3d00d6
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 9 deletions.
diff --git a/internal/grpc/services/ocmshareprovider/ocmshareprovider.go b/internal/grpc/services/ocmshareprovider/ocmshareprovider.go
@@ -320,7 +320,6 @@ func (s *service) RemoveOCMShare(ctx context.Context, req *ocm.RemoveOCMShareReq
 			return &ocm.RemoveOCMShareResponse{
 				Status: status.NewNotFound(ctx, "share does not exist"),
 			}, nil
-
 		}
 		return &ocm.RemoveOCMShareResponse{
 			Status: status.NewInternal(ctx, err, "error removing share"),
@@ -340,7 +339,6 @@ func (s *service) GetOCMShare(ctx context.Context, req *ocm.GetOCMShareRequest)
 			return &ocm.GetOCMShareResponse{
 				Status: status.NewNotFound(ctx, "share does not exist"),
 			}, nil
-
 		}
 		return &ocm.GetOCMShareResponse{
 			Status: status.NewInternal(ctx, err, "error getting share"),
@@ -377,7 +375,6 @@ func (s *service) UpdateOCMShare(ctx context.Context, req *ocm.UpdateOCMShareReq
 			return &ocm.UpdateOCMShareResponse{
 				Status: status.NewNotFound(ctx, "share does not exist"),
 			}, nil
-
 		}
 		return &ocm.UpdateOCMShareResponse{
 			Status: status.NewInternal(ctx, err, "error updating share"),
@@ -414,7 +411,6 @@ func (s *service) UpdateReceivedOCMShare(ctx context.Context, req *ocm.UpdateRec
 			return &ocm.UpdateReceivedOCMShareResponse{
 				Status: status.NewNotFound(ctx, "share does not exist"),
 			}, nil
-
 		}
 		return &ocm.UpdateReceivedOCMShareResponse{
 			Status: status.NewInternal(ctx, err, "error updating received share"),
@@ -435,7 +431,6 @@ func (s *service) GetReceivedOCMShare(ctx context.Context, req *ocm.GetReceivedO
 			return &ocm.GetReceivedOCMShareResponse{
 				Status: status.NewNotFound(ctx, "share does not exist"),
 			}, nil
-
 		}
 		return &ocm.GetReceivedOCMShareResponse{
 			Status: status.NewInternal(ctx, err, "error getting received share"),

diff --git a/pkg/ocm/share/repository/json/json.go b/pkg/ocm/share/repository/json/json.go
@@ -342,10 +342,7 @@ func (m *mgr) DeleteShare(ctx context.Context, user *userpb.User, ref *ocm.Share
 		if sharesEqual(ref, share) {
 			if utils.UserEqual(user.Id, share.Owner) || utils.UserEqual(user.Id, share.Creator) {
 				delete(m.model.Shares, id)
-				if err := m.save(); err != nil {
-					return err
-				}
-				return nil
+				return m.save()
 			}
 		}
 	}