rvmiller89 / multitype

Eclipse Plug-in for collaborative code editing similar to Google Docs
0 stars 0 forks source link

Client Crashes if File/User Window not Visible #35

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Connect to a server with the file window open, but not visible.
2. FileList will not be correctly updated

Client Log:

Received: FEU: NF | Type: Connection_Succeed | ASGN ID: 2

GetUpdate: FEU: NF | Type: Connection_Succeed | ASGN ID: 2
Sent: FEU: NF | Type: User_Connected | UID: 2 | UNAME: john6

Received: FEU: NF | Type: User_Connected | UID: 2 | UNAME: john6

Received: FEU: NF | Type: User_Connected | UID: 0 | UNAME: rodrigo

Received: FEU: NF | Type: User_Connected | UID: 1 | UNAME: rharagut

Received: FEU: NF | Type: User_Connected | UID: 2 | UNAME: john6

Received: FEU: NF | Type: New_Host | UID: 0 | FID: -1 | CONTENT:  | 

GetUpdate: FEU: NF | Type: User_Connected | UID: 2 | UNAME: john6
GetUpdate: FEU: NF | Type: User_Connected | UID: 0 | UNAME: rodrigo
GetUpdate: FEU: NF | Type: User_Connected | UID: 1 | UNAME: rharagut
GetUpdate: FEU: NF | Type: User_Connected | UID: 2 | UNAME: john6
GetUpdate: FEU: NF | Type: New_Host | UID: 0 | FID: -1 | CONTENT:  | 
Received: FEU: NF | Type: New_Shared_File | UID: -1 | FID: 0 | CONTENT: 
test2.txt | 

GetUpdate: FEU: NF | Type: New_Shared_File | UID: -1 | FID: 0 | CONTENT: 
test2.txt | 
Exception in thread "Thread-5" java.lang.NullPointerException
    at multitype.views.ViewManager.receive(ViewManager.java:61)
    at multitype.FEUManager.dispatchFEU(FEUManager.java:39)
    at multitype.FEUListener$1.run(FEUListener.java:38)
    at java.lang.Thread.run(Thread.java:662)
Received: FEU: NF | Type: User_Connected | UID: 3 | UNAME: John

Original issue reported on code.google.com by wedgepla...@gmail.com on 16 Apr 2011 at 5:56

GoogleCodeExporter commented 9 years ago
In an effort to prevent us having to redo the front-end of the plug-in, could 
we just keep it a requirement that the file list and user list have to be 
present to use our plugin.  This is the functionality we described in our 
Requirements, Design, and Project Plan documents.

Also, i've already got an issue open for Yeong to explore how the userlist 
reacts if it is hidden.

Original comment by rvmille...@gmail.com on 16 Apr 2011 at 6:06

GoogleCodeExporter commented 9 years ago
P.S.: Our plug-in will come packaged with a perspective that will make sure the 
File List / User List are present on the screen before you can connect.

Original comment by rvmille...@gmail.com on 16 Apr 2011 at 6:07

GoogleCodeExporter commented 9 years ago
Just for the sake of our testing, can we at least add a check before connected 
to prompt the user to open these windows?

Original comment by wedgepla...@gmail.com on 16 Apr 2011 at 6:10

GoogleCodeExporter commented 9 years ago
Yes, keep this issue open and we should do that

Comment #3 on issue 35 by wedgeplayssoccer: Client Crashes if File/User
Window not Visible

http://code.google.com/p/multitype/issues/detail?id=35
Just for the sake of our testing, can we at least add a check before
connected to prompt the user to open these windows?

Original comment by rvmille...@gmail.com on 16 Apr 2011 at 6:13

GoogleCodeExporter commented 9 years ago

Original comment by rvmille...@gmail.com on 19 Apr 2011 at 12:04

GoogleCodeExporter commented 9 years ago

Original comment by mrcorndo...@gmail.com on 19 Apr 2011 at 1:39