archive/phabricator
1
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

Allow GitCommitChangeParser to parse 'T' (type change) messages in

Browse Source 
git history.
This commit is contained in:
epriestley
2011-04-14 14:16:53 -07:00
parent 6e713ad784
commit bc1be958bd
1 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/applications/repository/worker/commitchangeparser/git/PhabricatorRepositoryGitCommitChangeParserWorker.php
View File

@@ -101,6 +101,10 @@ class PhabricatorRepositoryGitCommitChangeParserWorker
$change_target = $src_path; $change_target = $src_path;
$move_away[$change_target][] = $change_path; $move_away[$change_target][] = $change_path;
break; break;
case 'T':
// Type of the file changed, fall through and treat it as a
// modification. Not 100% sure this is the right thing to do but it
// seems reasonable.
case 'M': case 'M':
if ($file_type == DifferentialChangeType::FILE_DIRECTORY) { if ($file_type == DifferentialChangeType::FILE_DIRECTORY) {
$change_type = DifferentialChangeType::TYPE_CHILD; $change_type = DifferentialChangeType::TYPE_CHILD;
@@ -109,6 +113,8 @@ class PhabricatorRepositoryGitCommitChangeParserWorker
$change_type = DifferentialChangeType::TYPE_CHANGE; $change_type = DifferentialChangeType::TYPE_CHANGE;
} }
break; break;
// NOTE: "U" (unmerged) and "X" (unknown) statuses are also possible
// in theory but shouldn't appear here.
default: default:
throw new Exception("Failed to parse line '{$line}'."); throw new Exception("Failed to parse line '{$line}'.");
} }