configpaths: Add get_plugin_dirs() method

This helps with mypy because otherwise get() has multiple return types

This reduces our usage of cast()
This commit is contained in:
Philipp Hörist 2018-09-23 14:40:49 +02:00
parent 5db0178a15
commit ad5f59ba49
4 changed files with 12 additions and 15 deletions

View File

@ -24,7 +24,6 @@ from typing import List
from typing import Generator from typing import Generator
from typing import Optional # pylint: disable=unused-import from typing import Optional # pylint: disable=unused-import
from typing import Tuple from typing import Tuple
from typing import Union
import os import os
import sys import sys
@ -37,14 +36,16 @@ from gajim.common.const import PathType, PathLocation
from gajim.common.types import PathTuple from gajim.common.types import PathTuple
def get(key: str) -> Union[str, List[str]]: def get(key: str) -> str:
if key == 'PLUGINS_DIRS': return _paths[key]
def get_plugin_dirs() -> List[str]:
if gajim.IS_FLATPAK: if gajim.IS_FLATPAK:
return ['/app/plugins', return ['/app/plugins',
_paths['PLUGINS_BASE']] _paths['PLUGINS_BASE']]
return [_paths['PLUGINS_BASE'], return [_paths['PLUGINS_BASE'],
_paths['PLUGINS_USER']] _paths['PLUGINS_USER']]
return _paths[key]
def get_paths(type_: PathType) -> Generator[str, None, None]: def get_paths(type_: PathType) -> Generator[str, None, None]:

View File

@ -17,8 +17,6 @@
# You should have received a copy of the GNU General Public License # You should have received a copy of the GNU General Public License
# along with Gajim. If not, see <http://www.gnu.org/licenses/>. # along with Gajim. If not, see <http://www.gnu.org/licenses/>.
from typing import cast
import sys import sys
import os import os
import traceback import traceback
@ -42,7 +40,7 @@ if __name__ == '__main__':
glade_file = os.path.join('data', 'gui', 'exception_dialog.ui') glade_file = os.path.join('data', 'gui', 'exception_dialog.ui')
else: else:
from gajim.common import configpaths from gajim.common import configpaths
gui_path = cast(str, configpaths.get('GUI')) gui_path = configpaths.get('GUI')
glade_file = os.path.join(gui_path, 'exception_dialog.ui') glade_file = os.path.join(gui_path, 'exception_dialog.ui')

View File

@ -107,7 +107,7 @@ class PluginManager(metaclass=Singleton):
Registered names with instances of encryption Plugins. Registered names with instances of encryption Plugins.
''' '''
for path in reversed(configpaths.get('PLUGINS_DIRS')): for path in reversed(configpaths.get_plugin_dirs()):
pc = PluginManager.scan_dir_for_plugins(path) pc = PluginManager.scan_dir_for_plugins(path)
self.add_plugins(pc) self.add_plugins(pc)

View File

@ -14,8 +14,6 @@
# You should have received a copy of the GNU General Public License # You should have received a copy of the GNU General Public License
# along with Gajim. If not, see <http://www.gnu.org/licenses/>. # along with Gajim. If not, see <http://www.gnu.org/licenses/>.
from typing import cast
import os import os
import locale import locale
import gettext import gettext
@ -23,7 +21,7 @@ import gettext
from gajim.common import configpaths from gajim.common import configpaths
APP = 'gajim_plugins' APP = 'gajim_plugins'
plugin_user_dir = cast(str, configpaths.get('PLUGINS_USER')) plugin_user_dir = configpaths.get('PLUGINS_USER')
plugins_locale_dir = os.path.join(plugin_user_dir, 'locale') plugins_locale_dir = os.path.join(plugin_user_dir, 'locale')
if os.name != 'nt': if os.name != 'nt':