Homec4science

Set default content to "" (empty string), not null, in PhrictionDocumentEditor

Authored by epriestley <git@epriestley.com> on Sep 12 2011, 20:11.

Description

Set default content to "" (empty string), not null, in PhrictionDocumentEditor

Summary: phriction.edit allows you to omit the content string, meaning "don't
edit content". If you're also creating the page, we currently break in a
terrible, horrible, no-good, very-bad way because 'content' can't be null.
Default to empty string instead so phriction.edit creates an empty page instead
of a broken mess.

Test Plan: Called phriction.edit on a new page with no content.

Reviewers: skrul, jungejason, nh, tuomaspelkonen, aran

Reviewed By: skrul

CC: aran, skrul

Differential Revision: 920

Details

Committed
epriestley <git@epriestley.com>Sep 13 2011, 17:48
Pushed
aubortJan 31 2017, 17:16
Parents
rPHbd778b4c8e08: Allow Diffusion to display PDF files
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH0366936d4c64: Set default content to "" (empty string), not null, in PhrictionDocumentEditor (authored by epriestley <git@epriestley.com>).Sep 13 2011, 17:48