coder
diff --git a/‎coderd/database/querier.go
Lines changed: 1 addition & 1 deletion b/‎coderd/database/querier.go
Lines changed: 1 addition & 1 deletion
diff --git a/‎coderd/database/queries.sql.go
Lines changed: 18 additions & 4 deletions b/‎coderd/database/queries.sql.go
Lines changed: 18 additions & 4 deletions
diff --git a/‎coderd/database/queries/workspaces.sql
Lines changed: 3 additions & 2 deletions b/‎coderd/database/queries/workspaces.sql
Lines changed: 3 additions & 2 deletions
diff --git a/‎coderd/workspaces.go
Lines changed: 4 additions & 5 deletions b/‎coderd/workspaces.go
Lines changed: 4 additions & 5 deletions
@@ -112,14 +112,15 @@ SET
 WHERE
 	id = $1;
 
--- name: UpdateWorkspace :exec
+-- name: UpdateWorkspace :one
 UPDATE
 	workspaces
 SET
 	name = $2
 WHERE
 	id = $1
-	AND deleted = false;
+	AND deleted = false
+RETURNING *;
 
 -- name: UpdateWorkspaceAutostart :exec
 UPDATE
 
@@ -501,18 +501,17 @@ func (api *API) patchWorkspace(rw http.ResponseWriter, r *http.Request) {
 		name = req.Name
 	}
 
-	err := api.Database.UpdateWorkspace(r.Context(), database.UpdateWorkspaceParams{
+	_, err := api.Database.UpdateWorkspace(r.Context(), database.UpdateWorkspaceParams{
 		ID:   workspace.ID,
 		Name: name,
 	})
 	if err != nil {
 		// The query protects against updating deleted workspaces and
 		// the existence of the workspace is checked in the request,
-		// the only conclusion we can make is that we're trying to
-		// update a deleted workspace.
+		// if we get ErrNoRows it means the workspace was deleted.
 		//
-		// We could do this check earlier but since we're not in a
-		// transaction, it's pointless.
+		// We could do this check earlier but we'd need to start a
+		// transaction.
 		if errors.Is(err, sql.ErrNoRows) {
 			httpapi.Write(rw, http.StatusMethodNotAllowed, codersdk.Response{
 				Message: fmt.Sprintf("Workspace %q is deleted and cannot be updated.", workspace.Name),