Merge pull request #54 from henriquemattos/master

Allow dynamic Facebook API version via Strategy parameters

Author: YuzuruS

.gitignore

@@ -1,2 +1,2 @@
 .DS_Store
-
+.idea

FacebookStrategy.php

@@ -8,16 +8,21 @@ class FacebookStrategy extends OpauthStrategy {
         'scope' => 'email', // Default scope for necessary permissions
     );
 
-    private $api_version = 'v12.0'; // Update to the latest supported version
+    private $api_version = 'v17.0'; // Update to the latest supported version
 
     public function request() {
-        $url = "https://www.facebook.com/{$this->api_version}/dialog/oauth";
         $params = array(
             'client_id' => $this->strategy['app_id'],
             'redirect_uri' => $this->strategy['redirect_uri'],
             'scope' => $this->strategy['scope'],
         );
 
+        if (!empty($this->strategy['api_version'])) {
+            $params['api_version'] = $this->strategy['api_version'];
+            $this->api_version = $this->strategy['api_version'];
+        }
+        $url = "https://www.facebook.com/{$this->api_version}/dialog/oauth";
+
         // Other optional parameters
         if (!empty($this->strategy['state'])) $params['state'] = $this->strategy['state'];
         if (!empty($this->strategy['response_type'])) $params['response_type'] = $this->strategy['response_type'];
@@ -114,6 +119,10 @@ private function me($access_token) {
             $fields = $this->strategy['fields'];
         }
 
+        if (!empty($this->strategy['api_version'])) {
+            $this->api_version = $this->strategy['api_version'];
+        }
+
         $me = $this->serverGet(
             "https://graph.facebook.com/{$this->api_version}/me",
             array(

composer.json

@@ -2,13 +2,18 @@
 	"name": "opauth/facebook",
 	"description": "Facebook strategy for Opauth",
 	"keywords": ["authentication","auth","facebook"],
-	"homepage": "http://opauth.org",
+	"homepage": "https://opauth.org",
 	"license": "MIT",
 	"authors": [
 		{
 			"name": "U-Zyn Chua",
 			"email": "[email protected]",
 			"homepage": "http://uzyn.com"
+		},
+		{
+			"name": "Henrique Mattos",
+			"email": "[email protected]",
+			"homepage": "https://www.visualworks.com.br"
 		}
 	],
 	"require": {