* same as their serialized form. Examples would be JavaScript and CSS code. As of now,
* this also applies to wikitext (mediawiki's default content type), but wikitext
* content may be represented by a DOM or AST structure in the future.
- *
+ *
+ * TODO: add documentation
*/
abstract class ContentHandler {
+ /**
+ * @abstract
+ * @param Content $content
+ * @param null $format
+ * @return String
+ */
+ public abstract function serialize( Content $content, $format = null );
+
+ /**
+ * TODO: calling unserialize on a ContentHandler returns a Content?!! Something looks wrong here...
+ *
+ * @abstract
+ * @param $blob String
+ * @param null $format
+ * @return Content
+ */
+ public abstract function unserialize( $blob, $format = null );
+
+ /**
+ * FIXME: bad method name: suggests it empties the content of an instance rather then creating a new empty one
+ */
+ public abstract function emptyContent();
+
public static function getContentText( Content $content = null ) {
global $wgContentHandlerTextFallback;
- if ( !$content ) return '';
+ if ( is_null( $content ) ) {
+ return '';
+ }
if ( $content instanceof TextContent ) {
return $content->getNativeData();
}
- if ( $wgContentHandlerTextFallback == 'fail' ) throw new MWException( "Attempt to get text from Content with model " . $content->getModelName() );
- if ( $wgContentHandlerTextFallback == 'serialize' ) return $content->serialize();
+ if ( $wgContentHandlerTextFallback == 'fail' ) {
+ throw new MWException( "Attempt to get text from Content with model " . $content->getModelName() );
+ }
+
+ if ( $wgContentHandlerTextFallback == 'serialize' ) {
+ return $content->serialize();
+ }
return null;
}
public static function makeContent( $text, Title $title, $modelName = null, $format = null ) {
- if ( !$modelName ) {
+
+ if ( is_null( $modelName ) ) {
$modelName = $title->getContentModelName();
}
public static function getDefaultModelFor( Title $title ) {
global $wgNamespaceContentModels;
- # NOTE: this method must not rely on $title->getContentModelName() directly or indirectly,
- # because it is used to initialized the mContentModelName memebr.
+ // NOTE: this method must not rely on $title->getContentModelName() directly or indirectly,
+ // because it is used to initialized the mContentModelName memebr.
$ns = $title->getNamespace();
$model = $wgNamespaceContentModels[ $ns ];
}
- # hook can determin default model
+ // hook can determin default model
if ( !wfRunHooks( 'DefaultModelFor', array( $title, &$model ) ) ) { #FIXME: document new hook!
- if ( $model ) return $model;
+ if ( !is_null( $model ) ) {
+ return $model;
+ }
}
- # Could this page contain custom CSS or JavaScript, based on the title?
- $isCssOrJsPage = ( NS_MEDIAWIKI == $ns && preg_match( "!\.(css|js)$!u", $title->getText(), $m ) );
- if ( $isCssOrJsPage ) $ext = $m[1];
+ // Could this page contain custom CSS or JavaScript, based on the title?
+ $isCssOrJsPage = NS_MEDIAWIKI == $ns && preg_match( '!\.(css|js)$!u', $title->getText(), $m );
+ if ( $isCssOrJsPage ) {
+ $ext = $m[1];
+ }
- # hook can force js/css
- wfRunHooks( 'TitleIsCssOrJsPage', array( $title, &$isCssOrJsPage, &$ext ) ); #FIXME: add $ext to hook interface spec
+ // hook can force js/css
+ wfRunHooks( 'TitleIsCssOrJsPage', array( $title, &$isCssOrJsPage ) );
- # Is this a .css subpage of a user page?
- $isJsCssSubpage = ( NS_USER == $ns && !$isCssOrJsPage && preg_match( "/\\/.*\\.(js|css)$/", $title->getText(), $m ) );
- if ( $isJsCssSubpage ) $ext = $m[1];
+ // Is this a .css subpage of a user page?
+ $isJsCssSubpage = NS_USER == $ns && !$isCssOrJsPage && preg_match( "/\\/.*\\.(js|css)$/", $title->getText(), $m );
+ if ( $isJsCssSubpage ) {
+ $ext = $m[1];
+ }
- # is this wikitext, according to $wgNamespaceContentModels or the DefaultModelFor hook?
- $isWikitext = ( $model == CONTENT_MODEL_WIKITEXT || $model === null );
- $isWikitext = ( $isWikitext && !$isCssOrJsPage && !$isJsCssSubpage );
+ // is this wikitext, according to $wgNamespaceContentModels or the DefaultModelFor hook?
+ $isWikitext = is_null( $model ) || $model == CONTENT_MODEL_WIKITEXT;
+ $isWikitext = $isWikitext && !$isCssOrJsPage && !$isJsCssSubpage;
- # hook can override $isWikitext
+ // hook can override $isWikitext
wfRunHooks( 'TitleIsWikitextPage', array( $title, &$isWikitext ) );
if ( !$isWikitext ) {
-
- if ( $ext == 'js' )
- return CONTENT_MODEL_JAVASCRIPT;
- else if ( $ext == 'css' )
- return CONTENT_MODEL_CSS;
-
- if ( $model )
- return $model;
- else
- return CONTENT_MODEL_TEXT;
+ switch ( $ext ) {
+ case 'js':
+ return CONTENT_MODEL_JAVASCRIPT;
+ case 'css':
+ return CONTENT_MODEL_CSS;
+ default:
+ return is_null( $model ) ? CONTENT_MODEL_TEXT : $model;
+ }
}
- # we established that is must be wikitext
+ // we established that is must be wikitext
+
return CONTENT_MODEL_WIKITEXT;
}
if ( empty( $wgContentHandlers[$modelName] ) ) {
$handler = null;
- wfRunHooks( "ContentHandlerForModelName", array( $modelName, &$handler ) ); #FIXME: document new hook
- if ( $handler ) { # NOTE: may be a string or an object, either is fine!
+ // TODO: document new hook
+ wfRunHooks( 'ContentHandlerForModelName', array( $modelName, &$handler ) );
+
+ if ( $handler ) { // NOTE: may be a string or an object, either is fine!
$wgContentHandlers[$modelName] = $handler;
} else {
throw new MWException( "No handler for model $modelName registered in \$wgContentHandlers" );
return $wgContentHandlers[$modelName];
}
- # ----------------------------------------------------------------------------------------------------------
+ // ----------------------------------------------------------------------------------------------------------
public function __construct( $modelName, $formats ) {
$this->mModelName = $modelName;
$this->mSupportedFormats = $formats;
}
public function getModelName() {
- # for wikitext: wikitext; in the future: wikiast, wikidom?
- # for wikidata: wikidata
+ // for wikitext: wikitext; in the future: wikiast, wikidom?
+ // for wikidata: wikidata
return $this->mModelName;
}
}
public function getSupportedFormats() {
- # for wikitext: "text/x-mediawiki-1", "text/x-mediawiki-2", etc
- # for wikidata: "application/json", "application/x-php", etc
+ // for wikitext: "text/x-mediawiki-1", "text/x-mediawiki-2", etc
+ // for wikidata: "application/json", "application/x-php", etc
return $this->mSupportedFormats;
}
}
public function isSupportedFormat( $format ) {
- if ( !$format ) return true; # this means "use the default"
+
+ if ( !$format ) {
+ return true; // this means "use the default"
+ }
return in_array( $format, $this->mSupportedFormats );
}
}
}
- /**
- * @abstract
- * @param Content $content
- * @param null $format
- * @return String
- */
- public abstract function serialize( Content $content, $format = null );
-
- /**
- * @abstract
- * @param $blob String
- * @param null $format
- * @return Content
- */
- public abstract function unserialize( $blob, $format = null );
-
- public abstract function emptyContent();
-
/**
* Return an Article object suitable for viewing the given object
*
* NOTE: does *not* do special handling for Image and Category pages!
* Use Article::newFromTitle() for that!
*
- * @param type $title
- * @return \Article
+ * @param Title $title
+ * @return Article
* @todo Article is being refactored into an action class, keep track of that
*/
public function createArticle( Title $title ) {
/**
* Return an EditPage object suitable for editing the given object
- *
- * @param type $article
- * @return \EditPage
+ *
+ * @param Article $article
+ * @return EditPage
*/
public function createEditPage( Article $article ) {
- $this->checkModelName( $article->getContentObject()->getModelName() );
+ $this->checkModelName( $article->getContentModelName() );
$editPage = new EditPage( $article );
return $editPage;
/**
* Return an ExternalEdit object suitable for editing the given object
*
- * @param type $article
- * @return \ExternalEdit
+ * @param IContextSource $context
+ * @return ExternalEdit
*/
public function createExternalEdit( IContextSource $context ) {
$this->checkModelName( $context->getTitle()->getModelName() );
* @param $rcid Integer ??? FIXME (default 0)
* @param $refreshCache boolean If set, refreshes the diff cache
* @param $unhide boolean If set, allow viewing deleted revs
+ *
+ * @return DifferenceEngine
*/
public function getDifferenceEngine( IContextSource $context, $old = 0, $new = 0, $rcid = 0, #FIMXE: use everywhere!
$refreshCache = false, $unhide = false ) {
$this->checkModelName( $context->getTitle()->getModelName() );
- $de = new DifferenceEngine( $context, $old, $new, $rcid, $refreshCache, $unhide );
+ $diffEngineClass = $this->getDiffEngineClass();
- return $de;
+ return new $diffEngineClass( $context, $old, $new, $rcid, $refreshCache, $unhide );
}
+ /**
+ * Returns the name of the diff engine to use.
+ *
+ * @since 0.1
+ *
+ * @return string
+ */
+ protected function getDiffEngineClass() {
+ return 'DifferenceEngine';
+ }
+
/**
* attempts to merge differences between three versions.
* Returns a new Content object for a clean merge and false for failure or a conflict.
/**
* Return an applicable autosummary if one exists for the given edit.
*
- * @param $oldContent Content: the previous text of the page.
- * @param $newContent Content: The submitted text of the page.
+ * @param $oldContent Content|null: the previous text of the page.
+ * @param $newContent Content|null: The submitted text of the page.
* @param $flags Int bitmask: a bitmask of flags submitted for the edit.
*
* @return string An appropriate autosummary, or an empty string.
*/
- public function getAutosummary( Content $oldContent, Content $newContent, $flags ) {
+ public function getAutosummary( Content $oldContent = null, Content $newContent = null, $flags ) {
global $wgContLang;
- # Decide what kind of autosummary is needed.
+ // Decide what kind of autosummary is needed.
+
+ // Redirect autosummaries
- # Redirect autosummaries
- $ot = $oldContent->getRedirectTarget();
- $rt = $newContent->getRedirectTarget();
+ $ot = !empty( $ot ) ? $oldContent->getRedirectTarget() : false;
+ $rt = !empty( $rt ) ? $newContent->getRedirectTarget() : false;
if ( is_object( $rt ) && ( !is_object( $ot ) || !$rt->equals( $ot ) || $ot->getFragment() != $rt->getFragment() ) ) {
return wfMsgForContent( 'autoredircomment', $rt->getFullText(), $truncatedtext );
}
- # New page autosummaries
+ // New page autosummaries
if ( $flags & EDIT_NEW && $newContent->getSize() > 0 ) {
- # If they're making a new article, give its text, truncated, in the summary.
+ // If they're making a new article, give its text, truncated, in the summary.
$truncatedtext = $newContent->getTextForSummary(
200 - strlen( wfMsgForContent( 'autosumm-new' ) ) );
return wfMsgForContent( 'autosumm-new', $truncatedtext );
}
- # Blanking autosummaries
+ // Blanking autosummaries
if ( $oldContent->getSize() > 0 && $newContent->getSize() == 0 ) {
return wfMsgForContent( 'autosumm-blank' );
} elseif ( $oldContent->getSize() > 10 * $newContent->getSize() && $newContent->getSize() < 500 ) {
- # Removing more than 90% of the article
+ // Removing more than 90% of the article
$truncatedtext = $newContent->getTextForSummary(
200 - strlen( wfMsgForContent( 'autosumm-replace' ) ) );
return wfMsgForContent( 'autosumm-replace', $truncatedtext );
}
- # If we reach this point, there's no applicable autosummary for our case, so our
- # autosummary is empty.
+ // If we reach this point, there's no applicable autosummary for our case, so our
+ // autosummary is empty.
+
return '';
}
* if no revision occurred
*/
public function getAutoDeleteReason( Title $title, &$hasHistory ) {
- global $wgContLang;
-
- try {
- $dbw = wfGetDB( DB_MASTER );
+ $dbw = wfGetDB( DB_MASTER );
- // Get the last revision
- $rev = Revision::newFromTitle( $title );
+ // Get the last revision
+ $rev = Revision::newFromTitle( $title );
- if ( is_null( $rev ) ) {
- return false;
- }
+ if ( is_null( $rev ) ) {
+ return false;
+ }
- // Get the article's contents
- $content = $rev->getContent();
- $blank = false;
+ // Get the article's contents
+ $content = $rev->getContent();
+ $blank = false;
- // If the page is blank, use the text from the previous revision,
- // which can only be blank if there's a move/import/protect dummy revision involved
- if ( $content->getSize() == 0 ) {
- $prev = $rev->getPrevious();
+ // If the page is blank, use the text from the previous revision,
+ // which can only be blank if there's a move/import/protect dummy revision involved
+ if ( $content->getSize() == 0 ) {
+ $prev = $rev->getPrevious();
- if ( $prev ) {
- $content = $rev->getContent();
- $blank = true;
- }
+ if ( $prev ) {
+ $content = $rev->getContent();
+ $blank = true;
}
+ }
- // Find out if there was only one contributor
- // Only scan the last 20 revisions
- $res = $dbw->select( 'revision', 'rev_user_text',
- array( 'rev_page' => $title->getArticleID(), $dbw->bitAnd( 'rev_deleted', Revision::DELETED_USER ) . ' = 0' ),
- __METHOD__,
- array( 'LIMIT' => 20 )
- );
-
- if ( $res === false ) {
- // This page has no revisions, which is very weird
- return false;
- }
+ // Find out if there was only one contributor
+ // Only scan the last 20 revisions
+ $res = $dbw->select( 'revision', 'rev_user_text',
+ array( 'rev_page' => $title->getArticleID(), $dbw->bitAnd( 'rev_deleted', Revision::DELETED_USER ) . ' = 0' ),
+ __METHOD__,
+ array( 'LIMIT' => 20 )
+ );
+
+ if ( $res === false ) {
+ // This page has no revisions, which is very weird
+ return false;
+ }
- $hasHistory = ( $res->numRows() > 1 );
- $row = $dbw->fetchObject( $res );
-
- if ( $row ) { // $row is false if the only contributor is hidden
- $onlyAuthor = $row->rev_user_text;
- // Try to find a second contributor
- foreach ( $res as $row ) {
- if ( $row->rev_user_text != $onlyAuthor ) { // Bug 22999
- $onlyAuthor = false;
- break;
- }
- }
- } else {
- $onlyAuthor = false;
- }
+ $hasHistory = ( $res->numRows() > 1 );
+ $row = $dbw->fetchObject( $res );
- // Generate the summary with a '$1' placeholder
- if ( $blank ) {
- // The current revision is blank and the one before is also
- // blank. It's just not our lucky day
- $reason = wfMsgForContent( 'exbeforeblank', '$1' );
- } else {
- if ( $onlyAuthor ) {
- $reason = wfMsgForContent( 'excontentauthor', '$1', $onlyAuthor );
- } else {
- $reason = wfMsgForContent( 'excontent', '$1' );
+ if ( $row ) { // $row is false if the only contributor is hidden
+ $onlyAuthor = $row->rev_user_text;
+ // Try to find a second contributor
+ foreach ( $res as $row ) {
+ if ( $row->rev_user_text != $onlyAuthor ) { // Bug 22999
+ $onlyAuthor = false;
+ break;
}
}
+ } else {
+ $onlyAuthor = false;
+ }
- if ( $reason == '-' ) {
- // Allow these UI messages to be blanked out cleanly
- return '';
+ // Generate the summary with a '$1' placeholder
+ if ( $blank ) {
+ // The current revision is blank and the one before is also
+ // blank. It's just not our lucky day
+ $reason = wfMsgForContent( 'exbeforeblank', '$1' );
+ } else {
+ if ( $onlyAuthor ) {
+ $reason = wfMsgForContent( 'excontentauthor', '$1', $onlyAuthor );
+ } else {
+ $reason = wfMsgForContent( 'excontent', '$1' );
}
+ }
- // Max content length = max comment length - length of the comment (excl. $1)
- $text = $content->getTextForSummary( 255 - ( strlen( $reason ) - 2 ) );
+ if ( $reason == '-' ) {
+ // Allow these UI messages to be blanked out cleanly
+ return '';
+ }
- // Now replace the '$1' placeholder
- $reason = str_replace( '$1', $text, $reason );
+ // Max content length = max comment length - length of the comment (excl. $1)
+ $text = $content->getTextForSummary( 255 - ( strlen( $reason ) - 2 ) );
- return $reason;
- } catch (MWException $e) {
- # if a page is horribly broken, we still want to be able to delete it. so be lenient about errors here.
- wfDebug("Error while building auto delete summary: $e");
- }
+ // Now replace the '$1' placeholder
+ $reason = str_replace( '$1', $text, $reason );
- return '';
+ return $reason;
}
/**
$undoafter_content = $undoafter->getContent();
if ( $cur_content->equals( $undo_content ) ) {
- # No use doing a merge if it's just a straight revert.
+ // No use doing a merge if it's just a straight revert.
return $undoafter_content;
}
return $undone_content;
}
-
- #TODO: how to handle extra message for JS/CSS previews??
- #TODO: Article::showCssOrJsPage ---> specialized classes!
-
- #XXX: ImagePage and CategoryPage... wrappers that use ContentHandler? or ContentHandler creates wrappers?
-
- #TODO: hook into dump generation to serialize and record model and format!
- #TODO: cover action=raw
- #TODO: make sure we cover lucene search / wikisearch.
- #TODO: nice&sane integration of GeSHi syntax highlighting
- # [11:59] <vvv> Hooks are ugly; make CodeHighlighter interface and a config to set the class which handles syntax highlighting
- # [12:00] <vvv> And default it to a DummyHighlighter
- #TODO: make sure we cover the external editor interface (does anyone actually use that?!)
}
$ok = wfMerge( $old, $mine, $yours, $result );
- if ( !$ok ) return false;
- if ( !$result ) return $this->emptyContent();
+ if ( !$ok ) {
+ return false;
+ }
+
+ if ( !$result ) {
+ return $this->emptyContent();
+ }
$mergedContent = $this->unserialize( $result, $format );
return $mergedContent;
}
public function emptyContent() {
- return new WikitextContent( "" );
+ return new WikitextContent( '' );
}
class JavaScriptContentHandler extends TextContentHandler {
public function __construct( $modelName = CONTENT_MODEL_WIKITEXT ) {
- parent::__construct( $modelName, array( 'text/javascript' ) );
+ parent::__construct( $modelName, array( 'text/javascript' ) ); #XXX: or use $wgJsMimeType? this is for internal storage, not HTTP...
}
public function unserialize( $text, $format = null ) {
}
public function emptyContent() {
- return new JavaScriptContent( "" );
+ return new JavaScriptContent( '' );
}
}
}
public function emptyContent() {
- return new CssContent( "" );
+ return new CssContent( '' );
}
}