This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 9341 - "Diff" and "Add" commands throw exceptions on an empty JavaCVS filesystem.
Summary: "Diff" and "Add" commands throw exceptions on an empty JavaCVS filesystem.
Status: VERIFIED FIXED
Alias: None
Product: versioncontrol
Classification: Unclassified
Component: CVS (show other bugs)
Version: 3.x
Hardware: Other Windows ME/2000
: P4 normal (vote)
Assignee: issues@versioncontrol
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2001-02-01 15:22 UTC by Jiri Kovalsky
Modified: 2007-01-04 17:14 UTC (History)
0 users

See Also:
Issue Type: ENHANCEMENT
Exception Reporter:

Attachments
Attachment for bug #9341. (2.29 KB, text/plain)
2001-07-20 20:47 UTC, Jiri Kovalsky 		Details
picture of popup menu actions (103.62 KB, image/jpeg)
2001-07-20 20:39 UTC, dmladek 		Details
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 Jiri Kovalsky 2001-02-01 15:22:28 UTC 
Development build of NetBeans #116 (2001-02-01) on Windows 2000 with JDK 1.3.

Description:
============
Two above mentioned commands throw exceptions if performed on completely empty
JavaCVS filesystem. "Diff" throws NPE and "Add" throws two java.lang.String-
IndexOutOfBoundsExceptions. For more details please see attached file containing
all exceptions.

Steps to reproduce:
===================
1. Prepare completely empty directory and mount it as JavaCVS filesystem.
2. Invoke "Diff" command from the root's popup menu and push "Run Command".
3. NPE will be thrown. The same way invoke "Add" command and explore console.
Comment 1 Jiri Kovalsky 2001-02-01 15:27:59 UTC 
Created attachment 532 [details]
Attachment for bug #9341.
Comment 2 Jan Chalupa 2001-03-12 10:48:23 UTC 
Version: 'Dev' -> 3.2
Comment 3 Milos Kleint 2001-03-14 13:38:11 UTC 
fixed the Diff bug, by disabling the difftreeview component.
I couldn't reproduce the add command exception in release32 build 2.
Comment 4 dmladek 2001-06-08 18:28:27 UTC 
verified on [NBdev-206] => NPE is newer thrown and command ADD isn't in popup
menu. See attached picture.

But I don't feel well with this solution and rather reopen this bug then create
a new one, sorry. And I'm changing it to ENHANCMENT too:-)

My suggestion is:
=================
Disbale all action which couldn't be performed in this and other situations.
For instance if you have realy emty working dir, disable this actions:
Commit,
Status,
Log,
Refresh+R.Recursively,
Tag,
Diff,
mayby update?


so it will be the best.

other solution(dificult, and laboured) could be:
for each individual command like diff throw some kinda dialog/Localized E.

but IMHO it is nonsence to allow to run diff command on your empty working and 
notice somhow user that it is nonsence.
Comment 5 dmladek 2001-06-08 18:29:33 UTC 
Created attachment 1530 [details]
picture of popup menu actions
Comment 6 Peter Pis 2005-04-27 14:35:26 UTC 
Seems to be OK, Martine, can you change to resolution to fixed, please?
Comment 7 Martin Entlicher 2005-04-27 15:09:09 UTC 
I do not remember exactly in which release JavaCVS filesystem was removed, but
this was just "fixed" by removal of the JavaCVS filesystem. There are no more
exceptions and in 4.1 release there are only context actions which have sense
according to files state.
Comment 8 Jiri Kovalsky 2005-04-27 15:36:26 UTC 
Yes, I guess this is indeed resolved ... :-)