MokoConsulting/MokoWaaS
1
1
Fork 0
Code Issues 13 Pull Requests Actions 6 Licenses 1 Releases 5 Wiki Activity

fix: docblock syntax errors and crontab reset schedule #91

Merged
jmiller merged 13 commits from dev into main 2026-05-30 21:20:45 +00:00
Conversation 0 Commits 13 Files Changed 37
+272 -61
jmiller commented 2026-05-30 21:14:01 +00:00
Owner
Copy Link
Copy Source

Summary

  • Fix PHP syntax errors from */ sequences in docblock comments (closes block comment prematurely)
  • Add crontab-style reset schedule with preset dropdown (hourly, 4h, 6h, daily, weekly, etc.) plus custom option
  • Banner countdown now uses stored next-reset timestamp instead of relative timer
  • Added cron expression parser with support for wildcards, steps, ranges, comma lists

Test plan

  • Verified install on dev site (02.23.00 on Joomla 6.1.1 / PHP 8.4.20)
  • Health endpoint responding correctly
  • Verify clean build includes plg_task_mokowaasdemo
## Summary - Fix PHP syntax errors from `*/` sequences in docblock comments (closes block comment prematurely) - Add crontab-style reset schedule with preset dropdown (hourly, 4h, 6h, daily, weekly, etc.) plus custom option - Banner countdown now uses stored next-reset timestamp instead of relative timer - Added cron expression parser with support for wildcards, steps, ranges, comma lists ## Test plan - [x] Verified install on dev site (02.23.00 on Joomla 6.1.1 / PHP 8.4.20) - [x] Health endpoint responding correctly - [ ] Verify clean build includes plg_task_mokowaasdemo
jmiller added 8 commits 2026-05-30 21:14:01 +00:00
chore: sync updates.xml 02.23.00 from main [skip ci] c37e6d9637
feat(demo): crontab-style reset schedule with preset dropdown and stored countdown
Generic: Repo Health / Site Health (push) Has been skipped
Details
Generic: Repo Health / Access control (push) Successful in 2s
Details
Universal: Auto Version Bump / Version Bump (push) Failing after 4s
Details
Update Server / Update Server (push) Successful in 13s
Details
Generic: Repo Health / Release configuration (push) Has been cancelled
Details
Generic: Repo Health / Scripts governance (push) Has been cancelled
Details
Generic: Repo Health / Repository health (push) Has been cancelled
Details
65e986344e 
- Replace hours interval with preset schedule dropdown (hourly, 4h, 6h,
  12h, daily, weekly, monthly) plus custom crontab option
- Banner countdown now uses stored demo_next_reset timestamp calculated
  from cron schedule on save, not a relative timer from page load
- Added cron parser (calculateNextCronRun) supporting wildcards, steps,
  ranges, and comma-separated values

Authored-by: Moko Consulting
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
chore(version): auto-bump 02.23.01-dev [skip ci] 689bf1712f
chore: update development channel 02.23.01-dev [skip ci] 88817690e5
fix: docblock */ in (*/N) prematurely closes comment, causing PHP syntax error 1673616523 
The string "*/N" inside a docblock contains */ which PHP interprets as
the end of the block comment, making "N" an unexpected identifier.

Authored-by: Moko Consulting
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
fix: remove all */ sequences from comments to prevent docblock termination
Generic: Repo Health / Site Health (push) Has been skipped
Details
Generic: Repo Health / Access control (push) Successful in 8s
Details
Universal: Auto Version Bump / Version Bump (push) Failing after 18s
Details
Update Server / Update Server (push) Successful in 26s
Details
Generic: Repo Health / Release configuration (push) Has been cancelled
Details
Generic: Repo Health / Scripts governance (push) Has been cancelled
Details
Generic: Repo Health / Repository health (push) Has been cancelled
Details
5245d15b9d 
Cron-related comments contained */ in examples like "*/5" and "*/N"
which PHP interprets as closing the block comment, causing syntax errors.

Authored-by: Moko Consulting
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
chore(version): auto-bump 02.23.02-dev [skip ci] ccdab8b5da
chore: update development channel 02.23.02-dev [skip ci] 1dff862d2b
jmiller added 1 commit 2026-05-30 21:17:05 +00:00
fix: add plugin attribute to task plugin manifest to set element in DB
Generic: Repo Health / Site Health (push) Has been skipped
Details
Generic: Repo Health / Access control (push) Successful in 1s
Details
Universal: PR Check / Branch Policy (pull_request) Successful in 1s
Details
Generic: Repo Health / Site Health (pull_request) Has been skipped
Details
Generic: Repo Health / Access control (pull_request) Successful in 1s
Details
Joomla: Extension CI / Release Readiness Check (pull_request) Failing after 6s
Details
Universal: Secret Scanning / Gitleaks Secret Scan (pull_request) Successful in 7s
Details
Joomla: Extension CI / Lint & Validate (pull_request) Failing after 8s
Details
Universal: Auto Version Bump / Version Bump (push) Failing after 6s
Details
Universal: PR Check / Validate PR (pull_request) Successful in 8s
Details
Update Server / Update Server (push) Successful in 15s
Details
Generic: Repo Health / Release configuration (push) Has been cancelled
Details
Generic: Repo Health / Scripts governance (push) Has been cancelled
Details
Generic: Repo Health / Repository health (push) Has been cancelled
Details
Joomla: Extension CI / Tests (PHP 8.2) (pull_request) Has been cancelled
Details
Joomla: Extension CI / Tests (PHP 8.3) (pull_request) Has been cancelled
Details
Joomla: Extension CI / PHPStan Analysis (pull_request) Has been cancelled
Details
Joomla: Extension CI / Build RC Pre-Release (pull_request) Has been cancelled
Details
Universal: PR Check / Build RC Package (pull_request) Has been cancelled
Details
Generic: Repo Health / Release configuration (pull_request) Has been cancelled
Details
Generic: Repo Health / Scripts governance (pull_request) Has been cancelled
Details
Generic: Repo Health / Repository health (pull_request) Has been cancelled
Details
d9012ffddb 
Joomla requires a <filename plugin="mokowaasdemo"> in the files block
to populate the element field in #__extensions. Without it, the DB
INSERT fails with "Field 'element' doesn't have a default value".

Authored-by: Moko Consulting
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
jmiller added 1 commit 2026-05-30 21:17:20 +00:00
chore(version): auto-bump 02.23.03-dev [skip ci] b1519cf12a
jmiller added 1 commit 2026-05-30 21:17:23 +00:00
chore: update development channel 02.23.03-dev [skip ci] 45cbd5cad4
jmiller added 2 commits 2026-05-30 21:20:19 +00:00
chore: merge main into dev (resolve version conflicts) [skip ci] 6cceb85be6 
Authored-by: Moko Consulting
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Merge branch 'dev' of https://git.mokoconsulting.tech/MokoConsulting/MokoWaaS into dev
Generic: Repo Health / Site Health (push) Has been skipped
Details
Generic: Repo Health / Access control (push) Successful in 21s
Details
Generic: Repo Health / Site Health (pull_request) Has been skipped
Details
Universal: PR Check / Branch Policy (pull_request) Successful in 11s
Details
Generic: Repo Health / Access control (pull_request) Successful in 4s
Details
Universal: Auto Version Bump / Version Bump (push) Failing after 28s
Details
Universal: Build & Release / Promote to RC (pull_request) Has been skipped
Details
Branch Cleanup / Delete merged branch (pull_request) Has been skipped
Details
Joomla: Extension CI / Lint & Validate (pull_request) Failing after 34s
Details
Universal: PR Check / Validate PR (pull_request) Successful in 19s
Details
Universal: Secret Scanning / Gitleaks Secret Scan (pull_request) Successful in 29s
Details
Universal: Build & Release / Build & Release Pipeline (pull_request) Successful in 1m14s
Details
Joomla: Extension CI / Release Readiness Check (pull_request) Failing after 17m48s
Details
Generic: Repo Health / Release configuration (push) Has been cancelled
Details
Generic: Repo Health / Scripts governance (push) Has been cancelled
Details
Generic: Repo Health / Repository health (push) Has been cancelled
Details
Joomla: Extension CI / Tests (PHP 8.2) (pull_request) Has been cancelled
Details
Joomla: Extension CI / Tests (PHP 8.3) (pull_request) Has been cancelled
Details
Joomla: Extension CI / PHPStan Analysis (pull_request) Has been cancelled
Details
Joomla: Extension CI / Build RC Pre-Release (pull_request) Has been cancelled
Details
Universal: PR Check / Build RC Package (pull_request) Has been cancelled
Details
Generic: Repo Health / Release configuration (pull_request) Has been cancelled
Details
Generic: Repo Health / Scripts governance (pull_request) Has been cancelled
Details
Generic: Repo Health / Repository health (pull_request) Has been cancelled
Details
586b7bc105
jmiller merged commit 22acb25bbe into main 2026-05-30 21:20:45 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
automation
Automated processes or scripts
 breaking-change
Breaking API or functionality change
 build
Build system changes
 ci-cd
CI/CD pipeline changes
 config
Configuration file changes
 css
CSS/styling changes
 dependencies
Dependency updates
 deploy-failure
Automated deploy failure tracking
 docker
Docker configuration changes
 documentation
Documentation changes
 dolibarr
Dolibarr module or extension
 generic
Generic project or library
 good first issue
Good for newcomers
 health-check
Repository health check results
 health: excellent
Health score 90-100
 health: fair
Health score 50-69
 health: good
Health score 70-89
 health: poor
Health score below 50
 help wanted
Extra attention needed
 html
HTML template changes
 javascript
JavaScript code changes
 joomla
Joomla extension or component
 major-release
Major version release (breaking changes)
 minor-release
Minor version release (XX.YY.00)
 mokostandards
MokoStandards compliance
 needs-review
Awaiting code review
 needs-testing
Requires manual or automated testing
 patch-release
Patch version release (XX.YY.ZZ)
 php
PHP code changes
 priority: critical
Critical priority, must be addressed immediately
 priority: high
High priority
 priority: low
Low priority
 priority: medium
Medium priority
 push-failure
File push failure requiring attention
 python
Python code changes
 regression
Regression from a previous working state
 release-candidate
Release candidate build
 security
Security-related changes
 size/l
Large change (101-300 lines)
 size/m
Medium change (31-100 lines)
 size/s
Small change (11-30 lines)
 size/xl
Extra large change (301-1000 lines)
 size/xs
Extra small change (1-10 lines)
 size/xxl
Extremely large change (1000+ lines)
 standards-drift
Repository drifted from MokoStandards
 standards-update
MokoStandards sync update
 standards-violation
Standards compliance failure
 status: blocked
Blocked by another issue or dependency
 status: in-progress
Currently being worked on
 status: needs-review
Awaiting code review
 status: on-hold
Temporarily on hold
 status: pending
Pending action or decision
 status: wontfix
This will not be worked on
 sync-failure
Bulk sync failure requiring attention
 sync-report
Bulk sync run report
 template-validation-failure
Template workflow validation failure
 test-failure
Automated test failure
 tests
Test suite changes
 type: bug
Something isn't working
 type: chore
Maintenance tasks
 type: enhancement
Enhancement to existing feature
 type: feature
New feature or request
 type: refactor
Code refactoring
 type: release
Release preparation or tracking
 type: test
Test suite additions or changes
 type: version
Version-related change
 typescript
TypeScript code changes
 version
Version bump or release
 version-branch
Version branch related
 version-drift
Version mismatch detected
 work-in-progress
Work in progress, not ready for merge
bug
Something is not working
 chore
Maintenance and housekeeping
 documentation
Documentation improvements
 enhancement
Improvement to existing functionality
 feature
New feature or request
 pending: dependency
Blocked by another issue or external dependency
 pending: deployment
Tested and approved, awaiting deployment to production
 pending: design
Needs UI/UX or architecture design before implementation
 pending: documentation
Feature works, needs documentation/wiki update
 pending: feedback
Awaiting feedback or decision from stakeholder
 pending: review
Implementation complete, awaiting code review
 pending: testing
Feature implemented but not yet tested
 priority: critical
Must fix immediately
 priority: high
Should fix soon
 priority: low
Nice to have
 priority: medium
Fix when convenient
 refactor
Code restructuring without behavior change
 roadmap
Planned feature or enhancement tracked on the roadmap
 scope: client
Client-specific work
 scope: dolibarr
Dolibarr modules and customizations
 scope: infrastructure
Server, CI, backups, monitoring
 scope: joomla
Joomla templates and extensions
 scope: waas
MokoWaaS platform
 security
Security vulnerability or hardening
 status: blocked
Waiting on external dependency
 status: duplicate
Duplicate of another issue
 status: in-progress
Being worked on
 status: needs-review
Ready for review
 status: wontfix
Will not be addressed
No labels
Milestone
No items
No Milestone
Assignees
Clear assignees
jmiller (Jonathan Miller)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: MokoConsulting/MokoWaaS#91
Delete Branch "dev"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?