Page MenuHomePhabricator

Fix Diffusion crash
ClosedPublic

Authored by hach-que on Apr 30 2014, 2:05 AM.
Tags
None
Referenced Files
F19353489: D8906.id.diff
Fri, Dec 26, 5:28 AM
F19310060: D8906.id21141.diff
Wed, Dec 24, 9:54 PM
F19018216: D8906.id.diff
Nov 23 2025, 10:22 AM
F18841546: D8906.id21140.diff
Oct 28 2025, 9:51 AM
F18841545: D8906.id21141.diff
Oct 28 2025, 9:51 AM
F18829291: D8906.diff
Oct 24 2025, 8:59 PM
F18822552: D8906.id21141.diff
Oct 23 2025, 7:24 AM
F18816896: D8906.id.diff
Oct 21 2025, 10:21 AM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rP466af3314790: Fix Diffusion crash
Summary

This fixes a crash that happens when visiting Diffusion pages due to an undefined variable. $title is only defined if it has a status to show, but then it uses it anyway and fails.

Test Plan

Pages stopped crashing and people stopped complaining.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

hach-que retitled this revision from to Fix Diffusion crash.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Apr 30 2014, 2:06 AM
epriestley updated this revision to Diff 21141.
epriestley added a subscriber: Unknown Object (User).

Closed by commit rP466af3314790 (authored by @jamesr, committed by @epriestley).