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

Removing reordering code that wasn't needed

Browse source 
Summary:
Remove reordering code for package array as it's ordered in the first place

Test Plan:
Called API through conduit console, still works

Reviewed By: epriestley
Reviewers: epriestley, tuomaspelkonen
CC: ju, dpepper, epriestley
Blame Revision:
126

Differential Revision: 141
This commit is contained in:
leon 2011-04-14 16:17:39 -07:00
parent bff6aef87a
commit dcb2e43960
1 changed files with 2 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
src/applications/owners/storage/package/PhabricatorOwnersPackage.php
View file

@ -135,15 +135,9 @@ class PhabricatorOwnersPackage extends PhabricatorOwnersDAO {
$packages = $package->loadAllWhere('id in (%Ld)', array_keys($order));
$result = array();
// Reorder packages according to specificity.
foreach ($packages as $package) {
$result[$package->getID()] = $package;
}
$packages = array_select_keys($packages, array_keys($order));
$result = array_select_keys($result, array_keys($order));
return $result;
return $packages;
}
public function save() {