@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
re-use files if you forget to name a mock
Summary: good title
Test Plan: made a mock with no title and the files i had attached showed up again correctly. named the mock and it worked!
Reviewers: epriestley, chad
Reviewed By: chad
CC: aran, Korvin
Maniphest Tasks: T2642
Differential Revision: https://secure.phabricator.com/D5261
+2
-1
+2
-1
src/applications/pholio/controller/PholioMockEditController.php
+2
-1
src/applications/pholio/controller/PholioMockEditController.php
···
53
53
$v_view = $mock->getViewPolicy();
54
54
$v_cc = PhabricatorSubscribersQuery::loadSubscribersForPHID(
55
55
$mock->getPHID());
56
+
$files = array();
56
57
57
58
if ($request->isFormPost()) {
58
59
$xactions = array();
···
81
82
if ($is_new) {
82
83
// TODO: Make this transactional and allow edits?
83
84
84
-
$files = array();
85
85
86
86
$file_phids = $request->getArr('file_phids');
87
87
if ($file_phids) {
···
193
193
->setUser($user))
194
194
->appendChild(
195
195
id(new AphrontFormDragAndDropUploadControl($request))
196
+
->setValue($files)
196
197
->setName('file_phids')
197
198
->setLabel(pht('Images'))
198
199
->setError($e_images))