revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-10 08:52:39 +01:00
Code Issues Releases Wiki Activity

Fix an issue with creating new Conpherences

Browse source 
Summary: The participant list can sometimes be `null`, which fails when we try to `array_fuse()` it.

Test Plan: Created a new thread cleanly.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Differential Revision: https://secure.phabricator.com/D11969
This commit is contained in:
epriestley 2015-03-04 14:36:13 -08:00
parent 174dd220df
commit 803a050824
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/applications/conpherence/editor/ConpherenceEditor.php
View file

@ -490,7 +490,7 @@ final class ConpherenceEditor extends PhabricatorApplicationTransactionEditor {
foreach ($xactions as $xaction) { foreach ($xactions as $xaction) {
$phids = $this->getPHIDTransactionNewValue( $phids = $this->getPHIDTransactionNewValue(
$xaction, $xaction,
$object->getParticipantPHIDs()); nonempty($object->getParticipantPHIDs(), array()));
if (!$phids) { if (!$phids) {
continue; continue;

2
src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php
View file

@ -1413,7 +1413,7 @@ abstract class PhabricatorApplicationTransactionEditor
PhabricatorApplicationTransaction $xaction, PhabricatorApplicationTransaction $xaction,
$old = null) { $old = null) {
if ($old) { if ($old !== null) {
$old = array_fuse($old); $old = array_fuse($old);
} else { } else {
$old = array_fuse($xaction->getOldValue()); $old = array_fuse($xaction->getOldValue());