From a7cbcb3495ee66d2d266778be644783a221cafd9 Mon Sep 17 00:00:00 2001
From: Bob Trahan <bob.trahan@gmail.com>
Date: Wed, 6 Mar 2013 13:40:50 -0800
Subject: [PATCH] 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
---
 .../pholio/controller/PholioMockEditController.php             | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/applications/pholio/controller/PholioMockEditController.php b/src/applications/pholio/controller/PholioMockEditController.php
index 00e35c4364..6e63841d97 100644
--- a/src/applications/pholio/controller/PholioMockEditController.php
+++ b/src/applications/pholio/controller/PholioMockEditController.php
@@ -53,6 +53,7 @@ final class PholioMockEditController extends PholioController {
     $v_view = $mock->getViewPolicy();
     $v_cc = PhabricatorSubscribersQuery::loadSubscribersForPHID(
       $mock->getPHID());
+    $files = array();
 
     if ($request->isFormPost()) {
       $xactions = array();
@@ -81,7 +82,6 @@ final class PholioMockEditController extends PholioController {
       if ($is_new) {
         // TODO: Make this transactional and allow edits?
 
-        $files = array();
 
         $file_phids = $request->getArr('file_phids');
         if ($file_phids) {
@@ -193,6 +193,7 @@ final class PholioMockEditController extends PholioController {
           ->setUser($user))
       ->appendChild(
         id(new AphrontFormDragAndDropUploadControl($request))
+          ->setValue($files)
           ->setName('file_phids')
           ->setLabel(pht('Images'))
           ->setError($e_images))