Page MenuHomePhorge

Do not hardcode default Priority names in Project Reports tooltip
Needs ReviewPublic

Authored by aklapper on Fri, Apr 26, 10:33.
Tags
None
Referenced Files
Unknown Object (File)
Sun, May 5, 23:31
Unknown Object (File)
Sun, May 5, 23:31
Unknown Object (File)
Sun, May 5, 01:58
Unknown Object (File)
Thu, May 2, 17:02
Unknown Object (File)
Wed, May 1, 22:43
Unknown Object (File)
Tue, Apr 30, 06:33
Unknown Object (File)
Tue, Apr 30, 04:26
Unknown Object (File)
Mon, Apr 29, 22:52

Details

Summary

Pull the names of Priority field values instead of hardcoding them.

Closes T15799

Test Plan
  • As an admin, go to /config/edit/maniphest.priorities/ and change the value of "name" of a Priority value with a value < 50 (e.g.: Low, Wishlist)
  • Open /maniphest/report/project/
  • Hover over the Oldest (Pri) column and check the tooltip text before and after applying this patch

Diff Detail

Repository
rP Phorge
Branch
T15799repPrioTooltip (branched from master)
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/maniphest/controller/ManiphestReportController.php:706XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 1197
Build 1197: arc lint + arc unit