golem
/
Math
9
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Merge "Fix MathFormatter failing on new SnakFormatter format MIME types"

This commit is contained in:
jenkins-bot 2018-04-13 16:11:03 +00:00 committed by Gerrit Code Review
parent 3291177420 1f87afa8e6
commit 0a8608399a
2 changed files with 8 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
MathFormatter.php
View File

@ -25,20 +25,9 @@ class MathFormatter implements ValueFormatter {
* Loads format to distinguish the type of formatting
*
* @param string $format One of the SnakFormatter::FORMAT_... constants.
*
* @throws InvalidArgumentException
*/
public function __construct( $format ) {
switch ( $format ) {
case SnakFormatter::FORMAT_PLAIN:
case SnakFormatter::FORMAT_WIKI:
case SnakFormatter::FORMAT_HTML:
case SnakFormatter::FORMAT_HTML_DIFF:
$this->format = $format;
break;
default:
throw new InvalidArgumentException( 'Unsupported output format: ' . $format );
}
$this->format = $format;
}
/**
@ -59,6 +48,8 @@ class MathFormatter implements ValueFormatter {
return $tex;
case SnakFormatter::FORMAT_WIKI:
return "<math>$tex</math>";
// Intentionally fall back to MathML output in all other, possibly unknown cases.
default:
$renderer = new MathMathML( $tex );
@ -72,7 +63,6 @@ class MathFormatter implements ValueFormatter {
$html = $this->formatDetails( $html, $tex );
}
// TeX string is not valid or rendering failed
return $html;
}
}

10
tests/MathFormatterTest.php
View File

@ -58,11 +58,11 @@ class MathFormatterTest extends MediaWikiTestCase {
$formatter->format( null );
}
/**
* @expectedException InvalidArgumentException
*/
public function testUnknownFormat() {
new MathFormatter( 'unknown/unknown' );
public function testUnknownFormatFallsBackToMathMl() {
$formatter = new MathFormatter( 'unknown/unknown' );
$value = new StringValue( self::SOME_TEX );
$resultFormat = $formatter->format( $value );
$this->assertContains( '</math>', $resultFormat );
}
public function testFormatPlain() {