Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Blank plugins preferences in git master #667

Closed
glitsj16 opened this issue Sep 3, 2018 · 11 comments
Closed

Blank plugins preferences in git master #667

glitsj16 opened this issue Sep 3, 2018 · 11 comments
Assignees
Labels

Comments

@glitsj16
Copy link
Contributor

glitsj16 commented Sep 3, 2018

By coincidence I wanted to check Liferea's plugins preferences, only to notice the view is empty (in Liferea compiled from current git master). Plugins still work though, only the plugins preferences pane is affected.

After some rebuilds and more testing it looks like this recent plugin-related commit is causing the issue for me (on both Arch Linux and Ubuntu 16.04 LTS). At least the issue disappears when reverting the aforementioned commit. I hoped to offer some kind of debug log, yet running with --debug-all didn't produce anything helpful (or I missed it). If anyone has pointers on how to debug this further, I would be more than happy to do so.

Regards

@lwindolf
Copy link
Owner

lwindolf commented Sep 3, 2018

Ah yes. I forgot to strip the "Plugins" tab from the preferences dialog.

The idea is to have only one place for "Plugins" which is the plugin installer available from menu via "Tools" -> "Plugins".

@lwindolf lwindolf self-assigned this Sep 3, 2018
@lwindolf lwindolf added the bug label Sep 3, 2018
@Leiaz
Copy link
Collaborator

Leiaz commented Sep 3, 2018

But the plugin installer is itself a plugin. If it replaces the plugins preference tab, maybe it should be made part of the app first.

@lwindolf
Copy link
Owner

lwindolf commented Sep 3, 2018

It is a mandatory and hidden plugin. It cannot be disabled by the user.

@sillyslux
Copy link

Now i have neither Tools->Preferences->Plugins nor Tools->Plugins. Am i missing some python dependency?

@glitsj16
Copy link
Contributor Author

@sillyslux Seeing the same thing. I don't think it's a missing dependency issue. Looks more like something was overlooked in afc3c04. Until properly fixed, you should be able to get/set active plugins via dconf. Here are example commands to toggle on/off the headerbar plugin:

  • get list of active plugins:
    $ gsettings get net.sf.liferea.plugins active-plugins
    ['media-player']
  • enable headerbar plugin:
    $ gsettings set net.sf.liferea.plugins active-plugins "['headerbar', 'media-player']"
  • disable headerbar plugin:
    $ gsettings set net.sf.liferea.plugins active-plugins "['media-player']"

@lwindolf
Copy link
Owner

@sillyslux @glitsj16 Above commit ensures the plugin installer cannot be disabled. I hope this helps.

@glitsj16
Copy link
Contributor Author

@lwindolf Thanks for looking in to this. There seems to be some confusion though. As sillyslux mentioned above, the issue is not that plugins aren't loaded/unloaded or accidentally disabled somehow. Current git master simply does not show a GUI menu item 'Tools > Plugins' (not even when including your latest commit).

Looking at https://github.com/lwindolf/liferea/blob/master/plugins/plugin-installer.py now to figure out why that menu item is still missing.

@Leiaz
Copy link
Collaborator

Leiaz commented Sep 16, 2018

@lwindolf You made the plugin installer plugin Builtin, but according to the doc it means it is Liferea's responsibility to load it, and currently I don't see anything ensuring it is loaded, regardless of user's settings (for users who didn't enable it previously).

@lwindolf
Copy link
Owner

@Leiaz @glitsj16 I think now I understand.

A partial fix (adding the plugin installer as installed per default in the DConf schema) was added with the commit above. There needs to be additional mandatory enabling in the code itself...

@lwindolf lwindolf reopened this Sep 17, 2018
@Leiaz
Copy link
Collaborator

Leiaz commented Sep 17, 2018

I think it should be loaded in liferea_plugins_engine_init at the end after the g_settings_bind.

@lwindolf
Copy link
Owner

Agreed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants