Merge 3282072 into 84a934e

sugarcrm · Feb 6, 2019 · 6f4158d · 6f4158d
2 parents 84a934e + 3282072
commit 6f4158d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/Helpers/Helpers.php b/src/Helpers/Helpers.php
@@ -8,7 +8,7 @@
 class Helpers
 {
     const API_VERSION = 10;
-    const API_URL = '/rest/v%d/';
+    const API_URL = '/rest/v%s/';
 
     /**
      * Given a sugarcrm server/instance generate the Rest/v10 API Url
@@ -19,8 +19,8 @@ class Helpers
     public static function configureAPIURL($instance, $version = null)
     {
         $url = 0;
-        $version = ($version === null ? self::API_VERSION : intval($version));
-        $instance = preg_replace('/\/rest\/v\d+/', '', $instance);
+        $version = ($version === null ? self::API_VERSION : $version);
+        $instance = preg_replace('/\/rest\/v.+/', '', $instance);
         $url = rtrim($instance,"/").sprintf(self::API_URL, $version);
         if (preg_match('/^(http|https):\/\//i', $url) === 0) {
             $url = "http://".$url;

diff --git a/tests/Helpers/HelpersTest.php b/tests/Helpers/HelpersTest.php
@@ -79,6 +79,7 @@ public function testConfigureAPIUrl()
         $this->assertEquals('http://localhost/Sugar/Test/rest/v10/',Helpers::configureAPIURL('http://localhost/Sugar/Test/rest/v10/rest/v10/rest/v10','10'));
         $this->assertEquals('http://localhost/SugarTest/rest/v10/',Helpers::configureAPIURL('http://localhost/SugarTest/rest/v10/rest/v10/rest/v10',10));
         $this->assertEquals('http://localhost/SugarTest/rest/v11/',Helpers::configureAPIURL('http://localhost/SugarTest/rest/v10/rest/v10/rest/v10',11));
+        $this->assertEquals('http://localhost/SugarTest/rest/v11_1/',Helpers::configureAPIURL('http://localhost/SugarTest/rest/v10/rest/v10/rest/v10','11_1'));
 
     }