@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
Limit number of affected files in Differential e-mails
Summary: This triggers mainly for SVN branch copy.
Test Plan: Ran this code separately.
Reviewers: epriestley
Reviewed By: epriestley
CC: aran, Korvin
Differential Revision: https://secure.phabricator.com/D5126
+8
-1
+8
-1
src/applications/differential/mail/DifferentialReviewRequestMail.php
+8
-1
src/applications/differential/mail/DifferentialReviewRequestMail.php
···
2
2
3
3
abstract class DifferentialReviewRequestMail extends DifferentialMail {
4
4
5
+
const MAX_AFFECTED_FILES = 1000;
6
+
5
7
protected $comments;
6
8
7
9
private $patch;
···
58
60
$changesets = $this->getChangesets();
59
61
if ($changesets) {
60
62
$body[] = 'AFFECTED FILES';
61
-
foreach ($changesets as $changeset) {
63
+
$max = self::MAX_AFFECTED_FILES;
64
+
foreach (array_values($changesets) as $i => $changeset) {
65
+
if ($i == $max) {
66
+
$body[] = ' ('.(count($changesets) - $max).' more files)';
67
+
break;
68
+
}
62
69
$body[] = ' '.$changeset->getFilename();
63
70
}
64
71
$body[] = null;