[copr] master: Handle situation where copr was deleted before admin reviewed its legal issues (4868d07)

bkabrda at fedoraproject.org bkabrda at fedoraproject.org
Tue Apr 2 13:24:08 UTC 2013


Repository : http://git.fedorahosted.org/cgit/copr.git

On branch  : master

>---------------------------------------------------------------

commit 4868d0756b5fdbc161b2189983df1e4f89a3f483
Author: Bohuslav Kabrda <bkabrda at redhat.com>
Date:   Tue Apr 2 13:38:32 2013 +0200

    Handle situation where copr was deleted before admin reviewed its legal issues


>---------------------------------------------------------------

 .../coprs/views/admin_ns/admin_general.py          |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/coprs_frontend/coprs/views/admin_ns/admin_general.py b/coprs_frontend/coprs/views/admin_ns/admin_general.py
index e439809..c1b6146 100644
--- a/coprs_frontend/coprs/views/admin_ns/admin_general.py
+++ b/coprs_frontend/coprs/views/admin_ns/admin_general.py
@@ -27,7 +27,9 @@ def legal_flag():
     # is not very good (and sqlalchemy complains about it)
     coprs = coprs_logic.CoprsLogic.get_multiple(flask.g.user, ids=ids).all() if ids else []
     for flag in legal_flags:
-        flag.copr = filter(lambda x: flag.object_id == x.id, coprs)[0]
+        # handle the situation where copr was deleted in the meanwhile
+        copr = filter(lambda x: flag.object_id == x.id, coprs)
+        flag.copr = copr[0] if copr else None
 
     return flask.render_template('admin/legal-flag.html',
                                  legal_flags=legal_flags)



More information about the copr-devel mailing list