1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-10 08:52:39 +01:00

Fix a hidden input ID issue mucking with project edits

Summary:
In D1926 this got converted to use the new InsetView, but we lost the 'id="resources"' on the hidden input which is required by the JS.

Just use phutil_render_tag() to make sure the `id` shows up.

Test Plan: Edited a project without bumping into an exception.

Reviewers: hsb, btrahan

Reviewed By: hsb

CC: aran, epriestley

Differential Revision: https://secure.phabricator.com/D1957
This commit is contained in:
epriestley 2012-03-19 16:06:52 -07:00
parent 3d3e63e4fc
commit a03efc1030
2 changed files with 9 additions and 1 deletions

View file

@ -269,7 +269,14 @@ final class PhabricatorProjectProfileEditController
'mustcapture' => true,
),
'Add New Resource'))
->addHiddenInput('resources', 'resources')
->appendChild(
phutil_render_tag(
'input',
array(
'type' => 'hidden',
'name' => 'resources',
'id' => 'resources',
)))
->setContent(javelin_render_tag(
'table',
array(

View file

@ -34,6 +34,7 @@ phutil_require_module('phabricator', 'view/form/error');
phutil_require_module('phabricator', 'view/form/inset');
phutil_require_module('phabricator', 'view/layout/panel');
phutil_require_module('phutil', 'markup');
phutil_require_module('phutil', 'utils');