change marathon group parsing method

fixes Difrex#9
w1r0x · Dec 6, 2016 · dec5e79 · dec5e79
1 parent 9a439fb
commit dec5e79
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 10 deletions.
diff --git a/AUTHORS b/AUTHORS
@@ -1 +1,2 @@
 Denis Zheleztsov <difrex.punk@gmail.com>
+Denis Ryabyy <vv1r0x@gmail.com>
diff --git a/surok/discovery.py b/surok/discovery.py
@@ -89,13 +89,7 @@ def get_group(service, app):
 # Return marathon.group
 def parse_marathon_app_id(marathon_app_id):
     marathon_app_id = marathon_app_id.split('/')
-    group = ''
-    counter = len(marathon_app_id) - 2
-    i = 0
-    while counter > i:
-        group = group + marathon_app_id[counter]
-        if counter != i + 1:
-            group += '.'
-            counter -= 1
-
-    return group
+    del(marathon_app_id[-1])
+    marathon_app_id.reverse()
+    group = ".".join(marathon_app_id)[:-1]
+    return(group)
Original file line number	Diff line number	Diff line change
		@@ -1 +1,2 @@
		Denis Zheleztsov <difrex.punk@gmail.com>
		Denis Ryabyy <vv1r0x@gmail.com>