Pin plugins v1.6.21, which follows ReportManagement into FOG\Pages - #1529
Merged
Conversation
GH-1528 moved the last 52 flat-namespace core classes into src/{Pages,Hooks,Reports,Events}, so \FOG\ReportManagement became \FOG\Pages\ReportManagement. Eight bundled plugin reports extend it and fatal with "Class FOG\ReportManagement not found" the moment someone opens one -- the autoloader refuses the flat spelling and logs the correct FQCN rather than resolving it, so the failure is legible, but the report page is gone. fog-plugins v1.6.21 carries the eight edits. Without this bump an install still fetches v1.6.20 against a core that cannot load it, so the fix exists and does not reach anybody: bin/fetch-plugins.sh reads the pinned tag straight out of this constant. Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
One constant.
bin/fetch-plugins.shreads the pinned tag straight out ofFOG_PLUGINS_VERSION, so this is what decides which plugin release an installactually fetches.
#1528 moved the last 52 flat-namespace core classes into
src/{Pages,Hooks,Reports,Events}, which made\FOG\ReportManagementinto\FOG\Pages\ReportManagement. Eight bundled plugin reports extend it — ldap,location, ou, subnetgroup, taskstateedit, tasktypeedit, windowskey and
wolbroadcast —
and on today's pin they fatal with
Class "FOG\ReportManagement" not foundthemoment someone opens one. The autoloader refuses the flat spelling and logs the
correct FQCN rather than resolving it, so the failure names its own fix, but the
report page is gone.
fog-plugins v1.6.21
(FOGProject/fog-plugins#32) carries the eight edits. Without this bump the fix
exists and reaches nobody.
🤖 Generated with Claude Code
https://claude.ai/code/session_01VTEiBnDpNrgzu1tSqt1318