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

	F18464999: D11756.id.diff
	Tue, Sep 2, 6:08 AM

	F18454968: D11756.diff
	Mon, Sep 1, 6:16 AM

	F18192675: D11756.diff
	Sun, Aug 17, 4:29 AM

	F18191861: D11756.id.diff
	Sun, Aug 17, 3:17 AM

	F18050293: D11756.id28699.diff
	Aug 3 2025, 7:18 PM

	F18048397: D11756.id.diff
	Aug 3 2025, 1:33 PM

	F18023278: D11756.id28340.diff
	Aug 2 2025, 8:53 PM

	F17959381: D11756.diff
	Aug 1 2025, 8:11 AM

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...