Org update should throw error if not found (#7066)

* Org update should give error if not found * Used affected rows

Org update should throw error if not found (#7066)
* Org update should give error if not found * Used affected rows
65057212 · Utkarsh Bhatnagar · Torkel Ödegaard · d8ebebc6 · 65057212
Commit 65057212 authored Jan 03, 2017 by Utkarsh Bhatnagar Committed by Torkel Ödegaard Jan 03, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 1 deletions

pkg/services/sqlstore/org.go
+7 -1

No files found.
--- a/pkg/services/sqlstore/org.go
+++ b/pkg/services/sqlstore/org.go
@@ -133,10 +133,16 @@ func UpdateOrg(cmd *m.UpdateOrgCommand) error {
 			Updated: time.Now(),
 		}

-		if _, err := sess.Id(cmd.OrgId).Update(&org); err != nil {
+		affectedRows, err := sess.Id(cmd.OrgId).Update(&org)
+
+		if err != nil {
 			return err
 		}

+		if affectedRows == 0 {
+			return m.ErrOrgNotFound
+		}
+
 		sess.publishAfterCommit(&events.OrgUpdated{
 			Timestamp: org.Updated,
 			Id:        org.Id,