BetterBrief / silverstripe-opauth

code/OpauthAuthenticator.php

Doc Comments +1 added lines

@@ -55,6 +55,7 @@
 	/**
 	 * get_strategy_segment
 	 * Works around Opauth's weird URL scheme - GoogleStrategy => /google/
+	 * @param string $strategy
 	 * @return string
 	 */
 	public static function get_strategy_segment($strategy) {

code/OpauthController.php

Doc Comments +4 added lines

@@ -291,6 +291,7 @@ 
 	/**
 	 * Validates the Oauth response for Opauth.
 	 * @throws InvalidArgumentException
+	 * @param Opauth $opauth
 	 */
 	protected function validateOpauthResponse($opauth, $response) {
 		if(!empty($response['error'])) {
@@ -385,6 +386,9 @@ 
 		return unserialize(base64_decode($this->request->{$method.'Var'}('opauth')));
 	}
 
+	/**
+	 * @param string $action
+	 */
 	public function Link($action = null) {
 		return Controller::join_links(
 			self::config()->opauth_path,

code/OpauthLoginForm.php

Doc Comments +4 added lines

@@ -28,6 +28,10 @@
 			'httpSubmission',
 		);
 
+	/**
+	 * @param Controller $controller
+	 * @param string $name
+	 */
 	public function __construct($controller, $name) {
 		parent::__construct($controller, $name, $this->getFields(), $this->getActions());
 		$this->configureBackURL();

code/OpauthResponseHelper.php

Doc Comments +1 added lines

@@ -55,6 +55,7 @@
 
 	/**
 	 * Try very hard to get a locale for this user. Helps for i18n etc.
+	 * @param string $language
 	 * @return string
 	 */
 	public static function get_smart_locale($language = null) {