#31: Fix

resources/i18n/en-GB.edn

@@ -75,6 +75,8 @@
  :email-prompt          "Email address" ;; text of the email widget prompt on edit user page
  :file-or-directory     "File or directory"
                                         ;; used in sanity check report
+ :file-summary-prompt   "Description/what's changed"
+                                        ;; prompt for the file upload summary input
  :file-upload-link-text "You may link to this file using a link of the form"
                                         ;; Text introducing the link to an uploaded file
  :file-upload-prompt    "File to upload" ;; prompt string for the file upload widget

resources/public/content/stylesheet.css

@@ -209,6 +209,10 @@ th {
 	display: block;
 }
 
+#uploaded-image {
+  float: right;
+}
+
 .change {
   background-color: rgb( 223, 223, 223);
   border: thin solid silver;

resources/templates/upload.html

@@ -3,12 +3,12 @@
         <div id="content" class="auth">
         {% if uploaded %}
           {% if is-image %}
-            <img alt="Uploaded image" src="uploads/{{uploaded}}"/>
-
             <p>
+              <img id="uploaded-image" alt="Uploaded image" src="content/uploads/{{uploaded}}"/>
+
               {% i18n file-upload-link-text %}:
 
-              <code>![Uploaded image](uploads/{{uploaded}})</code>
+              <code>![Uploaded image](content/uploads/{{uploaded}})</code>
             </p>
           {% else %}
             <p>
@@ -24,6 +24,10 @@
                     <label for="upload">{% i18n file-upload-prompt %}</label>
                     <input name="upload" id="upload" type="file" required/>
                 </p>
+                <p class="widget">
+                    <label for="summary">{% i18n file-summary-prompt %}</label>
+                    <input name="summary" id="summary" type="text" required/>
+                </p>
                 <p class="widget">
                     <label for="submit">{% i18n save-prompt %}</label>
                     <input name="submit" id="submit" type="submit" class="action" value="{% i18n save-label %}"/>

src/smeagol/routes/wiki.clj

@@ -156,9 +156,17 @@
   "Render a form to allow the upload of a file."
   [request]
   (let [params (keywordize-keys (:params request))
-        data-path (str (io/resource-path) "/uploads/")
+        data-path (str (io/resource-path) "/content/uploads/")
+        git-repo (get-git-repo)
         upload (:upload params)
-        uploaded (if upload (ul/store-upload params))]
+        uploaded (if upload (ul/store-upload params data-path))
+        user (session/get :user)
+        summary (format "%s: %s" user (or (:summary params) "no summary"))]
+    (if
+      uploaded
+      (do
+        (git/git-add git-repo uploaded)
+        (git/git-commit git-repo summary {:name user :email (auth/get-email user)})))
     (layout/render "upload.html"
                    (merge (util/standard-params request)
                           {:title (util/get-message :file-upload-title request)

src/smeagol/uploads.clj

@@ -29,19 +29,20 @@
 ;;;;
 ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
 
-(defn avoid-name-collisions
+;; No longer used as uploaded files now go into Git.
-  "Find a filename within this `path`, based on this `file-name`, that does not
+;; (defn avoid-name-collisions
-   reference an existing file. It is assumed that `path` ends with a path separator.
+;;   "Find a filename within this `path`, based on this `file-name`, that does not
-   Returns a filename hwich does not currently reference a file within the path."
+;;    reference an existing file. It is assumed that `path` ends with a path separator.
-  [path file-name]
+;;    Returns a filename hwich does not currently reference a file within the path."
-  (if (.exists (File. (str path file-name)))
+;;   [path file-name]
-    (let [parts (cs/split file-name #"\.")
+;;   (if (.exists (File. (str path file-name)))
-          prefix (cs/join "." (butlast parts))
+;;     (let [parts (cs/split file-name #"\.")
-          suffix (last parts)]
+;;           prefix (cs/join "." (butlast parts))
-      (first
+;;           suffix (last parts)]
-       (filter #(not (.exists (File. (str path %))))
+;;       (first
-               (map #(str prefix "." % "." suffix) (range)))))
+;;        (filter #(not (.exists (File. (str path %))))
-    file-name))
+;;                (map #(str prefix "." % "." suffix) (range)))))
+;;     file-name))
 
 
 (defn store-upload
@@ -49,11 +50,10 @@
   The issue with storing an upload is moving it into place.
   If `params` are passed as a map, it is expected that this is a map from
   an HTTP POST operation of a form with type `multipart/form-data`."
-  [params]
+  [params path]
   (let [upload (:upload params)
         tmp-file (:tempfile upload)
-        path (str (io/resource-path) "uploads/")
+        filename (:filename upload)]
-        filename (avoid-name-collisions path (:filename upload))]
     (timbre/info
       (str "Storing upload file: " upload))
       (if tmp-file