Remove hard mod_booking dependencies from local_wunderbyte_table#120
Draft
georgmaisser with Copilot wants to merge 2 commits into
Draft
Remove hard mod_booking dependencies from local_wunderbyte_table#120georgmaisser with Copilot wants to merge 2 commits into
georgmaisser with Copilot wants to merge 2 commits into
Conversation
Agent-Logs-Url: https://github.com/Wunderbyte-GmbH/moodle-local_wunderbyte_table/sessions/69af7b8e-bf37-4a62-a34f-12a0b6ba6efc Co-authored-by: georgmaisser <7524621+georgmaisser@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Start implementation to eliminate wunderbyte table dependency
Remove hard mod_booking dependencies from local_wunderbyte_table
Apr 7, 2026
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
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.
local_wunderbyte_tablehad unconditional top-leveluseimports ofmod_bookingclasses, causing PHP autoloader failures whenmod_bookingis not installed.Changes
classes/wunderbyte_table.php: Droppeduse mod_booking\singleton_service;. Wrapped the booking session-fetching block incalendarouthtml()with aclass_existsguard;rowswithdatesdegrades to an empty array whenmod_bookingis absent.classes/local/performance/performance.php: Droppeduse mod_booking\local\performance\performance_measurer;. The surroundingget_plugin_info+class_existsguards were already in place; replaced the short-name references with fully-qualified\mod_booking\local\performance\performance_measurercalls.Original prompt
Start implementation
[Chronological Review: The conversation began with the user requesting to eliminate the dependency on the wunderbyte table by integrating its functionality directly into the mod_mooduell plugin. The user asked for a comparison of the pros and cons of creating a subplugin versus including everything directly. The user then requested to start the implementation, specifying to focus only on changes in mod/mooduell and local/wunderbyte_table. The conversation continued with the user reiterating the request to start implementation, indicating a desire to move forward with the task.][Intent Mapping:
[Technical Inventory:
[Code Archaeology:
[Progress Assessment:
[Context Validation: All critical information for continuation is captured, including user goals and specific areas of focus.]
[Recent Commands Analysis:
1. Conversation Overview: - Primary Objectives: - "I want to get rid of the dependency of wunderbyte table..." - "Weigh pro and cons for a subplugin." - "Start implementation." - "nur die änderungen in mod/mooduell und local/wunderbyte_table beachten!" - Session Context: The conversation has focused on integrating the wunderbyte table functionality into the mod_mooduell plugin, with discussions on whether to create a subplugin or include everything directly. - User Intent Evolution: The user has moved from analysis of options to a clear directive to start implementation.- Technical Foundation:
- Moodle: The platform being modified.
- mod_mooduell: The main plugin being worked on.
- local/wunderbyte_table: The external dependency being integrated.
- Codebase Status:
- /var/www/moodle/public/mod/mooduell/version.php:
- Purpose: Important for the mod_mooduell plugin functionality.
- Current State: Awaiting implementation of integration changes.
- Key Code Segments: Specific functions and classes to be integrated from wunderbyte_table.
- Dependencies: Will reduce reliance on external tables.
- Problem Resolution:
- Issues Encountered: Dependency on wunderbyte table.
- Solutions Implemented: User's plan to integrate necessary components directly into mod_mooduell.
- Debugging Context: No specific debugging issues noted yet.
- Lessons Learned: Importance of evaluating dependency management in plugin development.
- Progress Tracking:
- Completed Tasks: User has outlined the task and specified focus areas.
- Partially Complete Work: Implementation of integration is pending.
- Validated Outcomes: None yet, as implementation has not started.
- Active Work State:
- Current Focus: Preparing to implement changes in mod_mooduell and local/wunderbyte_table.
- Recent Context: User has reiterated the need to start implementation and focus on specific areas.
- Working Code: No specific code snippets were discussed recently.
- Immediate Context: User is ready to begin coding the integration.
- Recent Operations:
- Last Agent Commands: User requested to "Start implementation" and "nur die änderungen in mod/mooduell und local/wunderbyte_table beachten!"
- Tool Results Summary: No specific tool results were generated.
- Pre-Summary State: The agent was preparing to implement changes in the specified files.
- Operation Context: Commands were executed to initiate the integration of the wunderbyte table functionality into the mod_mooduell plu...
Erstellt aus VS Code.