From c2d2f6ded13797a8b7f055ae295517578fdc6349 Mon Sep 17 00:00:00 2001 From: Sigurd Holsen Date: Mon, 21 May 2012 11:41:34 +0200 Subject: [PATCH] bugfix: mercurial commit worker had an array error It tried to get $path['targetPath'] istead of $change['targetPath'] when $path was a string --- ...habricatorRepositoryMercurialCommitChangeParserWorker.php | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/applications/repository/worker/commitchangeparser/mercurial/PhabricatorRepositoryMercurialCommitChangeParserWorker.php b/src/applications/repository/worker/commitchangeparser/mercurial/PhabricatorRepositoryMercurialCommitChangeParserWorker.php index baaa02b559..3b365b88e3 100644 --- a/src/applications/repository/worker/commitchangeparser/mercurial/PhabricatorRepositoryMercurialCommitChangeParserWorker.php +++ b/src/applications/repository/worker/commitchangeparser/mercurial/PhabricatorRepositoryMercurialCommitChangeParserWorker.php @@ -103,8 +103,9 @@ final class PhabricatorRepositoryMercurialCommitChangeParserWorker $away = array(); foreach ($changes as $path => $change) { - if ($path['targetPath']) { - $away[$path['targetPath']][] = $path; + $target_path = $change['targetPath']; + if ($target_path) { + $away[$target_path][] = $path; } }