Paths

Table of Contentst

Differential D20176

Fix a URI construction in remarkup macro/meme rules
ClosedPublic
Actions

Authored by epriestley on Feb 15 2019, 12:56 PM.

Tags

None

Referenced Files

	F19795256: D20176.diff
	Thu, Feb 26, 11:05 PM

	F19507925: D20176.diff
	Jan 9 2026, 11:58 PM

	F19470144: D20176.diff
	Jan 5 2026, 10:11 AM

	F19383861: D20176.id48171.diff
	Dec 27 2025, 1:27 AM

	F19236660: D20176.id48171.diff
	Dec 21 2025, 7:23 PM

	F19082914: D20176.id48181.diff
	Dec 2 2025, 12:27 PM

	F19001933: D20176.id.diff
	Nov 21 2025, 4:18 AM

	F18989200: D20176.diff
	Nov 18 2025, 11:19 AM

View All Files

Subscribers

None

Details

Reviewers

amckinley

Maniphest Tasks

T13250: Some typeaheads use nonscalar HTTP parameters, which fatal under new "phutil_build_http_query()" rules

Commits

rP66060b294bce: Fix a URI construction in remarkup macro/meme rules

Summary

Ref T13250. Some of these parameters may be NULL, and alter() is no longer happy about that.

Test Plan

Ran daemon tasks that happened to render some memes.

Diff Detail

Repository

rP Phabricator

Branch

meme1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 22016
Build 30074: Run Core Tests
Build 30073: arc lint + arc unit

Event Timeline

epriestley created this revision.Feb 15 2019, 12:56 PM

Harbormaster completed remote builds in B22016: Diff 48171.Feb 15 2019, 12:58 PM

epriestley requested review of this revision.Feb 15 2019, 12:58 PM

amckinley accepted this revision.Feb 15 2019, 4:37 PM

This revision is now accepted and ready to land.Feb 15 2019, 4:37 PM

Closed by commit rP66060b294bce: Fix a URI construction in remarkup macro/meme rules (authored by epriestley). · Explain WhyFeb 15 2019, 10:08 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

macro/

engine/

PhabricatorMemeEngine.php

11 lines

Diff 48171

View Options

src/applications/macro/engine/PhabricatorMemeEngine.php

Fix a URI construction in remarkup macro/meme rulesClosedPublicActions