Skip to content

docs(changelog): add the 2.7.4 Beta 1 release notes - #197

Merged
mambax7 merged 1 commit into
XOOPS:masterfrom
mambax7:docs/changelog-2.7.4-beta1
Sep 10, 2026
Merged

docs(changelog): add the 2.7.4 Beta 1 release notes#197
mambax7 merged 1 commit into
XOOPS:masterfrom
mambax7:docs/changelog-2.7.4-beta1

Conversation

@mambax7

@mambax7 mambax7 commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Summary

Curated release notes for 2.7.4 Beta 1 in docs/changelog.270.txt, above the 2.7.3 Final block: the seven security changes (#186 to #189, #194, #196), four bug fixes (#184, #188, #190, #195), the TinyMCE 7.9.3 update with the note that TinyMCE 5 stays unpatched (#193), the front-end menu label move (#185), and the executable guard tests.

docs/lang_diff.txt already carries its 2.7.4-Beta1 section (#185, #190) and include/version.php already reads 2.7.4-Beta1 (#184). CHANGELOG.md is regenerated by the workflow when the release is published and is not edited here. README and the release post are Final-only, as at 2.7.3 Beta 1.

Release gate

php -l clean on PHP 8.5 over class, kernel, include, the system module and upgrade. tests/ and upgrade/tests at baseline on PHP 8.4 and 8.5. No open issues or pull requests remain upstream.

Summary by Sourcery

Documentation:

  • Add curated 2.7.4 Beta 1 release notes covering security updates, bug fixes, dependency updates, UI changes, and related testing changes.

Copilot AI lite review requested due to automatic review settings September 10, 2026 03:54
@sourcery-ai

sourcery-ai Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor
Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

This documentation-only change adds the curated 2.7.4 Beta 1 release notes to the versioned changelog, covering security fixes, bug fixes, dependency and UI updates, and test coverage while preserving the repository’s existing release-file workflow.

File-Level Changes

Change Details Files
Adds curated 2.7.4 Beta 1 release notes before the 2.7.3 Final section.
  • Documents seven security changes, four bug fixes, the TinyMCE 7.9.3 update, and the TinyMCE 5 unpatched-status note.
  • Records the front-end menu label move and executable guard tests.
  • Leaves generated and already-updated release metadata files unchanged.
docs/changelog.270.txt

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@coderabbitai

coderabbitai Bot commented Sep 10, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: b3cc0782-35f1-4adf-b0af-1264b2b56c7b

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨

@gitar-bot

gitar-bot Bot commented Sep 10, 2026

Copy link
Copy Markdown

Important

You are using the Gitar free plan. Upgrade to unlock code review, CI analysis, auto-apply, custom automations, and more.

Gitar

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The change is limited to adding a well-structured curated changelog entry and does not introduce code or behavioral changes requiring further verification.

Pull request overview

Adds the curated 2.7.4 Beta 1 release notes to the 2.7.x narrative changelog, keeping the documented release history aligned with the already-landed security/bugfix work in the codebase.

Changes:

  • Add a new “2.7.4 Beta 1 (2026-09)” section above the 2.7.3 Final block.
  • Summarize included security fixes, bug fixes, TinyMCE 7.9.3 update (with TinyMCE 5 remaining unpatched), language move, and related guard-test work.
File summaries
File Description
docs/changelog.270.txt Adds curated 2.7.4 Beta 1 release notes above the existing 2.7.3 Final section.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@greptile-apps

greptile-apps Bot commented Sep 10, 2026

Copy link
Copy Markdown

RetriggerConfidence Score: 4/5

The documentation-only PR appears safe to merge after correcting the non-blocking overstatement about which remember-me cookie-clearing paths are guarded.

Findings

  1. P2 Cookie fix scope is overstated

Summary

  • Documents seven security changes and four bug fixes.
  • Records the TinyMCE 7.9.3 update and TinyMCE 5 caveat.
  • Describes language-label relocation and executable guard tests.

Reviews (1) · Last reviewed commit: "docs(changelog): add the 2.7.4 Beta 1 re..."

Comment thread docs/changelog.270.txt
Bug fixes:
- report a stalled upgrade patch task instead of re-queuing it silently: a task whose apply returned true without satisfying its own check was selected again on every click with no message, and every patch after it was never reached; XoopsUpgrade::apply() now names the task that fails or throws and re-checks after the loop, exception text and undeletable-file lists are shown without absolute server paths, and apply_rmindexhtml() in the 2.5.11 patch reports the files it could not delete instead of claiming success (#184, closes #183)
- define _AM_SYSTEM_USERS_NO_SUCH_USER, used by the users admin page since 2.5 but never defined; on PHP 8 an unknown user id produced a fatal page instead of a redirect with a message (#190)
- clear the remember-me cookie on logout, on session restore and in the users admin only when a cookie name is configured; the name is empty when the feature is disabled and setcookie() rejects an empty name on PHP 8, so a logout or a deactivated account on such a site produced an error (#188, #195, closes #191)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Cookie fix scope is overstated

The note says empty cookie names are avoided on logout and in the users admin, but the directly reachable profile logout still clears the configured cookie without an empty-name guard, and neither users-admin implementation clears this cookie. This gives operators an inaccurate account of which paths were fixed.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Both points hold. The profile module's own user.php logout carried the same two unguarded calls, and the users admin clears no cookie; the third guarded site is the login handler. Fixed in #198: the guard is applied to the profile logout, the regression test covers both logout pages, and the release note now names the two logout pages and the session restore.

@codecov

codecov Bot commented Sep 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 20.63%. Comparing base (4b5504d) to head (ea4627f).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #197      +/-   ##
============================================
- Coverage     20.64%   20.63%   -0.01%     
  Complexity     7995     7995              
============================================
  Files           675      675              
  Lines         43506    43514       +8     
============================================
  Hits           8981     8981              
- Misses        34525    34533       +8     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mambax7
mambax7 merged commit feb6767 into XOOPS:master Sep 10, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants