@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
Fix a loopy comment
Summary: I wrote this earlier in D14680 but have now realized that it's the same sentence twice when read carefully.
Test Plan: read more carefully
Reviewers: chad
Reviewed By: chad
Differential Revision: https://secure.phabricator.com/D14687
+3
-1
+3
-1
src/applications/transactions/view/PhabricatorApplicationTransactionView.php
+3
-1
src/applications/transactions/view/PhabricatorApplicationTransactionView.php
···
386
386
$results = array();
387
387
388
388
// Sort transactions within the group by action strength, then by
389
-
// within actions of similar strength.
389
+
// chronological order. This makes sure that multiple actions of the
390
+
// same type (like a close, then a reopen) render in the order they
391
+
// were performed.
390
392
$strength_groups = mgroup($group, 'getActionStrength');
391
393
krsort($strength_groups);
392
394
foreach ($strength_groups as $strength_group) {