Maniphest T319521

Add Unicode test from tests/All.js and fix it in php
Open, LowPublic
Actions

Assigned To

Authored By

	Stegmujo
	Oct 6 2022, 10:40 AM

Description

Currently, the expression in Unicode test is not recognized by the regular expressions in PHP TexVC.
A Syntax Error is thrown. If its necessary to forward such expressions through MediaWiki then the regular expressions / check functions should be updated.

Here is a gist which extracts the error case and some variations of the check without having to modify the grammar:
https://gist.github.com/Hyper-Node/04a8c5c9abf3c28a140ecd0708e9f029

Check if the Testcase is necessary to cover in MediaWiki scenario (are unicode utf-16 chars forwarded to TexVC PHP?)
If yes update regex or check functions

Details

	Subject	Repo	Branch	Lines +/-
	Remove unreachable grammar rules for 4 byte chars	mediawiki/extensions/Math	master	+506 -551

Customize query in gerrit

Related Objects
Search...

Status	Assigned	Task
Declined	Stegmujo	T310373 Make Mathoid Client a standalone class
Open	None	T327098 Implement Math Accessibility Features necessary for Intent attribute usage
Resolved	Physikerwelt	T302628 Implement native MathML rendering in Math
Resolved	Stegmujo	T312528 Convert Grammar related javascript to php in texvcjs
Resolved	Stegmujo	T315978 Reminder Final Checks in Implementation
Open	Physikerwelt	T319521 Add Unicode test from tests/All.js and fix it in php

Event Timeline

Stegmujo created this task.Oct 6 2022, 10:40 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptOct 6 2022, 10:40 AM

Stegmujo added a parent task: T315978: Reminder Final Checks in Implementation.Oct 6 2022, 10:41 AM

Stegmujo updated the task description. (Show Details)Oct 6 2022, 10:44 AM

Stegmujo renamed this task from Add Unicode test from tests/All.js to Add Unicode test from tests/All.js and fix it in php.Oct 6 2022, 11:49 AM

Physikerwelt triaged this task as Low priority.Nov 30 2023, 9:42 AM

Physikerwelt claimed this task.Jul 4 2024, 6:16 PM

Change #1052168 had a related patch set uploaded (by Physikerwelt; author: Physikerwelt):

[mediawiki/extensions/Math@master] Remove unreachable grammar rules for 4 byte chars

https://gerrit.wikimedia.org/r/1052168

gerritbot added a project: Patch-For-Review.Jul 4 2024, 9:26 PM

Change #1052168 merged by jenkins-bot: