Merge "Add separate database table for LaTeXML (step 3)"

This commit is contained in:
Frédéric Wang 2014-05-27 15:49:42 +00:00 committed by Gerrit Code Review
commit 423d80ef35
5 changed files with 32 additions and 14 deletions

View File

@ -192,7 +192,7 @@ class MathLaTeXML extends MathRenderer {
/**
* Picks a LaTeXML daemon.
* If more than one demon are availible one is chosen from the
* If more than one daemon are available one is chosen from the
* $wgMathLaTeXMLUrl array.
* @return string
*/
@ -347,4 +347,8 @@ class MathLaTeXML extends MathRenderer {
return Xml::tags( 'span', $attribs, $mml );
}
protected function getMathTableName() {
return 'mathlatexml';
}
}

View File

@ -496,5 +496,12 @@ abstract class MathRenderer {
public function getUserInputTex() {
return $this->userInputTex;
}
}
protected abstract function getMathTableName();
public function getModeStr() {
$names = MathHooks::getMathNames();
return $names[ $this->getMode() ];
}
}

View File

@ -46,4 +46,7 @@ class MathSource extends MathRenderer {
);
}
protected function getMathTableName() {
throw new MWException ( 'in math source mode no database caching should happen');
}
}

View File

@ -360,4 +360,8 @@ class MathTexvc extends MathRenderer {
return false;
}
protected function getMathTableName() {
return 'math';
}
}

View File

@ -26,7 +26,7 @@ class MathRendererTest extends MediaWikiTestCase {
*/
public function testWriteCacheSkip() {
$renderer = $this->getMockBuilder( 'MathRenderer' )
->setMethods( array( 'writeToDatabase' , 'render' ) )
->setMethods( array( 'writeToDatabase' , 'render', 'getMathTableName' ) )
->disableOriginalConstructor()
->getMock();
$renderer->expects( $this->never() )
@ -40,7 +40,7 @@ class MathRendererTest extends MediaWikiTestCase {
*/
public function testWriteCache() {
$renderer = $this->getMockBuilder( 'MathRenderer' )
->setMethods( array( 'writeToDatabase' , 'render' ) )
->setMethods( array( 'writeToDatabase' , 'render', 'getMathTableName' ) )
->disableOriginalConstructor()
->getMock();
$renderer->expects( $this->never() )
@ -54,7 +54,7 @@ class MathRendererTest extends MediaWikiTestCase {
*/
public function testChangeHash() {
$renderer = $this->getMockBuilder( 'MathRenderer' )
->setMethods( array( 'render' ) )
->setMethods( array( 'render', 'getMathTableName' ) )
->disableOriginalConstructor()
->getMock();
$this->assertEquals(
@ -71,7 +71,7 @@ class MathRendererTest extends MediaWikiTestCase {
public function testSetPurge() {
$renderer = $this->getMockBuilder( 'MathRenderer' )
->setMethods( array( 'render' ) )
->setMethods( array( 'render', 'getMathTableName' ) )
->disableOriginalConstructor()
->getMock();
$renderer->setPurge();