The abstract class is documented to use null for doing nothing, not void
Change-Id: I99f0747117f83f4f2ae554c759a8e1737e7dc8f7
/**
* Print the history page for an article.
/**
* Print the history page for an article.
*/
function onView() {
$out = $this->getOutput();
*/
function onView() {
$out = $this->getOutput();
* Allow client caching.
*/
if ( $out->checkLastModified( $this->page->getTouched() ) ) {
* Allow client caching.
*/
if ( $out->checkLastModified( $this->page->getTouched() ) ) {
- return; // Client cache fresh and headers sent, nothing more to do.
+ return null; // Client cache fresh and headers sent, nothing more to do.
}
$this->preCacheMessages();
}
$this->preCacheMessages();
$feedType = $request->getRawVal( 'feed' );
if ( $feedType !== null ) {
$this->feed( $feedType );
$feedType = $request->getRawVal( 'feed' );
if ( $feedType !== null ) {
$this->feed( $feedType );
}
$ts = $this->getTimestampFromRequest( $request );
}
$ts = $this->getTimestampFromRequest( $request );
$pager->getNavigationBar()
);
$out->preventClickjacking( $pager->getPreventClickjacking() );
$pager->getNavigationBar()
);
$out->preventClickjacking( $pager->getPreventClickjacking() );
/**
* @suppress SecurityCheck-XSS Non html mime type
/**
* @suppress SecurityCheck-XSS Non html mime type
*/
function onView() {
$this->getOutput()->disable();
*/
function onView() {
$this->getOutput()->disable();
$config = $this->context->getConfig();
if ( !$request->checkUrlExtension() ) {
$config = $this->context->getConfig();
if ( !$request->checkUrlExtension() ) {
}
if ( $this->getOutput()->checkLastModified( $this->page->getTouched() ) ) {
}
if ( $this->getOutput()->checkLastModified( $this->page->getTouched() ) ) {
- return; // Client cache fresh and headers sent, nothing more to do.
+ return null; // Client cache fresh and headers sent, nothing more to do.
}
$contentType = $this->getContentType();
}
$contentType = $this->getContentType();