DataCollection: remove old exception artifacts for ILIAS 10#10558
Merged
iszmais merged 1 commit intoILIAS-eLearning:release_10from Nov 28, 2025
Merged
DataCollection: remove old exception artifacts for ILIAS 10#10558iszmais merged 1 commit intoILIAS-eLearning:release_10from
iszmais merged 1 commit intoILIAS-eLearning:release_10from
Conversation
katringross
added a commit
to kroepelin-projekte/ILIAS
that referenced
this pull request
Dec 17, 2025
* Remove usage of domxml_open_mem method * [LTI] update: add updated LTI roadmap with short-, mid-, and long-term goals * LegalDocuments: Fix ilCtrl issue See: https://mantis.ilias.de/view.php?id=46159 * TermsOfService: Activate tab on confirmation screen * Test: Fix Class-Namespace See: https://mantis.ilias.de/view.php?id=44710#c119693 * media objects: fixed null issue in strpos call * Test: Fix Typo in Lang Files * Init/Error: Improve error handling in `error.php` This commit improves the global error handling in `error.php`: 1. Remove building a global template from `tpl.main.html` since it only contains one placeholder (see 2b8eafc) and provides no added value 2. Ensure a proper HTTP 500 status code is sent for normal error handling 3. Ensure a proper HTTP 500 status code is sent if an unexpected error occurs while rendering the error page 4. Log errors to `error_log` with minimal user-facing output, including an incident ID, timestamp, and message * Init: Redirect to platform URL if `ilCtrlException` bubbles up Mantis Issue: https://mantis.ilias.de/view.php?id=44170 / https://mantis.ilias.de/view.php?id=45707 * session: fixed array access * UIComponent/Explorer: Get rid of unused "Max Tree Depth" determination This PR suggests to remove the `ilTree::getMaximumDepth` method call in `\ilExplorer::getMaximumTreeDepth`. The calculated value is not used at all, but results (depending on the size of the `tree` table) in an expensive database call. * Access: fix display options in advmd admin for active plugin (33207) * fix dc file copy and deletion (ILIAS-eLearning#10434) * Object: Fix Export of Tile Images See: https://mantis.ilias.de/view.php?id=45574 * Test: Generate Table Filter Ids Based on Context * Test: Fix Applying withKioskMode From Defaults See: https://mantis.ilias.de/view.php?id=45956 * Calendar: Fix Missing Parameters On Redirect See: https://mantis.ilias.de/view.php?id=46106 * Test: Preserve Keys When Transforming Arrays See: https://mantis.ilias.de/view.php?id=46128 * Test: Add Login to Scoring by Question Table See: https://mantis.ilias.de/view.php?id=45304 * Test: Show Warning on Import Invalid Question File See: https://mantis.ilias.de/view.php?id=46037 * Adds null check for points in corrections * Fixed Mantis #45826: Text ist sehr nichtssagend in EN and DE * 0046165: SCORM: Trying to access certificate template configuration crashes for SCORM repository objects (ILIAS-eLearning#10439) * SCORM Debug Tool without YUI2 * check if array key exists * Scorm: Change sequence of parameters when transforming datetime (cherry-picked from release_9: 1111a6e) * Possible type error fixed (https://mantis.ilias.de/view.php?id=45306) * Type error fixed (https://mantis.ilias.de/view.php?id=45429) * Revert "Scorm: Change sequence of parameters when transforming datetime (ILIAS-eLearning#9825)" This reverts commit 2be2ba5. * Mantis #45149 - Fix Scorm1.2 favicons again for Ilias9 * Scorm: Change sequence of parameters when transforming datetime * Type fixes (#40842) * Fix 0046165: SCORM: Trying to access certificate template configuration crashes for SCORM repository objects * Tracking: fix export of matrix table (46182) * Contact: Remove invalid `ilCtrl` route annotation * dc fix presentation of deleted/unvisible il refs (ILIAS-eLearning#10447) * 45933: Not possible to exclude usages in history from ilObjMediaObject::_getMobsOfObject * Test: Remove ilCtrl-Annotation for Invalid Class * Test: import throws "Undefined variable" (46030) https://mantis.ilias.de/view.php?id=46030 * Test: Do Not Try to Migrate Invalid AddTime See: https://mantis.ilias.de/view.php?id=46198 * Chatroom: Bumb socket.io-client * copage: fixed type * Saml: stringify ILIAS\Data\URI * 45974: Error Call to a member function getTitle() on null when deleting user accounts * 45896: Insert Interactive-Image, 'Attempt to read property nodeName on null' * 46222: Failed test: Begriff hinzufügen * 11/Badge/46220 (ILIAS-eLearning#10471) * UI: 45648, add padding to asterisks and adjust property_form template. (ILIAS-eLearning#10144) https://mantis.ilias.de/view.php?id=45648 * Forum/Import: Deriver parent id from "Nested Set" structured as fallback See: https://mantis.ilias.de/view.php?id=46246 * fixed 46200: Screen ID mit \ im Namen lassen sich nicht nutzen; 46255: Manche Screen ID funktionieren nicht * 45742: Accordions containing Content-Snippets crash ILIAS when clicked in the page editor * Object/InfoScreen: fix presentation of precondition operator (46224) * Init: Silence ilCtrl error only for baseClass issues * Init/PasswordAssistance: Fix missing checks of second email address against `null` * [FIX] UICore: wrong removals of `ilCtrlStructure` entries. (ILIAS-eLearning#10428) * Bump symfony/http-foundation from 6.4.24 to 6.4.29 Bumps [symfony/http-foundation](https://github.com/symfony/http-foundation) from 6.4.24 to 6.4.29. - [Release notes](https://github.com/symfony/http-foundation/releases) - [Changelog](https://github.com/symfony/http-foundation/blob/7.3/CHANGELOG.md) - [Commits](symfony/http-foundation@v6.4.24...v6.4.29) --- updated-dependencies: - dependency-name: symfony/http-foundation dependency-version: 6.4.29 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> * BT: Fix exmpty user specific data handling during export * Fixes custom user column data not being shown * Drop wrong files from previous commit 99abe2c * Removed unnecessary code parts * Fixed Mantis #41922: Message regarding missing date for course period or availability period is misleading * Fixed Mantis #46214: Name des Repositoriums --> Name der Installation * Improves news renderer factory for future use * 46130 fix update when custom language file is installed Co-authored-by: Keven Clausen <keven.clausen@kroeplin-projekte.de> * Init: Improve exception handling/message * only load single record fields on require (ILIAS-eLearning#10534) * fix: icons in object releases tables In ILIAS 9 Fabian Kruse found an error in this table inside the configuration about LTI. It was reported in Mantis https://mantis.ilias.de/view.php?id=40731. After some testing the same error was found in ILIAS 10. This change fix the icon in that table. * Session: preload registration for access checks (45807) * fix cp notice * remove artifacts of tableview order (ILIAS-eLearning#10554) * dc remove old execption artifacts (ILIAS-eLearning#10558) * Restructure devguide. Unnecessary folder tutorial removed. * Rename devguide to tutorial * Bump js-yaml Bumps and [js-yaml](https://github.com/nodeca/js-yaml). These dependencies needed to be updated together. Updates `js-yaml` from 4.1.0 to 4.1.1 - [Changelog](https://github.com/nodeca/js-yaml/blob/master/CHANGELOG.md) - [Commits](nodeca/js-yaml@4.1.0...4.1.1) Updates `js-yaml` from 3.14.1 to 3.14.2 - [Changelog](https://github.com/nodeca/js-yaml/blob/master/CHANGELOG.md) - [Commits](nodeca/js-yaml@4.1.0...4.1.1) --- updated-dependencies: - dependency-name: js-yaml dependency-version: 4.1.1 dependency-type: indirect - dependency-name: js-yaml dependency-version: 3.14.2 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> * Repair Italian language file by copying and synchronizing from release_9. * Fixed Mantis #44894 and #44908 - label improvements to support OER * Test: Fix Missing Array Index in Settings See: https://mantis.ilias.de/view.php?id=46384 * Fixes issue with being undefinded when time runs out in test while working on it in multiple tabs * Removes byline for maximum duration of test in minutes * Fixes empty plugin name not being recognized as question type available * Streamline plugin question type discovery based on type tag and plugin attribute * Fixes issue with divisible by error * add aria-describedby attribute to file wizard input (ILIAS-eLearning#10153) * fix copyright doc (ILIAS-eLearning#10589) * [FIX] #46364 UICore: type-error inside `ilCtrlExistingPath::ensureValidCidPath()`. * Fixes https://mantis.ilias.de/view.php?id=46364 * Always pass an array to `in_array()` * Improves participant data export and import and also fixes inconsistent participant naming in different tables * Adds fix for issue found during review * Object: Always Load LangModule For Repository See: https://mantis.ilias.de/view.php?id=46299 * [FIX] YUI: remove component and all includes. (ILIAS-eLearning#9331) * remove last remains of yui (ILIAS-eLearning#10596) * Form: make autocomplete work for multi-text inputs (44429) (ILIAS-eLearning#9181) * AccessControl: Fix Autocomplete for Owner See: https://mantis.ilias.de/view.php?id=46423 * Forum: Fix behavior of first/root post in Thread * User: Fix vCard Download See: https://mantis.ilias.de/view.php?id=46357 * OnScreenChat: Fix sync issue with receiveMessage & onHistory * AccessControl: Fix Log Table See: https://mantis.ilias.de/view.php?id=46403 * Clear errors immediately after getting errors * Test: Fix Export Results With time See: https://mantis.ilias.de/view.php?id=46434 * dc fix table name limitation (ILIAS-eLearning#10618) * Maps: stop ServiceOpenLayers.js from overwriting Map (46368) (ILIAS-eLearning#10594) (ILIAS-eLearning#10622) Co-authored-by: Tim Schmitz <104776863+schmitz-ilias@users.noreply.github.com> * dc preserve table view on select (ILIAS-eLearning#10627) * Test: Fix Shuffle&Lock Constraint See: https://mantis.ilias.de/view.php?id=46384#c120712 * Test: Fix Shuffle&Lock Constraint See: https://mantis.ilias.de/view.php?id=46384#c120712 * Test: Show Intermediate Solution in Error-Text See: https://mantis.ilias.de/view.php?id=46218 * User: Add a Space in E-Mail Change E-Mail See: https://mantis.ilias.de/view.php?id=46149 * Sets show manual scoring to true for manual scoring by question * Test: Fix Essay-Quetion too Small See: https://mantis.ilias.de/view.php?id=46482 * Adds proper error message when submitting answer while already locked * Test: Disable QPL-Creation on Missing Rights See: https://mantis.ilias.de/view.php?id=46481 * fix: ilLTIAppEventListener changed Due to the rename of the LTI module to LTIProvider the class dedicated to detect the events was obsolete. After the last updates of ILIAS 10, ILIAS is able to detect when one module is listening, but it doesn't have the proper listener class. For that reason, we include this new class in order to have again the listener ready. Without this change the LTI connection is impossible, due to the error that ILIAS raise. This was detected in the report [0046456](https://mantis.ilias.de/bug_update_page.php) * fix: ilLTIAppEventListener changed Due to the rename of the LTI module to LTIProvider the class dedicated to detect the events was obsolete. * [FIX] exception message on wrong parameters * [FIX] Authorization Header Forwarding, see PR ILIAS-eLearning#10524 commit 1964748e363b83bf8da488dd92d5a973f0874c6a Author: Sergio Santiago Sánchez <54277294+sergiosant02@users.noreply.github.com> Date: Fri Nov 28 09:16:02 2025 +0100 fix: update comment in .httpaccess commit 704909e6bb0e15e253c13e5a89487bd60f73d3ca Author: Sergio Santiago <sergiosantiago0403@gmail.com> Date: Thu Nov 27 11:01:12 2025 +0100 fix: change comment in order to specify the reason of each option in httpaccess commit 10f23c86ec3e9e8d9e4deb80556785582d898263 Author: Sergio Santiago <sergiosantiago0403@gmail.com> Date: Mon Oct 20 08:17:28 2025 +0200 fix: Allow Authorization header to be passed through Apache for LTI 1.1 support Added SetEnvIf directive in .htaccess to expose the Authorization header to PHP. By default, Apache may strip or block the Authorization header, which prevents ILIAS from receiving OAuth credentials required for LTI 1.1 authentication and learning progress transmission. This change ensures that the header is passed through to the PHP environment as HTTP_AUTHORIZATION, enabling proper LTI request handling. The configuration aligns with ILIAS 10’s .htaccess setup and also maintains the existing WebDAV-specific directive added in ILIAS-eLearning@a77578c. # Conflicts: # components/ILIAS/Init/resources/.htaccess * add returntype never to http close * FileInfo: make sprintf robust against invalid translation placeholders (safeSprintf fallback) * Merge pull request ILIAS-eLearning#10631 from QualitusHartwig/9/bug/38803 Fix Mantis #38803: Contradictory SCORM Learning Progress * Cherry picked language improvements for Badges * Test: Fix Print View See: https://mantis.ilias.de/view.php?id=46552 * User: Fix Links in Role Lists See: https://mantis.ilias.de/view.php?id=46252 * Apply offline flag when creating forum via addObject (ILIAS-eLearning#10701) Apply offline flag when creating forum via addObject * Admin: fix lang vars (#46537) https://mantis.ilias.de/view.php?id=46537 * Revert "Admin: fix lang vars (#46537)" This reverts commit 544ce2e. * Test: Fix Downloads of UploadFiles See: https://mantis.ilias.de/view.php?id=46554 * NW: Performance optimization (ILIAS-eLearning#10625) * refactor(news): introduce data transfer objects and factory * refactor(news): introduce UserContextResolver and cache stubs * refactor(news): introduce NewsAggregator and aggregation strategies * refactor: cleanup NewsContext object and improve aggregation * refactor(news): separate recursive and iterative strategies for context aggregation * refactor(news): restructure cache layer and introduce NewsCollectionService * refactor(news): introduce methods to retain legacy compatibility * refactor(news): introduce NewsRepository * refactor(news): integrate service and data structures in NewsBlockGUI-classes * refactor(news): implement NewsCache and introduce optimized subset cache lookup * refactor(news): some bug fixes and remove usage of legacy methods * refactor(news): fixes cache key overflow bug, fixes missing news criteria bug, fixes file aggregation bug * refactor(news): introduce lazy loading collection to enhance caching end reduce memory consumption * refactor(news): remove legacy cache references * refactor(news): introduce news items database index * refactor(news): refactor timeline and mark legacy implementations as deprecated * refactor(news): fix missing user read status in lazy collection * refactor(news): fix potential cache memory overflow * fix: refactor code style and small improvements after review * fix: refactor queries and remove slow join operations * fix: remove index structure and improve cache overhead to reduce memory consumption * fix: wrong cache payload * fix: use iterative tree method to aggregate course and group items to avoid memory overflow --------- Co-authored-by: Lukas Eichenauer <leichenauer@databay.de> * NW: Fix cherry-pick from release_9 * Implements sync back to qpl for IPE Feedback questions * Fixes issue with item groups in page editor due to missing keys * [LTI] fix: show multiple LTI users in Authentication and Registration section (ILIAS-eLearning#10713) This change fix the report https://mantis.ilias.de/view.php?id=38537 which happens in ILIAS 9 and ILIAS 10. We should show every lti access in this section. * Fixes sizing issue with checkboxes and radio elements in print view of single and multiple choice questions * Improves some messages in the booking pool settings regarding user booking limits * Allows non owner with write permission to add new participants to booking pool inside of a course * Fixes cloze question changing answers and wrongly interpreting best answer * dc remove xlsx export notification (ILIAS-eLearning#10722) * dc add selected presentation to tableview (ILIAS-eLearning#10726) * dc set text detailed link to same view target (ILIAS-eLearning#10730) * Hides grade and passes status when test is yet unfinished * Reverts changes to ParticipantTable * Goes by default to last finished attempt not just newest attempt in test attempt overview * Displays result points in attempt overview * Redirects user after booking an item in bookingpool to week view * Improves redirect after booking of an item and adds missing language variables in booking pool * [FIX] array access * Chatroom: Fix message import on same installation See: https://mantis.ilias.de/view.php?id=46593 * Introduces own booking module language variable for participant * sync language files * FileUpload: Prohibit Inclusion of Foreign Objects Signed-off-by: Releasemanager <webmaster@ilias.de> * 0045883, 0045884, 0045900 Several BGT Issues Signed-off-by: Releasemanager <webmaster@ilias.de> * ilServer tika dependencies Signed-off-by: Releasemanager <webmaster@ilias.de> * Fix(SOAP): Add authorization check to getLearningProgressChanges. Signed-off-by: Releasemanager <webmaster@ilias.de> * Fix(SOAP): Add missing permission check to prevent moving objects from foreign containers Signed-off-by: Releasemanager <webmaster@ilias.de> * Fix(Soap): Add RBAC 'read' permission check to getSCORMCompletionStatus and hasSCORMCertificate to prevent unauthorized data access. Signed-off-by: Releasemanager <webmaster@ilias.de> * Release 10.4 --------- Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Releasemanager <webmaster@ilias.de> Co-authored-by: Marvin Beym <mBeym@databay.de> Co-authored-by: Sergio Santiago Sánchez <54277294+sergiosant02@users.noreply.github.com> Co-authored-by: mjansen <mjansen@databay.de> Co-authored-by: Lukas Scharmer <lscharmer@databay.de> Co-authored-by: Stephan Kergomard <webmaster@kergomard.ch> Co-authored-by: Alex Killing <killing@leifos.de> Co-authored-by: Tim Schmitz <schmitz@leifos.de> Co-authored-by: iszmais <45942348+iszmais@users.noreply.github.com> Co-authored-by: Christoph Ludolf <127395945+chlulei@users.noreply.github.com> Co-authored-by: Thomas Joußen <tjoussen@databay.de> Co-authored-by: Matheus Zych <mzych@databay.de> Co-authored-by: Aaron Bidzan <abidzan@databay.de> Co-authored-by: Matthias Kunkel <mkunkel@me.com> Co-authored-by: qualitus-dahme <dahme@qualitus.de> Co-authored-by: fneumann <fneumann@databay.de> Co-authored-by: Daniel Cazalla <86362063+ZallaxDev@users.noreply.github.com> Co-authored-by: Guido Vollbach <gvollbach@databay.de> Co-authored-by: Luka Stocker <67695434+lukastocker@users.noreply.github.com> Co-authored-by: Fabian Helfer <fhelfer@databay.de> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Keven Clausen <keven.clausen@kroeplin-projekte.de> Co-authored-by: Sergio Santiago <sergiosantiago0403@gmail.com> Co-authored-by: Fabian Wolf <wolf@ilias.de> Co-authored-by: Ahmed Hamouda <ahm3dhamouda@gmail.com> Co-authored-by: Thibeau Fuhrer <thibeau@sr.solutions> Co-authored-by: Tim Schmitz <104776863+schmitz-ilias@users.noreply.github.com> Co-authored-by: Fabian Schmid <fabian@sr.solutions> Co-authored-by: iszmais <iszmais@databay.de> Co-authored-by: Wolfgang Hübsch <wolfganghuebsch@users.noreply.github.com> Co-authored-by: Stefan Meyer <meyer@leifos.de> Co-authored-by: ingoj <120396930+ingoj@users.noreply.github.com> Co-authored-by: Sagun Karki <51822939+sKarki999@users.noreply.github.com> Co-authored-by: Lukas Eichenauer <leichenauer@databay.de> Co-authored-by: Releasemanager <webmaster@ilias.de> Co-authored-by: sagun <sagun.karki@minervis.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.
Sister PR to #10557