Diffusion Phabricator 2bc045bab8d7

Fix another stray "msort()/msortv()" issue
2bc045bab8d7
Actions

Description

Fix another stray "msort()/msortv()" issue

Summary: Ref T13303. See B22967. This should be "msortv()" but didn't get updated properly.

Test Plan: The system works!

Reviewers: amckinley

Reviewed By: amckinley

Maniphest Tasks: T13303

Differential Revision: https://secure.phabricator.com/D20585

Details

Provenance

epriestley	Authored on Jun 17 2019, 8:10 PM
epriestley	Pushed on Jun 17 2019, 8:20 PM

Reviewer

amckinley

Differential Revision

D20585: Fix another stray "msort()/msortv()" issue

Parents

rP874282db7589: Correct "msort()" vs "msortv()" to more fully stabilize transaction sorts after…

Branches

Unknown

Tags

Unknown

Tasks

T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes

Build Status

Buildable 22969
Build 31524: Run Core Tests

Event Timeline

epriestley committed rP2bc045bab8d7: Fix another stray "msort()/msortv()" issue (authored by epriestley).Jun 17 2019, 8:20 PM

epriestley added a task: T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes.

epriestley added an edge: D20585: Fix another stray "msort()/msortv()" issue.

Harbormaster completed building B22969: rP2bc045bab8d7: Fix another stray "msort()/msortv()" issue.Jun 17 2019, 8:21 PM

Changes (1)

Path

Size

src/

applications/

differential/

command/

DifferentialActionEmailCommand.php

rP2bc045bab8d7

View Options

src/applications/differential/command/DifferentialActionEmailCommand.php