Merge branch 'devel'

cmd/calls/articles.go

@@ -10,6 +10,27 @@ import (
 	b "streifling.com/jason/cpolis/cmd/backend"
 )
 
+func incrementClicks(db *b.DB, a *b.Article) error {
+	a.Clicks++
+	if err := db.UpdateAttributes(&b.Attribute{Table: "articles", ID: a.ID, AttName: "clicks", Value: a.Clicks}); err != nil {
+		return fmt.Errorf("error updating click attribute of article %v: %v", a.ID, err)
+	}
+
+	if a.IsInIssue {
+		issue, err := db.GetArticle(a.IssueID)
+		if err != nil {
+			return fmt.Errorf("error getting issue %v: %v", a.IssueID, err)
+		}
+
+		issue.Clicks++
+		if err := db.UpdateAttributes(&b.Attribute{Table: "articles", ID: issue.ID, AttName: "clicks", Value: issue.Clicks}); err != nil {
+			return fmt.Errorf("error updating click attribute of issue %v: %v", issue.ID, err)
+		}
+	}
+
+	return nil
+}
+
 func ServeArticle(c *b.Config, db *b.DB) http.HandlerFunc {
 	return func(w http.ResponseWriter, r *http.Request) {
 		if !tokenIsVerified(w, r, c) {
@@ -50,6 +71,41 @@ func ServeArticle(c *b.Config, db *b.DB) http.HandlerFunc {
 			return
 		}
 
-		fmt.Fprint(w, content)
+		if err = incrementClicks(db, article); err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+
+		if _, err = fmt.Fprint(w, content); err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+	}
+}
+
+func ServeClicks(db *b.DB) http.HandlerFunc {
+	return func(w http.ResponseWriter, r *http.Request) {
+		idString := r.PathValue("id")
+		id, err := strconv.ParseInt(idString, 10, 64)
+		if err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+
+		article, err := db.GetArticle(id)
+		if err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+
+		if _, err = fmt.Fprint(w, article.Clicks); err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
 	}
 }

cmd/frontend/articles.go

@@ -121,6 +121,7 @@ func SubmitArticle(c *b.Config, db *b.DB, s *b.CookieStore) http.HandlerFunc {
 			Rejected:      false,
 			IsInIssue:     r.PostFormValue("issue") == "on",
 			AutoGenerated: false,
+			EditedID:      0,
 		}
 
 		if len(article.Title) == 0 {

cmd/main.go

@@ -65,6 +65,7 @@ func main() {
 	mux.HandleFunc("GET /article/review-rejected/{id}", f.ReviewRejectedArticle(config, db, store))
 	mux.HandleFunc("GET /article/review-unpublished/{id}", f.ReviewArticle(config, db, store, "publish", "Artikel veröffentlichen", "Veröffentlichen"))
 	mux.HandleFunc("GET /article/serve/{id}", c.ServeArticle(config, db))
+	mux.HandleFunc("GET /article/serve/{id}/clicks", c.ServeClicks(db))
 	mux.HandleFunc("GET /article/write", f.WriteArticle(config, db, store))
 	mux.HandleFunc("GET /atom/serve", c.ServeAtomFeed(config))
 	mux.HandleFunc("GET /hub", f.ShowHub(config, db, store))

create_db.sql

@@ -34,7 +34,8 @@ CREATE TABLE articles (
     rejected BOOL NOT NULL,
     creator_id INT NOT NULL,
     issue_id INT NOT NULL,
-    edited_id INT,
+    edited_id INT NOT NULL,
+    clicks INT NOT NULL,
     is_in_issue BOOL NOT NULL,
     auto_generated BOOL NOT NULL,
     PRIMARY KEY (id),
@@ -49,24 +50,24 @@ CREATE TABLE tags (
 );
 
 CREATE TABLE articles_authors (
-    article_id INT,
-    author_id INT,
+    article_id INT NOT NULL,
+    author_id INT NOT NULL,
     PRIMARY KEY (article_id, author_id),
     FOREIGN KEY (article_id) REFERENCES articles (id),
     FOREIGN KEY (author_id) REFERENCES users (id)
 );
 
 CREATE TABLE articles_contributors (
-    article_id INT,
-    contributor_id INT,
+    article_id INT NOT NULL,
+    contributor_id INT NOT NULL,
     PRIMARY KEY (article_id, contributor_id),
     FOREIGN KEY (article_id) REFERENCES articles (id),
     FOREIGN KEY (contributor_id) REFERENCES users (id)
 );
 
 CREATE TABLE articles_tags (
-    article_id INT,
-    tag_id INT,
+    article_id INT NOT NULL,
+    tag_id INT NOT NULL,
     PRIMARY KEY (article_id, tag_id),
     FOREIGN KEY (article_id) REFERENCES articles (id),
     FOREIGN KEY (tag_id) REFERENCES tags (id)