revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-23 05:50:55 +01:00
Code Issues Releases Wiki Activity

Don't store empty copy:lines

Browse source 
Summary: The [[ https://secure.phabricator.com/diffusion/P/browse/master/src/applications/differential/parser/DifferentialChangesetParser.php;8d0918885da2c22b$1364 | callsite ]] is fine with that.

Test Plan: This diff.

Reviewers: btrahan, epriestley

Reviewed By: epriestley

CC: aran, Korvin

Differential Revision: https://secure.phabricator.com/D3537
This commit is contained in:
vrana 2012-09-21 11:35:24 -07:00
parent 48fa3eb3b3
commit 49f75d2554
1 changed files with 6 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
src/applications/differential/storage/DifferentialDiff.php
View file

@ -243,9 +243,12 @@ final class DifferentialDiff extends DifferentialDAO {
}
}
}
$metadata = $changeset->getMetadata();
$metadata['copy:lines'] = array_filter($copies);
$changeset->setMetadata($metadata);
$copies = array_filter($copies);
if ($copies) {
$metadata = $changeset->getMetadata();
$metadata['copy:lines'] = $copies;
$changeset->setMetadata($metadata);
}
}
}