use _return_single_option whenever there is a option lookup needed

This commit is contained in:
Andreas Zweili 2019-01-28 11:14:43 +01:00
parent 91eb47a536
commit a37125ec87
1 changed files with 6 additions and 6 deletions

View File

@ -32,11 +32,11 @@ class Config(QDialog):
def full_path(self):
"""returns the repository path or the repository server path if a
server was provided in the configuration."""
if 'repository_path' in self.config['borgqt']:
if 'server' in self.config['borgqt']:
return self._create_server_path()
if self._return_single_option('repository_path'):
if self._return_single_option('server'):
return self._create_server_path()
else:
return self.config['borgqt']['repository_path']
return self._return_single_option('repository_path')
else:
return ""
@ -106,9 +106,9 @@ class Config(QDialog):
def _create_server_path(self):
"""creates the full server path from the server, user and port
options."""
if 'user' not in self.config['borgqt']:
if not self._return_single_option('user'):
raise BorgException("User is missing in config.")
if 'port' not in self.config['borgqt']:
if not self._return_single_option('port'):
raise BorgException("Port is missing in config.")
server_path = (self.config['borgqt']['user']
+ "@"