From ed1e1d9f440acfbb99c7a6e3d2b5c11fdcfa27ce Mon Sep 17 00:00:00 2001
From: Robin Millette <millette@controlyourself.ca>
Date: Wed, 27 May 2009 03:47:45 -0400
Subject: [PATCH] Handles local aliases (redirection) for file uploads attached
 to notices.

---
 .gitignore                   |  6 ++++++
 actions/file.php             | 40 ++++++++++++++++++++++++++++++++++++
 actions/newnotice.php        | 15 +++++++++-----
 classes/File_redirection.php |  2 +-
 lib/router.php               |  2 --
 lib/util.php                 |  5 +++++
 6 files changed, 62 insertions(+), 8 deletions(-)
 create mode 100644 actions/file.php

diff --git a/.gitignore b/.gitignore
index da6947bfdc..3418d8ee54 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 avatar/*
 files/*
+file/*
 _darcs/*
 logs/*
 config.php
@@ -16,3 +17,8 @@ dataobject.ini
 .buildpath
 .project
 .settings
+TODO.rym
+config-*.php
+good-config.php
+lac08.log
+php.log
diff --git a/actions/file.php b/actions/file.php
new file mode 100644
index 0000000000..1179dbe9ac
--- /dev/null
+++ b/actions/file.php
@@ -0,0 +1,40 @@
+<?php
+/*
+ * Laconica - a distributed open-source microblogging tool
+ * Copyright (C) 2008, Controlez-Vous, Inc.
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+if (!defined('LACONICA')) { exit(1); }
+
+require_once(INSTALLDIR.'/actions/shownotice.php');
+
+class FileAction extends ShowNoticeAction
+{
+    function showPage() {
+        $source_url = common_local_url('file', array('notice' => $this->notice->id));
+        $query = "select file_redirection.url as url from file join file_redirection on file.id = file_redirection.file_id where file.url = '$source_url'";
+        $file = new File_redirection;
+        $file->query($query);
+        $file->fetch();
+        if (empty($file->url)) {
+            die('nothing attached here');
+        } else {
+            header("Location: {$file->url}");
+            die();
+        }
+    }
+}
+
diff --git a/actions/newnotice.php b/actions/newnotice.php
index d7507c1182..aebdab3cca 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 {
diff --git a/classes/File_redirection.php b/classes/File_redirection.php
index 0eae681783..212cc36158 100644
--- a/classes/File_redirection.php
+++ b/classes/File_redirection.php
@@ -133,7 +133,7 @@ class File_redirection extends Memcached_DataObject
         $file->limit(1);
         $file->orderBy('len');
         $file->find(true);
-        if (!empty($file->id)) {
+        if (!empty($file->url) && (strlen($file->url) < strlen($long_url))) {
             return $file->url;
         }
 
diff --git a/lib/router.php b/lib/router.php
index bd482eafab..456d1793e3 100644
--- a/lib/router.php
+++ b/lib/router.php
@@ -164,11 +164,9 @@ class Router
                     array('action' => 'newnotice'),
                     array('replyto' => '[A-Za-z0-9_-]+'));
 
-/*
         $m->connect('notice/:notice/file', 
             array('action' => 'file'), 
             array('notice' => '[0-9]+'));
-*/
         
         $m->connect('notice/:notice',
                     array('action' => 'shownotice'),
diff --git a/lib/util.php b/lib/util.php
index d56f44f7b4..ab12723098 100644
--- a/lib/util.php
+++ b/lib/util.php
@@ -499,6 +499,11 @@ function common_linkify($url) {
 
 // if this URL is an attachment, then we set class='attachment' and id='attahcment-ID'
 // where ID is the id of the attachment for the given URL.
+//
+// we need a better test telling what can be shown as an attachment
+// we're currently picking up oembeds only.
+// I think the best option is another file_view table in the db
+// and associated dbobject.
     $query = "select file_oembed.file_id as file_id from file join file_oembed on file.id = file_oembed.file_id where file.url='$longurl'";
     $file = new File;
     $file->query($query);