X-Git-Url: http://git.roojs.org/?a=blobdiff_plain;f=GitRepo.vala;h=6954437cd13cc3971096b4128846e211125717ef;hb=c49672c25237ec593d6bf9983d8aa6e9b131ff9f;hp=936ecd55ced119a9df58ad0f6ed6abbcffa54273;hpb=df7967d0f640ee80ad070016b6a2d1020ed01328;p=gitlive

diff --git a/GitRepo.vala b/GitRepo.vala
index 936ecd55..6954437c 100644
--- a/GitRepo.vala
+++ b/GitRepo.vala
@@ -29,6 +29,7 @@ public class GitRepo : Object
     public Gee.HashMap<string,GitBranch> branches; // accessed in GitBranch..
 	public RooTicket? activeTicket;
     public  Gee.HashMap<string,GitRepo> cache;
+    public  Gee.HashMap<string,string> config_cache;
     
     
     
@@ -155,7 +156,8 @@ public class GitRepo : Object
      *
      */
      
-    private GitRepo(string path) {
+    private GitRepo(string path) 
+    {
         // cal parent?
         this.name =   File.new_for_path(path).get_basename();
         this.ignore_files = new Gee.HashMap<string,bool>();
@@ -176,7 +178,7 @@ public class GitRepo : Object
     	var r = this.git({ "remote" , "get-url" , "--push" , "origin"});
     	var uri = new Soup.URI(r);    	
     	this.host = uri.get_host();
-
+		this.init_config();
     	
     	this.loadBranches();
     	this.loadActiveTicket();
@@ -191,6 +193,7 @@ public class GitRepo : Object
     }
     public void init_config()
     {
+    	this.config_cache = new Gee.HashMap<string,string>();
     	// managed = 
     	if (this.get_config("managed") == "") {
     		this.set_config("managed", this.host == "git.roojs.com" ? "1" : "0");
@@ -198,16 +201,35 @@ public class GitRepo : Object
     	if (this.get_config("autocommit") == "") {
     		this.set_config("autocommit", this.host == "git.roojs.com" ? "1" : "0");
 		}
-    
+    	if (this.get_config("autopush") == "") {
+    		this.set_config("autopush", this.host == "git.roojs.com" ? "1" : "0");
+		}
+		
     }
     
     
     
     public string get_config(string key) {
-    	return this.git({ "config" , "gitlive." + key });
+    
+    	if (this.config_cache.has_key(key)) {
+    		//GLib.debug("get_config %s = '%s'", key, this.config_cache.get(key));
+    		return this.config_cache.get(key);
+		}
+    	try {
+	    	var ret =   this.git({ "config" , "gitlive." + key }).strip();
+	    	this.config_cache.set(key, ret);
+    		//GLib.debug("get_config %s = '%s'", key, ret);
+	    	return ret;
+    	} catch (Error e) {
+	    	this.config_cache.set(key, "");
+    		//GLib.debug("get_config (fail) %s = '%s'", key, "");
+    		return ""; // happens when there is nothing set...
+    	}
+
 	}
-    public string set_config(string key, string value) {
-    	return this.git({ "config" , "gitlive." + key, value });
+    public void set_config(string key, string value) {
+    	this.git({ "config" , "gitlive." + key, value });
+    	this.config_cache.set(key,value);
 	}
     
     public bool is_managed()
@@ -243,52 +265,19 @@ public class GitRepo : Object
     
     public void set_auto_branch(bool val)
     {
-
-		var cur = this.is_auto_branch();
-		GLib.debug("SET auto branch : %s <= %s", val ? "ON" : "OFF",  cur  ? "ON" : "OFF");
-		
-		if (cur == val) {
-			return; // no change..
-		}
-    	if (this.host != "git.roojs.com") { // we can only push to this url. -- unless we have forced it to be managed.
-	    	// remote is not our server..
-	    	if (val) {
-	    		FileUtils.set_contents(this.gitdir + "/.gitlive-managed" , "x");
-    		} else {
-    			FileUtils.remove(this.gitdir + "/.gitlive-managed" ); 
-			}
+		if (this.name == "gitlog") {
     		return;
-
-    	}
-    	
-    	if (val) {
-			FileUtils.remove(this.gitdir + "/.gitlive-unmanaged" ); 
-		} else {
-    		FileUtils.set_contents(this.gitdir + "/.gitlive-unmanaged" , "x");		
-
 		}
-    	  
+		this.set_config("managed", val ? "1" : "0");
     
     }
     public bool is_autopush ()
     {
-    	return !FileUtils.test(this.gitdir + "/.gitlive-disable-autopush" , FileTest.EXISTS);
+    	return this.get_config("autopush") == "1";
     }
     public void set_autopush(bool val)
     {
-
-		var cur = this.is_autopush();
-		GLib.debug("SET auto push : %s <= %s", val ? "ON" : "OFF",  cur  ? "ON" : "OFF");
-		if (cur == val) {
-			return; // no change..
-		}
-		if (!val) {
-			FileUtils.set_contents(this.gitdir + "/.gitlive-disable-autopush" , "");
-		} else {
-			// it exists...
-			FileUtils.remove(this.gitdir + "/.gitlive-disable-autopush" ); 
-		}
-    
+		this.set_config("autopush", val ? "1" : "0");
     }
     
     
@@ -441,11 +430,8 @@ public class GitRepo : Object
     public void loadActiveTicket()
     {
     	this.activeTicket = null;
-    	if (!FileUtils.test(this.gitdir + "/.gitlive-active-ticket" , FileTest.EXISTS)) {
-	    	return;
-    	}
-    	string ticket_id;
-    	FileUtils.get_contents(this.gitdir + "/.gitlive-active-ticket" , out ticket_id);  
+		var ticket_id = this.get_config("ticket");
+    	
     	if (ticket_id.length < 1) {
     		return;
 		}
@@ -458,14 +444,11 @@ public class GitRepo : Object
     
     public bool setActiveTicket(RooTicket? ticket, string branchname)
     {
+    	this.set_config("ticket", "");
     	if (!this.createBranchNamed(branchname)) {
     		return false;
 		}
-		if (ticket != null) {
-	    	FileUtils.set_contents(this.gitdir + "/.gitlive-active-ticket" , ticket.id);
-    	} else {
-	    	FileUtils.remove(this.gitdir + "/.gitlive-active-ticket" );
-    	}
+		this.set_config("ticket", ticket == null ? "": ticket.id);
     	this.activeTicket = ticket;
     	return true;
     }
@@ -809,6 +792,8 @@ public class GitRepo : Object
        // may throw error...
         var sp = new Spawn(cfg);
       
+	     //GLib.debug( "GOT result: %d" , sp.result);
+      
       	// diff output is a bit big..
 		if (args_in[0] != "diff") {
 	        GLib.debug( "GOT: %s" , sp.output);
@@ -902,7 +887,7 @@ public class GitRepo : Object
        		}
            repo.update_async(updateAllCallback); 
         } 
-
+		GLib.debug("calls total = %d", (int) update_all_total);
     }
     public static void  updateAllCallback(GitRepo repo, int err, string res)
     {
@@ -910,14 +895,19 @@ public class GitRepo : Object
     	repo.loadStatus();
     	
     	update_all_total--;
+		GLib.debug("calls remaining = %d", (int)update_all_total);    	
     	if (update_all_total > 0 ) {
+
     		return;
 		}
+		GLib.debug("call after load = %s", update_all_after);    
+		
 		switch (update_all_after) {
 			case "show_clones":
 				Clones.singleton().show();
 				break;
 			default:
+				GLib.debug("Unkown call after load = %s", update_all_after);    	
 				break;
 		}
 		return;