Bug 233057 - [remotecdt] Connection Properties button should be disabled if no connection is available or Local is selected
Summary: [remotecdt] Connection Properties button should be disabled if no connection ...
Status: RESOLVED FIXED
Alias: None
Product: Target Management
Classification: Tools
Component: RSE (show other bugs)
Version: 3.0   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 3.0 RC2   Edit
Assignee: Martin Oberhuber CLA
QA Contact: Martin Oberhuber CLA
URL:
Whiteboard:
Keywords: contributed
Depends on: 231827
Blocks:
  Show dependency tree
 
Reported: 2008-05-20 16:01 EDT by Martin Oberhuber CLA
Modified: 2011-05-25 07:34 EDT (History)
6 users (show)

See Also:


Attachments
proposed patch (2.33 KB, patch)
2008-05-26 09:24 EDT, Johann Draschwandtner CLA
no flags Details | Diff
reworked patch (2.31 KB, patch)
2008-05-26 09:52 EDT, Johann Draschwandtner CLA
mober.at+eclipse: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Oberhuber CLA 2008-05-20 16:01:58 EDT
+++ This bug was initially created as a clone of Bug #231827 +++

From bug 231827 comment 9:
One minor gotcha I found is that in case no connection at all is available in
RSE yet (i.e. "Local" is not installed and RSE is started the first time), the
connection dropdown can be empty with nothing selected. The Properties button
should be disabled in this case. This can also happen if none of the available
connections fits the needs of remotecdt (shell subsystem available).

From bug 231827 comment 10:
Another Gotcha is that when choosing Properties for Local, the dialog is shown
but no values are editable. The button should better be disabled in this case.
Comment 1 Johann Draschwandtner CLA 2008-05-26 09:24:40 EDT
Created attachment 101976 [details]
proposed patch
Comment 2 Martin Oberhuber CLA 2008-05-26 09:34:42 EDT
Hm... patch seems to address issue (1), but for the case of the "Local" connection selected, it still brings up a Properties dialog where nothing can be edited (and now, not even OK can be pressed).

Wouldn't it make more sense to disable the "Properties" button in case a "Local" connection is selected?

Also, please add your legal statement as per
http://www.eclipse.org/dsdp/tm/development/committer_howto.php#external_contrib
Thanks!
Comment 3 Johann Draschwandtner CLA 2008-05-26 09:52:39 EDT
Created attachment 101978 [details]
reworked patch

I've mis-interpreted the statement "The button should better be disabled in this case" :-) Fixed as suggested.

And here's the legal statement:

I, Johann Draschwandtner, declare that I developed attached code from scratch,
without referencing any 3rd party materials except material licensed under the
EPL. I am authorized by my employer to make this contribution under the EPL.
Comment 4 Martin Oberhuber CLA 2008-05-26 10:40:48 EDT
That was fast, thanks!
Patch applied: 15 added / 2 removed lines.

Next time, please also change the Copyright Header in the file, adding a line for your contribution (I did it this time).