Browse Source

bridges: Return parent::getName by default

logmanoriginal 7 years ago
parent
commit
c4169f1579

+ 5 - 5
bridges/AllocineFRBridge.php

@@ -44,11 +44,11 @@ class AllocineFRBridge extends BridgeAbstract {
 
 	public function getName(){
 		if(!is_null($this->getInput('category'))){
-		return self::NAME . ' : '
-			.array_search(
-				$this->getInput('category'),
-				self::PARAMETERS[$this->queriedContext]['category']['values']
-			);
+			return self::NAME . ' : '
+				.array_search(
+					$this->getInput('category'),
+					self::PARAMETERS[$this->queriedContext]['category']['values']
+				);
 		}
 
 		return parent::getName();

+ 4 - 1
bridges/AmazonBridge.php

@@ -51,8 +51,11 @@ class AmazonBridge extends BridgeAbstract {
 	));
 
 	public function getName(){
+		if(!is_null($this->getInput('tld')) && !is_null($this->getInput('q'))){
+			return 'Amazon.'.$this->getInput('tld').': '.$this->getInput('q');
+		}
 
-		return 'Amazon.'.$this->getInput('tld').': '.$this->getInput('q');
+		return parent::getName();
 	}
 
 	public function collectData() {

+ 5 - 1
bridges/AskfmBridge.php

@@ -57,7 +57,11 @@ class AskfmBridge extends BridgeAbstract {
 	}
 
 	public function getName(){
-		return self::NAME . ' : ' . $this->getInput('u');
+		if(!is_null($this->getInput('u'))){
+			return self::NAME . ' : ' . $this->getInput('u');
+		}
+
+		return parent::getName();
 	}
 
 	public function getURI(){

+ 5 - 1
bridges/BandcampBridge.php

@@ -50,6 +50,10 @@ class BandcampBridge extends BridgeAbstract {
 	}
 
 	public function getName(){
-		return $this->getInput('tag') . ' - Bandcamp Tag';
+		if(!is_null($this->getInput('tag'))){
+			return $this->getInput('tag') . ' - Bandcamp Tag';
+		}
+
+		return parent::getName();
 	}
 }

+ 5 - 1
bridges/BooruprojectBridge.php

@@ -32,6 +32,10 @@ class BooruprojectBridge extends GelbooruBridge {
 	}
 
 	public function getName(){
-		return static::NAME . ' ' . $this->getInput('i');
+		if(!is_null($this->getInput('i'))){
+			return static::NAME . ' ' . $this->getInput('i');
+		}
+
+		return parent::getName();
 	}
 }

+ 6 - 2
bridges/CNETBridge.php

@@ -83,7 +83,11 @@ topic found in some section URLs, else all topics are selected.';
 	}
 
 	public function getName(){
-		$topic = $this->getInput('topic');
-		return 'CNET News Bridge' . (empty($topic) ? '' : ' - ' . $topic);
+		if(!is_null($this->getInput('topic'))){
+			$topic = $this->getInput('topic');
+			return 'CNET News Bridge' . (empty($topic) ? '' : ' - ' . $topic);
+		}
+
+		return parent::getName();
 	}
 }

+ 5 - 1
bridges/CpasbienBridge.php

@@ -50,7 +50,11 @@ class CpasbienBridge extends BridgeAbstract {
 	}
 
 	public function getName(){
-		return $this->getInput('q') . ' : ' . self::NAME;
+		if(!is_null($this->getInput('q'))){
+			return $this->getInput('q') . ' : ' . self::NAME;
+		}
+
+		return parent::getName();
 	}
 
 	private function getCachedDate($url){

+ 6 - 1
bridges/FacebookBridge.php

@@ -257,6 +257,11 @@ EOD;
 	}
 
 	public function getName(){
-		return isset($this->extraInfos['name']) ? $this->extraInfos['name'] : $this->authorName . ' - Facebook Bridge';
+		if(!empty($this->authorName)){
+			return isset($this->extraInfos['name']) ? $this->extraInfos['name'] : $this->authorName
+			. ' - Facebook Bridge';
+		}
+
+		return parent::getName();
 	}
 }

+ 1 - 0
bridges/GithubIssueBridge.php

@@ -47,6 +47,7 @@ class GithubIssueBridge extends BridgeAbstract {
 		case 'Issue comments':
 			$name = static::NAME . ' ' . $name . ' #' . $this->getInput('i');
 			break;
+		default: return parent::getName();
 		}
 		return $name;
 	}

+ 5 - 1
bridges/GoComicsBridge.php

@@ -46,6 +46,10 @@ class GoComicsBridge extends BridgeAbstract {
 	}
 
 	public function getName(){
-		return $this->getInput('comicname') . ' - GoComics';
+		if(!is_null($this->getInput('comicname'))){
+			return $this->getInput('comicname') . ' - GoComics';
+		}
+
+		return parent::getName();
 	}
 }

+ 5 - 1
bridges/GoogleSearchBridge.php

@@ -55,6 +55,10 @@ class GoogleSearchBridge extends BridgeAbstract {
 	}
 
 	public function getName(){
-		return $this->getInput('q') . ' - Google search';
+		if(!is_null($this->getInput('q'))){
+			return $this->getInput('q') . ' - Google search';
+		}
+
+		return parent::getName();
 	}
 }

+ 9 - 5
bridges/HDWallpapersBridge.php

@@ -70,10 +70,14 @@ class HDWallpapersBridge extends BridgeAbstract {
 	}
 
 	public function getName(){
-		return 'HDWallpapers - '
-		. str_replace(['__', '_'], [' & ', ' '], $this->getInput('c'))
-		. ' ['
-		. $this->getInput('r')
-		. ']';
+		if(!is_null($this->getInput('c')) && !is_null($this->getInput('r'))){
+			return 'HDWallpapers - '
+			. str_replace(['__', '_'], [' & ', ' '], $this->getInput('c'))
+			. ' ['
+			. $this->getInput('r')
+			. ']';
+		}
+
+		return parent::getName();
 	}
 }

+ 5 - 1
bridges/IdenticaBridge.php

@@ -35,7 +35,11 @@ class IdenticaBridge extends BridgeAbstract {
 	}
 
 	public function getName(){
-		return $this->getInput('u') . ' - Identica Bridge';
+		if(!is_null($this->getInput('u'))){
+			return $this->getInput('u') . ' - Identica Bridge';
+		}
+
+		return parent::getName();
 	}
 
 	public function getURI(){

+ 5 - 1
bridges/InstagramBridge.php

@@ -53,7 +53,11 @@ class InstagramBridge extends BridgeAbstract {
 	}
 
 	public function getName(){
-		return $this->getInput('u') . ' - Instagram Bridge';
+		if(!is_null($this->getInput('u'))){
+			return $this->getInput('u') . ' - Instagram Bridge';
+		}
+
+		return parent::getName();
 	}
 
 	public function getURI(){

+ 1 - 1
bridges/KununuBridge.php

@@ -70,7 +70,7 @@ class KununuBridge extends BridgeAbstract {
 			return ($this->companyName ?: $company) . ' - ' . self::NAME;
 		}
 
-		return paren::getName();
+		return parent::getName();
 	}
 
 	public function collectData(){

+ 5 - 1
bridges/MixCloudBridge.php

@@ -16,7 +16,11 @@ class MixCloudBridge extends BridgeAbstract {
 	));
 
 	public function getName(){
-		return 'MixCloud - ' . $this->getInput('u');
+		if(!is_null($this->getInput('u'))){
+			return 'MixCloud - ' . $this->getInput('u');
+		}
+
+		return parent::getName();
 	}
 
 	public function collectData(){

+ 4 - 0
bridges/NovelUpdatesBridge.php

@@ -56,6 +56,10 @@ class NovelUpdatesBridge extends BridgeAbstract {
 	}
 
 		public function getName(){
+			if(!empty($this->seriesTitle)){
 				return $this->seriesTitle . ' - ' . static::NAME;
+			}
+
+			return parent::getName();
 		}
 }

+ 13 - 9
bridges/ParuVenduImmoBridge.php

@@ -84,15 +84,19 @@ class ParuVenduImmoBridge extends BridgeAbstract {
 	}
 
 	public function getName(){
-		$request = '';
-		$minarea = $this->getInput('minarea');
-		if(!empty($minarea)){
-			$request .= ' ' . $minarea . ' m2';
-		}
-		$location = $this->getInput('lo');
-		if(!empty($location)){
-			$request .= ' In: ' . $location;
+		if(!is_null($this->getInput('minarea'))){
+			$request = '';
+			$minarea = $this->getInput('minarea');
+			if(!empty($minarea)){
+				$request .= ' ' . $minarea . ' m2';
+			}
+			$location = $this->getInput('lo');
+			if(!empty($location)){
+				$request .= ' In: ' . $location;
+			}
+			return 'Paru Vendu Immobilier' . $request;
 		}
-		return 'Paru Vendu Immobilier' . $request;
+
+		return parent::getName();
 	}
 }

+ 11 - 7
bridges/PickyWallpapersBridge.php

@@ -82,12 +82,16 @@ class PickyWallpapersBridge extends BridgeAbstract {
 	}
 
 	public function getName(){
-		$subcategory = $this->getInput('s');
-		return 'PickyWallpapers - '
-		. $this->getInput('c')
-		. ($subcategory ? ' > ' . $subcategory : '')
-		. ' ['
-		. $this->getInput('r')
-		. ']';
+		if(!is_null($this->getInput('s'))){
+			$subcategory = $this->getInput('s');
+			return 'PickyWallpapers - '
+			. $this->getInput('c')
+			. ($subcategory ? ' > ' . $subcategory : '')
+			. ' ['
+			. $this->getInput('r')
+			. ']';
+		}
+
+		return parent::getName();
 	}
 }

+ 5 - 1
bridges/RTBFBridge.php

@@ -53,6 +53,10 @@ class RTBFBridge extends BridgeAbstract {
 	}
 
 	public function getName(){
-		return $this->getInput('c') .' - RTBF Bridge';
+		if(!is_null($this->getInput('c'))){
+			return $this->getInput('c') .' - RTBF Bridge';
+		}
+
+		return parent::getName();
 	}
 }

+ 5 - 1
bridges/SoundcloudBridge.php

@@ -55,6 +55,10 @@ class SoundCloudBridge extends BridgeAbstract {
 
 	}
 	public function getName(){
-		return self::NAME . ' - ' . $this->getInput('u');
+		if(!is_null($this->getInput('u'))){
+			return self::NAME . ' - ' . $this->getInput('u');
+		}
+
+		return parent::getName();
 	}
 }

+ 5 - 1
bridges/SuperbWallpapersBridge.php

@@ -61,6 +61,10 @@ class SuperbWallpapersBridge extends BridgeAbstract {
 	}
 
 	public function getName(){
-		return self::NAME . '- ' . $this->getInput('c') . ' [' . $this->getInput('r') . ']';
+		if(!is_null($this->getInput('c')) && !is_null($this->getInput('r'))){
+			return self::NAME . '- ' . $this->getInput('c') . ' [' . $this->getInput('r') . ']';
+		}
+
+		return parent::getName();
 	}
 }

+ 5 - 1
bridges/TagBoardBridge.php

@@ -40,6 +40,10 @@ class TagBoardBridge extends BridgeAbstract {
 	}
 
 	public function getName(){
-		return 'tagboard - ' . $this->getInput('u');
+		if(!is_null($this->getInput('u'))){
+			return 'tagboard - ' . $this->getInput('u');
+		}
+
+		return parent::getName();
 	}
 }

+ 5 - 1
bridges/Torrent9Bridge.php

@@ -81,7 +81,11 @@ class Torrent9Bridge extends BridgeAbstract {
 
 
 	public function getName(){
-		return $this->getInput('q') . ' : ' . self::NAME;
+		if(!is_null($this->getInput('q'))){
+			return $this->getInput('q') . ' : ' . self::NAME;
+		}
+
+		return parent::getName();
 	}
 
 	private function getCachedDate($url){

+ 1 - 0
bridges/TwitterBridge.php

@@ -46,6 +46,7 @@ class TwitterBridge extends BridgeAbstract {
 			$specific = '@';
 			$param = 'u';
 			break;
+		default: return parent::getName();
 		}
 		return 'Twitter ' . $specific . $this->getInput($param);
 	}

+ 11 - 7
bridges/WallpaperStopBridge.php

@@ -92,12 +92,16 @@ class WallpaperStopBridge extends BridgeAbstract {
 	}
 
 	public function getName(){
-		$subcategory = $this->getInput('s');
-		return 'WallpaperStop - '
-		. $this->getInput('c')
-		. (!empty($subcategory) ? ' > ' . $subcategory : '')
-		. ' ['
-		. $this->getInput('r')
-		. ']';
+		if(!is_null($this->getInput('s')) && !is_null($this->getInput('c')) && !is_null($this->getInput('r'))){
+			$subcategory = $this->getInput('s');
+			return 'WallpaperStop - '
+			. $this->getInput('c')
+			. (!empty($subcategory) ? ' > ' . $subcategory : '')
+			. ' ['
+			. $this->getInput('r')
+			. ']';
+		}
+
+		return parent::getName();
 	}
 }

+ 1 - 3
bridges/WikipediaBridge.php

@@ -56,9 +56,7 @@ class WikipediaBridge extends BridgeAbstract {
 			case 'dyk':
 				$subject = WIKIPEDIA_SUBJECT_DYK;
 				break;
-			default:
-				$subject = WIKIPEDIA_SUBJECT_TFA;
-				break;
+			default: return parent::getName();
 		}
 
 		switch($subject){

+ 1 - 1
bridges/WorldOfTanksBridge.php

@@ -38,7 +38,7 @@ class WorldOfTanksBridge extends BridgeAbstract {
 	}
 
 	public function getName(){
-		return $this->title ?: self::NAME;
+		return $this->title ?: parent::getName();
 	}
 
 	public function collectData(){

+ 1 - 1
lib/FeedExpander.php

@@ -194,6 +194,6 @@ abstract class FeedExpander extends BridgeAbstract {
 	}
 
 	public function getName(){
-		return $this->name;
+		return $this->name ?: parent::getName();
 	}
 }