azine / AzineHybridAuthBundle

Controller/AzineHybridAuthJsonController.php

Unused Use Statements -5 removed lines

@@ -3,15 +3,10 @@
 namespace Azine\HybridAuthBundle\Controller;
 
 use Symfony\Component\HttpFoundation\RedirectResponse;
-
 use Azine\HybridAuthBundle\Services\AzineHybridAuth;
-
 use Azine\HybridAuthBundle\Services\AzineMergedBusinessNetworksProvider;
-
 use Symfony\Component\HttpFoundation\JsonResponse;
-
 use Symfony\Component\HttpFoundation\Request;
-
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
 
 class AzineHybridAuthJsonController extends Controller {

Controller/HybridEndPointController.php

Doc Comments +3 added lines

@@ -238,6 +238,9 @@
 		}

 	}

 

+	/**
+	 * @param string $provider_id
+	 */
 	private function returnToCallbackUrl($provider_id) {

 		// get the stored callback url

 		$callback_url = $this->hybridAuth->storage()->get( "hauth_session.$provider_id.hauth_return_to" );

Unused Use Statements +5 added lines, -9 removed lines

@@ -2,15 +2,11 @@
 

 namespace Azine\HybridAuthBundle\Controller;

 

-use Azine\HybridAuthBundle\Services\AzineHybridAuth;

-use Symfony\Component\HttpFoundation\ParameterBag;

-

-use Symfony\Component\HttpFoundation\RedirectResponse;

-

-use Symfony\Component\HttpFoundation\Response;

-

-use Symfony\Component\HttpFoundation\Request;

-

+use Azine\HybridAuthBundle\Services\AzineHybridAuth;
+use Symfony\Component\HttpFoundation\ParameterBag;
+use Symfony\Component\HttpFoundation\RedirectResponse;
+use Symfony\Component\HttpFoundation\Response;
+use Symfony\Component\HttpFoundation\Request;
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;

 

 class HybridEndPointController extends Controller {

Entity/UserContact.php

Doc Comments +5 added lines

@@ -43,6 +43,11 @@
 	/* For Xing & LinkedIn usually job-title @ main ompany */
 	public $headline = NULL;
 	
+	/**
+	 * @param string $gender
+	 * @param string $firstName
+	 * @param string $lastName
+	 */
 	public function __construct($provider,
 								$gender = null,
 								$firstName = null, 

Services/AzineGenderGuesser.php

Doc Comments +3 added lines

@@ -37,6 +37,9 @@
 	
 	private $debugMsg = '';
 	
+	/**
+	 * @param string[] $matchList
+	 */
 	public function __construct($femaleNames=null, $malesNames=null, $merge = false, $matchList = null){
 		// Use the double_metaphone module if it's available, or fall back to the
 		// standard metaphone() PHP call if not

Unused Use Statements -2 removed lines

@@ -1,8 +1,6 @@
 <?php
 namespace Azine\HybridAuthBundle\Services;
 
-use Azine\HybridAuthBundle\Entity\UserContact;
-
 /**
  * 	https://github.com/petewarden/genderfromname/blob/master/genderfromname.php
 	

Services/AzineHybridAuth.php

Doc Comments +4 added lines, -4 removed lines

@@ -64,7 +64,7 @@ 
 	}
 
 	/**
-	 * @param $sessionData
+	 * @param string|null $sessionData
 	 */
 	public function storeHybridAuthSessionData($sessionData){
 		// todo implement this!
@@ -90,7 +90,7 @@ 
 	 *
 	 * @param string $provider_id
 	 * @param boolean $require_login
-	 * @return \Hybrid_Provider_Model
+	 * @return \Hybrid_Provider_Adapter
 	 */
 	public function getProvider($provider_id, $require_login = true){
 		$adapter = $this->hybridAuth->getAdapter($provider_id);
@@ -113,7 +113,7 @@ 
 	
 	/**
      * Get the Xing Adapter
-     * @return \Hybrid_Providers_XING
+     * @return \Hybrid_Provider_Adapter
      */
 	public function getXing(){
 		return $this->getProvider("xing");
@@ -131,7 +131,7 @@ 
 	/**
 	 * Get the LinkedIn Adapter
 	 *
-	 * @return \Hybrid_Providers_LinkedIn
+	 * @return \Hybrid_Provider_Adapter
 	 */
 	public function getLinkedIn(){
 		return $this->getProvider("linkedin");

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 namespace Azine\HybridAuthBundle\Services;
 
 use Azine\HybridAuthBundle\DependencyInjection\AzineHybridAuthExtension;
-
 use Doctrine\Common\Persistence\ObjectManager;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\Routing\Generator\UrlGeneratorInterface;

Services/AzineMergedBusinessNetworksProvider.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 namespace Azine\HybridAuthBundle\Services;
 
 use Azine\HybridAuthBundle\Entity\UserContact;
-
 use Symfony\Component\HttpFoundation\Session\Session;
 
 class AzineMergedBusinessNetworksProvider {

Services/GenderGuesser.php

Doc Comments +2 added lines, -2 removed lines

@@ -24,9 +24,9 @@
 	/**
 	 * Guess the gender for a given firstname. The guesser will return an array with
 	 * 
-	 * @param \string $firstName
+	 * @param string $firstName
 	 * @param int $looseness
-	 * @return \string sex => m | f
+	 * @return string sex => m | f
 	 */
 	public function gender($firstName, $looseness=1);	
 }
\ No newline at end of file

Unused Use Statements -8 removed lines

@@ -1,14 +1,6 @@
 <?php
 namespace Azine\HybridAuthBundle\Services;
 
-use Azine\HybridAuthBundle\Entity\UserContact;
-
-use Symfony\Component\HttpFoundation\Session\Session;
-
-use Azine\HybridAuthBundle\DependencyInjection\AzineHybridAuthExtension;
-
-use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
-
 interface GenderGuesser {
 
 	/**

Tests/Services/AzineGenderGuesserTest.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php
 namespace Azine\HybridAuthBundle\Services;
 
-use Azine\HybridAuthBundle\Entity\UserContact;
 use Symfony\Bundle\FrameworkBundle\Tests\TestCase;
 
 class AzineGenderGuesserTest extends TestCase {