Merge pull request #110 from eiriksm/feat/assignee-based-on-roles

Only allow assignees if the role is correct
eiriksm · May 1, 2019 · fa14c75 · fa14c75
2 parents 138a7b0 + b79c789
commit fa14c75
Showing 1 changed file with 15 additions and 1 deletion.
diff --git a/src/CosyComposer.php b/src/CosyComposer.php
@@ -138,7 +138,6 @@ class CosyComposer
      */
     protected $project;
 
-
     /**
      * @var \Http\Adapter\Guzzle6\Client
      */
@@ -890,6 +889,21 @@ public function run()
                         $assignees = $cdata->extra->violinist->assignees;
                     }
                 }
+                $assignees_allowed_roles = [
+                    'agency',
+                    'enterprise',
+                ];
+                $assignees_allowed = false;
+                if ($this->project && $this->project->getRoles()) {
+                    foreach ($this->project->getRoles() as $role) {
+                        if (in_array($role, $assignees_allowed_roles)) {
+                            $assignees_allowed = true;
+                        }
+                    }
+                }
+                if (!$assignees_allowed) {
+                    $assignees = [];
+                }
                 $pr_params = [
                     'base'  => $default_branch,
                     'head'  => $head,