From 8e92cebaf2087abac3884882cfd9663c0a3669ef Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Adrian=20Schr=C3=B6ter?= <adrian@suse.de>
Date: Thu, 9 Sep 2010 22:07:50 +0200
Subject: [PATCH] don't crash on add_role to a project (untested)

---
 osc/commandline.py | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/osc/commandline.py b/osc/commandline.py
index b0c36462..2283a96b 100644
--- a/osc/commandline.py
+++ b/osc/commandline.py
@@ -1221,15 +1221,17 @@ Please submit there instead, or use --nodevelproject to force direct submission.
         user = args[0]
         role = args[1]
         project = args[2]
+        actionxml = """ <action type="add_role"> <target project="%s" /> <person name="%s" role="%s" /> </action> """ % \
+                (project, user, role)
+
         if len(args) > 3:
             package = args[3]
+            actionxml = """ <action type="add_role"> <target project="%s" package="%s" /> <person name="%s" role="%s" /> </action> """ % \
+                (project, package, user, role)
 
         if get_user_meta(apiurl, user) == None:
             raise oscerr.WrongArgs('osc: an error occured.')
 
-        actionxml = """ <action type="add_role"> <target project="%s" package="%s" /> <person name="%s" role="%s" /> </action> """ % \
-                (project, package, user, role)
-
         return actionxml
 
     def _set_bugowner(self, args, opts):