backlogops-gui 0.8__tar.gz → 1.0__tar.gz

{backlogops_gui-0.8 → backlogops_gui-1.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: backlogops-gui
-Version: 0.8
+Version: 1.0
 Summary: Graphical user interface for backlog operations.
 Author: Tom Björkholm
 Author-email: Tom Björkholm <klausuler_linnet0q@icloud.com>
@@ -21,10 +21,11 @@ Classifier: Typing :: Typed
 Requires-Python: >=3.12
 Description-Content-Type: text/markdown
 License-File: LICENSE.txt
-Requires-Dist: backlogops>=0.8
-Requires-Dist: argcomplete>=3.7.1
+Requires-Dist: backlogops>=1.0
+Requires-Dist: argcomplete>=3.7.2
 Requires-Dist: wizard-ui-bridge>=1.3
 Requires-Dist: wizard-tk-bridge>=1.3
+Requires-Dist: edit-cfg-json-tk==0.0.4
 Requires-Dist: versionreporter>=0.4
 Dynamic: author
 Dynamic: license-file
@@ -285,9 +286,20 @@ is in the menus.
       closed for vacation, and preset configuration for the inputs or
       outputs you want to use.
 
+    - Edit configuration...: Show the whole configuration in a folding
+      editor, so you can change one value without stepping through the
+      wizard. It asks whether to edit the configuration the application is
+      using or one in a file you pick. Saving validates the configuration
+      first, keeps what it writes over as a `.bak` file, and makes what it
+      wrote the configuration in use.
+
     - Create IO preset file...: Create a stand-alone input or output preset
       configuration file via a wizard.
 
+    - Edit IO preset file...: Show a stand-alone input or output preset file
+      in the same folding editor. Whether the file is an input or an output
+      preset is detected from the file itself.
+
     - Migrate IO preset file...: Migrate an older input or output preset
       file to the current file format.
 
@@ -350,10 +362,10 @@ one with the list of releases. You will want to use the menus.
 
 ## Test summary
 
-- Test result: 2173 passed, 1 deselected in 35s
+- Test result: 2234 passed, 1 deselected in 45s
 - No flake8 warnings.
 - No mypy errors found.
 - No pylint warnings.
 - No python layout warnings.
-- Built version(s): 0.8
-- Build and test using Python 3.14.6
+- Built version(s): 1.0
+- Build and test using Python 3.14.7

{backlogops_gui-0.8 → backlogops_gui-1.0}/README_pypi.md

@@ -252,9 +252,20 @@ is in the menus.
       closed for vacation, and preset configuration for the inputs or
       outputs you want to use.
 
+    - Edit configuration...: Show the whole configuration in a folding
+      editor, so you can change one value without stepping through the
+      wizard. It asks whether to edit the configuration the application is
+      using or one in a file you pick. Saving validates the configuration
+      first, keeps what it writes over as a `.bak` file, and makes what it
+      wrote the configuration in use.
+
     - Create IO preset file...: Create a stand-alone input or output preset
       configuration file via a wizard.
 
+    - Edit IO preset file...: Show a stand-alone input or output preset file
+      in the same folding editor. Whether the file is an input or an output
+      preset is detected from the file itself.
+
     - Migrate IO preset file...: Migrate an older input or output preset
       file to the current file format.
 
@@ -317,10 +328,10 @@ one with the list of releases. You will want to use the menus.
 
 ## Test summary
 
-- Test result: 2173 passed, 1 deselected in 35s
+- Test result: 2234 passed, 1 deselected in 45s
 - No flake8 warnings.
 - No mypy errors found.
 - No pylint warnings.
 - No python layout warnings.
-- Built version(s): 0.8
-- Build and test using Python 3.14.6
+- Built version(s): 1.0
+- Build and test using Python 3.14.7

{backlogops_gui-0.8 → backlogops_gui-1.0}/backlogops_gui.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: backlogops-gui
-Version: 0.8
+Version: 1.0
 Summary: Graphical user interface for backlog operations.
 Author: Tom Björkholm
 Author-email: Tom Björkholm <klausuler_linnet0q@icloud.com>
@@ -21,10 +21,11 @@ Classifier: Typing :: Typed
 Requires-Python: >=3.12
 Description-Content-Type: text/markdown
 License-File: LICENSE.txt
-Requires-Dist: backlogops>=0.8
-Requires-Dist: argcomplete>=3.7.1
+Requires-Dist: backlogops>=1.0
+Requires-Dist: argcomplete>=3.7.2
 Requires-Dist: wizard-ui-bridge>=1.3
 Requires-Dist: wizard-tk-bridge>=1.3
+Requires-Dist: edit-cfg-json-tk==0.0.4
 Requires-Dist: versionreporter>=0.4
 Dynamic: author
 Dynamic: license-file
@@ -285,9 +286,20 @@ is in the menus.
       closed for vacation, and preset configuration for the inputs or
       outputs you want to use.
 
+    - Edit configuration...: Show the whole configuration in a folding
+      editor, so you can change one value without stepping through the
+      wizard. It asks whether to edit the configuration the application is
+      using or one in a file you pick. Saving validates the configuration
+      first, keeps what it writes over as a `.bak` file, and makes what it
+      wrote the configuration in use.
+
     - Create IO preset file...: Create a stand-alone input or output preset
       configuration file via a wizard.
 
+    - Edit IO preset file...: Show a stand-alone input or output preset file
+      in the same folding editor. Whether the file is an input or an output
+      preset is detected from the file itself.
+
     - Migrate IO preset file...: Migrate an older input or output preset
       file to the current file format.
 
@@ -350,10 +362,10 @@ one with the list of releases. You will want to use the menus.
 
 ## Test summary
 
-- Test result: 2173 passed, 1 deselected in 35s
+- Test result: 2234 passed, 1 deselected in 45s
 - No flake8 warnings.
 - No mypy errors found.
 - No pylint warnings.
 - No python layout warnings.
-- Built version(s): 0.8
-- Build and test using Python 3.14.6
+- Built version(s): 1.0
+- Build and test using Python 3.14.7

{backlogops_gui-0.8 → backlogops_gui-1.0}/backlogops_gui.egg-info/SOURCES.txt

@@ -18,6 +18,7 @@ src/backlogops_gui/backlog_io.py
 src/backlogops_gui/backlog_window.py
 src/backlogops_gui/blog_version_reporter.py
 src/backlogops_gui/choice_dialogs.py
+src/backlogops_gui/config_edit.py
 src/backlogops_gui/file_choosers.py
 src/backlogops_gui/format_dialogs.py
 src/backlogops_gui/jira_actions.py

{backlogops_gui-0.8 → backlogops_gui-1.0}/backlogops_gui.egg-info/requires.txt

@@ -1,5 +1,6 @@
-backlogops>=0.8
-argcomplete>=3.7.1
+backlogops>=1.0
+argcomplete>=3.7.2
 wizard-ui-bridge>=1.3
 wizard-tk-bridge>=1.3
+edit-cfg-json-tk==0.0.4
 versionreporter>=0.4

{backlogops_gui-0.8 → backlogops_gui-1.0}/setup.py

@@ -5,7 +5,7 @@ from setuptools import setup
 
 setup(
   name='backlogops-gui',
-  version='0.8',
+  version='1.0',
   description='Graphical user interface for backlog operations.',
   author='Tom Björkholm',
   author_email='klausuler_linnet0q@icloud.com',
@@ -19,10 +19,11 @@ setup(
     ]
   },
   install_requires=[
-    'backlogops >= 0.8',
-    'argcomplete >= 3.7.1',
+    'backlogops >= 1.0',
+    'argcomplete >= 3.7.2',
     'wizard-ui-bridge >= 1.3',
     'wizard-tk-bridge >= 1.3',
+    'edit-cfg-json-tk == 0.0.4',
     'versionreporter >= 0.4'
   ]
 )

{backlogops_gui-0.8 → backlogops_gui-1.0}/src/backlogops_gui/application.py

@@ -3,7 +3,9 @@
 
 The application opens a main window whose menu reads a backlog from a file
 or from Jira, loads or replaces the active configuration from a file, runs
-the teams configuration wizard, creates a stand-alone input or output
+the teams configuration wizard, edits a configuration or a stand-alone
+preset file in the folding editor of
+:mod:`backlogops_gui.config_edit`, creates a stand-alone input or output
 preset file, migrates a stand-alone preset file to the current format,
 writes the running configuration to a file, and creates a demonstration
 backlog. The configuration wizard and the preset wizard first ask whether
@@ -41,10 +43,10 @@ from config_as_json import Config, migrate_cfg
 from config_as_json.file_extension import fix_file_extension
 from wizard_tk_bridge import WizardUiBridgeTk
 from backlogops import (
-    AvailableTeams, BacklogOpsConfig, BacklogReleases, GuiDisplayConfig,
-    InputFormatConfig, Levels, OutputFormatConfig, Status, get_demo_backlog,
-    get_backlog_ops_config, backlog_ops_wizard, preset_wizard,
-    read_backlog_ops_config, read_io_preset, safe_write_config,
+    AvailableTeams, BacklogOpsConfig, BacklogReleases, CONFIG_EXTENSION,
+    GuiDisplayConfig, InputFormatConfig, Levels, OutputFormatConfig, Status,
+    get_demo_backlog, get_backlog_ops_config, backlog_ops_wizard,
+    preset_wizard, read_backlog_ops_config, read_io_preset, safe_write_config,
     encrypt_token_file, encrypt_token_to_file)
 from backlogops_gui.backlog_io import read_backlog
 from backlogops_gui.backlog_window import (
@@ -53,6 +55,7 @@ from backlogops_gui.blog_version_reporter import BloGuiVersionReporter
 from backlogops_gui.choice_dialogs import (
     ConfigChoice, PresetKind, SourceChoice, ask_no_config_choice,
     ask_preset_kind, ask_source_choice)
+from backlogops_gui.config_edit import edit_config, edit_preset_file
 from backlogops_gui.file_choosers import (
     choose_config_file, choose_existing_config, choose_existing_preset,
     choose_input_file, choose_migrated_preset, choose_preset_to_migrate)
@@ -66,17 +69,17 @@ from backlogops_gui.python_version import check_python_version
 from backlogops_gui.tcltk_version import check_tcltk_version
 
 APP_TITLE = 'Backlog operations GUI'
-CONFIG_EXTENSION = '.cfg'
 WRAP_LENGTH = 520
 LOG_REFRESH_MS = 800
 HEADING_FONT = ('TkDefaultFont', 14, 'bold')
 INSTRUCTIONS = (
     'Use the menus to read a backlog from a file or from Jira, load a '
-    'configuration file, run the teams wizard, create a stand-alone input '
-    'or output preset file, migrate a preset file to the current format, '
-    'write the current configuration to a file, or create a demonstration '
-    'backlog. Each backlog opens in its own window. On macOS the menu bar '
-    'is at the top of the display.')
+    'configuration file, run the teams wizard, edit a configuration or a '
+    'preset file, create a stand-alone input or output preset file, '
+    'migrate a preset file to the current format, write the current '
+    'configuration to a file, or create a demonstration backlog. Each '
+    'backlog opens in its own window. On macOS the menu bar is at the top '
+    'of the display.')
 DESCRIPTION = 'Graphical user interface for backlog operations'
 CONFIG_ERRORS = (FileNotFoundError, NotADirectoryError, RuntimeError,
                  ValueError, TypeError, KeyError, OSError)
@@ -155,7 +158,7 @@ class BacklogApp:
 
     def __init__(self, root: tk.Tk,
                  config: Optional[BacklogOpsConfig] = None) -> None:
-        """Store the window, config, log and Jira collaborators."""
+        """Store the window, config, log and the action collaborators."""
         self.root = root
         self.config = config
         self.log = LogBuffer()
@@ -164,6 +167,21 @@ class BacklogApp:
         self._status: Optional[tk.StringVar] = None
         self.jira = JiraActions(self)
 
+    def adopt_config(self, config: BacklogOpsConfig, source: str) -> None:
+        """Make one configuration the active one and say where it came from.
+
+        Every way a configuration becomes the active one goes through here —
+        loading a file, the wizard, and the editor — so the status line
+        cannot end up saying one thing while another is in use.
+
+        Args:
+            config: The configuration to use from now on.
+            source: Where it came from, as a file name or a short phrase.
+        """
+        self.config = config
+        self.config_source = source
+        self._update_status()
+
     def in_presets(self) -> Optional[dict[str, InputFormatConfig]]:
         """Return the input presets of the current configuration."""
         return self.config.input_configs if self.config else None
@@ -214,9 +232,8 @@ class BacklogApp:
         """
         config, error = initial_config(config_arg, self.log)
         if config is not None:
-            self.config = config
-            self.config_source = (config_arg if config_arg is not None
-                                  else 'the default location')
+            self.adopt_config(config, config_arg if config_arg is not None
+                              else 'the default location')
             return True
         if config_arg is not None and error is not None:
             self.show_error('Configuration error', error)
@@ -244,8 +261,7 @@ class BacklogApp:
         config = self.run_wizard()
         if config is None:
             return False
-        self.config = config
-        self.config_source = 'the wizard'
+        self.adopt_config(config, 'the wizard')
         return True
 
     def _adopt_loaded_config(self) -> bool:
@@ -258,8 +274,7 @@ class BacklogApp:
             self.show_error('Configuration error',
                             error or 'Could not load the configuration.')
             return False
-        self.config = config
-        self.config_source = path
+        self.adopt_config(config, path)
         return True
 
     def _run_bridge_wizard(self, wizard: Callable[..., _WizardConfig],
@@ -352,9 +367,7 @@ class BacklogApp:
             self.show_error('Configuration error',
                             error or 'Could not load the configuration.')
             return
-        self.config = config
-        self.config_source = path
-        self._update_status()
+        self.adopt_config(config, path)
         self.show_info('Configuration loaded',
                        f'Loaded configuration from {path}.')
 
@@ -374,11 +387,17 @@ class BacklogApp:
             return
         config = self.run_wizard(default)
         if config is not None:
-            self.config = config
-            self.config_source = 'the wizard'
-            self._update_status()
+            self.adopt_config(config, 'the wizard')
             self.show_info('Wizard', 'The new configuration is now active.')
 
+    def run_config_editor(self) -> None:
+        """Edit the configuration in use, or one in a file, in the editor."""
+        edit_config(self)
+
+    def run_preset_editor(self) -> None:
+        """Edit a stand-alone input or output preset file in the editor."""
+        edit_preset_file(self)
+
     def create_preset_file(self) -> None:
         """Ask the source, run the IO preset wizard, and write the preset.
 
@@ -651,8 +670,12 @@ class BacklogApp:
         menu.add_separator()
         menu.add_command(label='Run configuration wizard…',
                          command=self.run_config_wizard)
+        menu.add_command(label='Edit configuration…',
+                         command=self.run_config_editor)
         menu.add_command(label='Create IO preset file…',
                          command=self.create_preset_file)
+        menu.add_command(label='Edit IO preset file…',
+                         command=self.run_preset_editor)
         menu.add_command(label='Migrate IO preset file…',
                          command=self.migrate_preset_file)
         menu.add_command(label='Write configuration…',

{backlogops_gui-0.8 → backlogops_gui-1.0}/src/backlogops_gui/choice_dialogs.py

@@ -6,8 +6,10 @@ selecting one enumerated value, with no OK or Cancel. The no-configuration
 dialog offers to run the wizard, load a file, or exit at startup. The
 preset-kind dialog asks whether a stand-alone preset file is an input or
 an output preset before it is migrated. The source dialog asks whether to
-start a wizard from scratch, base it on an existing file, or cancel. All
-three are built from the same :class:`ButtonChoiceDialog`.
+start a wizard from scratch, base it on an existing file, or cancel. The
+edit-target dialog asks whether the configuration editor opens the
+configuration the application is using or one in a file. All four are built
+from the same :class:`ButtonChoiceDialog`.
 """
 
 # Copyright (c) 2026, Tom Björkholm
@@ -26,6 +28,11 @@ NO_CONFIG_TEXT = (
 PRESET_KIND_TEXT = (
     'Is the preset file an input format preset or an output format '
     'preset? The kind decides how the file is migrated.')
+EDIT_TARGET_TITLE = 'Edit configuration'
+EDIT_TARGET_TEXT = (
+    'Edit the configuration the application is using, or one in a file '
+    'you pick? Saving the configuration in use writes the file it was '
+    'loaded from, and Save as… in the editor writes any other file.')
 _Choice = TypeVar('_Choice')
 
 
@@ -52,6 +59,14 @@ class SourceChoice(Enum):
     CANCEL = 'cancel'
 
 
+class EditTargetChoice(Enum):
+    """Whether the editor opens the configuration in use or one in a file."""
+
+    IN_USE = 'in_use'
+    FROM_FILE = 'from_file'
+    CANCEL = 'cancel'
+
+
 # pylint: disable-next=too-few-public-methods
 class ButtonChoiceDialog(Generic[_Choice]):
     """Modal dialog presenting a column of single-choice buttons.
@@ -130,3 +145,12 @@ def ask_source_choice(parent: tk.Misc, title: str, text: str) -> SourceChoice:
                ('Cancel', SourceChoice.CANCEL)]
     return ButtonChoiceDialog(parent, title, text, options,
                               SourceChoice.CANCEL).choice
+
+
+def ask_edit_target(parent: tk.Misc) -> EditTargetChoice:
+    """Ask whether to edit the configuration in use or one in a file."""
+    options = [('The configuration in use', EditTargetChoice.IN_USE),
+               ('A configuration file…', EditTargetChoice.FROM_FILE),
+               ('Cancel', EditTargetChoice.CANCEL)]
+    return ButtonChoiceDialog(parent, EDIT_TARGET_TITLE, EDIT_TARGET_TEXT,
+                              options, EditTargetChoice.CANCEL).choice

backlogops_gui-1.0/src/backlogops_gui/config_edit.py

@@ -0,0 +1,261 @@
+#! /usr/local/bin/python3
+"""Edit a configuration in a window of the application.
+
+The two edit actions of the configuration menu live here.
+:func:`edit_config` opens the configuration the application is using, or one
+in a file the user picks; :func:`edit_preset_file` opens a stand-alone input
+or output preset file, whose direction is detected from the file itself.
+What can be edited, what a save writes, and what each member is for all
+belong to :mod:`backlogops.config_editing`, so the editor of the terminal
+interface shows exactly the same configuration.
+
+These are functions taking the application rather than a collaborator
+object, because an editing session keeps nothing between two of them: the
+session belongs to the window and everything else is the application's.
+
+The editor is mounted in a :class:`tkinter.Toplevel` this module creates,
+rather than started through ``edit_cfg_json_tk.edit``. That entry point
+creates a ``tkinter.Tk`` and an event loop of its own, which is for an
+application that runs neither yet: a second Tcl interpreter shares nothing
+with the first, and a nested loop would not end when the editor window
+closed, because Tcl runs its loop while any window of the process lives.
+``TkEditorPanel`` is what the library offers for an application that already
+runs Tk. It is given the window as its ``area``, so that the window stays
+this module's — its title names the file being edited, and the editor
+destroys only what it built itself.
+
+The window is not made modal. The editor opens dialogs of its own — a file
+chooser for Save as…, a question before it overwrites a file, and one asking
+for the key of a new entry — and a grab held by the editor window would keep
+their clicks and keys from reaching them.
+"""
+
+# Copyright (c) 2026, Tom Björkholm
+# MIT License
+
+import tkinter as tk
+from pathlib import Path
+from typing import NamedTuple, Optional, TYPE_CHECKING
+from config_as_json import Config, PathOrStr
+from edit_cfg_json import ConfigLoadError, EditModel, default_config
+from edit_cfg_json_tk import TkEditorPanel
+from wizard_tk_bridge.close_binding import bind_close
+from backlogops import (
+    BacklogOpsConfig, EDIT_SETTINGS, descriptions_for, io_preset_class)
+from backlogops_gui.choice_dialogs import EditTargetChoice, ask_edit_target
+from backlogops_gui.file_choosers import (
+    choose_config_to_edit, choose_preset_to_edit)
+
+if TYPE_CHECKING:
+    from backlogops_gui.application import BacklogApp
+
+EDIT_ERRORS = (ConfigLoadError, ValueError, TypeError, KeyError, OSError)
+"""Errors raised when a configuration cannot be opened for editing."""
+
+OPEN_FAIL_TITLE = 'Could not open the configuration'
+NO_CONFIG_TEXT = 'There is no configuration to edit.'
+NOT_SAVED_TEXT = 'The editor was closed without saving anything.'
+EDITOR_TITLE = 'Configuration editor'
+ADOPTED = 'The saved configuration is now the one in use.'
+
+
+class EditorWindow(NamedTuple):
+    """One editor window and the panel running the session inside it."""
+
+    window: tk.Toplevel
+    panel: TkEditorPanel
+
+
+def editor_window(app: 'BacklogApp', config: Config, title: str, *,
+                  in_file: Optional[PathOrStr] = None,
+                  out_file: Optional[PathOrStr] = None) -> EditorWindow:
+    """Create a window of the application with the editor mounted in it.
+
+    Every way out of the editor — its Close button, its key, the close
+    button of the window and the platform close key — goes through the
+    editor's own close action, so none of them can drop an unsaved change
+    without asking. Closing takes the editor off this window, and the
+    window then goes with it.
+
+    Args:
+        app: The application, whose window this one belongs to.
+        config: Configuration object of the class to edit, holding the
+            values to start from when there is no input file.
+        title: Title of the window, saying what is being edited.
+        in_file: Configuration file to read, or None to edit the values
+            that ``config`` holds.
+        out_file: Configuration file a save writes, or None to write the
+            input file. With neither, the editor asks the user for one.
+
+    Returns:
+        The window and the panel that the editor runs in.
+
+    Raises:
+        ConfigLoadError: The input file cannot be opened for editing. The
+            window is taken away again before this is raised.
+    """
+    window = tk.Toplevel(app.root)
+    window.title(title)
+    window.transient(app.root)
+    try:
+        panel = TkEditorPanel(config, area=window, modal=False,
+                              on_close=window.destroy,
+                              descriptions=descriptions_for(config),
+                              in_file=in_file, out_file=out_file,
+                              settings=EDIT_SETTINGS, stderr_file=app.log)
+    except EDIT_ERRORS:
+        window.destroy()
+        raise
+    window.protocol('WM_DELETE_WINDOW', panel.close)
+    bind_close(window, panel.close)
+    return EditorWindow(window=window, panel=panel)
+
+
+def edit_in_window(app: 'BacklogApp', config: Config, title: str, *,
+                   in_file: Optional[PathOrStr] = None,
+                   out_file: Optional[PathOrStr] = None) -> EditModel:
+    """Show one configuration in a window of its own until it is closed.
+
+    Args:
+        app, config, title, in_file, out_file: As of :func:`editor_window`.
+
+    Returns:
+        The model of the session that has just ended, which says what it
+        saved.
+
+    Raises:
+        ConfigLoadError: The input file cannot be opened for editing.
+    """
+    mounted = editor_window(app, config, title, in_file=in_file,
+                            out_file=out_file)
+    mounted.window.wait_window()
+    return mounted.panel.model
+
+
+def open_editor_window(app: 'BacklogApp', config: Config, title: str, *,
+                       in_file: Optional[PathOrStr] = None,
+                       out_file: Optional[PathOrStr] = None
+                       ) -> Optional[EditModel]:
+    """Run one editing session in a window, or report why it cannot run.
+
+    Args:
+        app, config, title, in_file, out_file: As of :func:`editor_window`.
+
+    Returns:
+        The model of the session that ran, or None when the configuration
+        could not be opened for editing, which is then reported.
+    """
+    try:
+        return edit_in_window(app, config, title, in_file=in_file,
+                              out_file=out_file)
+    except EDIT_ERRORS as error:
+        _report_failure(app, error)
+        return None
+
+
+def edit_config(app: 'BacklogApp') -> None:
+    """Edit the configuration in use, or one in a file the user picks.
+
+    A configuration the editor saved becomes the active configuration,
+    whichever of the two was edited, because the user has just said that
+    those are the values they want. Cancelling any step, and closing the
+    editor without saving, leaves everything as it was.
+    """
+    choice = ask_edit_target(app.root)
+    if choice is EditTargetChoice.CANCEL:
+        return
+    model = (_edit_in_use(app) if choice is EditTargetChoice.IN_USE
+             else _edit_config_file(app))
+    if model is not None:
+        _adopt_saved(app, model)
+
+
+def edit_preset_file(app: 'BacklogApp') -> None:
+    """Edit a stand-alone input or output preset file.
+
+    The direction of the file is detected from its own contents, so the user
+    picks a preset file and nothing else. What the editor writes is a file
+    and not a configuration of the application, so nothing is adopted.
+    """
+    path = choose_preset_to_edit(app.root)
+    if path is None:
+        return
+    try:
+        config = default_config(io_preset_class(path))
+    except EDIT_ERRORS as error:
+        _report_failure(app, error)
+        return
+    model = open_editor_window(app, config, f'Edit preset {Path(path).name}',
+                               in_file=path)
+    if model is not None:
+        _report_saved(app, model, 'Preset saved')
+
+
+def _edit_in_use(app: 'BacklogApp') -> Optional[EditModel]:
+    """Edit the configuration the application uses, in a window.
+
+    A save writes the file the configuration was loaded from, when it came
+    from one; a configuration that came from the wizard has no file yet and
+    the editor asks for one before it can save.
+    """
+    config = app.config
+    if config is None:
+        app.show_error('No configuration', NO_CONFIG_TEXT)
+        return None
+    out_file = _loaded_file(app)
+    return open_editor_window(app, config, _config_title(out_file),
+                              out_file=out_file)
+
+
+def _edit_config_file(app: 'BacklogApp') -> Optional[EditModel]:
+    """Edit a configuration file the user picks, in a window."""
+    path = choose_config_to_edit(app.root)
+    if path is None:
+        return None
+    return open_editor_window(app, default_config(BacklogOpsConfig),
+                              _config_title(path), in_file=path)
+
+
+def _config_title(name: Optional[str]) -> str:
+    """Return the window title, naming the file when the session has one."""
+    if name is None:
+        return 'Edit configuration'
+    return f'Edit configuration {Path(name).name}'
+
+
+def _loaded_file(app: 'BacklogApp') -> Optional[str]:
+    """Return the file the active configuration was loaded from, if any.
+
+    The source of a configuration is either a file name or a phrase saying
+    where else it came from, so only a name that is a file now is a
+    destination that a save may write.
+    """
+    source = app.config_source
+    if source is None or not Path(source).is_file():
+        return None
+    return source
+
+
+def _report_failure(app: 'BacklogApp', error: Exception) -> None:
+    """Log and show why a configuration cannot be opened for editing."""
+    app.log.write(f'{OPEN_FAIL_TITLE}: {error}\n')
+    app.show_error(OPEN_FAIL_TITLE, str(error))
+
+
+def _adopt_saved(app: 'BacklogApp', model: EditModel) -> None:
+    """Make what the editor saved the configuration the application uses."""
+    saved = model.saved_config
+    if saved is None:
+        app.show_info(EDITOR_TITLE, NOT_SAVED_TEXT)
+        return
+    assert isinstance(saved, BacklogOpsConfig)
+    app.adopt_config(saved, str(model.out_file))
+    app.show_info('Configuration saved', f'{model.save_message}\n{ADOPTED}')
+
+
+def _report_saved(app: 'BacklogApp', model: EditModel, title: str) -> None:
+    """Report what the editor wrote, or that it wrote nothing."""
+    if model.saved_config is None:
+        app.show_info(EDITOR_TITLE, NOT_SAVED_TEXT)
+        return
+    app.show_info(title, model.save_message)

{backlogops_gui-0.8 → backlogops_gui-1.0}/src/backlogops_gui/file_choosers.py

@@ -48,6 +48,19 @@ def choose_existing_preset(parent: tk.Misc) -> Optional[str]:
     return name or None
 
 
+def choose_config_to_edit(parent: tk.Misc) -> Optional[str]:
+    """Ask for an existing configuration file to edit, or None to cancel."""
+    name = filedialog.askopenfilename(parent=parent,
+                                      title='Edit configuration file')
+    return name or None
+
+
+def choose_preset_to_edit(parent: tk.Misc) -> Optional[str]:
+    """Ask for an existing preset file to edit, or None when cancelled."""
+    name = filedialog.askopenfilename(parent=parent, title='Edit preset file')
+    return name or None
+
+
 def choose_preset_to_migrate(parent: tk.Misc) -> Optional[str]:
     """Ask for an existing preset file to migrate, or None when cancelled."""
     name = filedialog.askopenfilename(parent=parent,