revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2025-01-10 23:01:04 +01:00
Code Issues Releases Wiki Activity

Fix weird "0" notification numbers

Browse source 
Summary: The JsShrink change swapped an "== 0" for "=== 0" in JS, but we're currently sending a string down. Fixes T3230.

Test Plan: I'm not 100% sure this actually fixes it but probably? Let me know if you still see it.

Reviewers: chad, btrahan

Reviewed By: chad

CC: aran

Maniphest Tasks: T3230

Differential Revision: https://secure.phabricator.com/D5997
This commit is contained in:
epriestley 2013-05-21 15:44:44 -07:00
parent 49ef13e876
commit f8d8156ccc
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/applications/notification/controller/PhabricatorNotificationPanelController.php
View file

@ -43,7 +43,7 @@ final class PhabricatorNotificationPanelController
$json = array(
'content' => $content,
'number' => $unread_count,
'number' => (int)$unread_count,
);
return id(new AphrontAjaxResponse())->setContent($json);