aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDevan Franchini <twitch153@gentoo.org>2015-08-13 03:04:40 -0400
committerDevan Franchini <twitch153@gentoo.org>2015-08-13 03:04:44 -0400
commit3e582a86a5389c5013fd0b8c5a6cc964d3252a2c (patch)
tree595d596b148f7fb02d72291652e84a63512f28ae
parentsqlite_db.py: read_db() connects to database with "with" command (diff)
downloadlayman-3e582a86a5389c5013fd0b8c5a6cc964d3252a2c.tar.gz
layman-3e582a86a5389c5013fd0b8c5a6cc964d3252a2c.tar.bz2
layman-3e582a86a5389c5013fd0b8c5a6cc964d3252a2c.zip
sqlite_db.py: Modifies owner information gathering method
Due to the fact that including the owner name isn't necessary layman needed to modify the way it gathered owner information to prevent run-time errors.
-rw-r--r--layman/db_modules/sqlite_db/sqlite_db.py23
1 files changed, 9 insertions, 14 deletions
diff --git a/layman/db_modules/sqlite_db/sqlite_db.py b/layman/db_modules/sqlite_db/sqlite_db.py
index 14bfa25..6619699 100644
--- a/layman/db_modules/sqlite_db/sqlite_db.py
+++ b/layman/db_modules/sqlite_db/sqlite_db.py
@@ -169,15 +169,15 @@ class DBHandler(object):
WHERE Overlay_ID = ?''', (overlay_id,))
overlay['source'] = cursor.fetchall()
- cursor.execute('''SELECT Owner_Name, Owner_Email FROM
+ cursor.execute('''SELECT Owner_Email, Owner_Name FROM
Overlay_Owner JOIN Overlay USING (Overlay_ID) JOIN Owner USING
(Owner_ID) WHERE Overlay_ID = ?''', (overlay_id,))
- owner_info = cursor.fetchall()
+ owner_info = cursor.fetchall()[0]
- if len(owner_info):
- owner_info = owner_info[0]
- overlay['owner_name'] = owner_info[0]
- overlay['owner_email'] = owner_info[1]
+ overlay['owner_email'] = owner_info[0]
+
+ if len(owner_info) > 1:
+ overlay['owner_name'] = owner_info[1]
cursor.execute('''SELECT Description FROM Description JOIN
Overlay USING (Overlay_ID) WHERE Overlay_ID = ?''',
@@ -187,12 +187,7 @@ class DBHandler(object):
overlay['status'] = overlay_info[3]
overlay['quality'] = overlay_info[4]
overlay['priority'] = overlay_info[2]
-
- if overlay_info[7]:
- overlay['license'] = overlay_info[7]
- else:
- overlay['license'] = None
-
+ overlay['license'] = overlay_info[7]
overlay['homepage'] = overlay_info[5]
overlay['IRC'] = overlay_info[6]
@@ -248,8 +243,8 @@ class DBHandler(object):
VALUES ( ?, ? )''', (overlay.owner_name, overlay.owner_email,))
connection.commit()
- cursor.execute('''SELECT Owner_ID from Owner WHERE Owner_Name = ?;''',
- (overlay.owner_name,))
+ cursor.execute('''SELECT Owner_ID from Owner WHERE Owner_Email = ?;''',
+ (overlay.owner_email,))
owner_id = cursor.fetchone()[0]
for source in overlay.sources: