Fix miscellaneous stories bugs (#103)

* Fix course groups not filtered

* Fix created stories not linked to group

* Fix insufficient permissions for backup script

Author: RichDom2185

controller/stories/stories.go

@@ -140,7 +140,14 @@ func HandleCreate(w http.ResponseWriter, r *http.Request) error {
 		}
 	}
 
-	storyModel := *params.ToModel()
+	// Get group id from context
+	groupID, err := usergroups.GetGroupIDFrom(r)
+	if err != nil {
+		logrus.Error(err)
+		return err
+	}
+
+	storyModel := *params.ToModel(groupID)
 
 	// Get DB instance
 	db, err := database.GetDBFrom(r)

deployment/backups.sh

@@ -17,7 +17,7 @@ NEW_BACKUP_FILENAME="pgdump_`date +%Y%m%d-%H%M%S`.sql"
 BACKUP_FILE_PATH="$BACKUP_DIR/$NEW_BACKUP_FILENAME"
 
 BACKUP_COMMAND="'pg_dumpall -c -h \$PG_HOST -U \$PG_USER'"
-SQL_DUMP="$(docker compose -f "$COMPOSE_FILE_PATH" run --rm -it --entrypoint "bash -c $BACKUP_COMMAND" migrator)"
+SQL_DUMP="$(sudo docker compose -f "$COMPOSE_FILE_PATH" run --rm -it --entrypoint "bash -c $BACKUP_COMMAND" migrator)"
 
 if [[ -z "$LAST_BACKUP_FILENAME" ]]; then
     echo "No previous backups found, creating new backup..."

model/coursegroups.go

@@ -17,7 +17,10 @@ func GetGroupByCourseID(db *gorm.DB, courseID int) (Group, error) {
 	courseGroup := CourseGroup{
 		CourseID: uint(courseID),
 	}
-	err := db.Preload(clause.Associations).First(&courseGroup).Error
+	err := db.
+		Preload(clause.Associations).
+		Where(&courseGroup).
+		First(&courseGroup).Error
 	if err != nil {
 		return courseGroup.Group, database.HandleDBError(err, "courseGroup")
 	}

params/stories/create.go

@@ -16,9 +16,10 @@ func (params *Create) Validate() error {
 	return nil
 }
 
-func (params *Create) ToModel() *model.Story {
+func (params *Create) ToModel(associatedGroupID *uint) *model.Story {
 	return &model.Story{
 		AuthorID: params.AuthorID,
+		GroupID:  associatedGroupID,
 		Title:    params.Title,
 		Content:  params.Content,
 		PinOrder: params.PinOrder,

params/stories/create_test.go

@@ -16,7 +16,7 @@ func TestToModel(t *testing.T) {
 			AuthorID: 1,
 			Content:  "# Hi\n\nThis is a test story.",
 		}
-		model := params.ToModel()
+		model := params.ToModel(nil)
 		assert.Equal(t, params.AuthorID, model.AuthorID)
 		assert.Equal(t, params.Content, model.Content)
 	})