save / load CP generator settings doesn't work

Bug #685224 reported by zarl
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Hugin
Expired
Undecided
Unassigned

Bug Description

Applies to Harry's new beta2 of Hugin-2010.4.0 bundle as of 4 Dec 2010.

- clicking "Export" in Prefs -> CP Detectors if no entry is selected opens a file dialog that assumes that a setting could be exported. Instead, the Export button should be greyed if no entry is selected

- the 'Export' file dialog doesn't add the necessary '.setting' to a file name. This extension is needed for the 'import' file dialog to accept a file. Furthermore it might make sense to automatically use the description of the actual setting to name the settings file together with a date/time stamp, e.g. "cpfind-101204-1721.settings"

- Exported settings files are empty, so nothing will be imported yet (I also opened an exported .setting file with a text editor to make sure it's really empty).

- What will happen if I import a settings file with the same description (but different arguments) I already have in my prefs? Do I get the chance to rename it (favoured) or will it overwrite an existing setting (bad)?

Revision history for this message
tmodes (tmodes) wrote :

- "Export" saves all cp detector settings, not only the selected one (therefore point 1 and 4 are invalid)
- The missing extension is fixed in repository.
- I can not reproduce the empty setting file (on windows it is correctly saved). I added a "Flush" to explicitly save it. This hopefully fixes the issue, but I can not test.

Changed in hugin:
status: New → Triaged
tmodes (tmodes)
Changed in hugin:
status: Triaged → Incomplete
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for Hugin because there has been no activity for 60 days.]

Changed in hugin:
status: Incomplete → Expired
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.