Differential D25901

ManiphestReportController: Add more PHPDoc coverage
ClosedPublic
Actions

Authored by aklapper on Mon, Mar 3, 14:45.

Tags

None

Referenced Files

	F3251668: D25901.1742365376.diff
	Tue, Mar 18, 06:22

	F3249193: D25901.1742284284.diff
	Mon, Mar 17, 07:51

	F3236514: D25901.1742175172.diff
	Sun, Mar 16, 01:32

	F3225906: D25901.1742145240.diff
	Sat, Mar 15, 17:14

	F3225543: D25901.1742126368.diff
	Sat, Mar 15, 11:59

	F3224682: D25901.1742077095.diff
	Fri, Mar 14, 22:18

	F3224494: D25901.1742047790.diff
	Fri, Mar 14, 14:09

	F3223660: D25901.1741999394.diff
	Fri, Mar 14, 00:43

View All 25 Files

Subscribers

valerio.bozzolan

Details

Reviewers

valerio.bozzolan

Group Reviewers

O1: Blessed Committers

Commits

rP0f4ab2413d8b: ManiphestReportController: Add more PHPDoc coverage

Summary

Make things a bit easier to understand, for the start.

Test Plan

Check return values for their types and classes.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

aklapper created this revision.Mon, Mar 3, 14:45

Owners added a reviewer: O1: Blessed Committers.Mon, Mar 3, 14:45

Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan, tobiaswiese. · View Herald TranscriptMon, Mar 3, 14:45

aklapper requested review of this revision.Mon, Mar 3, 14:45

Harbormaster completed remote builds in B1760: Diff 2716.Mon, Mar 3, 14:45

Thanks

sgtm

src/applications/maniphest/controller/ManiphestReportController.php
519	I don't like hardcoding it also in PHPDoc lol Maybe better to say that "greater means big priority" (or vice-versa - I don't know lol)

This revision is now accepted and ready to land.Mon, Mar 3, 14:49

aklapper marked an inline comment as done.Mon, Mar 3, 14:51

aklapper added inline comments.

src/applications/maniphest/controller/ManiphestReportController.php
519	The docs cover the current behavior. ;) I'll leave understanding Priority integers as an exercise to admins and developers if anyone ever wants to dig into that.

Closed by commit rP0f4ab2413d8b: ManiphestReportController: Add more PHPDoc coverage. · Explain WhyMon, Mar 3, 14:53

This revision was automatically updated to reflect the committed changes.

aklapper marked an inline comment as done.

aklapper added a commit: rP0f4ab2413d8b: ManiphestReportController: Add more PHPDoc coverage.

valerio.bozzolan added inline comments.Mon, Mar 3, 14:53

src/applications/maniphest/controller/ManiphestReportController.php
519	This is a nightmare for my dreams but I have already approved so... asd But better to explain what it is than to explain its current value

Revision Contents
Changeset List

Path

Size

Packages

src/

applications/

maniphest/

controller/

ManiphestReportController.php

42 lines

O1: Blessed Committers

Diff 2717

src/applications/maniphest/controller/ManiphestReportController.php

Loading...