TSP Posted June 9, 2015 Share Posted June 9, 2015 When the legacyParser rebuilds earlier posts half of the time is spent by HtmlPurifier in two calls that are made in system/Text/parser.php. Any particular issues that've caused you to do this? Do you consider both calls really necessary? public function parse( $value ) { /* HTMLPurifier - remove bad stuff and make sure HTML is well formed */ if ( $this->htmlPurifier !== NULL ) { $value = $this->htmlPurifier->purify( $value ); } ... /* HTMLPurifier again, to catch anything bad from bbcodes */ if ( $this->htmlPurifier !== NULL ) { $value = $this->htmlPurifier->purify( $value ); } Link to comment Share on other sites More sharing options...
TSP Posted June 27, 2015 Author Share Posted June 27, 2015 Any comments on this? It spends at least half the rebuild time on running the two instances of HTMLPurifier::purify(). Maybe there is a point in one of them, but it still seems weird to have it be two times. Link to comment Share on other sites More sharing options...
Recommended Posts
Archived
This topic is now archived and is closed to further replies.