From 9d653acc0d5fda17cec2d8d42f6608092b0f16c1 Mon Sep 17 00:00:00 2001 From: Yishi Lin Date: Sat, 11 Mar 2017 02:14:53 +0800 Subject: [PATCH] Polish the "discard all local changes" feature --- pass/Controllers/AdvancedSettingsTableViewController.swift | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/pass/Controllers/AdvancedSettingsTableViewController.swift b/pass/Controllers/AdvancedSettingsTableViewController.swift index 0454c60..2ef6023 100644 --- a/pass/Controllers/AdvancedSettingsTableViewController.swift +++ b/pass/Controllers/AdvancedSettingsTableViewController.swift @@ -35,13 +35,15 @@ class AdvancedSettingsTableViewController: UITableViewController { tableView.deselectRow(at: indexPath, animated: true) } else if tableView.cellForRow(at: indexPath) == discardChangesTableViewCell { let alert = UIAlertController(title: "Discard All Changes?", message: "Do you want to permanently discard all changes to the local copy of your password data? You cannot undo this action.", preferredStyle: UIAlertControllerStyle.alert) - alert.addAction(UIAlertAction(title: "Discard All Changesa", style: UIAlertActionStyle.destructive, handler: {[unowned self] (action) -> Void in + alert.addAction(UIAlertAction(title: "Discard All Changes", style: UIAlertActionStyle.destructive, handler: {[unowned self] (action) -> Void in DispatchQueue.global(qos: .userInitiated).async { SVProgressHUD.show(withStatus: "Resetting ...") DispatchQueue.main.async { do { let numberDiscarded = try PasswordStore.shared.reset() - NotificationCenter.default.post(Notification(name: Notification.Name("passwordStoreChangeDiscarded"))) + if numberDiscarded > 0 { + NotificationCenter.default.post(Notification(name: Notification.Name("passwordStoreChangeDiscarded"))) + } self.navigationController!.popViewController(animated: true) SVProgressHUD.showSuccess(withStatus: "Discarded \(numberDiscarded) commits") SVProgressHUD.dismiss(withDelay: 1)