Merge pull request #33 from onelogin/copilot/fix-32

Fix missing app icons in getUserApps by upgrading to API v2 endpoint

Author: Subterrane

src/models/App.php

@@ -32,7 +32,7 @@ public function __construct($data)
     {
         $this->id = isset($data->id)? (int) $data->id : null;
         $this->name = $data->name;
-        $this->icon = $data->icon;
+        $this->icon = isset($data->icon) ? $data->icon : null;
         $this->provisioned = $data->provisioned;
         $this->extension = $data->extension;
         $this->loginId = $data->login_id;

src/util/Constants.php

@@ -18,7 +18,7 @@ class Constants
     // User URLs
     const GET_USERS_URL = "https://api.%s.onelogin.com/api/1/users";
     const GET_USER_URL = "https://api.%s.onelogin.com/api/1/users/%s";
-    const GET_APPS_FOR_USER_URL = "https://api.%s.onelogin.com/api/1/users/%s/apps";
+    const GET_APPS_FOR_USER_URL = "https://api.%s.onelogin.com/api/2/users/%s/apps";
     const GET_ROLES_FOR_USER_URL = "https://api.%s.onelogin.com/api/1/users/%s/roles";
     const GET_CUSTOM_ATTRIBUTES_URL = "https://api.%s.onelogin.com/api/1/users/custom_attributes";
     const CREATE_USER_URL = "https://api.%s.onelogin.com/api/1/users";