Merge pull request richp10#14 from richp10/martinduparc-patch-1

Code refactoring

Author: martinduparc

src/DiscourseAPI.php

@@ -43,9 +43,9 @@ public function getGroups()
          * @return mixed HTTP return code and API return object
          */
 
-        public function getGroup($group)
+        public function getGroup($groupname)
         {
-            return $this->_getRequest('/groups/' . $group . '.json');
+            return $this->_getRequest('/groups/' . $groupname . '.json');
         }
 
         /**
@@ -79,20 +79,12 @@ public function joinGroup($groupname, $username)
          */
         public function getGroupIdByGroupName($groupname)
         {
-            $obj = $this->getGroups();
+            $obj = $this->getGroup($groupname);
             if ($obj->http_code !== 200) {
                 return false;
             }
 
-            foreach ($obj->apiresult as $group) {
-                if ($group->name === $groupname) {
-                    $groupId = (int)$group->id;
-                    break;
-                }
-                $groupId = false;
-            }
-
-            return $groupId;
+            return $obj->apiresult->group->id;
         }
 
         /**