Paths

Table of Contentst

Differential D11756

Add abstract `getGroup()` method
ClosedPublic
Actions

Authored by joshuaspence on Feb 12 2015, 7:33 AM.

Tags

None

Referenced Files

	F15440369: D11756.id28340.diff
	Wed, Mar 26, 12:27 PM

	F15437226: D11756.id28699.diff
	Tue, Mar 25, 6:26 PM

	F15429920: D11756.id28340.diff
	Mon, Mar 24, 4:53 AM

	F15426108: D11756.diff
	Sun, Mar 23, 7:13 AM

	F15413566: D11756.diff
	Wed, Mar 19, 7:42 PM

	F15392952: D11756.diff
	Mar 15 2025, 6:37 PM

	F15362353: D11756.diff
	Mar 11 2025, 12:48 PM

	F15285816: D11756.id28340.diff
	Mar 4 2025, 4:11 PM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Commits

Restricted Diffusion Commit
rPeec2d145df03: Add abstract `getGroup()` method

Summary

In D11722, a getGroup() method was added to all subclasses of PhabricatorApplicationConfigOptions, but no abstract method was added to the base class. This will fail if a custom *ConfigOptions class does not provide a getGroup() method, in which case $group->getGroup() (in PhabricatorConfigListController) will fatal.

Test Plan

N/A

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

joshuaspence updated this revision to Diff 28340.Feb 12 2015, 7:33 AM

joshuaspence retitled this revision from to Add abstract `getGroup()` method.

joshuaspence updated this object.

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

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptFeb 12 2015, 7:33 AM

Herald added a subscriber: epriestley. · View Herald Transcript

WIP?

This revision is now accepted and ready to land.Feb 28 2015, 9:55 PM

joshuaspence updated this object.Feb 28 2015, 10:44 PM

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

joshuaspence edited edge metadata.

joshuaspence updated this object.Feb 28 2015, 10:46 PM

Closed by commit rPeec2d145df03: Add abstract `getGroup()` method (authored by joshuaspence, committed by joshuaspence). · Explain WhyFeb 28 2015, 10:47 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

config/

option/

PhabricatorApplicationConfigOptions.php

1 line

Diff 28699

src/applications/config/option/PhabricatorApplicationConfigOptions.php

Loading...