If you are browsing within a certain repository, the id of that repository is missing in the breadcrumb links, resulting in incorrect links. I propose the following fix:

  1. Change the function _subversion_breadcrumb($path) into _subversion_breadcrumb($path, $repo)
  2. Luckily the variable $repo is already defined in all calls to _subversion_breadcrumb, so we can change all references _subversion_breadcrumb($path); into _subversion_breadcrumb($path, $repo);. This should be done in functions subversion_page_browse, subversion_page_file and subversion_page_log.
  3. Now that the variable $repo is known in _subversion_breadcrumb, we can easily insert the repository id, by changing $breadcrumb[] = l($title, 'subversion/browse'. rtrim($p, '/')); into $breadcrumb[] = l($title, 'subversion/browse/'. $repo->rid. rtrim($p, '/'));

Comments

halkeye’s picture

Status: Needs review » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.