wikimedia/mediawiki-extensions-RSS
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/RSS synced 2024-11-23 15:26:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

Replace isset() with null check

Browse source 
isset() should only be used to suppress errors, not for null check.
When the variable is always defined, there is no need to use isset.
Found by a new phan plugin (2efea9f989)
https://www.mediawiki.org/wiki/Manual:Coding_conventions/PHP#isset

Change-Id: I4d3ebaf7fd3fc9613decd74fd36cc98582519a7c
This commit is contained in:
Umherirrender 2024-10-27 13:17:13 +01:00
parent 5404ab51f8
commit 3f09c46cee
1 changed files with 7 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
includes/RSSParser.php
View file

@ -30,9 +30,9 @@ class RSSParser {
protected $filter = [];
/** @var string[] */
protected $filterOut = [];
/** @var string */
/** @var string|null */
protected $itemTemplate;
/** @var string */
/** @var string|null */
protected $url;
/** @var string */
protected $etag;
@ -58,7 +58,7 @@ class RSSParser {
private $parserFactory;
/**
* @var RSSData
* @var RSSData|null
*/
public $rss;
@ -204,7 +204,7 @@ class RSSParser {
* @return Status object
*/
public function fetch() {
if ( !isset( $this->url ) ) {
if ( $this->url === null ) {
return Status::newFatal( 'rss-fetch-nourl' );
}
@ -266,7 +266,7 @@ class RSSParser {
protected function storeInCache( $key ) {
global $wgRSSCacheAge;
if ( !isset( $this->rss ) ) {
if ( $this->rss === null ) {
return false;
}
@ -393,7 +393,7 @@ class RSSParser {
$renderedFeed = '';
$wikitextFeed = '';
if ( isset( $this->itemTemplate ) && isset( $parser ) && isset( $frame ) ) {
if ( $this->itemTemplate !== null && $parser && $frame ) {
$headcnt = 0;
if ( $this->reversed ) {
$this->rss->items = array_reverse( $this->rss->items );
@ -427,7 +427,7 @@ class RSSParser {
* @return mixed
*/
protected function renderItem( $item, $parser ) {
$renderedItem = $this->itemTemplate;
$renderedItem = $this->itemTemplate ?? '';
// $info will only be an XML element name, so we're safe using it.
// $item[$info] is handled by the XML parser --