HomePhorge

Fix several error handling issues with Subversion commits in Diffusion

Description

Fix several error handling issues with Subversion commits in Diffusion

Summary:
Ref T9513. I checked this briefly but didn't do a very thorough job of it.

  • Don't try to query merges for Subversion, since it doesn't support them.
  • Fix up "existsquery" to work properly (and efficiently) for both hosted and imported repositories.
  • Fix up "parentsquery" to have similar behavior on invalid commits to other VCSes (throw an exception).

Test Plan:

  • No more merges warning on SVN.
  • Hosted SVN gets the right exists result now.
  • Visiting "r23980283789287" now 404's instead of "not parsed yet".

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T9513

Differential Revision: https://secure.phabricator.com/D14239

Details

Provenance
epriestleyAuthored on Oct 5 2015, 3:57 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rPde2bbfef7d14: Allow PhabricatorWorker->queueTask() to take full $options
Branches
Unknown
Tags
Unknown

Event Timeline