Bug 193182 - [UI] If disconnected, user should be prompted to reconnect
Summary: [UI] If disconnected, user should be prompted to reconnect
Status: RESOLVED WONTFIX
Alias: None
Product: ECF
Classification: RT
Component: ecf.ui (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 enhancement (vote)
Target Milestone: 1.0.2   Edit
Assignee: Mariot Chauvin CLA
QA Contact:
URL:
Whiteboard:
Keywords: bugday, helpwanted
Depends on:
Blocks:
 
Reported: 2007-06-18 15:30 EDT by Chris Aniszczyk CLA
Modified: 2014-05-09 12:42 EDT (History)
3 users (show)

See Also:

Attachments
patch for the ChatRoomManagerUI java file (2.25 KB, patch)
2007-08-31 19:17 EDT, Mariot Chauvin CLA 		no flags Details | Diff
patch for Messages.java and Messages.properties files (2.04 KB, patch)
2007-08-31 19:18 EDT, Mariot Chauvin CLA 		no flags Details | Diff
new Dialog to reconnect (2.85 KB, text/x-java)
2007-08-31 19:20 EDT, Mariot Chauvin CLA 		no flags Details
new Dialog, update messages (5.96 KB, patch)
2007-08-31 20:38 EDT, Mariot Chauvin CLA 		no flags Details | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Aniszczyk CLA 2007-06-18 15:30:18 EDT 
If a user is disconnected from a provider like IRC, it would be nice to get a popup informing the user he/she were disconnected and if they would like to reconnect using the same settings.
Comment 1 Scott Lewis CLA 2007-07-22 21:42:09 EDT 
Setting target milestone.  Hopefully this can be addressed on bugday.  Should only involve additional UI.
Comment 2 Mariot Chauvin CLA 2007-08-22 08:04:15 EDT 
Even if I am totally new with ECF, I think I can do it for the next eclipse bug day 31.08.2007. Is it ok ?
Comment 3 Remy Suen CLA 2007-08-27 10:07:59 EDT 
It's all yours.
Comment 4 Mariot Chauvin CLA 2007-08-31 19:17:10 EDT 
Created attachment 77520 [details]
patch for the ChatRoomManagerUI java file
Comment 5 Mariot Chauvin CLA 2007-08-31 19:18:05 EDT 
Created attachment 77521 [details]
patch for Messages.java and Messages.properties files
Comment 6 Mariot Chauvin CLA 2007-08-31 19:20:37 EDT 
Created attachment 77522 [details]
new Dialog  to reconnect
Comment 7 Mariot Chauvin CLA 2007-08-31 19:26:02 EDT 
Could someone review the patches attached. I think is is not really resolve the bug, because the user has to enter its password to reconnect (I don't know if username was necessary ?), but I don't find how to get these info from the lost connection.
Comment 8 Mariot Chauvin CLA 2007-08-31 20:38:06 EDT 
Created attachment 77524 [details]
new Dialog, update messages

Get automatically the saved password. code inspired by IRCConnectWizard
Comment 9 Remy Suen CLA 2007-09-02 00:16:03 EDT 
Hi Mariot, I'm a little curious as to how you got this code to compile because attachment 77520 [details] makes a call to IContainer's connect(ID, IConnectContext) which throws a checked ContainerConnectException but you have no try/catch clause around it anywhere.

Another issue I noticed was that the connect method is being called from the UI thread. This will hang the UI while the provider is trying to make the connection, which prevents the user from doing anything and gives them the impression that the program has hung (and for all we know maybe it really did hang).
Comment 10 Scott Lewis CLA 2014-05-09 12:42:38 EDT 
Resolving as wontfix due to lack of resources.  If contributor or committer resources become available, then please reopen.