Jaculabilis
/
amanuensis
1
1
Fork 0
Code Issues 19 Pull Requests Projects 1 Releases Activity

Unnecessary to pass both root and model factory

This commit is contained in:
Tim Van Baak 2020-04-24 10:53:19 -07:00
parent 6d0aa3eb34
commit 9252e524ae
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
amanuensis/cli/lexicon.py
View File

@ -43,7 +43,7 @@ def command_create(args):
return -1 return -1
# Perform command # Perform command
create_lexicon(root, args.model_factory, args.name, args.user) create_lexicon(root, args.name, args.user)
# Output already logged by create_lexicon # Output already logged by create_lexicon
return 0 return 0

2
amanuensis/lexicon/admin.py
View File

@ -28,7 +28,6 @@ def valid_name(name: str) -> bool:
def create_lexicon( def create_lexicon(
root: RootConfigDirectoryContext, root: RootConfigDirectoryContext,
model_factory: ModelFactory,
name: str, name: str,
editor: UserModel) -> LexiconModel: editor: UserModel) -> LexiconModel:
""" """
@ -74,6 +73,7 @@ def create_lexicon(
pass # Create an empry config file pass # Create an empry config file
# Load the lexicon and add the editor and default character # Load the lexicon and add the editor and default character
model_factory: ModelFactory = ModelFactory(root)
lexicon = model_factory.lexicon(lid) lexicon = model_factory.lexicon(lid)
with lexicon.ctx.edit_config() as cfg: with lexicon.ctx.edit_config() as cfg:
cfg.join.joined.append(editor.uid) cfg.join.joined.append(editor.uid)