[bugfix] Check the length of form.MediaIDs instead of just checking f…

…or null (#766)
superseriousbusiness · Aug 26, 2022 · e9b5ba0 · e9b5ba0
1 parent 79fb8ba
commit e9b5ba0
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/internal/api/client/status/statuscreate.go b/internal/api/client/status/statuscreate.go
@@ -105,11 +105,15 @@ func (m *Module) StatusCreatePOSTHandler(c *gin.Context) {
 }
 
 func validateCreateStatus(form *model.AdvancedStatusCreateForm) error {
-	if form.Status == "" && form.MediaIDs == nil && form.Poll == nil {
+	hasStatus := form.Status != ""
+	hasMedia := len(form.MediaIDs) != 0
+	hasPoll := form.Poll != nil
+
+	if !hasStatus && !hasMedia && !hasPoll {
 		return errors.New("no status, media, or poll provided")
 	}
 
-	if form.MediaIDs != nil && form.Poll != nil {
+	if hasMedia && hasPoll {
 		return errors.New("can't post media + poll in same status")
 	}