EdgeList Database Configuration is not saved

Bug #571263 reported by Mathieu Bastian on 2010-04-28
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Gephi
Fix Released
Medium
Unassigned
0.7
Fix Released
Medium
Unassigned

Bug Description

The EdgeList Database import window should save current configuration and propose it to the user.

This is not the case currently and force users to re-enter connection information.

André Panisson (panisson) wrote :

I would like to propose a patch I was working the last days while I am getting used with the environment.
Any suggestions about improving it are welcome!

Great, I tried your patch it works but I have maybe a suggestion?

The best wouldn't be to save the configuration automatically instead proposing it to the user ?

I explain: The user enters a configuration, with a configuration name (important) and all details. When he hits OK the configuration is saved. Then when the window is opened again the configuration has appeared in the configuration combo-box. If he changes the configuration name that creates a new configuration, or just save the details otherwise. Does that makes sense?

I think it provides better user experience than letting users manage configuration files.

André Panisson (panisson) wrote :

I'm attaching a new patch with Mathieu's suggestions, and other improvements like better internationalization and exception handling.

André Panisson (panisson) wrote :

Sorry, the last patch was incomplete. This one is complete.

André Panisson (panisson) wrote :

Sorry, the last patch was incomplete. This one is complete.

Great it works nicely, thank you.

I committed in the trunk in rev 1405.

Before committing I made a small change on Remove Configuration button. It has now an icon and is enabled/disabled instead of visible/hidden. I think this is more logical for users.

Changed in gephi:
status: Confirmed → Fix Committed
milestone: 0.7alpha3 → 0.7alpha4
Changed in gephi:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers