Skip to content

Core compatibility with ILIAS: Add helpers for platform-specific paths and input rendering - #1843

Open
Saaweel wants to merge 9 commits into
maths:devfrom
surlabs:proposed-moodle-core-changes-from-ilias
Open

Core compatibility with ILIAS: Add helpers for platform-specific paths and input rendering#1843
Saaweel wants to merge 9 commits into
maths:devfrom
surlabs:proposed-moodle-core-changes-from-ilias

Conversation

@Saaweel

@Saaweel Saaweel commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Hi all,

As part of the ongoing STACKForILIAS 11.0.0 work, this PR adds a few small extension points to STACK core that allow ILIAS to reuse the same code without carrying local copies of Moodle-specific paths and rendering details.

The changes are intentionally minimal and preserve existing Moodle behaviour:

  • Add stack_plot_url_base() and stack_cors_scripts_dir() in locallib.php.
  • Use these helpers when rendering question-library plots and checking JSXGraph/Parsons styles.
  • Extend replace_validation_tags() with an optional custom validation value.
  • Add get_name() to the base input API.
  • Keep equivalence validation markup unchanged in Moodle, while using valid block markup outside Moodle.
  • Purge cached CAS results when their STACK-Maxima version is incompatible.

These changes are needed by STACKForILIAS, but are neutral for Moodle and reduce future divergence between both integrations.

Validated with PHP lint and on Moodle 5.2 using the STACK healthcheck, including CAS, cache, plots, JSXGraph and Parsons.

Kind regards,
Saúl
SURLABS

@Saaweel Saaweel changed the title Proposed moodle core changes from ilias Core compatibility with ILIAS: Add helpers for platform-specific paths and input rendering Aug 10, 2026
@sangwinc
sangwinc requested a review from EJMFarrow August 13, 2026 14:03
@sangwinc

Copy link
Copy Markdown
Member

Thanks @Saaweel that's really very helpful. We'll review these (when the holidays are over) and sort out some mechanism to implement the changes you need. Please let us know of anything similar.

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.

3 participants