Merge pull request #163 from heiglandreas/hotfix/twitterlistAPI

Fixes twitter-list when an exception occurs

Author: heiglandreas

Diff for: src/module/Phpug/src/Phpug/Api/Rest/TwitterController.php

@@ -52,39 +52,47 @@ public function getList()
 
         $result = array();
         foreach ($twitters as $twitter) {
-            $group = $twitter->getGroup();
-            if (!$group) {
-                continue;
-            }
-            if (! $group instanceof Usergroup) {
-                continue;
-            }
-            if ($group->getState() != Usergroup::ACTIVE) {
-                continue;
-            }
-            if (! isset($result[$twitter->getName()])) {
-                $result[$twitter->getName()] = array(
-                    'screen_name' => $twitter->getName(),
-                    'name'        => $twitterService->getInfoForUser('name', $twitter->getName()),
-                    'url'         => $twitter->getUrl(),
-                    'icon_url'    => $twitterService->getInfoForUser('profile_image_url_https', $twitter->getName()),
-                    'groups'      => array(),
-                );
-            }
-            $groupMapUrl = $this->url()->fromRoute('home', array(), array('force_canonical' => true)) . '?center=' . $group->getShortName();
-            $groupApiUrl = $this->url()->fromRoute(
-                'api/rest', array(
+            try {
+                $group = $twitter->getGroup();
+                if (! $group) {
+                    continue;
+                }
+                if (! $group instanceof Usergroup) {
+                    continue;
+                }
+                if ($group->getState() != Usergroup::ACTIVE) {
+                    continue;
+                }
+                if (! isset($result[$twitter->getName()])) {
+                    $result[$twitter->getName()] = array(
+                        'screen_name' => $twitter->getName(),
+                        'name'        => $twitterService->getInfoForUser('name',
+                            $twitter->getName()),
+                        'url'         => $twitter->getUrl(),
+                        'icon_url'    => $twitterService->getInfoForUser('profile_image_url_https',
+                            $twitter->getName()),
+                        'groups'      => array(),
+                    );
+                }
+                $groupMapUrl                             = $this->url()->fromRoute('home',
+                        array(),
+                        array('force_canonical' => true)) . '?center=' . $group->getShortName();
+                $groupApiUrl                             = $this->url()->fromRoute(
+                    'api/rest', array(
                     'controller' => 'Usergroup',
-                    'id' => $group->getId(),
+                    'id'         => $group->getId(),
                 ),
-                array('force_canonical' => true)
-            );
-            $result[$twitter->getName()]['groups'][] = array(
-                'usergroup'           => $group->getName(),
-                'usergroup_url'       => $group->getUrl(),
-                'phpug_group_map_url' => $groupMapUrl,
-                'phpug_group_api_url' => $groupApiUrl,
-            );
+                    array('force_canonical' => true)
+                );
+                $result[$twitter->getName()]['groups'][] = array(
+                    'usergroup'           => $group->getName(),
+                    'usergroup_url'       => $group->getUrl(),
+                    'phpug_group_map_url' => $groupMapUrl,
+                    'phpug_group_api_url' => $groupApiUrl,
+                );
+            } catch (\Exception $e) {
+                
+            }
         }
 
         usort($result, function($a, $b){