Paths

Table of Contentst

Differential D11278

Explicitly declare method/property visibility
ClosedPublic
Actions

Authored by joshuaspence on Jan 8 2015, 8:55 AM.

Tags

None

Referenced Files

	F16682218: D11278.diff
	Tue, Jun 17, 2:44 PM

	F16644687: D11278.diff
	Tue, Jun 17, 10:58 AM

	Unknown Object (File)
	Sun, Jun 15, 9:25 PM

	Unknown Object (File)
	Apr 23 2025, 2:43 PM

	Unknown Object (File)
	Apr 21 2025, 1:57 PM

	Unknown Object (File)
	Mar 16 2025, 9:31 AM

	Unknown Object (File)
	Mar 15 2025, 10:52 PM

	Unknown Object (File)
	Mar 9 2025, 1:05 PM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Commits

Restricted Diffusion Commit
rP698b7f9ea39c: Explicitly declare method/property visibility

Summary

Self-explanatory.

Test Plan

arc lint

Diff Detail

Repository

Branch

master

Lint

Lint Passed

Unit

Test Failures

Build Status

Buildable 3683
Build 3693: [Placeholder Plan] Wait for 30 Seconds

Unit TestsFailed
View All

	Time	Test
	86 ms	AphrontIsolatedDatabaseConnectionTestCase::testTransactionRollback
	63 ms	AphrontMySQLDatabaseConnectionTestCase::testConnectionFailures
	60 ms	AphrontIsolatedDatabaseConnectionTestCase::testDeletePermitted
	79 ms	AphrontIsolatedDatabaseConnectionTestCase::testInsertGeneratesID
	70 ms	AphrontIsolatedDatabaseConnectionTestCase::testIsolation
		View Full Test Results (2 Failed · 7 Passed)

Event Timeline

joshuaspence updated this revision to Diff 27096.Jan 8 2015, 8:55 AM

joshuaspence retitled this revision from to Explicitly declare method/property visibility.

joshuaspence updated this object.

joshuaspence edited the test plan for this revision. (Show Details)

joshuaspence added a reviewer: epriestley.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJan 8 2015, 8:55 AM

Herald added subscribers: epriestley, Korvin, hach-que. · View Herald Transcript

Test issues clearly unrelated.

This revision is now accepted and ready to land.Jan 8 2015, 4:39 PM

Closed by commit rP698b7f9ea39c: Explicitly declare method/property visibility (authored by joshuaspence, committed by joshuaspence). · Explain WhyJan 11 2015, 9:21 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

landing/

DifferentialLandingStrategy.php

2 lines

drydock/

controller/

DrydockController.php

2 lines

owners/

storage/

__tests__/

PhabricatorOwnersPackageTestCase.php

2 lines

phragment/

controller/

PhragmentController.php

4 lines

PhragmentRevertController.php

2 lines

PhragmentSnapshotDeleteController.php

2 lines

PhragmentSnapshotPromoteController.php

2 lines

infrastructure/

log/

PhabricatorAccessLog.php

2 lines

PhabricatorSSHLog.php

2 lines

storage/

patch/

PhabricatorSQLPatchList.php

4 lines

Diff 27096

src/applications/differential/landing/DifferentialLandingStrategy.php

Loading...

src/applications/drydock/controller/DrydockController.php

Loading...

src/applications/owners/storage/tests/PhabricatorOwnersPackageTestCase.php

Loading...

src/applications/phragment/controller/PhragmentController.php

Loading...

src/applications/phragment/controller/PhragmentRevertController.php

Loading...

src/applications/phragment/controller/PhragmentSnapshotDeleteController.php

Loading...

src/applications/phragment/controller/PhragmentSnapshotPromoteController.php

Loading...

src/infrastructure/log/PhabricatorAccessLog.php

Loading...

src/infrastructure/log/PhabricatorSSHLog.php

Loading...

src/infrastructure/storage/patch/PhabricatorSQLPatchList.php

Loading...