Browse Source

[WikipediaBridge] remove useless code

Signed-off-by: Pierre Mazière <pierre.maziere@gmx.com>
Pierre Mazière 7 years ago
parent
commit
792578aefb
1 changed files with 1 additions and 27 deletions
  1. 1 27
      bridges/WikipediaBridge.php

+ 1 - 27
bridges/WikipediaBridge.php

@@ -51,7 +51,6 @@ class WikipediaBridge extends BridgeAbstract{
 
     public function getName(){
 		$params=$this->parameters[$this->queriedContext];
-        $subject = WIKIPEDIA_SUBJECT_TFA;
 		switch($params['subject']['value']){
 			case 'tfa':
 				$subject = WIKIPEDIA_SUBJECT_TFA;
@@ -80,16 +79,7 @@ class WikipediaBridge extends BridgeAbstract{
 
 	public function collectData(){
 		$params=$this->parameters[$this->queriedContext];
-		if(!isset($params['language']['value']))
-			$this->returnClientError('You must specify a valid language via \'&language=\'!');
 
-		if(!$this->CheckLanguageCode(strtolower($params['language']['value'])))
-			$this->returnClientError('The language code you provided (\'' . $params['language']['value'] . '\') is not supported!');
-
-		if(!isset($params['subject']['value']))
-			$this->returnClientError('You must specify a valid subject via \'&subject=\'!');
-
-		$subject = WIKIPEDIA_SUBJECT_TFA;
 		switch($params['subject']['value']){
 			case 'tfa':
 				$subject = WIKIPEDIA_SUBJECT_TFA;
@@ -102,9 +92,7 @@ class WikipediaBridge extends BridgeAbstract{
 				break;
 		}
 
-		$fullArticle = false;
-		if(isset($params['fullarticle']['value']))
-			$fullArticle = $params['fullarticle']['value'];
+		$fullArticle = $params['fullarticle']['value'];
 
 		// This will automatically send us to the correct main page in any language (try it!)
 		$html = $this->getSimpleHTMLDOM($this->getURI() . '/wiki');
@@ -129,20 +117,6 @@ class WikipediaBridge extends BridgeAbstract{
 	}
 
 	/**
-	* Returns true if the language code is part of the parameters list
-	*/
-	private function CheckLanguageCode($languageCode){
-		$languages = $this->parameters[0]['language']['values'];
-
-		$language_names = array();
-
-		foreach($languages as $name=>$value)
-			$language_names[] = $value;
-
-		return in_array($languageCode, $language_names);
-	}
-
-	/**
 	* Replaces all relative URIs with absolute ones
 	* @param $element A simplehtmldom element
 	* @return The $element->innertext with all URIs replaced