mirrors/dotdrop
1
0
Fork 0
mirror of https://github.com/deadc0de6/dotdrop.git synced 2026-02-04 17:24:46 +00:00
Code Activity

Using try-catch KeyError: instead of if in config parser

Browse Source
This commit is contained in:
Davide Laezza
2019-04-19 19:25:51 +02:00
parent f84920e2e6
commit 917b1a9ce0
1 changed files with 28 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

44
dotdrop/config.py
View File

@@ -244,9 +244,11 @@ class Cfg:
self.lnk_settings[self.key_workdir] = self._abs_path(self.curworkdir)
# load external variables/dynvariables
if self.key_import_vars in self.lnk_settings:
paths = self.lnk_settings[self.key_import_vars]
try:
paths = self.lnk_settings[self.key_import_vars] or []
self._load_ext_variables(paths, profile=profile)
except KeyError:
pass
# load global upignore
if self.key_upignore in self.lnk_settings:
@@ -263,28 +265,38 @@ class Cfg:
if self.debug:
self.log.dbg('loading actions from {}'.format(path))
content = self._load_yaml(path)
if self.key_actions in content and \
content[self.key_actions] is not None:
self._load_actions(content[self.key_actions])
# If external actions are None, replaces them with empty dict
try:
external_actions = content[self.key_actions] or {}
self._load_actions(external_actions)
except KeyError:
pass
except KeyError:
pass
# parse local actions
if self.key_actions in self.content and \
self.content[self.key_actions] is not None:
self._load_actions(self.content[self.key_actions])
# If local actions are None, replaces them with empty dict
try:
local_actions = self.content[self.key_actions] or {}
self._load_actions(local_actions)
except KeyError:
pass
# parse read transformations
if self.key_trans_r in self.content and \
self.content[self.key_trans_r] is not None:
for k, v in self.content[self.key_trans_r].items():
self.trans_r[k] = Transform(k, v)
# If read transformations are None, replaces them with empty dict
try:
read_trans = self.content[self.key_trans_r] or {}
self.trans_r = {k: Transform(k, v) for k, v in read_trans.items()}
except KeyError:
pass
# parse write transformations
if self.key_trans_w in self.content and \
self.content[self.key_trans_w] is not None:
for k, v in self.content[self.key_trans_w].items():
self.trans_w[k] = Transform(k, v)
# If write transformations are None, replaces them with empty dict
try:
read_trans = self.content[self.key_trans_w] or {}
self.trans_w = {k: Transform(k, v) for k, v in read_trans.items()}
except KeyError:
pass
# parse the dotfiles and construct the dict of objects per dotfile key
# ensures the dotfiles entry is a dict