Ticket #1011 (new defect)

Opened 5 years ago

Last modified 5 years ago

ActiveConnection should be removed

Reported by: Graham Cobb Owned by: dgollub
Priority: normal Milestone: OpenSync 0.40
Component: OpenSync: Application API Version: 0.38
Severity: minor Keywords:
Cc: tokoe, henrik

Description

The <ActiveConnection?> tag in the config file is a hack, which doesn't add much useful functionality and doesn't belong in the config file at all. It adds complexity to the config file to solve a UI problem and doesn't even solve it very well. It should be removed.

If it can't be removed, it should at least be made optional: it is only useful in the rare case where more than one connection, of different types, are specified in the config file. In the normal case it adds no value.

Change History

comment:1 Changed 5 years ago by dgollub

  • Component changed from OpenSync to OpenSync: Application API

This got introduced with r4286 due to #840

comment:2 Changed 5 years ago by dgollub

  • Cc tokoe, henrik added

Luckily there is no public API involved in this. So this doesn't block 0.39 This stays for 0.40

comment:3 Changed 5 years ago by henrik

I completely agree with Graham: This should be removed.

It is up to the frontend/GUI to keep track of different connections if the user wants to switch between different connections.

comment:4 Changed 5 years ago by henrik

If we are fixing #1013 anyway, it would make sense to fix this one as well at the same time...

It might not be a GUI change per se, but it certainly changes the way the GUI is used

comment:5 Changed 5 years ago by henrik

Sorry, please change "GUI" -> "API" in the above comment.

Note: See TracTickets for help on using tickets.