See https://discourse.phabricator-community.org/t/conduit-method-phriction-edit-requires-title-while-the-docs-say-its-optional/2176. Make code consistent with documentation by not requiring either content or title.
Details
Details
- Reviewers
epriestley - Commits
- rP00a7071e2d1d: Fix handling of Phriction conduit edits
Hit the method via the UI and no longer got an error on missing content or title fields.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- master
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 21279 Build 28950: Run Core Tests Build 28949: arc lint + arc unit
Event Timeline
Comment Actions
These should probably be strlen(...) tests -- as written, setting the title or content to 0 likely does not work.