Do not store application info in the DB
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Zeitgeist Framework |
Fix Released
|
Medium
|
Unassigned |
Bug Description
"""""""
# Insert the application
# FIXME: Is reading the .desktop file and storing that stuff into
# the DB really required?
app_info = DesktopEntry(
app_uri_id, app_content_id, app_source_id = self._get_
unicode(
app_item = self._get_
unicode(
try:
self.store.
(app_uri_id, unicode(
except sqlite3.
pass
"""""""
Why are we inserting this into the DB (and making insertion slower by loading all the .desktop files, which should probably be workarounded with a cache in case we keep it) when we don't use it for anything, and this isn't contemplated in the Database spec?
(Additionally, in case we keep it we need to contemplate that Exec= may be empty, as else this results in a crash).
Related branches
- Seif Lotfy: Pending requested
- Markus Korn: Pending requested
- Zeitgeist Framework Team: Pending requested
- Diff: None lines
Changed in zeitgeist: | |
status: | New → In Progress |
importance: | Undecided → Medium |
importance: | Medium → Undecided |
milestone: | none → 0.2 |
status: | In Progress → New |
status: | New → Confirmed |
importance: | Undecided → Medium |
Changed in zeitgeist: | |
status: | Confirmed → Fix Released |