Paths

Table of Contentst

Check for strlen instead of null for Maniphest title
ClosedPublic
Actions

Authored by chad on May 23 2017, 4:24 PM.

Tags

None

Referenced Files

	F19467905: D18002.diff
	Sat, Jan 3, 7:18 PM

	F19216251: D18002.id43293.diff
	Wed, Dec 17, 7:10 AM

	F19078245: D18002.diff
	Dec 1 2025, 5:36 PM

	F18856563: D18002.id43291.diff
	Nov 1 2025, 10:20 AM

	F18853924: D18002.id.diff
	Oct 31 2025, 8:38 PM

	F18850518: D18002.diff
	Oct 30 2025, 5:35 PM

	F18746157: D18002.diff
	Oct 3 2025, 11:35 AM

	F18733609: D18002.id.diff
	Sep 30 2025, 10:38 PM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley

Maniphest Tasks

T12744: Mail tag for newly created tasks is "[Maniphest] [Retitled]"

Commits

rP93d8b33ccade: Check for strlen instead of null for Maniphest title

Summary

Fixes T12744. Unclear why null doesn't work here but does for the title, but !strlen seems to work fine in both cases.

Test Plan

Create a new task, check mail folder, see [Created]

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad created this revision.May 23 2017, 4:24 PM

Herald added a subscriber: Korvin. · View Herald TranscriptMay 23 2017, 4:24 PM

Harbormaster completed remote builds in B17204: Diff 43291.May 23 2017, 4:25 PM

epriestley accepted this revision.May 23 2017, 4:58 PM

This revision is now accepted and ready to land.May 23 2017, 4:58 PM

Closed by commit rP93d8b33ccade: Check for strlen instead of null for Maniphest title (authored by chad, committed by chad). · Explain WhyMay 23 2017, 4:59 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

maniphest/

xaction/

ManiphestTaskTitleTransaction.php

7 lines

Diff 43293

View Options

src/applications/maniphest/xaction/ManiphestTaskTitleTransaction.php

Check for strlen instead of null for Maniphest titleClosedPublicActions