Add support for multiple authors and contributors

cmd/backend/articles.go

@@ -5,6 +5,7 @@ import (
 	"database/sql"
 	"fmt"
 	"log"
+	"os"
 	"time"
 )
 
@@ -14,7 +15,7 @@ type Article struct {
 	BannerLink    string
 	Summary       string
 	ID            int64
-	AuthorID      int64
+	CreatorID     int64
 	IssueID       int64
 	EditedID      int64
 	Published     bool
@@ -29,7 +30,7 @@ func (db *DB) AddArticle(a *Article) (int64, error) {
 	selectQuery := "SELECT id FROM issues WHERE published = false"
 	insertQuery := `
     INSERT INTO articles
-        (title, banner_link, summary, published, rejected, author_id, issue_id, edited_id, is_in_issue, auto_generated)
+        (title, banner_link, summary, published, rejected, creator_id, issue_id, edited_id, is_in_issue, auto_generated)
     VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
     `
 
@@ -47,7 +48,7 @@ func (db *DB) AddArticle(a *Article) (int64, error) {
 				return 0, fmt.Errorf("error getting issue ID when adding article to DB: %v", err)
 			}
 
-			result, err := tx.Exec(insertQuery, a.Title, a.BannerLink, a.Summary, a.Published, a.Rejected, a.AuthorID, id, a.EditedID, a.IsInIssue, a.AutoGenerated)
+			result, err := tx.Exec(insertQuery, a.Title, a.BannerLink, a.Summary, a.Published, a.Rejected, a.CreatorID, id, a.EditedID, a.IsInIssue, a.AutoGenerated)
 			if err != nil {
 				if rollbackErr := tx.Rollback(); rollbackErr != nil {
 					log.Fatalf("transaction error: %v, rollback error: %v", err, rollbackErr)
@@ -81,7 +82,7 @@ func (db *DB) AddArticle(a *Article) (int64, error) {
 
 func (db *DB) GetArticle(id int64) (*Article, error) {
 	query := `
-    SELECT title, created, banner_link, summary, published, author_id, issue_id, edited_id, is_in_issue, auto_generated
+    SELECT title, created, banner_link, summary, published, creator_id, issue_id, edited_id, is_in_issue, auto_generated
     FROM articles
     WHERE id = ?
     `
@@ -91,7 +92,7 @@ func (db *DB) GetArticle(id int64) (*Article, error) {
 	var created []byte
 	var err error
 
-	if err := row.Scan(&article.Title, &created, &article.BannerLink, &article.Summary, &article.Published, &article.AuthorID, &article.IssueID, &article.EditedID, &article.IsInIssue, &article.AutoGenerated); err != nil {
+	if err := row.Scan(&article.Title, &created, &article.BannerLink, &article.Summary, &article.Published, &article.CreatorID, &article.IssueID, &article.EditedID, &article.IsInIssue, &article.AutoGenerated); err != nil {
 		return nil, fmt.Errorf("error scanning article row: %v", err)
 	}
 
@@ -106,7 +107,7 @@ func (db *DB) GetArticle(id int64) (*Article, error) {
 
 func (db *DB) GetCertainArticles(attribute string, value bool) ([]*Article, error) {
 	query := fmt.Sprintf(`
-    SELECT id, title, created, banner_link, summary, author_id, issue_id, published, rejected, is_in_issue, auto_generated
+    SELECT id, title, created, banner_link, summary, creator_id, issue_id, published, rejected, is_in_issue, auto_generated
     FROM articles
     WHERE %s = ?
     `, attribute)
@@ -120,7 +121,7 @@ func (db *DB) GetCertainArticles(attribute string, value bool) ([]*Article, erro
 		article := new(Article)
 		var created []byte
 
-		if err = rows.Scan(&article.ID, &article.Title, &created, &article.BannerLink, &article.Summary, &article.AuthorID, &article.IssueID, &article.Published, &article.Rejected, &article.IsInIssue, &article.AutoGenerated); err != nil {
+		if err = rows.Scan(&article.ID, &article.Title, &created, &article.BannerLink, &article.Summary, &article.CreatorID, &article.IssueID, &article.Published, &article.Rejected, &article.IsInIssue, &article.AutoGenerated); err != nil {
 			return nil, fmt.Errorf("error scanning article row: %v", err)
 		}
 
@@ -140,7 +141,7 @@ func (db *DB) GetCurrentIssueArticles() ([]*Article, error) {
 	txOptions := &sql.TxOptions{Isolation: sql.LevelSerializable}
 	issueQuery := "SELECT id FROM issues WHERE published = false"
 	articlesQuery := `
-    SELECT id, title, created, banner_link, summary, author_id, auto_generated
+    SELECT id, title, created, banner_link, summary, auto_generated
     FROM articles
     WHERE issue_id = ? AND published = true AND is_in_issue = true
     `
@@ -173,7 +174,7 @@ func (db *DB) GetCurrentIssueArticles() ([]*Article, error) {
 				article := new(Article)
 				var created []byte
 
-				if err = rows.Scan(&article.ID, &article.Title, &created, &article.BannerLink, &article.Summary, &article.AuthorID, &article.AutoGenerated); err != nil {
+				if err = rows.Scan(&article.ID, &article.Title, &created, &article.BannerLink, &article.Summary, &article.AutoGenerated); err != nil {
 					if rollbackErr := tx.Rollback(); rollbackErr != nil {
 						log.Fatalf("transaction error: %v, rollback error: %v", err, rollbackErr)
 					}
@@ -269,3 +270,13 @@ func (db *DB) DeleteArticle(id int64) error {
 
 	return nil
 }
+
+func WriteArticleToFile(c *Config, articleID int64, content []byte) error {
+	articleAbsName := fmt.Sprint(c.ArticleDir, "/", articleID, ".md")
+
+	if err := os.WriteFile(articleAbsName, content, 0644); err != nil {
+		return fmt.Errorf("error writing article %v to file: %v", articleID, err)
+	}
+
+	return nil
+}

cmd/backend/articles_authors.go

@@ -0,0 +1,125 @@
+package backend
+
+import (
+	"fmt"
+	"log"
+)
+
+func (db *DB) WriteArticleAuthors(articleID int64, authorIDs []int64) error {
+	query := "INSERT INTO articles_authors (article_id, author_id) VALUES (?, ?)"
+
+	for i := 0; i < TxMaxRetries; i++ {
+		err := func() error {
+			tx, err := db.Begin()
+			if err != nil {
+				return fmt.Errorf("error starting transaction: %v", err)
+			}
+
+			for _, authorID := range authorIDs {
+				if _, err := tx.Exec(query, articleID, authorID); err != nil {
+					if rollbackErr := tx.Rollback(); rollbackErr != nil {
+						log.Fatalf("transaction error: %v, rollback error: %v", err, rollbackErr)
+					}
+					return fmt.Errorf("error inserting into articles_authors: %v", err)
+				}
+			}
+
+			if err = tx.Commit(); err != nil {
+				return fmt.Errorf("error committing transaction: %v", err)
+			}
+			return nil
+		}()
+		if err == nil {
+			return nil
+		}
+
+		log.Println(err)
+		wait(i)
+	}
+	return fmt.Errorf("error: %v unsuccessful retries for DB operation, aborting", TxMaxRetries)
+}
+
+func (db *DB) GetArticleAuthors(c *Config, articleID int64) ([]*User, error) {
+	query := `
+    SELECT u.id
+    FROM articles a
+        INNER JOIN articles_authors aa ON a.id = aa.article_id
+        INNER JOIN users u ON aa.author_id = u.id
+    WHERE a.id = ?
+    `
+	rows, err := db.Query(query, articleID)
+	if err != nil {
+		return nil, fmt.Errorf("error querying articles_authors: %v", err)
+	}
+
+	authors := make([]*User, 0)
+	for rows.Next() {
+		var authorID int64
+
+		if err = rows.Scan(&authorID); err != nil {
+			return nil, fmt.Errorf("error scanning rows: %v", err)
+		}
+
+		author, err := db.GetUser(c, authorID)
+		if err != nil {
+			return nil, fmt.Errorf("error getting user info for article author: %v", err)
+		}
+
+		authors = append(authors, author)
+	}
+
+	return authors, nil
+}
+
+func (db *DB) UpdateArticleAuthors(articleID int64, authorIDs []int64) error {
+	deleteQuery := "DELETE FROM articles_authors WHERE article_id = ?"
+	insertQuery := "INSERT INTO articles_authors (article_id, author_id) VALUES (?, ?)"
+
+	for i := 0; i < TxMaxRetries; i++ {
+		err := func() error {
+			tx, err := db.Begin()
+			if err != nil {
+				return fmt.Errorf("error starting transaction: %v", err)
+			}
+
+			if _, err := tx.Exec(deleteQuery, articleID); err != nil {
+				if rollbackErr := tx.Rollback(); rollbackErr != nil {
+					log.Fatalf("transaction error: %v, rollback error: %v", err, rollbackErr)
+				}
+				return fmt.Errorf("error deleting entries from articles_authors before inserting new ones: %v", err)
+			}
+
+			for _, authorID := range authorIDs {
+				if _, err := tx.Exec(insertQuery, articleID, authorID); err != nil {
+					if rollbackErr := tx.Rollback(); rollbackErr != nil {
+						log.Fatalf("transaction error: %v, rollback error: %v", err, rollbackErr)
+					}
+					return fmt.Errorf("error inserting new entries into articles_authors: %v", err)
+				}
+			}
+
+			if err = tx.Commit(); err != nil {
+				return fmt.Errorf("error committing transaction: %v", err)
+			}
+			return nil
+		}()
+		if err == nil {
+			return nil
+		}
+
+		log.Println(err)
+		wait(i)
+	}
+	return fmt.Errorf("error: %v unsuccessful retries for DB operation, aborting", TxMaxRetries)
+}
+
+func (db *DB) DeleteArticleAuthors(articleID int64) error {
+	query := "DELETE FROM articles_authors WHERE article_id = ?"
+
+	_, err := db.Exec(query, articleID)
+	if err != nil {
+		return fmt.Errorf("error deleting articles_authors %v from DB: %v", articleID, err)
+	}
+
+	return nil
+}

cmd/backend/articles_contributors.go

@@ -0,0 +1,125 @@
+package backend
+
+import (
+	"fmt"
+	"log"
+)
+
+func (db *DB) WriteArticleContributors(articleID int64, contributorIDs []int64) error {
+	query := "INSERT INTO articles_contributors (article_id, contributor_id) VALUES (?, ?)"
+
+	for i := 0; i < TxMaxRetries; i++ {
+		err := func() error {
+			tx, err := db.Begin()
+			if err != nil {
+				return fmt.Errorf("error starting transaction: %v", err)
+			}
+
+			for _, contributorID := range contributorIDs {
+				if _, err := tx.Exec(query, articleID, contributorID); err != nil {
+					if rollbackErr := tx.Rollback(); rollbackErr != nil {
+						log.Fatalf("transaction error: %v, rollback error: %v", err, rollbackErr)
+					}
+					return fmt.Errorf("error inserting into articles_contributors: %v", err)
+				}
+			}
+
+			if err = tx.Commit(); err != nil {
+				return fmt.Errorf("error committing transaction: %v", err)
+			}
+			return nil
+		}()
+		if err == nil {
+			return nil
+		}
+
+		log.Println(err)
+		wait(i)
+	}
+	return fmt.Errorf("error: %v unsuccessful retries for DB operation, aborting", TxMaxRetries)
+}
+
+func (db *DB) GetArticleContributors(c *Config, articleID int64) ([]*User, error) {
+	query := `
+    SELECT u.id
+    FROM articles a
+        INNER JOIN articles_contributors ac ON a.id = ac.article_id
+        INNER JOIN users u ON ac.contributor_id = u.id
+    WHERE a.id = ?
+    `
+	rows, err := db.Query(query, articleID)
+	if err != nil {
+		return nil, fmt.Errorf("error querying articles_contributors: %v", err)
+	}
+
+	contributors := make([]*User, 0)
+	for rows.Next() {
+		var contributorID int64
+
+		if err = rows.Scan(&contributorID); err != nil {
+			return nil, fmt.Errorf("error scanning rows: %v", err)
+		}
+
+		contributor, err := db.GetUser(c, contributorID)
+		if err != nil {
+			return nil, fmt.Errorf("error getting user info for article contributor: %v", err)
+		}
+
+		contributors = append(contributors, contributor)
+	}
+
+	return contributors, nil
+}
+
+func (db *DB) UpdateArticleContributors(articleID int64, contributorIDs []int64) error {
+	deleteQuery := "DELETE FROM articles_contributors WHERE article_id = ?"
+	insertQuery := "INSERT INTO articles_contributors (article_id, contributor_id) VALUES (?, ?)"
+
+	for i := 0; i < TxMaxRetries; i++ {
+		err := func() error {
+			tx, err := db.Begin()
+			if err != nil {
+				return fmt.Errorf("error starting transaction: %v", err)
+			}
+
+			if _, err := tx.Exec(deleteQuery, articleID); err != nil {
+				if rollbackErr := tx.Rollback(); rollbackErr != nil {
+					log.Fatalf("transaction error: %v, rollback error: %v", err, rollbackErr)
+				}
+				return fmt.Errorf("error deleting entries from articles_contributors before inserting new ones: %v", err)
+			}
+
+			for _, contributorID := range contributorIDs {
+				if _, err := tx.Exec(insertQuery, articleID, contributorID); err != nil {
+					if rollbackErr := tx.Rollback(); rollbackErr != nil {
+						log.Fatalf("transaction error: %v, rollback error: %v", err, rollbackErr)
+					}
+					return fmt.Errorf("error inserting new entries into articles_contributors: %v", err)
+				}
+			}
+
+			if err = tx.Commit(); err != nil {
+				return fmt.Errorf("error committing transaction: %v", err)
+			}
+			return nil
+		}()
+		if err == nil {
+			return nil
+		}
+
+		log.Println(err)
+		wait(i)
+	}
+	return fmt.Errorf("error: %v unsuccessful retries for DB operation, aborting", TxMaxRetries)
+}
+
+func (db *DB) DeleteArticleContributors(articleID int64) error {
+	query := "DELETE FROM articles_contributors WHERE article_id = ?"
+
+	_, err := db.Exec(query, articleID)
+	if err != nil {
+		return fmt.Errorf("error deleting articles_contributors %v from DB: %v", articleID, err)
+	}
+
+	return nil
+}

cmd/backend/atom.go

@@ -51,12 +51,33 @@ func GenerateAtomFeed(c *Config, db *DB) (*string, error) {
 			entry.Links[linkID].Type = "image/webp"
 		}
 
-		user, err := db.GetUser(c, article.AuthorID)
+		authors, err := db.GetArticleAuthors(c, article.ID)
 		if err != nil {
-			return nil, fmt.Errorf("error getting user user info for Atom feed: %v", err)
+			return nil, fmt.Errorf("error getting article's authors for Atom feed: %v", err)
 		}
+		for _, author := range authors {
+			user, err := db.GetUser(c, author.ID)
+			if err != nil {
+				return nil, fmt.Errorf("error getting user info for Atom feed: %v", err)
+			}
+
 			authorID := entry.AddAuthor(atom.NewPerson(user.FirstName + " " + user.LastName))
 			entry.Authors[authorID].URI = c.Domain + "/image/serve/" + user.ProfilePicLink
+		}
+
+		contributors, err := db.GetArticleContributors(c, article.ID)
+		if err != nil {
+			return nil, fmt.Errorf("error getting article's contributors for Atom feed: %v", err)
+		}
+		for _, contributor := range contributors {
+			user, err := db.GetUser(c, contributor.ID)
+			if err != nil {
+				return nil, fmt.Errorf("error getting user info for Atom feed: %v", err)
+			}
+
+			contributorID := entry.AddContributor(atom.NewPerson(user.FirstName + " " + user.LastName))
+			entry.Contributors[contributorID].URI = c.Domain + "/image/serve/" + user.ProfilePicLink
+		}
 
 		tags, err := db.GetArticleTags(article.ID)
 		if err != nil {

cmd/backend/users.go

@@ -448,7 +448,46 @@ func (db *DB) AddFirstUser(c *Config, u *User, pass string) (int64, error) {
 	return 0, fmt.Errorf("error: %v unsuccessful retries for DB operation, aborting", TxMaxRetries)
 }
 
-func (db *DB) GetAllUsers(c *Config) (map[int64]*User, error) {
+func (db *DB) GetAllUsers(c *Config) ([]*User, error) {
+	var aesFirstName, aesLastName, aesEmail string
+	var err error
+
+	query := "SELECT id, username, first_name, last_name, email, profile_pic_link, role FROM users"
+
+	rows, err := db.Query(query)
+	if err != nil {
+		return nil, fmt.Errorf("error getting all users from DB: %v", err)
+	}
+
+	users := make([]*User, 0)
+	for rows.Next() {
+		user := new(User)
+		if err = rows.Scan(&user.ID, &user.UserName, &aesFirstName, &aesLastName, &aesEmail, &user.ProfilePicLink, &user.Role); err != nil {
+			return nil, fmt.Errorf("error getting user info: %v", err)
+		}
+
+		user.FirstName, err = aesDecrypt(c, aesFirstName)
+		if err != nil {
+			return nil, fmt.Errorf("error decrypting first name: %v", err)
+		}
+
+		user.LastName, err = aesDecrypt(c, aesLastName)
+		if err != nil {
+			return nil, fmt.Errorf("error decrypting last name: %v", err)
+		}
+
+		user.Email, err = aesDecrypt(c, aesEmail)
+		if err != nil {
+			return nil, fmt.Errorf("error decrypting email: %v", err)
+		}
+
+		users = append(users, user)
+	}
+
+	return users, nil
+}
+
+func (db *DB) GetAllUsersMap(c *Config) (map[int64]*User, error) {
 	var aesFirstName, aesLastName, aesEmail string
 	var err error
 

cmd/frontend/articles.go

@@ -7,6 +7,7 @@ import (
 	"net/http"
 	"os"
 	"strconv"
+	"strings"
 	"time"
 
 	b "streifling.com/jason/cpolis/cmd/backend"
@@ -17,6 +18,17 @@ const (
 	PreviewMode
 )
 
+const (
+	None = iota
+	Author
+	Contributor
+)
+
+type ArticleUser struct {
+	*b.User
+	ArticleRole int
+}
+
 type EditorHTMLData struct {
 	Selected     map[int64]bool
 	Content      string
@@ -27,6 +39,10 @@ type EditorHTMLData struct {
 	HTMLContent  template.HTML
 	Article      *b.Article
 	Tags         []*b.Tag
+	ArticleUsers map[string]*ArticleUser // A map is way more efficient in ReviewRejectedArticle()
+	Creator      *ArticleUser
+	Authors      []*b.User
+	Contributors []*b.User
 }
 
 func WriteArticle(c *b.Config, db *b.DB, s *b.CookieStore) http.HandlerFunc {
@@ -40,11 +56,30 @@ func WriteArticle(c *b.Config, db *b.DB, s *b.CookieStore) http.HandlerFunc {
 
 		var data *EditorHTMLData
 		if session.Values["article"] == nil {
-			data = &EditorHTMLData{Action: "submit", Article: new(b.Article)}
+			data = &EditorHTMLData{Action: "submit", Article: new(b.Article), ArticleUsers: make(map[string]*ArticleUser)}
 		} else {
 			data = session.Values["article"].(*EditorHTMLData)
 		}
 
+		users, err := db.GetAllUsers(c)
+		if err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+		for _, user := range users {
+			data.ArticleUsers[fmt.Sprint(user.LastName, user.FirstName, user.ID)] = &ArticleUser{User: user, ArticleRole: None}
+		}
+
+		creator, err := db.GetUser(c, session.Values["id"].(int64))
+		if err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+		data.Creator = data.ArticleUsers[fmt.Sprint(creator.LastName, creator.FirstName, creator.ID)]
+		delete(data.ArticleUsers, fmt.Sprint(creator.LastName, creator.FirstName, creator.ID))
+
 		data.Tags, err = db.GetTagList()
 		if err != nil {
 			log.Println(err)
@@ -81,9 +116,9 @@ func SubmitArticle(c *b.Config, db *b.DB, s *b.CookieStore) http.HandlerFunc {
 			Title:         r.PostFormValue("article-title"),
 			BannerLink:    r.PostFormValue("article-banner-url"),
 			Summary:       r.PostFormValue("article-summary"),
+			CreatorID:     session.Values["id"].(int64),
 			Published:     false,
 			Rejected:      false,
-			AuthorID:      session.Values["id"].(int64),
 			IsInIssue:     r.PostFormValue("issue") == "on",
 			AutoGenerated: false,
 		}
@@ -97,6 +132,38 @@ func SubmitArticle(c *b.Config, db *b.DB, s *b.CookieStore) http.HandlerFunc {
 			return
 		}
 
+		r.ParseForm()
+		authors := make([]int64, 0)
+		contributors := make([]int64, 0)
+
+		for key, values := range r.Form {
+			if strings.HasPrefix(key, "user-") && len(values) > 0 {
+				id, err := strconv.ParseInt(strings.Split(key, "-")[1], 10, 64)
+				if err != nil {
+					log.Println(err)
+					http.Error(w, err.Error(), http.StatusBadRequest)
+					return
+				}
+
+				switch values[0] {
+				case "author":
+					authors = append(authors, id)
+				case "contributor":
+					contributors = append(contributors, id)
+				}
+			}
+		}
+
+		if r.PostFormValue("creator") == "contributor" {
+			contributors = append(contributors, article.CreatorID)
+		} else {
+			authors = append(authors, article.CreatorID)
+		}
+		if len(authors) == 0 {
+			http.Error(w, "Es muss mindestens einen Autor geben.", http.StatusBadRequest)
+			return
+		}
+
 		article.ID, err = db.AddArticle(article)
 		if err != nil {
 			log.Println(err)
@@ -109,23 +176,34 @@ func SubmitArticle(c *b.Config, db *b.DB, s *b.CookieStore) http.HandlerFunc {
 			http.Error(w, "Bitte den Artikel eingeben.", http.StatusBadRequest)
 			return
 		}
-
-		articleAbsName := fmt.Sprint(c.ArticleDir, "/", article.ID, ".md")
-		if err = os.WriteFile(articleAbsName, content, 0644); err != nil {
+		if err := b.WriteArticleToFile(c, article.ID, content); err != nil {
 			log.Println(err)
 			http.Error(w, err.Error(), http.StatusInternalServerError)
 			return
 		}
 
-		r.ParseForm()
+		if err = db.WriteArticleAuthors(article.ID, authors); err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+		if len(contributors) > 0 {
+			if err = db.WriteArticleContributors(article.ID, contributors); err != nil {
+				log.Println(err)
+				http.Error(w, err.Error(), http.StatusInternalServerError)
+				return
+			}
+		}
+
 		tags := make([]int64, 0)
 		for _, tag := range r.Form["tags"] {
 			tagID, err := strconv.ParseInt(tag, 10, 64)
 			if err != nil {
 				log.Println(err)
-				http.Error(w, err.Error(), http.StatusInternalServerError)
+				http.Error(w, err.Error(), http.StatusBadRequest)
 				return
 			}
+
 			tags = append(tags, tagID)
 		}
 		if err = db.WriteArticleTags(article.ID, tags); err != nil {
@@ -156,32 +234,68 @@ func ResubmitArticle(c *b.Config, db *b.DB, s *b.CookieStore) http.HandlerFunc {
 			return
 		}
 
-		id, err := strconv.ParseInt(r.PathValue("id"), 10, 64)
-		if err != nil {
-			log.Println(err)
-			http.Error(w, err.Error(), http.StatusInternalServerError)
-			return
+		article := &b.Article{
+			Title:      r.PostFormValue("article-title"),
+			BannerLink: r.PostFormValue("article-banner-url"),
+			Summary:    r.PostFormValue("article-summary"),
+			CreatorID:  session.Values["id"].(int64),
+			IsInIssue:  r.PostFormValue("issue") == "on",
 		}
 
-		title := r.PostFormValue("article-title")
-		if len(title) == 0 {
+		if len(article.Title) == 0 {
 			http.Error(w, "Bitte den Titel eingeben.", http.StatusBadRequest)
 			return
 		}
-
-		summary := r.PostFormValue("article-summary")
-		if len(summary) == 0 {
+		if len(article.Summary) == 0 {
 			http.Error(w, "Bitte die Beschreibung eingeben.", http.StatusBadRequest)
 			return
 		}
 
+		r.ParseForm()
+		authors := make([]int64, 0)
+		contributors := make([]int64, 0)
+
+		for key, values := range r.Form {
+			if strings.HasPrefix(key, "user-") && len(values) > 0 {
+				id, err := strconv.ParseInt(strings.Split(key, "-")[1], 10, 64)
+				if err != nil {
+					log.Println(err)
+					http.Error(w, err.Error(), http.StatusBadRequest)
+					return
+				}
+
+				switch values[0] {
+				case "author":
+					authors = append(authors, id)
+				case "contributor":
+					contributors = append(contributors, id)
+				}
+			}
+		}
+
+		if r.PostFormValue("creator") == "contributor" {
+			contributors = append(contributors, article.CreatorID)
+		} else {
+			authors = append(authors, article.CreatorID)
+		}
+		if len(authors) == 0 {
+			http.Error(w, "Es muss mindestens einen Autor geben.", http.StatusBadRequest)
+			return
+		}
+
+		article.ID, err = strconv.ParseInt(r.PathValue("id"), 10, 64)
+		if err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+
 		content := r.PostFormValue("article-content")
 		if len(content) == 0 {
 			http.Error(w, "Bitte den Artikel eingeben.", http.StatusBadRequest)
 			return
 		}
-
-		contentLink := fmt.Sprint(c.ArticleDir, "/", id, ".md")
+		contentLink := fmt.Sprint(c.ArticleDir, "/", article.ID, ".md")
 		if err = os.WriteFile(contentLink, []byte(content), 0644); err != nil {
 			log.Println(err)
 			http.Error(w, err.Error(), http.StatusInternalServerError)
@@ -189,18 +303,30 @@ func ResubmitArticle(c *b.Config, db *b.DB, s *b.CookieStore) http.HandlerFunc {
 		}
 
 		if err = db.UpdateAttributes(
-			&b.Attribute{Table: "articles", ID: id, AttName: "title", Value: title},
-			&b.Attribute{Table: "articles", ID: id, AttName: "banner_link", Value: r.PostFormValue("article-banner-url")},
-			&b.Attribute{Table: "articles", ID: id, AttName: "summary", Value: summary},
-			&b.Attribute{Table: "articles", ID: id, AttName: "rejected", Value: false},
-			&b.Attribute{Table: "articles", ID: id, AttName: "is_in_issue", Value: r.PostFormValue("issue") == "on"},
+			&b.Attribute{Table: "articles", ID: article.ID, AttName: "title", Value: article.Title},
+			&b.Attribute{Table: "articles", ID: article.ID, AttName: "banner_link", Value: article.BannerLink},
+			&b.Attribute{Table: "articles", ID: article.ID, AttName: "summary", Value: article.Summary},
+			&b.Attribute{Table: "articles", ID: article.ID, AttName: "rejected", Value: false},
+			&b.Attribute{Table: "articles", ID: article.ID, AttName: "is_in_issue", Value: article.IsInIssue},
 		); err != nil {
 			log.Println(err)
 			http.Error(w, err.Error(), http.StatusInternalServerError)
 			return
 		}
 
-		r.ParseForm()
+		if err = db.UpdateArticleAuthors(article.ID, authors); err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+		if len(contributors) > 0 {
+			if err = db.UpdateArticleContributors(article.ID, contributors); err != nil {
+				log.Println(err)
+				http.Error(w, err.Error(), http.StatusInternalServerError)
+				return
+			}
+		}
+
 		tags := make([]int64, 0)
 		for _, tag := range r.Form["tags"] {
 			tagID, err := strconv.ParseInt(tag, 10, 64)
@@ -211,7 +337,7 @@ func ResubmitArticle(c *b.Config, db *b.DB, s *b.CookieStore) http.HandlerFunc {
 			}
 			tags = append(tags, tagID)
 		}
-		if err = db.UpdateArticleTags(id, tags); err != nil {
+		if err = db.UpdateArticleTags(article.ID, tags); err != nil {
 			log.Println(err)
 			http.Error(w, err.Error(), http.StatusInternalServerError)
 			return
@@ -297,7 +423,7 @@ func ShowRejectedArticles(c *b.Config, db *b.DB, s *b.CookieStore) http.HandlerF
 
 		data.MyIDs = make(map[int64]bool)
 		for _, article := range data.RejectedArticles {
-			if article.AuthorID == session.Values["id"].(int64) {
+			if article.CreatorID == session.Values["id"].(int64) {
 				data.MyIDs[article.ID] = true
 			}
 		}
@@ -314,7 +440,8 @@ func ShowRejectedArticles(c *b.Config, db *b.DB, s *b.CookieStore) http.HandlerF
 
 func ReviewRejectedArticle(c *b.Config, db *b.DB, s *b.CookieStore) http.HandlerFunc {
 	return func(w http.ResponseWriter, r *http.Request) {
-		if _, err := GetSession(w, r, c, s); err != nil {
+		session, err := GetSession(w, r, c, s)
+		if err != nil {
 			log.Println(err)
 			http.Error(w, err.Error(), http.StatusInternalServerError)
 			return
@@ -353,6 +480,46 @@ func ReviewRejectedArticle(c *b.Config, db *b.DB, s *b.CookieStore) http.Handler
 			return
 		}
 
+		data.ArticleUsers = make(map[string]*ArticleUser)
+		users, err := db.GetAllUsers(c)
+		if err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+		for _, user := range users {
+			data.ArticleUsers[fmt.Sprint(user.LastName, user.FirstName, user.ID)] = &ArticleUser{User: user, ArticleRole: None}
+		}
+
+		authors, err := db.GetArticleAuthors(c, data.Article.ID)
+		if err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+		for _, author := range authors {
+			data.ArticleUsers[fmt.Sprint(author.LastName, author.FirstName, author.ID)].ArticleRole = Author
+		}
+
+		contributors, err := db.GetArticleContributors(c, data.Article.ID)
+		if err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+		for _, contributor := range contributors {
+			data.ArticleUsers[fmt.Sprint(contributor.LastName, contributor.FirstName, contributor.ID)].ArticleRole = Contributor
+		}
+
+		creator, err := db.GetUser(c, session.Values["id"].(int64))
+		if err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+		data.Creator = data.ArticleUsers[fmt.Sprint(creator.LastName, creator.FirstName, creator.ID)]
+		delete(data.ArticleUsers, fmt.Sprint(creator.LastName, creator.FirstName, creator.ID))
+
 		selectedTags, err := db.GetArticleTags(id)
 		if err != nil {
 			log.Println(err)
@@ -406,9 +573,9 @@ func PublishArticle(c *b.Config, db *b.DB, s *b.CookieStore) http.HandlerFunc {
 		}
 
 		if err = db.UpdateAttributes(
-			&b.Attribute{Table: "articles", ID: id, AttName: "published", Value: true},
-			&b.Attribute{Table: "articles", ID: id, AttName: "rejected", Value: false},
-			&b.Attribute{Table: "articles", ID: id, AttName: "created", Value: time.Now().Format("2006-01-02 15:04:05")},
+			&b.Attribute{Table: "articles", ID: article.ID, AttName: "published", Value: true},
+			&b.Attribute{Table: "articles", ID: article.ID, AttName: "rejected", Value: false},
+			&b.Attribute{Table: "articles", ID: article.ID, AttName: "created", Value: time.Now().Format("2006-01-02 15:04:05")},
 		); err != nil {
 			log.Println(err)
 			http.Error(w, err.Error(), http.StatusInternalServerError)
@@ -423,6 +590,18 @@ func PublishArticle(c *b.Config, db *b.DB, s *b.CookieStore) http.HandlerFunc {
 				return
 			}
 
+			if err = db.DeleteArticleContributors(oldArticle.ID); err != nil {
+				log.Println(err)
+				http.Error(w, err.Error(), http.StatusInternalServerError)
+				return
+			}
+
+			if err = db.DeleteArticleAuthors(oldArticle.ID); err != nil {
+				log.Println(err)
+				http.Error(w, err.Error(), http.StatusInternalServerError)
+				return
+			}
+
 			if err = db.DeleteArticle(oldArticle.ID); err != nil {
 				log.Println(err)
 				http.Error(w, err.Error(), http.StatusInternalServerError)
@@ -435,10 +614,7 @@ func PublishArticle(c *b.Config, db *b.DB, s *b.CookieStore) http.HandlerFunc {
 				return
 			}
 
-			if err = db.UpdateAttributes(
-				&b.Attribute{Table: "articles", ID: id, AttName: "content_link", Value: fmt.Sprint(c.Domain, "/article/serve/", article.ID)},
-				&b.Attribute{Table: "articles", ID: id, AttName: "edited_id", Value: 0},
-			); err != nil {
+			if err = db.UpdateAttributes(&b.Attribute{Table: "articles", ID: article.ID, AttName: "edited_id", Value: 0}); err != nil {
 				log.Println(err)
 				http.Error(w, err.Error(), http.StatusInternalServerError)
 				return
@@ -629,6 +805,22 @@ func ReviewArticle(c *b.Config, db *b.DB, s *b.CookieStore, action, title, butto
 		}
 		data.HTMLContent = template.HTML(data.Content)
 
+		data.Authors, err = db.GetArticleAuthors(c, data.Article.ID)
+		if err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+		sortUsersByName(data.Authors)
+
+		data.Contributors, err = db.GetArticleContributors(c, data.Article.ID)
+		if err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+		sortUsersByName(data.Contributors)
+
 		data.Tags, err = db.GetArticleTags(id)
 		if err != nil {
 			log.Println(err)
@@ -721,25 +913,59 @@ func AllowEditArticle(c *b.Config, db *b.DB, s *b.CookieStore) http.HandlerFunc
 			return
 		}
 
-		newArticle := oldArticle
+		newArticle := *oldArticle
 		newArticle.Published = false
 		newArticle.Rejected = true
 		newArticle.EditedID = oldArticle.ID
 
-		newID, err := db.AddArticle(newArticle)
+		newArticle.ID, err = db.AddArticle(&newArticle)
 		if err != nil {
 			log.Println(err)
 			http.Error(w, err.Error(), http.StatusInternalServerError)
 			return
 		}
 
-		if err = db.UpdateAttributes(&b.Attribute{Table: "articles", ID: oldID, AttName: "edited_id", Value: newID}); err != nil {
+		if err = db.UpdateAttributes(&b.Attribute{Table: "articles", ID: oldArticle.ID, AttName: "edited_id", Value: newArticle.ID}); err != nil {
 			log.Println(err)
 			http.Error(w, err.Error(), http.StatusInternalServerError)
 			return
 		}
 
-		if err = b.CopyFile(fmt.Sprint(c.ArticleDir, "/", oldID, ".md"), fmt.Sprint(c.ArticleDir, "/", newID, ".md")); err != nil {
+		src := fmt.Sprint(c.ArticleDir, "/", oldArticle.ID, ".md")
+		dst := fmt.Sprint(c.ArticleDir, "/", newArticle.ID, ".md")
+		if err = b.CopyFile(src, dst); err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+
+		authors, err := db.GetArticleAuthors(c, oldArticle.ID)
+		if err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+		authorIDs := make([]int64, len(authors))
+		for i, author := range authors {
+			authorIDs[i] = author.ID
+		}
+		if err = db.WriteArticleAuthors(newArticle.ID, authorIDs); err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+
+		contributors, err := db.GetArticleContributors(c, oldArticle.ID)
+		if err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+		contributorIDs := make([]int64, len(contributors))
+		for i, contributor := range contributors {
+			contributorIDs[i] = contributor.ID
+		}
+		if err = db.WriteArticleContributors(newArticle.ID, contributorIDs); err != nil {
 			log.Println(err)
 			http.Error(w, err.Error(), http.StatusInternalServerError)
 			return

cmd/frontend/issues.go

@@ -33,7 +33,6 @@ func PublishLatestIssue(c *b.Config, db *b.DB, s *b.CookieStore) http.HandlerFun
 			Published:     true,
 			Rejected:      false,
 			Created:       time.Now(),
-			AuthorID:      session.Values["id"].(int64),
 			AutoGenerated: true,
 		}
 
@@ -49,6 +48,22 @@ func PublishLatestIssue(c *b.Config, db *b.DB, s *b.CookieStore) http.HandlerFun
 			return
 		}
 
+		authorIDs := make([]int64, 1)
+		var ok bool
+
+		if authorIDs[0], ok = session.Values["id"].(int64); !ok {
+			msg := "fälschlicherweise session.Values[\"id\"].(int64) für authorIDs[0] angenommen"
+			log.Println(msg)
+			http.Error(w, msg, http.StatusInternalServerError)
+			return
+		}
+
+		if err = db.WriteArticleAuthors(article.ID, authorIDs); err != nil {
+			log.Println(err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+
 		content := []byte(r.PostFormValue("issue-content"))
 		if len(content) == 0 {
 			http.Error(w, "Bitte eine Beschreibung eingeben.", http.StatusBadRequest)

cmd/frontend/users.go

@@ -5,6 +5,7 @@ import (
 	"html/template"
 	"log"
 	"net/http"
+	"sort"
 	"strconv"
 
 	b "streifling.com/jason/cpolis/cmd/backend"
@@ -33,6 +34,15 @@ func checkUserStrings(user *b.User) (string, int, bool) {
 	}
 }
 
+func sortUsersByName(users []*b.User) {
+	sort.SliceStable(users, func(i, j int) bool {
+		if users[i].LastName == users[j].LastName {
+			return users[i].FirstName < users[j].FirstName
+		}
+		return users[i].LastName < users[j].LastName
+	})
+}
+
 func CreateUser(c *b.Config, s *b.CookieStore) http.HandlerFunc {
 	return func(w http.ResponseWriter, r *http.Request) {
 		if _, err := GetSession(w, r, c, s); err != nil {
@@ -332,14 +342,14 @@ func ShowAllUsers(c *b.Config, db *b.DB, s *b.CookieStore, action string) http.H
 		})
 
 		data.Action = action
-		data.Users, err = db.GetAllUsers(c)
+		data.Users, err = db.GetAllUsersMap(c)
 		if err != nil {
 			log.Println(err)
 			http.Error(w, err.Error(), http.StatusInternalServerError)
 			return
 		}
-
 		delete(data.Users, session.Values["id"].(int64))
+
 		tmpl, err := template.ParseFiles(c.WebDir + "/templates/show-all-users.html")
 		if err = template.Must(tmpl, err).ExecuteTemplate(w, "page-content", data); err != nil {
 			log.Println(err)

create_db.sql

@@ -1,4 +1,6 @@
 DROP TABLE IF EXISTS articles_tags;
+DROP TABLE IF EXISTS articles_contributors;
+DROP TABLE IF EXISTS articles_authors;
 DROP TABLE IF EXISTS tags;
 DROP TABLE IF EXISTS articles;
 DROP TABLE IF EXISTS issues;
@@ -30,13 +32,13 @@ CREATE TABLE articles (
     summary TEXT NOT NULL,
     published BOOL NOT NULL,
     rejected BOOL NOT NULL,
-    author_id       INT             NOT NULL,
+    creator_id INT NOT NULL,
     issue_id INT NOT NULL,
     edited_id INT,
     is_in_issue BOOL NOT NULL,
     auto_generated BOOL NOT NULL,
     PRIMARY KEY (id),
-    FOREIGN KEY (author_id) REFERENCES users (id),
+    FOREIGN KEY (creator_id) REFERENCES users (id),
     FOREIGN KEY (issue_id) REFERENCES issues (id)
 );
 
@@ -46,6 +48,22 @@ CREATE TABLE tags (
     PRIMARY KEY (id)
 );
 
+CREATE TABLE articles_authors (
+    article_id INT,
+    author_id INT,
+    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,
+    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,

web/templates/editor.html

@@ -7,7 +7,7 @@
 
         <div class="grid grid-cols-2 gap-4 items-center">
             <div class="flex flex-col">
-                <label for="article-title">Titel</label>
+                <h3>Titel</h3>
                 <input name="article-title" type="text" value="{{.Article.Title}}" />
             </div>
 
@@ -20,12 +20,18 @@
     </div>
 
     <div class="flex flex-col gap-1">
-        <label for="article-summary">Beschreibung</label>
+        <h3>Beschreibung</h3>
         <textarea name="article-summary">{{.Article.Summary}}</textarea>
     </div>
 
+    <div class="flex flex-col gap-1">
+        <h3>Artikel</h3>
+        <textarea id="easyMDE">{{.Content}}</textarea>
+        <input id="article-content" name="article-content" type="hidden" value="{{.Content}}" />
+    </div>
+
     <div>
-        <span>Tags</span>
+        <h3>Tags</h3>
         <div class="flex flex-wrap gap-4">
             <div>
                 <input id="issue" name="issue" type="checkbox" {{if .Article.IsInIssue}}checked{{end}} />
@@ -42,10 +48,36 @@
         </div>
     </div>
 
-    <div class="flex flex-col gap-1">
-        <label for="easyMDE">Artikel</label>
-        <textarea id="easyMDE">{{.Content}}</textarea>
-        <input id="article-content" name="article-content" type="hidden" value="{{.Content}}" />
+    <div>
+        <h3>Beteiligte</h3>
+        {{range .ArticleUsers}}
+        <div class="border border-slate-200 dark:border-slate-800 flex gap-4 px-2 py-1 rounded-md">
+            <span>{{.FirstName}} {{.LastName}}: </span>
+
+            <div>
+                <input id="{{.ID}}-author" name="user-{{.ID}}" type="radio" value="author" {{if eq .ArticleRole
+                    1}}checked{{end}} />
+                <label for="{{.ID}}-author">Autor</label>
+            </div>
+
+            <div>
+                <input id="{{.ID}}-contributor" name="user-{{.ID}}" type="radio" value="contributor" {{if eq
+                    .ArticleRole 2}}checked{{end}} />
+                <label for="{{.ID}}-contributor">Mitwirkender</label>
+            </div>
+
+            <div>
+                <input id="{{.ID}}-none" name="user-{{.ID}}" type="radio" {{if eq .ArticleRole 0}}checked{{end}} />
+                <label for="{{.ID}}-none">Unbeteiligt</label>
+            </div>
+        </div>
+        {{end}}
+    </div>
+
+    <div>
+        <input id="creator" name="creator" type="checkbox" value="contributor" {{if eq .Creator.ArticleRole
+            2}}checked{{end}} />
+        <label for="creator">Ich bin nicht der Autor.</label>
     </div>
 
     <div class="btn-area">

web/templates/review-article.html

@@ -6,24 +6,24 @@
         <img src="/image/serve/{{.Image}}" alt="Banner Image">
     </div>
 
-    <span>Titel</span>
+    <h3>Titel</h3>
     <div class="border border-slate-200 dark:border-slate-800 mb-3 px-2 py-2 rounded-md w-full">
         {{.Article.Title}}
     </div>
 
-    <span>Beschreibung</span>
+    <h3>Beschreibung</h3>
     <div class="border border-slate-200 dark:border-slate-800 mb-3 px-2 py-2 rounded-md w-full">
         {{.Article.Summary}}
     </div>
 
-    <span>Artikel</span>
+    <h3>Artikel</h3>
     <div class="border border-slate-200 dark:border-slate-800 mb-3 px-2 py-2 rounded-md w-full">
         <div class="prose text-slate-900 dark:text-slate-100">
             {{.HTMLContent}}
         </div>
     </div>
 
-    <span>Tags</span>
+    <h3>Tags</h3>
     <div class="border border-slate-200 dark:border-slate-800 mb-3 px-2 py-2 rounded-md w-full">
         {{if .Article.IsInIssue}}
         <span>Orient Express</span>
@@ -35,6 +35,22 @@
         {{end}}
     </div>
 
+    <h3>Autoren</h3>
+    <div class="border border-slate-200 dark:border-slate-800 mb-3 px-2 py-2 rounded-md w-full">
+        {{range .Authors}}
+        <span>{{.FirstName}} {{.LastName}}</span>
+        <br>
+        {{end}}
+    </div>
+
+    <h3>Mitwirkende</h3>
+    <div class="border border-slate-200 dark:border-slate-800 mb-3 px-2 py-2 rounded-md w-full">
+        {{range .Contributors}}
+        <span>{{.FirstName}} {{.LastName}}</span>
+        <br>
+        {{end}}
+    </div>
+
     {{if eq .Action "publish"}}
     <div class="btn-area-3">
         <input class="action-btn" type="submit" value="{{.ActionButton}}" hx-get="/article/{{.Action}}/{{.Article.ID}}"