@epriestley, do you see it can be easy enough so I can to submit some patch ?

My digging is that :

• DiffusionBrowseDirectoryController use DiffusionBrowseTableView
• DiffusionBrowseTableView does the think right (see here)
• DiffusionCommitController does not use DiffusionBrowseTableView but DiffusionHistoryTableView
• the build of table of content is done in DiffusionCommitController::buildTableOfContents() and in peculiar here, it does not take into account the case if the $file_type == DifferentialChangeType::FILE_SUBMODULE

Or for evrything else than DifferentialChangeType::FILE_SUBMODULE the browse link must be :

$browse_link = phutil_tag('strong', array(), $this->linkBrowse(
  $full_path.$dir_slash,
  array(
    'type' => $file_type,
    'name' => $browse_text,
  )));

but for DifferentialChangeType::FILE_SUBMODULE, it should be somethink like :

$browse_link = phutil_tag('strong', array(), $this->linkBrowse(
  null,
  array(
    'type' => $file_type,
    'name' => $browse_text,
    'hash' => $path->getHash(),
    'external' => $path->getExternalURI(),
  )));

Let me know if you agree to let me submit a simple patch with a switch case.
If you want some bigger refont like moving building table of content to a new view or more, I surely can help, but will need more help.

Of course I would double check it this time, and really try the last diff send in the production environnement where i get the problem, to be sure it don't break the world like last time.