Bug: schedule() allows re-scheduling already-posted items #121

New Issue

Closed

opened 2026-06-06 11:50:10 +00:00 by jmiller · 1 comment

jmiller commented 2026-06-06 11:50:10 +00:00

Owner

Copy Link

Copy Source

Audit Finding

Severity: Low

Issue:
PostsController::schedule() sets any selected post to status=queued regardless of current status. Posts with posted or posting status can be re-queued, causing duplicate cross-posts.

Fix: Add WHERE status filter to only allow scheduling of queued/failed/cancelled posts.

Files: PostsController.php

## Audit Finding **Severity:** Low **Issue:** `PostsController::schedule()` sets any selected post to `status=queued` regardless of current status. Posts with `posted` or `posting` status can be re-queued, causing duplicate cross-posts. **Fix:** Add WHERE status filter to only allow scheduling of queued/failed/cancelled posts. **Files:** `PostsController.php`

jmiller commented 2026-06-06 11:50:14 +00:00

Author

Owner

Copy Link

Copy Source

Branch created: feature/121-bug-schedule-allows-re-scheduling-alread

git fetch origin
git checkout feature/121-bug-schedule-allows-re-scheduling-alread

Branch created: [`feature/121-bug-schedule-allows-re-scheduling-alread`](https://git.mokoconsulting.tech/MokoConsulting/MokoJoomCross/src/branch/feature/121-bug-schedule-allows-re-scheduling-alread) ```bash git fetch origin git checkout feature/121-bug-schedule-allows-re-scheduling-alread ```

jmiller referenced this issue from a commit 2026-06-06 14:42:01 +00:00

fix: resolve remaining low-priority bugs (#121, #122, #123, #126)

jmiller closed this issue 2026-06-06 14:42:24 +00:00

Sign in to join this conversation.

Labels

No items

No labels

Priority High

Type Bug

Milestone

No items

No Milestone

Assignees

Clear assignees

jmiller (Jonathan Miller)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: MokoConsulting/MokoJoomCross#121