git.repo.base: substituted all usages of the active_branch property with self.head.* in order to more safely get the current commit. active_branch would only work on non-detached heads

Byron · Byron · commit 031271e89032 · 2011-01-17T20:41:20.000+01:00
diff --git a/git/repo/base.py b/git/repo/base.py
@@ -368,7 +368,7 @@ def commit(self, rev=None):
 		:param rev: revision specifier, see git-rev-parse for viable options.
 		:return: ``git.Commit``"""
 		if rev is None:
-			return self.active_branch.commit
+			return self.head.commit
 		else:
 			return self.rev_parse(str(rev)+"^0")
 		
@@ -391,7 +391,7 @@ def tree(self, rev=None):
 			it cannot know about its path relative to the repository root and subsequent 
 			operations might have unexpected results."""
 		if rev is None:
-			return self.active_branch.commit.tree
+			return self.head.commit.tree
 		else:
 			return self.rev_parse(str(rev)+"^{tree}")
 
@@ -415,7 +415,7 @@ def iter_commits(self, rev=None, paths='', **kwargs):
 
 		:return ``git.Commit[]``"""
 		if rev is None:
-			rev = self.active_branch
+			rev = self.head.commit
 		
 		return Commit.iter_items(self, rev, paths, **kwargs)
 
@@ -739,7 +739,7 @@ def archive(self, ostream, treeish=None, prefix=None,  **kwargs):
 		:raise GitCommandError: in case something went wrong
 		:return: self"""
 		if treeish is None:
-			treeish = self.active_branch
+			treeish = self.head.commit
 		if prefix and 'prefix' not in kwargs:
 			kwargs['prefix'] = prefix 
 		kwargs['output_stream'] = ostream
