aclindsa/moneygo
1
0
Fork 0
mirror of https://github.com/aclindsa/moneygo.git synced 2025-10-30 01:23:26 -04:00
Code Issues Releases Wiki Activity

users: Simplify deletion of prices and splits owned by a user

Browse Source 
The previous way was overly complex and didn't work with MySQL anyway
This commit is contained in:
Aaron Lindsay
2017-10-24 20:13:42 -04:00
parent 09a5cf9a99
commit bcbb003e40
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
internal/handlers/users.go
View File

@@ -143,11 +143,11 @@ func DeleteUser(tx *Tx, u *User) error {
if count != 1 { if count != 1 {
return fmt.Errorf("No user to delete") return fmt.Errorf("No user to delete")
} }
_, err = tx.Exec("DELETE FROM prices WHERE prices.PriceId IN (SELECT prices.PriceId FROM prices INNER JOIN securities ON prices.SecurityId=securities.SecurityId WHERE securities.UserId=?)", u.UserId) _, err = tx.Exec("DELETE FROM prices WHERE prices.SecurityId IN (SELECT securities.SecurityId FROM securities WHERE securities.UserId=?)", u.UserId)
if err != nil { if err != nil {
return err return err
} }
_, err = tx.Exec("DELETE FROM splits WHERE splits.SplitId IN (SELECT splits.SplitId FROM splits INNER JOIN transactions ON splits.TransactionId=transactions.TransactionId WHERE transactions.UserId=?)", u.UserId) _, err = tx.Exec("DELETE FROM splits WHERE splits.TransactionId IN (SELECT transactions.TransactionId FROM transactions WHERE transactions.UserId=?)", u.UserId)
if err != nil { if err != nil {
return err return err
} }