Support for deleting files

Closes #85

Author: csmith

git.go

@@ -351,7 +351,19 @@ func (g *GitBackend) DeletePage(name string, message string, user string) error
 	g.mutex.Lock()
 	defer g.mutex.Unlock()
 
-	_, gitPath, err := resolvePath(g.dir, fmt.Sprintf("%s.md", name))
+	return g.delete(fmt.Sprintf("%s.md", name), message, user)
+}
+
+
+func (g *GitBackend) DeleteFile(name string, message string, user string) error {
+	g.mutex.Lock()
+	defer g.mutex.Unlock()
+
+	return g.delete(name, message, user)
+}
+
+func (g *GitBackend) delete(name, message, user string) error {
+	_, gitPath, err := resolvePath(g.dir, name)
 	if err != nil {
 		return err
 	}

handlers_file.go

@@ -1,6 +1,7 @@
 package main
 
 import (
+	"fmt"
 	"io"
 	"log"
 	"mime"
@@ -105,3 +106,37 @@ func FileHandler(provider FileProvider) http.HandlerFunc {
 		_, _ = io.Copy(writer, reader)
 	}
 }
+
+type DeleteFileProvider interface {
+	DeleteFile(name string, message string, user string) error
+}
+
+func DeleteFileConfirmHandler(t *Templates) http.HandlerFunc {
+	return func(w http.ResponseWriter, r *http.Request) {
+		name := strings.TrimPrefix(r.URL.Path, "/files/delete/")
+		t.RenderDeleteFile(w, r, name)
+	}
+}
+
+func DeleteFileHandler(provider DeleteFileProvider) http.HandlerFunc {
+	return func(writer http.ResponseWriter, request *http.Request) {
+		name := strings.TrimPrefix(request.URL.Path, "/files/delete/")
+		confirm := request.FormValue("confirm")
+		if confirm == "" {
+			http.Redirect(writer, request, "/files/delete/"+name, http.StatusTemporaryRedirect)
+			return
+		}
+		message := request.FormValue("message")
+		username := "Anonymoose"
+		if user := getUserForRequest(request); user != nil {
+			username = user.Name
+		}
+		err := provider.DeleteFile(name, message, username)
+		if err != nil {
+			writer.WriteHeader(http.StatusInternalServerError)
+			return
+		}
+		putSessionKey(writer, request, sessionNoticeKey, fmt.Sprintf("Deleted file %s", name))
+		http.Redirect(writer, request, "/", http.StatusTemporaryRedirect)
+	}
+}

handlers_page.go

@@ -111,7 +111,8 @@ func DeletePageHandler(provider DeletePageProvider) http.HandlerFunc {
 			writer.WriteHeader(http.StatusInternalServerError)
 			return
 		}
-		http.Redirect(writer, request, "/", http.StatusOK)
+		putSessionKey(writer, request, sessionNoticeKey, fmt.Sprintf("Deleted page %s", name))
+		http.Redirect(writer, request, "/", http.StatusTemporaryRedirect)
 	}
 }
 

main.go

@@ -128,6 +128,8 @@ func main() {
 	wikiRouter.PathPrefix("/view/").Handler(read(ViewPageHandler(templates, renderer, gitBackend))).Methods(http.MethodGet)
 	wikiRouter.PathPrefix("/history/").Handler(read(PageHistoryHandler(templates, gitBackend))).Methods(http.MethodGet)
 	wikiRouter.PathPrefix("/files/view/").Handler(read(FileHandler(gitBackend))).Methods(http.MethodGet)
+	wikiRouter.PathPrefix("/files/delete/").Handler(write(DeleteFileConfirmHandler(templates))).Methods(http.MethodGet)
+	wikiRouter.PathPrefix("/files/delete/").Handler(write(DeleteFileHandler(gitBackend))).Methods(http.MethodPost)
 	wikiRouter.PathPrefix("/delete/").Handler(write(DeletePageConfirmHandler(templates))).Methods(http.MethodGet)
 	wikiRouter.PathPrefix("/delete/").Handler(write(DeletePageHandler(gitBackend))).Methods(http.MethodPost)
 	wikiRouter.PathPrefix("/rename/").Handler(write(RenamePageConfirmHandler(gitBackend, templates))).Methods(http.MethodGet)

resources/templates/delete_file.gohtml

@@ -0,0 +1,12 @@
+{{- /*gotype: github.com/mdbot/wiki.DeleteFileArgs*/ -}}
+{{template "header" .Common}}
+<form action="/files/delete/{{.Common.PageTitle}}" method="post" class="editor">
+    {{.Common.CsrfField}}
+    <input type="hidden" id="confirm" name="confirm" value="confirm" />
+    <div class="form-group">
+        <label for="message">Reason:</label>
+        <input id="message" type="text" name="message">
+    </div>
+    <button type="submit" class="btn btn-primary" value="Edit">Confirm delete</button>
+</form>
+{{template "footer" .Common}}

resources/templates/listfiles.gohtml

@@ -3,7 +3,7 @@
 All files:
 <ul>
     {{range .Files}}
-        <li><a href="/files/view/{{.Name}}">{{.Name}}</a> ({{.Size | bytes}})</li>
+        <li><a href="/files/view/{{.Name}}">{{.Name}}</a> ({{.Size | bytes}}) [<a href="/files/delete/{{.Name}}">delete</a>]</li>
     {{end}}
 </ul>
 {{template "footer" .Common}}

templates.go

@@ -131,6 +131,18 @@ func (t *Templates) RenderFileList(w http.ResponseWriter, r *http.Request, files
 	})
 }
 
+type DeleteFileArgs struct {
+	Common CommonArgs
+}
+
+func (t *Templates) RenderDeleteFile(w http.ResponseWriter, r *http.Request, fileName string) {
+	t.render("delete_file.gohtml", http.StatusOK, w, &DeleteFileArgs{
+		Common: t.populateArgs(w, r, CommonArgs{
+			PageTitle: fileName,
+		}),
+	})
+}
+
 type UploadFileArgs struct {
 	Common CommonArgs
 }