Paths

Table of Contentst

Test 0 and "" cases in Project Icon Config
ClosedPublic
Actions

Authored by chad on Jul 10 2017, 6:41 PM.

Tags

None

Referenced Files

	F19909054: D18197.diff
	Wed, Apr 1, 11:02 PM

	F18854384: D18197.diff
	Oct 31 2025, 11:51 PM

	F18808164: D18197.id43773.diff
	Oct 19 2025, 5:10 AM

	F18782580: D18197.diff
	Oct 13 2025, 2:02 AM

	F18112626: D18197.id43773.diff
	Aug 12 2025, 7:11 PM

	F18105922: D18197.id43774.diff
	Aug 10 2025, 8:35 PM

	F18089808: D18197.id43773.diff
	Aug 6 2025, 12:44 PM

	F17847830: D18197.id.diff
	Jul 27 2025, 6:35 AM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley

Commits

rPaf71c990eee1: Test 0 and "" cases in Project Icon Config

Summary

Better validation for setting a default image in project.icon

Test Plan

Test adding "0" and "" as image options in project.icon, see error.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad created this revision.Jul 10 2017, 6:41 PM

Herald added a subscriber: Korvin. · View Herald TranscriptJul 10 2017, 6:41 PM

Harbormaster completed remote builds in B17663: Diff 43773.Jul 10 2017, 6:42 PM

(The , null parameter has no effect for any input.)

This revision is now accepted and ready to land.Jul 10 2017, 6:57 PM

Closed by commit rPaf71c990eee1: Test 0 and "" cases in Project Icon Config (authored by chad, committed by chad). · Explain WhyJul 10 2017, 7:01 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

project/

icon/

PhabricatorProjectIconSet.php

3 lines

Diff 43774

View Options

src/applications/project/icon/PhabricatorProjectIconSet.php

Test 0 and "" cases in Project Icon ConfigClosedPublicActions