comparison mercurial/hgweb/webcommands.py @ 19534:983bb4069004

hgweb: cleaner if conditions in changelog() function

This removes unneeded extra nesting level and extra variable, which makes
the code easier to understand.

19533:9a020b354d93

def changelog(web, req, tmpl, shortlog=False):

    query = ''
    if 'node' in req.form:
        ctx = webutil.changectx(web.repo, req)
    else:
        if 'rev' in req.form:
            query = req.form['rev'][0]
            hi = query
        else:
            hi = 'tip'
        try:
            ctx = web.repo[hi]
        except (error.RepoError, error.LookupError):
            return _search(web, req, tmpl) # XXX redirect to 404 page?

    def changelist(latestonly, **map):
        revs = []
        if pos != -1:
            revs = web.repo.changelog.revs(pos, 0)

19534:983bb4069004

def changelog(web, req, tmpl, shortlog=False):

    query = ''
    if 'node' in req.form:
        ctx = webutil.changectx(web.repo, req)
    elif 'rev' in req.form:
        query = req.form['rev'][0]
        try:
            ctx = web.repo[query]
        except (error.RepoError, error.LookupError):
            return _search(web, req, tmpl) # XXX redirect to 404 page?
    else:
        ctx = web.repo['tip']

    def changelist(latestonly, **map):
        revs = []
        if pos != -1:
            revs = web.repo.changelog.revs(pos, 0)