Page MenuHomePhorge

Include number of modified files in total of attached files
Needs ReviewPublic

Authored by aklapper on Feb 21 2025, 12:43.
Tags
None
Referenced Files
F3749191: D25892.1745791839.diff
Sat, Apr 26, 22:10
F3733444: D25892.1745745769.diff
Sat, Apr 26, 09:22
F3712495: D25892.1745684078.diff
Fri, Apr 25, 16:14
F3388310: D25892.1744446171.diff
Fri, Apr 11, 08:22
F3388307: D25892.1744446157.diff
Fri, Apr 11, 08:22
F3388306: D25892.1744446152.diff
Fri, Apr 11, 08:22
F3378187: D25892.1744351103.diff
Thu, Apr 10, 05:58
F3364696: D25892.1744159513.diff
Tue, Apr 8, 00:45

Details

Summary

The second parameter in the message %s updated %s attached file(s), added %s: %s; removed %s: %s; modified %s: %s. did not take the number of modified files into account but only added and removed files.

Closes T15959

Test Plan

Read the code.
Additionally, update and remove and modify files attached to a task and count.

Diff Detail

Repository
rP Phorge
Branch
T15959 (branched from master)
Lint
Lint Passed
Unit
Test Failures
Build Status
Buildable 1738
Build 1738: arc lint + arc unit

Unit TestsFailed

TimeTest
768 msPhabricatorCelerityTestCase::testCelerityMaps
Assertion failed, expected 'true' (at PhabricatorCelerityTestCase.php:27): When this test fails, it means the Celerity resource map is out of date. Run `bin/celerity map` to rebuild it. ACTUAL VALUE
833 msPhabricatorConduitTestCase::testConduitMethods
1 assertion passed.
379 msPhabricatorInfrastructureTestCase::testApplicationsInstalled
1 assertion passed.
385 msPhabricatorInfrastructureTestCase::testRejectMySQLNonUTF8Queries
1 assertion passed.
2,583 msPhabricatorLibraryTestCase::testEverythingImplemented
1 assertion passed.
View Full Test Results (1 Failed · 6 Passed)