move alpm functions in a separate file, all handled by Transaction

src/Makefile

@@ -23,6 +23,7 @@ COMMON_SOURCES = 	../util/alpm-util.c \
 					alpm_config.vala \
 
 TRANSACTION_SOURCES = 	transaction.vala \
+						alpm_utils.vala \
 						aur.vala \
 						choose_provider_dialog.vala \
 						transaction_sum_dialog.vala \
@@ -90,7 +91,7 @@ libpamac.so: ../vapi/libalpm.vapi $(COMMON_SOURCES) $(TRANSACTION_SOURCES) $(PRE
 			$(TRANSACTION_SOURCES) \
 			$(PREFERENCES_SOURCES)
 
-pamac-manager: libpamac.so choose_dep_dialog.vala history_dialog.vala transaction_info_dialog.vala ../resources/manager_resources.c ../util/alpm-util.c alpm_utils.vala packages_model.vala aur_model.vala manager_window.vala manager.vala
+pamac-manager: libpamac.so choose_dep_dialog.vala history_dialog.vala transaction_info_dialog.vala ../resources/manager_resources.c ../util/alpm-util.c packages_model.vala aur_model.vala manager_window.vala manager.vala
 	valac 	-o pamac-manager \
 			$(COMMON_VALA_FLAGS) \
 			-X -I. \
@@ -105,7 +106,6 @@ pamac-manager: libpamac.so choose_dep_dialog.vala history_dialog.vala transactio
 			../util/alpm-util.c \
 			choose_dep_dialog.vala \
 			history_dialog.vala \
-			alpm_utils.vala \
 			packages_model.vala \
 			aur_model.vala \
 			manager_window.vala \

src/alpm_config.vala

@@ -17,27 +17,26 @@
  *  along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-namespace Pamac {
 [Compact]
-	public class Repo {
+public class AlpmRepo {
 	public string name;
 	public Alpm.Signature.Level siglevel;
 	public Alpm.Signature.Level siglevel_mask;
 	public Alpm.DB.Usage usage;
 	public GLib.List<string> urls;
 
-		public Repo (string name) {
+	public AlpmRepo (string name) {
 		this.name = name;
 		siglevel = Alpm.Signature.Level.USE_DEFAULT;
 		usage = 0;
 		urls = new GLib.List<string> ();
 	}
 
-		public static int compare_name (Repo a, Repo b) {
+	public static int compare_name (AlpmRepo a, AlpmRepo b) {
 		return strcmp (a.name, b.name);
 	}
 
-		public static int search_name (Repo a, string name) {
+	public static int search_name (AlpmRepo a, string name) {
 		return strcmp (a.name, name);
 	}
 
@@ -54,12 +53,12 @@ namespace Pamac {
 	public double deltaratio;
 	public int usesyslog;
 	public int checkspace;
-		public Alpm.List<string>? cachedirs;
+	public Alpm.List<string?>? cachedirs;
 	public Alpm.List<string?>? hookdirs;
-		public Alpm.List<string>? ignoregroups;
+	public Alpm.List<string?>? ignoregroups;
-		public Alpm.List<string>? ignorepkgs;
+	public Alpm.List<string?>? ignorepkgs;
-		public Alpm.List<string>? noextracts;
+	public Alpm.List<string?>? noextracts;
-		public Alpm.List<string>? noupgrades;
+	public Alpm.List<string?>? noupgrades;
 	public GLib.List<string>? holdpkgs;
 	public GLib.List<string>? syncfirsts;
 	public Alpm.Signature.Level siglevel;
@@ -68,7 +67,7 @@ namespace Pamac {
 	public Alpm.Signature.Level siglevel_mask;
 	public Alpm.Signature.Level localfilesiglevel_mask;
 	public Alpm.Signature.Level remotefilesiglevel_mask;
-		public GLib.List<Repo> repo_order;
+	public GLib.List<AlpmRepo> repo_order;
 	public Alpm.Handle? handle;
 
 	public AlpmConfig (string path) {
@@ -111,7 +110,7 @@ namespace Pamac {
 		siglevel = Alpm.Signature.Level.PACKAGE | Alpm.Signature.Level.PACKAGE_OPTIONAL | Alpm.Signature.Level.DATABASE | Alpm.Signature.Level.DATABASE_OPTIONAL;
 		localfilesiglevel = Alpm.Signature.Level.USE_DEFAULT;
 		remotefilesiglevel = Alpm.Signature.Level.USE_DEFAULT;
-			repo_order = new GLib.List<Repo> ();
+		repo_order = new GLib.List<AlpmRepo> ();
 		// parse conf file
 		parse_file (conf_path);
 		// if rootdir is set and dbpath/logfile are not
@@ -148,7 +147,7 @@ namespace Pamac {
 		}
 	}
 
-		public void get_handle () {
+	public void set_handle () {
 		Alpm.Errno error;
 		handle = Alpm.Handle.new (rootdir, dbpath, out error);
 		if (handle == null) {
@@ -177,7 +176,7 @@ namespace Pamac {
 		handle.noextracts = noextracts;
 		handle.noupgrades = noupgrades;
 		// register dbs
-			foreach (unowned Repo repo in repo_order) {
+		foreach (unowned AlpmRepo repo in repo_order) {
 			repo.siglevel = merge_siglevel (siglevel, repo.siglevel, repo.siglevel_mask);
 			unowned Alpm.DB db = handle.register_syncdb (repo.name, repo.siglevel);
 			foreach (unowned string url in repo.urls) {
@@ -214,8 +213,8 @@ namespace Pamac {
 					if (line[0] == '[' && line[line.length-1] == ']') {
 						current_section = line[1:-1];
 						if (current_section != "options") {
-								var repo = new Repo (current_section);
+							var repo = new AlpmRepo (current_section);
-								if (repo_order.find_custom (repo, Repo.compare_name) == null) {
+							if (repo_order.find_custom (repo, AlpmRepo.compare_name) == null) {
 								repo_order.append ((owned) repo);
 							}
 						}
@@ -293,9 +292,9 @@ namespace Pamac {
 							}
 						}
 					} else {
-							unowned GLib.List<Repo>? found = repo_order.search (current_section, (SearchFunc) Repo.search_name);
+						unowned GLib.List<AlpmRepo>? found = repo_order.search (current_section, (SearchFunc) AlpmRepo.search_name);
 						if (found != null) {
-								unowned Repo repo = found.data;
+							unowned AlpmRepo repo = found.data;
 							if (key == "Server") {
 								repo.urls.append (val);
 							} else if (key == "SigLevel") {
@@ -463,4 +462,3 @@ namespace Pamac {
 		return (sigmask != 0) ? (sigover & sigmask) | (sigbase & ~sigmask) : sigover;
 	}
 }
-}

src/alpm_utils.vala

@@ -1,7 +1,7 @@
 /*
  *  pamac-vala
  *
- *  Copyright (C) 2015-2016 Guillaume Benoit <guillaume@manjaro.org>
+ *  Copyright (C) 2014-2016 Guillaume Benoit <guillaume@manjaro.org>
  *
  *  This program is free software; you can redistribute it and/or modify
  *  it under the terms of the GNU General Public License as published by
@@ -17,22 +17,174 @@
  *  along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-int compare_name (Alpm.Package pkg_a, Alpm.Package pkg_b) {
+public int alpm_pkg_compare_name (Alpm.Package pkg_a, Alpm.Package pkg_b) {
 	return strcmp (pkg_a.name, pkg_b.name);
 }
 
-int compare_state (Alpm.Package pkg_a, Alpm.Package pkg_b) {
+public int alpm_pkg_compare_origin (Alpm.Package pkg_a, Alpm.Package pkg_b) {
 	return (int) (pkg_a.origin > pkg_b.origin) - (int) (pkg_a.origin < pkg_b.origin);
 }
 
-int compare_version (Alpm.Package pkg_a, Alpm.Package pkg_b) {
+public int alpm_pkg_compare_version (Alpm.Package pkg_a, Alpm.Package pkg_b) {
 	return Alpm.pkg_vercmp (pkg_a.version, pkg_b.version);
 }
 
-int compare_repo (Alpm.Package pkg_a, Alpm.Package pkg_b) {
+public int alpm_pkg_compare_db_name (Alpm.Package pkg_a, Alpm.Package pkg_b) {
 	return strcmp (pkg_a.db.name, pkg_b.db.name);
 }
 
-int compare_size (Alpm.Package pkg_a, Alpm.Package pkg_b) {
+public int alpm_pkg_compare_installed_size (Alpm.Package pkg_a, Alpm.Package pkg_b) {
 	return (int) (pkg_a.isize > pkg_b.isize) - (int) (pkg_a.isize < pkg_b.isize);
 }
+
+public class AlpmUtils {
+	AlpmConfig alpm_config;
+
+	public AlpmUtils (string conf_file_path) {
+		alpm_config = new AlpmConfig (conf_file_path);
+		alpm_config.set_handle ();
+	}
+
+	public bool reload () {
+		alpm_config.reload ();
+		alpm_config.set_handle ();
+		if (alpm_config.handle != null) {
+			return true;
+		}
+		return false;
+	}
+
+	public int get_checkspace () {
+		return alpm_config.checkspace;
+	}
+
+	public unowned GLib.List<string> get_holdpkgs () {
+		return alpm_config.holdpkgs;
+	}
+
+	public unowned Alpm.List<unowned string?> get_ignorepkgs () {
+		return alpm_config.ignorepkgs;
+	}
+
+	public unowned Alpm.Package? get_installed_pkg (string pkg_name) {
+		return alpm_config.handle.localdb.get_pkg (pkg_name);
+	}
+
+	public unowned Alpm.Package? get_sync_pkg (string pkg_name) {
+		unowned Alpm.Package? pkg = null;
+		foreach (var db in alpm_config.handle.syncdbs) {
+			pkg = db.get_pkg (pkg_name);
+			if (pkg != null) {
+				break;
+			}
+		}
+		return pkg;
+	}
+
+	public unowned Alpm.DB? get_localdb () {
+		return alpm_config.handle.localdb;
+	}
+
+	public unowned Alpm.List<unowned Alpm.DB?> get_syncdbs () {
+		return alpm_config.handle.syncdbs;
+	}
+
+	public Alpm.List<unowned Alpm.Package?> search_all_dbs (string search_string) {
+		var syncpkgs = new Alpm.List<unowned Alpm.Package?> ();
+		var needles = new Alpm.List<unowned string> ();
+		string[] splitted = search_string.split (" ");
+		foreach (unowned string part in splitted) {
+			needles.add (part);
+		}
+		var result = alpm_config.handle.localdb.search (needles);
+		foreach (var db in alpm_config.handle.syncdbs) {
+			if (syncpkgs.length == 0) {
+				syncpkgs = db.search (needles);
+			} else {
+				syncpkgs.join (db.search (needles).diff (syncpkgs, (Alpm.List.CompareFunc) alpm_pkg_compare_name));
+			}
+		}
+		result.join (syncpkgs.diff (result, (Alpm.List.CompareFunc) alpm_pkg_compare_name));
+		//result.sort ((Alpm.List.CompareFunc) alpm_pkg_compare_name);
+		return result;
+	}
+
+	public Alpm.List<unowned Alpm.Package?> get_group_pkgs (string group_name) {
+		var result = new Alpm.List<unowned Alpm.Package?> ();
+		unowned Alpm.Group? grp = alpm_config.handle.localdb.get_group (group_name);
+		if (grp != null) {
+			foreach (var pkg in grp.packages) {
+				result.add (pkg);
+			}
+		}
+		result.join (Alpm.find_group_pkgs (alpm_config.handle.syncdbs, group_name).diff (result, (Alpm.List.CompareFunc) alpm_pkg_compare_name));
+		//result.sort ((Alpm.List.CompareFunc) alpm_pkg_compare_name);
+		return result;
+	}
+
+	public Alpm.List<unowned Alpm.Package?> get_installed_pkgs () {
+		return alpm_config.handle.localdb.pkgcache.copy ();
+	}
+
+	public Alpm.List<unowned Alpm.Package?> get_orphans () {
+		var result = new Alpm.List<unowned Alpm.Package?> ();
+		foreach (var pkg in alpm_config.handle.localdb.pkgcache) {
+			if (pkg.reason == Alpm.Package.Reason.DEPEND) {
+				Alpm.List<string?> requiredby = pkg.compute_requiredby ();
+				if (requiredby.length == 0) {
+					Alpm.List<string?> optionalfor = pkg.compute_optionalfor ();
+					if (optionalfor.length == 0) {
+						result.add (pkg);
+					}
+					optionalfor.free_data ();
+				}
+				requiredby.free_data ();
+			}
+		}
+		return result;
+	}
+
+	public Alpm.List<unowned Alpm.Package?> get_local_pkgs () {
+		var result = new Alpm.List<unowned Alpm.Package?> ();
+		foreach (var pkg in alpm_config.handle.localdb.pkgcache) {
+			if (get_sync_pkg (pkg.name) == null) {
+				result.add (pkg);
+			}
+		}
+		return result;
+	}
+
+	public Alpm.List<unowned Alpm.Package?> get_repo_pkgs (string repo_name) {
+		var result = new Alpm.List<unowned Alpm.Package?> ();
+		foreach (var db in alpm_config.handle.syncdbs) {
+			if (db.name == repo_name) {
+				foreach (var sync_pkg in db.pkgcache) {
+					unowned Alpm.Package?local_pkg = alpm_config.handle.localdb.get_pkg (sync_pkg.name);
+					if (local_pkg != null) {
+						result.add (local_pkg);
+					} else {
+						result.add (sync_pkg);
+					}
+				}
+			}
+		}
+		return result;
+	}
+
+	public Alpm.List<unowned Alpm.Package?> get_all_pkgs () {
+		var syncpkgs = new Alpm.List<unowned Alpm.Package?> ();
+		var result = new Alpm.List<unowned Alpm.Package?> ();
+		result = alpm_config.handle.localdb.pkgcache.copy ();
+		foreach (var db in alpm_config.handle.syncdbs) {
+			if (syncpkgs.length == 0)
+				syncpkgs = db.pkgcache.copy ();
+			else {
+				syncpkgs.join (db.pkgcache.diff (syncpkgs, (Alpm.List.CompareFunc) alpm_pkg_compare_name));
+			}
+		}
+		result.join (syncpkgs.diff (result, (Alpm.List.CompareFunc) alpm_pkg_compare_name));
+		//result.sort ((Alpm.List.CompareFunc) alpm_pkg_compare_name);
+		return result;
+	}
+
+}

src/aur_model.vala

@@ -25,8 +25,8 @@ int aur_compare_name (Json.Object pkg_a, Json.Object pkg_b) {
 }
 
 int aur_compare_state (Json.Object pkg_a, Json.Object pkg_b) {
-	unowned Alpm.Package? alpm_pkg_a = manager_window.transaction.alpm_config.handle.localdb.get_pkg (pkg_a.get_string_member ("Name"));
+	unowned Alpm.Package? alpm_pkg_a = manager_window.transaction.alpm_utils.get_installed_pkg (pkg_a.get_string_member ("Name"));
-	unowned Alpm.Package? alpm_pkg_b = manager_window.transaction.alpm_config.handle.localdb.get_pkg (pkg_b.get_string_member ("Name"));
+	unowned Alpm.Package? alpm_pkg_b = manager_window.transaction.alpm_utils.get_installed_pkg (pkg_b.get_string_member ("Name"));
 	if (pkg_a != null) {
 		if (pkg_b != null) {
 			return (int) (alpm_pkg_a.origin > alpm_pkg_b.origin) - (int) (alpm_pkg_a.origin < alpm_pkg_b.origin);
@@ -100,9 +100,9 @@ namespace Pamac {
 				case 1:
 					val = Value (typeof (Object));
 					if (pkg_info != null) {
-						unowned Alpm.Package? pkg = manager_window.transaction.alpm_config.handle.localdb.get_pkg (pkg_info.get_string_member ("Name"));
+						unowned Alpm.Package? pkg = manager_window.transaction.alpm_utils.get_installed_pkg (pkg_info.get_string_member ("Name"));
 						if (pkg != null) {
-							if (manager_window.transaction.alpm_config.holdpkgs.find_custom (pkg.name, strcmp) != null) {
+							if (manager_window.transaction.alpm_utils.get_holdpkgs ().find_custom (pkg.name, strcmp) != null) {
 								val.set_object (manager_window.locked_icon);
 							} else if (manager_window.transaction.to_add.contains (pkg.name)) {
 								val.set_object (manager_window.to_reinstall_icon);
@@ -121,7 +121,7 @@ namespace Pamac {
 				case 2:
 					val = Value (typeof (string));
 					if (pkg_info != null) {
-						unowned Alpm.Package? pkg = manager_window.transaction.alpm_config.handle.localdb.get_pkg (pkg_info.get_string_member ("Name"));
+						unowned Alpm.Package? pkg = manager_window.transaction.alpm_utils.get_installed_pkg (pkg_info.get_string_member ("Name"));
 						if (pkg != null) {
 							val.set_string (pkg.version);
 						} else {

src/daemon.vala

@@ -131,7 +131,7 @@ namespace Pamac {
 		}
 
 		private void refresh_handle () {
-			alpm_config.get_handle ();
+			alpm_config.set_handle ();
 			if (alpm_config.handle == null) {
 				current_error = ErrorInfos () {
 					message = _("Failed to initialize alpm library"),

src/manager_window.vala

@@ -132,10 +132,13 @@ namespace Pamac {
 
 		public SortInfo sortinfo;
 
+		bool refreshing;
+
 		public ManagerWindow (Gtk.Application application) {
 			Object (application: application);
 
 			support_aur (false, false);
+			refreshing = false;
 
 			Timeout.add (100, populate_window);
 		}
@@ -196,7 +199,7 @@ namespace Pamac {
 			transaction.write_pamac_config_finished.connect (on_write_pamac_config_finished);
 			transaction.set_pkgreason_finished.connect (on_set_pkgreason_finished);
 
-			unowned Alpm.Package? pkg = Alpm.find_satisfier (transaction.alpm_config.handle.localdb.pkgcache, "yaourt");
+			unowned Alpm.Package? pkg = Alpm.find_satisfier (transaction.alpm_utils.get_installed_pkgs (), "yaourt");
 			if (pkg != null) {
 				support_aur (transaction.pamac_config.enable_aur, transaction.pamac_config.search_aur);
 			}
@@ -219,7 +222,7 @@ namespace Pamac {
 			if (recurse) {
 				transaction.flags |= Alpm.TransFlag.RECURSE;
 			}
-			unowned Alpm.Package? pkg = Alpm.find_satisfier (transaction.alpm_config.handle.localdb.pkgcache, "yaourt");
+			unowned Alpm.Package? pkg = Alpm.find_satisfier (transaction.alpm_utils.get_installed_pkgs (), "yaourt");
 			if (pkg != null) {
 				support_aur (enable_aur, search_aur);
 			}
@@ -246,115 +249,6 @@ namespace Pamac {
 			cancel_button.set_sensitive (sensitive);
 		}
 
-		async Alpm.List<unowned Alpm.Package?> search_all_dbs (string search_string) {
-			var syncpkgs = new Alpm.List<unowned Alpm.Package?> ();
-			var needles = new Alpm.List<unowned string> ();
-			string[] splitted = search_string.split (" ");
-			foreach (unowned string part in splitted) {
-				needles.add (part);
-			}
-			var result = transaction.alpm_config.handle.localdb.search (needles);
-			foreach (var db in transaction.alpm_config.handle.syncdbs) {
-				if (syncpkgs.length == 0) {
-					syncpkgs = db.search (needles);
-				} else {
-					syncpkgs.join (db.search (needles).diff (syncpkgs, (Alpm.List.CompareFunc) compare_name));
-				}
-			}
-			result.join (syncpkgs.diff (result, (Alpm.List.CompareFunc) compare_name));
-			//result.sort ((Alpm.List.CompareFunc) compare_name);
-			return result;
-		}
-
-		async Alpm.List<unowned Alpm.Package?> get_group_pkgs (string grp_name) {
-			var result = new Alpm.List<unowned Alpm.Package?> ();
-			unowned Alpm.Group? grp = transaction.alpm_config.handle.localdb.get_group (grp_name);
-			if (grp != null) {
-				foreach (var pkg in grp.packages) {
-					result.add (pkg);
-				}
-			}
-			result.join (Alpm.find_group_pkgs (transaction.alpm_config.handle.syncdbs, grp_name).diff (result, (Alpm.List.CompareFunc) compare_name));
-			//result.sort ((Alpm.List.CompareFunc) compare_name);
-			return result;
-		}
-
-		async Alpm.List<unowned Alpm.Package?> get_installed_pkgs () {
-			return transaction.alpm_config.handle.localdb.pkgcache.copy ();
-		}
-
-		async Alpm.List<unowned Alpm.Package?> get_orphans () {
-			var result = new Alpm.List<unowned Alpm.Package?> ();
-			foreach (var pkg in transaction.alpm_config.handle.localdb.pkgcache) {
-				if (pkg.reason == Alpm.Package.Reason.DEPEND) {
-					Alpm.List<string?> requiredby = pkg.compute_requiredby ();
-					if (requiredby.length == 0) {
-						Alpm.List<string?> optionalfor = pkg.compute_optionalfor ();
-						if (optionalfor.length == 0) {
-							result.add (pkg);
-						}
-						optionalfor.free_data ();
-					}
-					requiredby.free_data ();
-				}
-			}
-			return result;
-		}
-
-		async Alpm.List<unowned Alpm.Package?> get_local_pkgs () {
-			var result = new Alpm.List<unowned Alpm.Package?> ();
-			foreach (var pkg in transaction.alpm_config.handle.localdb.pkgcache) {
-				if (get_sync_pkg (pkg.name) == null) {
-					result.add (pkg);
-				}
-			}
-			return result;
-		}
-
-		async Alpm.List<unowned Alpm.Package?> get_repo_pkgs (string reponame) {
-			var result = new Alpm.List<unowned Alpm.Package?> ();
-			foreach (var db in transaction.alpm_config.handle.syncdbs) {
-				if (db.name == reponame) {
-					foreach (var sync_pkg in db.pkgcache) {
-						unowned Alpm.Package?local_pkg = transaction.alpm_config.handle.localdb.get_pkg (sync_pkg.name);
-						if (local_pkg != null) {
-							result.add (local_pkg);
-						} else {
-							result.add (sync_pkg);
-						}
-					}
-				}
-			}
-			return result;
-		}
-
-//~ 		async Alpm.List<unowned Alpm.Package?> get_all_pkgs () {
-//~ 			var syncpkgs = new Alpm.List<unowned Alpm.Package?> ();
-//~ 			var result = new Alpm.List<unowned Alpm.Package?> ();
-//~ 			result = transaction.alpm_config.handle.localdb.pkgcache.copy ();
-//~ 			foreach (var db in transaction.alpm_config.handle.syncdbs) {
-//~ 				if (syncpkgs.length == 0)
-//~ 					syncpkgs = db.pkgcache.copy ();
-//~ 				else {
-//~ 					syncpkgs.join (db.pkgcache.diff (syncpkgs, (Alpm.List.CompareFunc) compare_name));
-//~ 				}
-//~ 			}
-//~ 			result.join (syncpkgs.diff (result, (Alpm.List.CompareFunc) compare_name));
-//~ 			//result.sort ((Alpm.List.CompareFunc) compare_name);
-//~ 			return result;
-//~ 		}
-
-		unowned Alpm.Package? get_sync_pkg (string pkgname) {
-			unowned Alpm.Package? pkg = null;
-			foreach (var db in transaction.alpm_config.handle.syncdbs) {
-				pkg = db.get_pkg (pkgname);
-				if (pkg != null) {
-					break;
-				}
-			}
-			return pkg;
-		}
-
 		void show_default_pkgs () {
 			this.get_window ().set_cursor (new Gdk.Cursor.for_display (Gdk.Display.get_default (), Gdk.CursorType.WATCH));
 			get_installed_pkgs.begin ((obj, res) => {
@@ -369,7 +263,7 @@ namespace Pamac {
 			selection = repos_treeview.get_selection ();
 			selection.changed.disconnect (on_repos_treeview_selection_changed);
 			var groups_names = new GLib.List<string> ();
-			foreach (var db in transaction.alpm_config.handle.syncdbs) {
+			foreach (var db in transaction.alpm_utils.get_syncdbs ()) {
 				repos_list.insert_with_values (out iter, -1, 0, db.name);
 				foreach (var group in db.groupcache) {
 					if (groups_names.find_custom (group.name, strcmp) == null) {
@@ -384,7 +278,7 @@ namespace Pamac {
 
 			selection = groups_treeview.get_selection ();
 			selection.changed.disconnect (on_groups_treeview_selection_changed);
-			foreach (var group in transaction.alpm_config.handle.localdb.groupcache) {
+			foreach (var group in transaction.alpm_utils.get_localdb ().groupcache) {
 				if (groups_names.find_custom (group.name, strcmp) == null) {
 					groups_names.append (group.name);
 				}
@@ -481,7 +375,7 @@ namespace Pamac {
 				list = deps;
 				string optdep_str = list.data.compute_string ();
 				var optdep = new StringBuilder (optdep_str);
-				if (Alpm.find_satisfier (transaction.alpm_config.handle.localdb.pkgcache, optdep_str) != null) {
+				if (Alpm.find_satisfier (transaction.alpm_utils.get_installed_pkgs (), optdep_str) != null) {
 					optdep.append (" [");
 					optdep.append (dgettext (null, "Installed"));
 					optdep.append ("]");
@@ -492,7 +386,7 @@ namespace Pamac {
 				for (list = list.next (); list != null; list = list.next ()) {
 					optdep_str = list.data.compute_string ();
 					optdep = new StringBuilder (optdep_str);
-					if (Alpm.find_satisfier (transaction.alpm_config.handle.localdb.pkgcache, optdep_str) != null) {
+					if (Alpm.find_satisfier (transaction.alpm_utils.get_installed_pkgs (), optdep_str) != null) {
 						optdep.append (" [");
 						optdep.append (dgettext (null, "Installed"));
 						optdep.append ("]");
@@ -620,7 +514,7 @@ namespace Pamac {
 				deps = node.get_array ();
 				string optdep_str = deps.get_string_element (0);
 				var optdep = new StringBuilder (optdep_str);
-				if (Alpm.find_satisfier (transaction.alpm_config.handle.localdb.pkgcache, optdep_str) != null) {
+				if (Alpm.find_satisfier (transaction.alpm_utils.get_installed_pkgs (), optdep_str) != null) {
 					optdep.append (" [");
 					optdep.append (dgettext (null, "Installed"));
 					optdep.append ("]");
@@ -633,7 +527,7 @@ namespace Pamac {
 				while (i < length) {
 					optdep_str = deps.get_string_element (i);
 					optdep = new StringBuilder (optdep_str);
-					if (Alpm.find_satisfier (transaction.alpm_config.handle.localdb.pkgcache, optdep_str.split (": ", 2)[0]) != null) {
+					if (Alpm.find_satisfier (transaction.alpm_utils.get_installed_pkgs (), optdep_str.split (": ", 2)[0]) != null) {
 						optdep.append (" [");
 						optdep.append (dgettext (null, "Installed"));
 						optdep.append ("]");
@@ -984,7 +878,7 @@ namespace Pamac {
 					if (pkg_info == null) {
 						return;
 					}
-					unowned Alpm.Package? pkg = transaction.alpm_config.handle.localdb.get_pkg (pkg_info.get_string_member ("Name"));
+					unowned Alpm.Package? pkg = transaction.alpm_utils.get_installed_pkg (pkg_info.get_string_member ("Name"));
 					if (pkg == null) {
 						files_scrolledwindow.visible = false;
 						switch (properties_notebook.get_current_page ()) {
@@ -1050,7 +944,7 @@ namespace Pamac {
 				} else if (transaction.to_remove.remove (pkg.name)) {
 				} else {
 					if (pkg.origin == Alpm.Package.From.LOCALDB) {
-						if (transaction.alpm_config.holdpkgs.find_custom (pkg.name, strcmp) == null) {
+						if (transaction.alpm_utils.get_holdpkgs ().find_custom (pkg.name, strcmp) == null) {
 							transaction.to_remove.add (pkg.name);
 						}
 					} else {
@@ -1071,10 +965,10 @@ namespace Pamac {
 		void on_aur_treeview_row_activated (Gtk.TreeView treeview, Gtk.TreePath path, Gtk.TreeViewColumn column) {
 			unowned Json.Object? pkg_info = aur_list.get_pkg_at_path (path);
 			if (pkg_info != null) {
-				unowned Alpm.Package? pkg = transaction.alpm_config.handle.localdb.get_pkg (pkg_info.get_string_member ("Name"));
+				unowned Alpm.Package? pkg = transaction.alpm_utils.get_installed_pkg (pkg_info.get_string_member ("Name"));
 				if (pkg != null) {
 					if (pkg.origin == Alpm.Package.From.LOCALDB) {
-						if (transaction.alpm_config.holdpkgs.find_custom (pkg.name, strcmp) == null) {
+						if (transaction.alpm_utils.get_holdpkgs ().find_custom (pkg.name, strcmp) == null) {
 							transaction.to_remove.add (pkg.name);
 						}
 					}
@@ -1121,7 +1015,7 @@ namespace Pamac {
 		void on_remove_item_activate () {
 			foreach (var pkg in selected_pkgs) {
 				transaction.to_add.remove (pkg.name);
-				if (transaction.alpm_config.holdpkgs.find_custom (pkg.name, strcmp) == null) {
+				if (transaction.alpm_utils.get_holdpkgs ().find_custom (pkg.name, strcmp) == null) {
 					if (pkg.origin == Alpm.Package.From.LOCALDB) {
 						transaction.to_remove.add (pkg.name);
 					}
@@ -1156,7 +1050,7 @@ namespace Pamac {
 				Gtk.TreeIter iter;
 				int length = 0;
 				foreach (var optdep in pkg.optdepends) {
-					if (Alpm.find_satisfier (transaction.alpm_config.handle.localdb.pkgcache, optdep.name) == null) {
+					if (Alpm.find_satisfier (transaction.alpm_utils.get_installed_pkgs (), optdep.name) == null) {
 						length++;
 						choose_dep_dialog.deps_list.insert_with_values (out iter, -1,
 											0, false,
@@ -1175,7 +1069,7 @@ namespace Pamac {
 						if ((bool) val) {
 							// get value at column 1 to get the pkg name
 							choose_dep_dialog.deps_list.get_value (iter, 1, out val);
-							unowned Alpm.Package? sync_pkg = get_sync_pkg ((string) val);
+							unowned Alpm.Package? sync_pkg = transaction.alpm_utils.get_sync_pkg ((string) val);
 							if (sync_pkg != null) {
 								transaction.to_add.add (sync_pkg.name);
 							}
@@ -1289,7 +1183,7 @@ namespace Pamac {
 					clicked_pkg = selected_pkgs.data;
 					if (clicked_pkg.origin == Alpm.Package.From.LOCALDB) {
 						foreach (var optdep in clicked_pkg.optdepends) {
-							if (Alpm.find_satisfier (transaction.alpm_config.handle.localdb.pkgcache, optdep.name) == null) {
+							if (Alpm.find_satisfier (transaction.alpm_utils.get_installed_pkgs (), optdep.name) == null) {
 								install_optional_deps_item.set_sensitive (true);
 								break;
 							}
@@ -1297,7 +1191,7 @@ namespace Pamac {
 						if (clicked_pkg.reason == Alpm.Package.Reason.DEPEND) {
 							explicitly_installed_item.set_sensitive (true);
 						}
-						unowned Alpm.Package? find_pkg = get_sync_pkg (clicked_pkg.name);
+						unowned Alpm.Package? find_pkg = transaction.alpm_utils.get_sync_pkg (clicked_pkg.name);
 						if (find_pkg != null) {
 							if (Alpm.pkg_vercmp (find_pkg.version, clicked_pkg.version) == 0) {
 								reinstall_item.set_sensitive (true);
@@ -1339,7 +1233,7 @@ namespace Pamac {
 				selected_pkgs = new Alpm.List<unowned Alpm.Package?> ();
 				selected_aur = new GLib.List<unowned Json.Object> ();
 				foreach (unowned Gtk.TreePath path in selected_paths) {
-					unowned Alpm.Package? pkg = transaction.alpm_config.handle.localdb.get_pkg (clicked_pkg_info.get_string_member ("Name"));
+					unowned Alpm.Package? pkg = transaction.alpm_utils.get_installed_pkg (clicked_pkg_info.get_string_member ("Name"));
 					if (pkg != null) {
 						selected_pkgs.add (pkg);
 						// there is for sure a pkg to remove
@@ -1573,6 +1467,30 @@ namespace Pamac {
 			on_search_entry_activate ();
 		}
 
+		async Alpm.List<unowned Alpm.Package?> search_all_dbs (string search_string) {
+			return transaction.alpm_utils.search_all_dbs (search_string);
+		}
+
+		async Alpm.List<unowned Alpm.Package?> get_group_pkgs (string group_name) {
+			return transaction.alpm_utils.get_group_pkgs (group_name);
+		}
+
+		async Alpm.List<unowned Alpm.Package?> get_installed_pkgs () {
+			return transaction.alpm_utils.get_installed_pkgs ();
+		}
+
+		async Alpm.List<unowned Alpm.Package?> get_orphans () {
+			return transaction.alpm_utils.get_orphans ();
+		}
+
+		async Alpm.List<unowned Alpm.Package?> get_local_pkgs () {
+			return transaction.alpm_utils.get_local_pkgs ();
+		}
+
+		async Alpm.List<unowned Alpm.Package?> get_repo_pkgs (string repo_name) {
+			return transaction.alpm_utils.get_repo_pkgs (repo_name);
+		}
+
 		[GtkCallback]
 		void on_search_treeview_selection_changed () {
 			Gtk.TreeModel model;
@@ -1644,9 +1562,9 @@ namespace Pamac {
 				if (state == dgettext (null, "To install")) {
 					var pkgs = new Alpm.List<unowned Alpm.Package?> ();
 					foreach (unowned string pkgname in transaction.to_add) {
-						unowned Alpm.Package? pkg = transaction.alpm_config.handle.localdb.get_pkg (pkgname);
+						unowned Alpm.Package? pkg = transaction.alpm_utils.get_installed_pkg (pkgname);
 						if (pkg == null) {
-							pkg = get_sync_pkg (pkgname);
+							pkg = transaction.alpm_utils.get_sync_pkg (pkgname);
 						}
 						if (pkg != null) {
 							pkgs.add (pkg);
@@ -1656,7 +1574,7 @@ namespace Pamac {
 				} else if (state == dgettext (null, "To remove")) {
 					var pkgs = new Alpm.List<unowned Alpm.Package?> ();
 					foreach (unowned string pkgname in transaction.to_remove) {
-						unowned Alpm.Package? pkg = transaction.alpm_config.handle.localdb.get_pkg (pkgname);
+						unowned Alpm.Package? pkg = transaction.alpm_utils.get_installed_pkg (pkgname);
 						if (pkg != null) {
 							pkgs.add (pkg);
 						}
@@ -1815,17 +1733,20 @@ namespace Pamac {
 		[GtkCallback]
 		void on_refresh_button_clicked () {
 			this.get_window ().set_cursor (new Gdk.Cursor.for_display (Gdk.Display.get_default (), Gdk.CursorType.WATCH));
+			refreshing = true;
 			transaction.start_refresh (false);
 		}
 
-		void on_transaction_finished (bool database_modified) {
+		void on_transaction_finished (bool success) {
-			if (database_modified) {
 			set_buttons_sensitive (false);
 			refresh_packages_list ();
-			} else {
-				this.get_window ().set_cursor (null);
-			}
 			transaction.to_load.remove_all ();
+			if (refreshing) {
+				if (success) {
+					transaction.sysupgrade (false);
+				}
+				refreshing = false;
+			}
 		}
 	}
 }

src/packages_model.vala

@@ -62,7 +62,7 @@ namespace Pamac {
 					val = Value (typeof (Object));
 					if (pkg != null) {
 						if (pkg.origin == Alpm.Package.From.LOCALDB) {
-							if (manager_window.transaction.alpm_config.holdpkgs.find_custom (pkg.name, strcmp) != null) {
+							if (manager_window.transaction.alpm_utils.get_holdpkgs ().find_custom (pkg.name, strcmp) != null) {
 								val.set_object (manager_window.locked_icon);
 							} else if (manager_window.transaction.to_add.contains (pkg.name)) {
 								val.set_object (manager_window.to_reinstall_icon);
@@ -184,7 +184,7 @@ namespace Pamac {
 
 		// custom sort functions
 		public void sort_by_name (Gtk.SortType order) {
-			pkgs.sort ((Alpm.List.CompareFunc) compare_name);
+			pkgs.sort ((Alpm.List.CompareFunc) alpm_pkg_compare_name);
 			if (order == Gtk.SortType.DESCENDING) {
 				pkgs.reverse ();
 			}
@@ -199,7 +199,7 @@ namespace Pamac {
 		}
 
 		public void sort_by_state (Gtk.SortType order) {
-			pkgs.sort ((Alpm.List.CompareFunc) compare_state);
+			pkgs.sort ((Alpm.List.CompareFunc) alpm_pkg_compare_origin);
 			if (order == Gtk.SortType.DESCENDING) {
 				pkgs.reverse ();
 			}
@@ -214,7 +214,7 @@ namespace Pamac {
 		}
 
 		public void sort_by_version (Gtk.SortType order) {
-			pkgs.sort ((Alpm.List.CompareFunc) compare_version);
+			pkgs.sort ((Alpm.List.CompareFunc) alpm_pkg_compare_version);
 			if (order == Gtk.SortType.DESCENDING) {
 				pkgs.reverse ();
 			}
@@ -229,7 +229,7 @@ namespace Pamac {
 		}
 
 		public void sort_by_repo (Gtk.SortType order) {
-			pkgs.sort ((Alpm.List.CompareFunc) compare_repo);
+			pkgs.sort ((Alpm.List.CompareFunc) alpm_pkg_compare_db_name);
 			if (order == Gtk.SortType.DESCENDING) {
 				pkgs.reverse ();
 			}
@@ -244,7 +244,7 @@ namespace Pamac {
 		}
 
 		public void sort_by_size (Gtk.SortType order) {
-			pkgs.sort ((Alpm.List.CompareFunc) compare_size);
+			pkgs.sort ((Alpm.List.CompareFunc) alpm_pkg_compare_installed_size);
 			if (order == Gtk.SortType.DESCENDING) {
 				pkgs.reverse ();
 			}

src/preferences_dialog.vala

@@ -68,7 +68,7 @@ namespace Pamac {
 			this.transaction = transaction;
 			refresh_period_label.set_markup (dgettext (null, "How often to check for updates, value in hours") +":");
 			remove_unrequired_deps_button.active = transaction.pamac_config.recurse;
-			check_space_button.active = (transaction.alpm_config.checkspace == 1);
+			check_space_button.active = (transaction.alpm_utils.get_checkspace () == 1);
 			if (transaction.pamac_config.refresh_period == 0) {
 				check_updates_button.active = false;
 				refresh_period_label.sensitive = false;
@@ -87,8 +87,8 @@ namespace Pamac {
 
 			// populate ignorepkgs_liststore
 			Gtk.TreeIter iter;
-			for (unowned Alpm.List<string> list = transaction.alpm_config.ignorepkgs; list != null; list = list.next ()) {
+			foreach (unowned string ignorepkg in transaction.alpm_utils.get_ignorepkgs ()) {
-				ignorepkgs_liststore.insert_with_values (out iter, -1, 0, list.data);
+				ignorepkgs_liststore.insert_with_values (out iter, -1, 0, ignorepkg);
 			}
 			remove_unrequired_deps_button.state_set.connect (on_remove_unrequired_deps_button_state_set);
 			check_space_button.state_set.connect (on_check_space_button_state_set);
@@ -98,7 +98,7 @@ namespace Pamac {
 			no_update_hide_icon_checkbutton.toggled.connect (on_no_update_hide_icon_checkbutton_toggled);
 			transaction.write_pamac_config_finished.connect (on_write_pamac_config_finished);
 
-			unowned Alpm.Package? pkg = Alpm.find_satisfier (transaction.alpm_config.handle.localdb.pkgcache, "pacman-mirrorlist");
+			unowned Alpm.Package? pkg = Alpm.find_satisfier (transaction.alpm_utils.get_installed_pkgs (), "pacman-mirrorlist");
 			if (pkg == null) {
 				mirrors_config_box.visible = false;
 			} else {
@@ -126,7 +126,7 @@ namespace Pamac {
 				transaction.write_mirrors_config_finished.connect (on_write_mirrors_config_finished);
 			}
 
-			pkg = Alpm.find_satisfier (transaction.alpm_config.handle.localdb.pkgcache, "yaourt");
+			pkg = Alpm.find_satisfier (transaction.alpm_utils.get_installed_pkgs (), "yaourt");
 			if (pkg == null) {
 				aur_config_box.visible = false;
 			} else {
@@ -248,9 +248,9 @@ namespace Pamac {
 		[GtkCallback]
 		void on_add_ignorepkgs_button_clicked () {
 			var choose_ignorepkgs_dialog = new ChooseIgnorepkgsDialog (this);
-			foreach (var pkg in transaction.alpm_config.handle.localdb.pkgcache) {
+			foreach (var pkg in transaction.alpm_utils.get_installed_pkgs ()) {
 				Gtk.TreeIter iter;
-				if (transaction.alpm_config.ignorepkgs.find_str (pkg.name) == null) {
+				if (transaction.alpm_utils.get_ignorepkgs ().find_str (pkg.name) == null) {
 					choose_ignorepkgs_dialog.pkgs_list.insert_with_values (out iter, -1, 0, false, 1, pkg.name);
 				} else {
 					choose_ignorepkgs_dialog.pkgs_list.insert_with_values (out iter, -1, 0, true, 1, pkg.name);
@@ -311,8 +311,8 @@ namespace Pamac {
 			// re-populate ignorepkgs_liststore
 			Gtk.TreeIter iter;
 			ignorepkgs_liststore.clear ();
-			for (unowned Alpm.List<string> list = transaction.alpm_config.ignorepkgs; list != null; list = list.next ()) {
+			foreach (unowned string ignorepkg in transaction.alpm_utils.get_ignorepkgs ()) {
-				ignorepkgs_liststore.insert_with_values (out iter, -1, 0, list.data);
+				ignorepkgs_liststore.insert_with_values (out iter, -1, 0, ignorepkg);
 			}
 		}
 

src/transaction.vala

@@ -84,7 +84,7 @@ namespace Pamac {
 
 		Daemon daemon;
 
-		public AlpmConfig alpm_config;
+		public AlpmUtils alpm_utils;
 		public Pamac.Config pamac_config;
 
 		public Alpm.TransFlag flags;
@@ -113,6 +113,7 @@ namespace Pamac {
 		uint64 rates_nb;
 		Timer timer;
 		bool database_modified;
+		bool success;
 
 		//dialogs
 		TransactionSumDialog transaction_sum_dialog;
@@ -121,7 +122,7 @@ namespace Pamac {
 		//parent window
 		public Gtk.ApplicationWindow? application_window;
 
-		public signal void finished (bool database_modified);
+		public signal void finished (bool success);
 		public signal void set_pkgreason_finished ();
 		public signal void get_updates_finished (Updates updates);
 		public signal void write_pamac_config_finished (bool recurse, uint64 refresh_period, bool no_update_hide_icon,
@@ -131,8 +132,7 @@ namespace Pamac {
 		public signal void write_mirrors_config_finished (string choosen_country, string choosen_generation_method);
 
 		public Transaction (Gtk.ApplicationWindow? application_window) {
-			alpm_config = new AlpmConfig ("/etc/pacman.conf");
+			alpm_utils = new AlpmUtils ("/etc/pacman.conf");
-			refresh_handle ();
 			pamac_config = new Pamac.Config ("/etc/pamac.conf");
 			flags = Alpm.TransFlag.CASCADE;
 			if (pamac_config.recurse) {
@@ -162,6 +162,7 @@ namespace Pamac {
 			sysupgrade_after_trans = false;
 			timer = new Timer ();
 			database_modified = false;
+			success = false;
 		}
 
 		public async void run_preferences_dialog () {
@@ -182,13 +183,6 @@ namespace Pamac {
 			daemon.disconnect (handler_id);
 		}
 
-		public void refresh_handle () {
-			alpm_config.get_handle ();
-			if (alpm_config.handle == null) {
-				stderr.printf (dgettext (null, "Failed to initialize alpm library"));
-			}
-		}
-
 		public ErrorInfos get_current_error () {
 			try {
 				return daemon.get_current_error ();
@@ -280,15 +274,29 @@ namespace Pamac {
 				stderr.printf ("IOError: %s\n", e.message);
 				daemon.refresh_finished.disconnect (on_refresh_finished);
 				database_modified = true;
+				success = false;
 				finish_transaction ();
 			}
 		}
 
 		public void start_get_updates () {
+			daemon.get_updates_finished.connect (on_get_updates_finished);
 			try {
 				daemon.start_get_updates (pamac_config.enable_aur && pamac_config.check_aur_updates);
 			} catch (IOError e) {
 				stderr.printf ("IOError: %s\n", e.message);
+				success = false;
+				finish_transaction ();
+			}
+		}
+
+		void start_get_updates_for_sysupgrade () {
+			daemon.get_updates_finished.connect (on_get_updates_for_sysupgrade_finished);
+			try {
+				daemon.start_get_updates (pamac_config.enable_aur && pamac_config.check_aur_updates);
+			} catch (IOError e) {
+				stderr.printf ("IOError: %s\n", e.message);
+				success = false;
 				finish_transaction ();
 			}
 		}
@@ -325,7 +333,7 @@ namespace Pamac {
 		void sysupgrade_simple (bool enable_downgrade) {
 			progress_dialog.progressbar.set_fraction (0);
 			progress_dialog.cancel_button.set_visible (true);
-			bool success = init (0);
+			success = init (0);
 			if (success) {
 				try {
 					success = daemon.trans_sysupgrade (enable_downgrade);
@@ -343,6 +351,7 @@ namespace Pamac {
 					} catch (IOError e) {
 						stderr.printf ("IOError: %s\n", e.message);
 						release ();
+						success = false;
 						finish_transaction ();
 					}
 				} else {
@@ -366,12 +375,16 @@ namespace Pamac {
 			while (Gtk.events_pending ()) {
 				Gtk.main_iteration ();
 			}
-			// sysupgrade
+			start_get_updates_for_sysupgrade ();
-			daemon.get_updates_finished.connect (on_get_updates_finished);
-			start_get_updates ();
 		}
 
 		void on_get_updates_finished (Updates updates) {
+			daemon.get_updates_finished.disconnect (on_get_updates_finished);
+			get_updates_finished (updates);
+		}
+
+		void on_get_updates_for_sysupgrade_finished (Updates updates) {
+			daemon.get_updates_finished.disconnect (on_get_updates_for_sysupgrade_finished);
 			// get syncfirst updates
 			if (updates.is_syncfirst) {
 				clear_lists ();
@@ -402,8 +415,6 @@ namespace Pamac {
 					on_trans_prepare_finished (true);
 				}
 			}
-			daemon.get_updates_finished.disconnect (on_get_updates_finished);
-			get_updates_finished (updates);
 		}
 
 		public void clear_lists () {
@@ -432,7 +443,7 @@ namespace Pamac {
 				// there only AUR packages to build so no need to prepare transaction
 				on_trans_prepare_finished (true);
 			} else {
-				bool success = false;
+				success = false;
 				try {
 					success = daemon.trans_init (flags);
 				} catch (IOError e) {
@@ -476,6 +487,7 @@ namespace Pamac {
 						} catch (IOError e) {
 							stderr.printf ("IOError: %s\n", e.message);
 							release ();
+							success = false;
 							finish_transaction ();
 						}
 					} else {
@@ -647,6 +659,7 @@ namespace Pamac {
 			} catch (IOError e) {
 				stderr.printf ("IOError: %s\n", e.message);
 				database_modified = true;
+				success = false;
 				finish_transaction ();
 			}
 		}
@@ -1150,31 +1163,23 @@ namespace Pamac {
 
 		void finish_transaction () {
 			if (database_modified) {
-				refresh_handle ();
+				alpm_utils.reload ();
+				database_modified = false;
 			}
 			if (progress_dialog.expander.get_expanded ()) {
 				progress_dialog.cancel_button.set_visible (false);
 				progress_dialog.close_button.set_visible (true);
 			} else {
-				finished (database_modified);
+				on_progress_dialog_close_button_clicked ();
-				progress_dialog.hide ();
-				while (Gtk.events_pending ()) {
-					Gtk.main_iteration ();
 			}
 		}
-			database_modified = false;
-		}
 
 		void on_refresh_finished (bool success) {
 			database_modified = true;
-			if (success) {
+			this.success = success;
-				if (mode == Mode.UPDATER) {
-					finish_transaction ();
-				} else {
-					refresh_handle ();
 			clear_lists ();
-					sysupgrade (false);
+			if (success) {
-				}
+				finish_transaction ();
 			} else {
 				handle_error (get_current_error ());
 			}
@@ -1183,11 +1188,12 @@ namespace Pamac {
 		}
 
 		void on_progress_dialog_close_button_clicked () {
-			finished (database_modified);
+			finished (success);
 			progress_dialog.hide ();
 			while (Gtk.events_pending ()) {
 				Gtk.main_iteration ();
 			}
+			success = false;
 		}
 
 		void on_progress_dialog_cancel_button_clicked () {
@@ -1201,6 +1207,7 @@ namespace Pamac {
 		}
 
 		void on_trans_prepare_finished (bool success) {
+			this.success = success;
 			if (success) {
 				show_warnings ();
 				Type type = set_transaction_sum ();
@@ -1228,6 +1235,7 @@ namespace Pamac {
 						release ();
 						//to_build.remove_all ();
 						sysupgrade_after_trans = false;
+						success = false;
 						finish_transaction ();
 					}
 				} else {
@@ -1245,6 +1253,7 @@ namespace Pamac {
 		}
 
 		void on_trans_commit_finished (bool success) {
+			this.success = success;
 			if (success) {
 				if (to_build.length != 0) {
 					if (to_add.length != 0
@@ -1270,13 +1279,11 @@ namespace Pamac {
 			} else {
 				// if it is an authentication error, database was not modified
 				var err = get_current_error ();
-				if (err.message == dgettext (null, "Authentication failed")) {
+				if (err.message != dgettext (null, "Authentication failed")) {
-					handle_error (err);
-				} else {
 					clear_lists ();
 					database_modified = true;
-					handle_error (err);
 				}
+				handle_error (err);
 			}
 			total_download = 0;
 			already_downloaded = 0;
@@ -1289,21 +1296,25 @@ namespace Pamac {
 			// let the time to the daemon to update databases
 			Timeout.add (1000, () => {
 				if (status == 0) {
+					success = true;
 					unowned string action = dgettext (null, "Transaction successfully finished");
 					progress_dialog.spawn_in_term ({"echo", action + ".\n"});
 					progress_dialog.action_label.set_text (action);
 				} else {
+					success = false;
 					progress_dialog.spawn_in_term ({"echo"});
 				}
 				progress_dialog.progressbar.set_fraction (1);
-				database_modified = true;
+				alpm_utils.reload ();
-				finish_transaction ();
+				database_modified = false;
+				progress_dialog.cancel_button.set_visible (false);
+				progress_dialog.close_button.set_visible (true);
 				return false;
 			});
 		}
 
 		void on_set_pkgreason_finished () {
-			refresh_handle ();
+			alpm_utils.reload ();
 			set_pkgreason_finished ();
 		}
 
@@ -1317,7 +1328,7 @@ namespace Pamac {
 		}
 
 		void on_write_alpm_config_finished (bool checkspace) {
-			alpm_config.reload ();
+			alpm_utils.reload ();
 			write_alpm_config_finished (checkspace);
 		}
 

src/tray.vala

@@ -321,7 +321,7 @@ namespace Pamac {
 			Notify.init (_("Update Manager"));
 
 			var alpm_config = new AlpmConfig ("/etc/pacman.conf");
-			alpm_config.get_handle ();
+			alpm_config.set_handle ();
 			lockfile = GLib.File.new_for_path (alpm_config.handle.lockfile);
 			start_daemon ();
 			Timeout.add (200, check_pacman_running);

src/updater_window.vala

@@ -69,8 +69,7 @@ namespace Pamac {
 
 			transaction = new Transaction (this as Gtk.ApplicationWindow);
 			transaction.mode = Mode.UPDATER;
-			transaction.finished.connect (on_transaction_finished);
+			transaction.finished.connect (populate_updates_list);
-
 			transaction.get_updates_finished.connect (on_get_updates_finished);
 
 			on_refresh_button_clicked ();
@@ -159,14 +158,6 @@ namespace Pamac {
 			this.application.quit ();
 		}
 
-		void on_transaction_finished (bool database_modified) {
-			if (database_modified) {
-				populate_updates_list ();
-			} else {
-				this.get_window ().set_cursor (null);
-			}
-		}
-
 		void populate_updates_list () {
 			this.get_window ().set_cursor (new Gdk.Cursor.for_display (Gdk.Display.get_default (), Gdk.CursorType.WATCH));
 			transaction.start_get_updates ();