Handles local aliases (redirection) for file uploads attached to notices.

1 files changed, 10 insertions, 5 deletions

diff --git a/actions/newnotice.php b/actions/newnotice.php
index d7507c118..aebdab3cc 100644
--- a/actions/newnotice.php
+++ b/actions/newnotice.php
@@ -110,7 +110,7 @@ class NewnoticeAction extends Action
     }
 
     function isFileAttached() {
-        return $_FILES['attach']['error'] === UPLOAD_ERR_OK;
+        return isset($_FILES['attach']['error']) && ($_FILES['attach']['error'] === UPLOAD_ERR_OK);
     }
 
     /**
@@ -205,14 +205,19 @@ class NewnoticeAction extends Action
         $destination = "file/{$notice->id}-$filename";
         if (move_uploaded_file($_FILES['attach']['tmp_name'], INSTALLDIR . "/$destination")) {
             $file = new File;
-//            $file->url = common_local_url('file', array('notice' => $notice->id));
-            $file->url = common_path($destination);
+            $file->url = common_local_url('file', array('notice' => $notice->id));
+//            $file->url = common_path($destination);
             $file->size = filesize(INSTALLDIR . "/$destination");
             $file->date = time();
             $file->mimetype = $_FILES['attach']['type'];
-            if ($ok = $file->insert()) {
+            if ($file_id = $file->insert()) {
+                $file_redir = new File_redirection;
+                $file_redir->url = common_path($destination);
+                $file_redir->file_id = $file_id;
+                $file_redir->insert();
+
                 $f2p = new File_to_post;
-                $f2p->file_id = $ok; 
+                $f2p->file_id = $file_id; 
                 $f2p->post_id = $notice->id; 
                 $f2p->insert();
             } else {