revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2025-04-02 15:38:18 +02:00
Code Issues Releases Wiki Activity

Make macro audio errors more clear

Browse source 
Summary: Ref T12685. I bamboozled myself.

Test Plan: {F4945786}

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12685

Differential Revision: https://secure.phabricator.com/D17847
This commit is contained in:
epriestley 2017-05-08 13:29:55 -07:00
parent 32af1d9a9f
commit ff205964cf
1 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/applications/macro/controller/PhabricatorMacroAudioController.php
View file

@ -50,7 +50,9 @@ final class PhabricatorMacroAudioController extends PhabricatorMacroController {
if ($file) { if ($file) {
if (!$file->isAudio()) { if (!$file->isAudio()) {
$errors[] = pht('You must upload audio.'); $errors[] = pht(
'The file you uploaded is invalid: it is not recognizable as '.
'a valid audio file.');
$e_file = pht('Invalid'); $e_file = pht('Invalid');
} else { } else {
$xactions[] = id(new PhabricatorMacroTransaction()) $xactions[] = id(new PhabricatorMacroTransaction())
@ -59,7 +61,9 @@ final class PhabricatorMacroAudioController extends PhabricatorMacroController {
->setNewValue($file->getPHID()); ->setNewValue($file->getPHID());
} }
} else { } else {
$errors[] = pht('You must upload an audio file.'); $errors[] = pht(
'To change the audio for a macro, you must upload an audio '.
'file.');
$e_file = pht('Required'); $e_file = pht('Required');
} }
} }