Fix PluginsManager context menu crashing

[jjw24]

Repro:
Going to any context menu other than pm install and select open website will get the exception pop up

Fix:

• Added handling when the object can not be cast as UserPlugins. This means the selected result was not provided the plugins list from the manifest, as a result we bypass creating menu items for them.
• Added context menu for pm update

Tested:

• pm install and pm update all can use the context menu
• all other context menus will go to the default items with out those options available in install and update

[taooceros]

Could you also add the contextdata in RequestUninstall

Flow.Launcher/Plugins/Flow.Launcher.Plugin.PluginsManager/PluginsManager.cs

Lines 390 to 418 in 2777795

	var results = Context.API
	.GetAllPlugins()
	.Select(x =>
	new Result
	{
	Title = $"{x.Metadata.Name} by {x.Metadata.Author}",
	SubTitle = x.Metadata.Description,
	IcoPath = x.Metadata.IcoPath,
	Action = e =>
	{
	string message = string.Format(
	Context.API.GetTranslation("plugin_pluginsmanager_uninstall_prompt"),
	x.Metadata.Name, x.Metadata.Author,
	Environment.NewLine, Environment.NewLine);
	if (MessageBox.Show(message,
	Context.API.GetTranslation("plugin_pluginsmanager_uninstall_title"),
	MessageBoxButton.YesNo) == MessageBoxResult.Yes)
	{
	Application.Current.MainWindow.Hide();
	Uninstall(x.Metadata);
	Context.API.RestartApp();
	return true;
	}
	return false;
	}
	});

[jjw24]

Could you also add the contextdata in RequestUninstall

Flow.Launcher/Plugins/Flow.Launcher.Plugin.PluginsManager/PluginsManager.cs

Lines 390 to 418 in 2777795

	var results = Context.API
	.GetAllPlugins()
	.Select(x =>
	new Result
	{
	Title = $"{x.Metadata.Name} by {x.Metadata.Author}",
	SubTitle = x.Metadata.Description,
	IcoPath = x.Metadata.IcoPath,
	Action = e =>
	{
	string message = string.Format(
	Context.API.GetTranslation("plugin_pluginsmanager_uninstall_prompt"),
	x.Metadata.Name, x.Metadata.Author,
	Environment.NewLine, Environment.NewLine);
	if (MessageBox.Show(message,
	Context.API.GetTranslation("plugin_pluginsmanager_uninstall_title"),
	MessageBoxButton.YesNo) == MessageBoxResult.Yes)
	{
	Application.Current.MainWindow.Hide();
	Uninstall(x.Metadata);
	Context.API.RestartApp();
	return true;
	}
	return false;
	}
	});

Because we are not creating results from UserPlugins object, it's going to require a whole lot of extra code to match up, bare in mind default plugins are not in the manifest list so it's not in the UserPlugins object.

Just thought this is not so necessary to implement unless you feel otherwise.

[taooceros]

Because we are not creating results from UserPlugins object, it's going to require a whole lot of extra code to match up, bare in mind default plugins are not in the manifest list so it's not in the UserPlugins object.

Just thought this is not so necessary to implement unless you feel otherwise.

Yeah probably not that easy to do so. Leave it in the future if needed.