[gnome-shell/wip/rstrode/rhel-7.9: 67/86] js: Always use AppSystem to lookup apps
- From: Ray Strode <halfline src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell/wip/rstrode/rhel-7.9: 67/86] js: Always use AppSystem to lookup apps
- Date: Fri, 12 Feb 2021 19:04:07 +0000 (UTC)
commit 7e212461838a78e55e0c289e3dfd7a7c0897ad1a
Author: Florian Müllner <fmuellner gnome org>
Date: Sat Mar 14 14:45:42 2020 +0100
js: Always use AppSystem to lookup apps
There is no good reason for bypassing the application cache in
AppSystem and loading .desktop files again.
https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1093
js/ui/appDisplay.js | 4 ++--
js/ui/calendar.js | 16 ++++++++++------
2 files changed, 12 insertions(+), 8 deletions(-)
---
diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js
index 57d144968b..b5a5d7cb30 100644
--- a/js/ui/appDisplay.js
+++ b/js/ui/appDisplay.js
@@ -1111,8 +1111,8 @@ var AppSearchProvider = new Lang.Class({
let results = [];
groups.forEach(group => {
group = group.filter(appID => {
- let app = Gio.DesktopAppInfo.new(appID);
- return app && app.should_show();
+ const app = this._appSys.lookup_app(appID);
+ return app && app.app_info.should_show();
});
results = results.concat(group.sort(
(a, b) => usage.compare('', a, b)
diff --git a/js/ui/calendar.js b/js/ui/calendar.js
index 651aac610a..0b2c61bf56 100644
--- a/js/ui/calendar.js
+++ b/js/ui/calendar.js
@@ -834,8 +834,9 @@ var EventsSection = new Lang.Class({
this._title.connect('clicked', this._onTitleClicked.bind(this));
this._title.connect('key-focus-in', this._onKeyFocusIn.bind(this));
- Shell.AppSystem.get_default().connect('installed-changed',
- this._appInstalledChanged.bind(this));
+ this._appSys = Shell.AppSystem.get_default();
+ this._appSys.connect('installed-changed',
+ this._appInstalledChanged.bind(this));
this._appInstalledChanged();
},
@@ -933,10 +934,13 @@ var EventsSection = new Lang.Class({
Main.overview.hide();
Main.panel.closeCalendar();
- let app = this._getCalendarApp();
- if (app.get_id() == 'evolution.desktop')
- app = Gio.DesktopAppInfo.new('evolution-calendar.desktop');
- app.launch([], global.create_app_launch_context(0, -1));
+ let appInfo = this._getCalendarApp();
+ if (app.get_id() == 'evolution.desktop') {
+ let app = this._appSys.lookup_app('evolution-calendar.desktop');
+ if (app)
+ appInfo = app.app_info;
+ }
+ appInfo.launch([], global.create_app_launch_context(0, -1));
},
setDate(date) {
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]