Page MenuHomePhabricator

Prevent "Spaces" field from being set to inconsistent values
ClosedPublic

Authored by epriestley on Dec 17 2015, 7:02 PM.
Tags
None
Referenced Files
F18846457: D14812.id35809.diff
Wed, Oct 29, 6:59 PM
F18812713: D14812.id35808.diff
Mon, Oct 20, 11:46 AM
F18767640: D14812.id35809.diff
Oct 7 2025, 11:33 PM
F18767628: D14812.id35808.diff
Oct 7 2025, 11:30 PM
F18766754: D14812.diff
Oct 7 2025, 6:21 PM
F18764495: D14812.id.diff
Oct 7 2025, 6:22 AM
F18712982: D14812.id.diff
Sep 29 2025, 5:35 AM
F18671277: D14812.diff
Sep 25 2025, 3:19 AM
Subscribers
None

Details

Summary

At least for now, the "Space" field is just a subfield of the "Visible To" field, so:

  • it doesn't get any separate settings; and
  • it always uses the "Visible To" settings.
Test Plan
  • Created a form with a hidden view policy field.
  • Created stuff with no "you must pick a space" errors.
  • Created stuff with a normal form.
  • Prefilled "Space" on a noraml form.
  • Verified that trying to prefill "Space" on a form with "Visible To" hidden does nothing.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable