From 370e83bb0dfd0c70de268c93bf07ad5ee0897192 Mon Sep 17 00:00:00 2001 From: Pierre Schmitz Date: Fri, 15 Aug 2008 01:29:47 +0200 Subject: Update auf 1.13.0 --- includes/api/ApiFormatBase.php | 51 +++++++++++++++++++++++++----------------- 1 file changed, 31 insertions(+), 20 deletions(-) (limited to 'includes/api/ApiFormatBase.php') diff --git a/includes/api/ApiFormatBase.php b/includes/api/ApiFormatBase.php index 768a18ac..db58fe52 100644 --- a/includes/api/ApiFormatBase.php +++ b/includes/api/ApiFormatBase.php @@ -30,12 +30,12 @@ if (!defined('MEDIAWIKI')) { /** * This is the abstract base class for API formatters. - * - * @addtogroup API + * + * @ingroup API */ abstract class ApiFormatBase extends ApiBase { - private $mIsHtml, $mFormat, $mUnescapeAmps, $mHelp; + private $mIsHtml, $mFormat, $mUnescapeAmps, $mHelp, $mCleared; /** * Create a new instance of the formatter. @@ -50,6 +50,7 @@ abstract class ApiFormatBase extends ApiBase { else $this->mFormat = $format; $this->mFormat = strtoupper($this->mFormat); + $this->mCleared = false; } /** @@ -62,7 +63,7 @@ abstract class ApiFormatBase extends ApiBase { /** * If formatter outputs data results as is, the results must first be sanitized. * An XML formatter on the other hand uses special tags, such as "_element" for special handling, - * and thus needs to override this function to return true. + * and thus needs to override this function to return true. */ public function getNeedsRawData() { return false; @@ -82,8 +83,8 @@ abstract class ApiFormatBase extends ApiBase { /** * Returns true when an HTML filtering printer should be used. - * The default implementation assumes that formats ending with 'fm' - * should be formatted in HTML. + * The default implementation assumes that formats ending with 'fm' + * should be formatted in HTML. */ public function getIsHtml() { return $this->mIsHtml; @@ -111,7 +112,7 @@ abstract class ApiFormatBase extends ApiBase { -mUnescapeAmps) { +mUnescapeAmps) { ?> MediaWiki API MediaWiki API Result @@ -127,7 +128,7 @@ abstract class ApiFormatBase extends ApiBase { You are looking at the HTML representation of the mFormat ); ?> format.
HTML is good for debugging, but probably is not suitable for your application.
-See complete documentation, or +See complete documentation, or API help for more information. complete documentation, or if ($this->getIsHtml()) echo $this->formatHTML($text); else + { + // For non-HTML output, clear all errors that might have been + // displayed if display_errors=On + // Do this only once, of course + if(!$this->mCleared) + { + ob_clean(); + $this->mCleared = true; + } echo $text; + } } /** @@ -175,7 +186,7 @@ See complete documentation, or public function setHelp( $help = true ) { $this->mHelp = true; } - + /** * Prety-print various elements in HTML format, such as xml tags and URLs. * This method also replaces any '<' with < @@ -188,16 +199,17 @@ See complete documentation, or $text = preg_replace('/\<(!--.*?--|.*?)\>/', '<\1>', $text); // identify URLs $protos = "http|https|ftp|gopher"; - $text = ereg_replace("($protos)://[^ \\'\"()<\n]+", '\\0', $text); + # This regex hacks around bug 13218 (" included in the URL) + $text = preg_replace("#(($protos)://.*?)(")?([ \\'\"()<\n])#", '\\1\\3\\4', $text); // identify requests to api.php - $text = ereg_replace("api\\.php\\?[^ \\()<\n\t]+", '\\0', $text); + $text = preg_replace("#api\\.php\\?[^ \\()<\n\t]+#", '\\0', $text); if( $this->mHelp ) { // make strings inside * bold $text = ereg_replace("\\*[^<>\n]+\\*", '\\0', $text); // make strings inside $ italic $text = ereg_replace("\\$[^<>\n]+\\$", '\\0', $text); } - + /* Temporary fix for bad links in help messages. As a special case, * XML-escaped metachars are de-escaped one level in the help message * for legibility. Should be removed once we have completed a fully-html @@ -220,13 +232,13 @@ See complete documentation, or } public static function getBaseVersion() { - return __CLASS__ . ': $Id: ApiFormatBase.php 30222 2008-01-28 19:05:26Z catrope $'; + return __CLASS__ . ': $Id: ApiFormatBase.php 36153 2008-06-10 15:20:22Z tstarling $'; } } /** - * This printer is used to wrap an instance of the Feed class - * @addtogroup API + * This printer is used to wrap an instance of the Feed class + * @ingroup API */ class ApiFormatFeedWrapper extends ApiFormatBase { @@ -275,13 +287,12 @@ class ApiFormatFeedWrapper extends ApiFormatBase { $feed->outItem($item); $feed->outFooter(); } else { - // Error has occured, print something usefull - // TODO: make this error more informative using ApiBase :: dieDebug() or similar - wfHttpError(500, 'Internal Server Error', ''); + // Error has occured, print something useful + ApiBase::dieDebug( __METHOD__, 'Invalid feed class/item' ); } } - + public function getVersion() { - return __CLASS__ . ': $Id: ApiFormatBase.php 30222 2008-01-28 19:05:26Z catrope $'; + return __CLASS__ . ': $Id: ApiFormatBase.php 36153 2008-06-10 15:20:22Z tstarling $'; } } -- cgit v1.2.2