Fixed incorrect handling of rev_parse if it was fed with branches that looked like hexshas. Thanks to Ning (frostyplanet) who pointed this issue out

Author: Byron

git/repo/fun.py

@@ -59,7 +59,11 @@ def name_to_object(repo, name, return_ref=False):
 		else:
 			hexsha = name
 		# END handle short shas
-	else:
+	#END find sha if it matches
+	
+	# if we couldn't find an object for what seemed to be a short hexsha 
+	# try to find it as reference anyway, it could be named 'aaa' for instance
+	if hexsha is None:
 		for base in ('%s', 'refs/%s', 'refs/tags/%s', 'refs/heads/%s', 'refs/remotes/%s', 'refs/remotes/%s/HEAD'):
 			try:
 				hexsha = SymbolicReference.dereference_recursive(repo, base % name)

git/test/test_repo.py

@@ -464,6 +464,12 @@ def _assert_rev_parse(self, name):
 
 		return orig_obj
 
+	@with_rw_repo('HEAD', bare=False)
+	def test_rw_rev_parse(self, rwrepo):
+		# verify it does not confuse branches with hexsha ids
+		ahead = rwrepo.create_head('aaaaaaaa')
+		assert(rwrepo.rev_parse(str(ahead)) == ahead.commit)
+		
 	def test_rev_parse(self):
 		rev_parse = self.rorepo.rev_parse
 
@@ -573,7 +579,6 @@ def test_rev_parse(self):
 		# currently, nothing more is supported
 		self.failUnlessRaises(NotImplementedError, rev_parse, "@{1 week ago}")
 
-		
 	def test_repo_odbtype(self):
 		target_type = GitDB
 		if sys.version_info[1] < 5: