Refactor part of GetUploadableBranches to lookup one specific branch

This way project.GetUploadableBranch(project.CurrentBranch) can tell
us how (if at all) to upload the currently checked out branch.

Signed-off-by: Shawn O. Pearce <sop@google.com>
This commit is contained in:
Shawn O. Pearce 2008-11-11 17:03:13 -08:00
parent 5d40e26201
commit 35f2596c27

View File

@ -428,14 +428,22 @@ class Project(object):
if branch in pubed and pubed[branch] == id: if branch in pubed and pubed[branch] == id:
continue continue
branch = self.GetBranch(branch) rb = self.GetUploadableBranch(branch)
base = branch.LocalMerge if rb:
if branch.LocalMerge: ready.append(rb)
rb = ReviewableBranch(self, branch, base)
if rb.commits:
ready.append(rb)
return ready return ready
def GetUploadableBranch(self, branch_name):
"""Get a single uploadable branch, or None.
"""
branch = self.GetBranch(branch_name)
base = branch.LocalMerge
if branch.LocalMerge:
rb = ReviewableBranch(self, branch, base)
if rb.commits:
return rb
return None
def UploadForReview(self, branch=None): def UploadForReview(self, branch=None):
"""Uploads the named branch for code review. """Uploads the named branch for code review.
""" """