Merge pull request #781 from anchor/per-page-uploads

Implement the ability to store uploads on a per-page basis

lib/gollum/app.rb

@@ -163,7 +163,7 @@ module Precious
         tempfile = params[:file][:tempfile]
       end
 
-      dir = 'uploads'
+      dir = wiki.per_page_uploads ? params[:upload_dest] : 'uploads'
       ext = ::File.extname(fullname)
       format = ext.split('.').last || 'txt'
       filename = ::File.basename(fullname, ext)
@@ -173,7 +173,7 @@ module Precious
       head = wiki.repo.head
 
       options = {
-        :message => "Uploaded file to uploads/#{reponame}",
+        :message => "Uploaded file to #{dir}/#{reponame}",
         :parent => wiki.repo.head.commit,
       }
       author = session['gollum.author']
@@ -447,6 +447,10 @@ module Precious
         @page = page
         @name = name
         @content  = page.formatted_data
+        @upload_dest = settings.wiki_options[:allow_uploads] ?
+                         (settings.wiki_options[:per_page_uploads] ?
+                            @name : 'uploads'
+                         ) : ''
 
         # Extensions and layout data
         @editable = true

lib/gollum/public/gollum/javascript/gollum.dialog.js

@@ -99,6 +99,8 @@
 
       html += '<form method=post enctype="multipart/form-data" ' +
         'action="' + action + '" ' + 'id="' + id + '">';
+      html += '<input type="hidden" name="upload_dest" value="' +
+        uploadDest + '">';
       html += '<input type=file name="' + name + '">';
       html += '</form>';
 

lib/gollum/public/gollum/javascript/gollum.js

@@ -155,7 +155,7 @@ $(document).ready(function() {
         fields: [
           {
             type:   'file',
-            context: 'Your uploaded file will be accessible at /uploads/[filename]',
+            context: 'Your uploaded file will be accessible at /'+uploadDest+'/[filename]',
             action: baseUrl + '/uploadFile'
           }
         ],

lib/gollum/templates/layout.mustache

@@ -17,6 +17,7 @@
       var baseUrl = '{{base_url}}';
       {{#page}}
       var pageFullPath = '{{url_path_display}}';
+      var uploadDest   = '{{upload_dest}}';
       {{/page}}
   </script>
   <script type="text/javascript" src="{{base_url}}/javascript/jquery-1.7.2.min.js"></script>

lib/gollum/views/page.rb

@@ -46,6 +46,10 @@ module Precious
       def allow_uploads
         @allow_uploads
       end
+      
+      def upload_dest
+        @upload_dest
+      end
 
       def has_header
         @header = (@page.header || false) if @header.nil?