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.
Summary: | AssertionError at org.netbeans.modules.apisupport.project.ui.customizer.SingleModuleProperties.refresh | ||
---|---|---|---|
Product: | apisupport | Reporter: | sreimers <sreimers> |
Component: | Project | Assignee: | rmichalsky <rmichalsky> |
Status: | RESOLVED FIXED | ||
Severity: | blocker | ||
Priority: | P3 | ||
Version: | 6.x | ||
Hardware: | All | ||
OS: | All | ||
URL: | http://statistics.netbeans.org/exceptions/detail.do?id=154553 | ||
Issue Type: | DEFECT | Exception Reporter: | 154553 |
Attachments: | stacktrace |
Description
sreimers
2009-07-30 08:44:52 UTC
Created attachment 85508 [details]
stacktrace
Cannot reproduce. Does it happen consistently on some project? Could you by chance attach some sample project on which it happens regularly? I think it may have been caused by a module beloging to a suite project, which did not exist on my filesystem (path to suite in properties pointed to invalid not existing directory) Having fixed this everything works as expected now - cannot reproduce this anymore It has to be some combination of factors, just having suite.dir pointing to nonexistent platform is not enough, I tried and IDE just warns about missing platform (and of course the project is not buildable). Closing, please reopen if you the problem shows up again. As a rule, AEs should not be closed as WORKSFORME. The assertion is clearly not valid under all possible conditions, so even if you cannot reproduce, the code must be written more defensively - or the assert message enhanced to include information that might point you to the right fix if it happens again. In this case, assert getSuiteDirectory() != null; ModuleList.refreshModuleListForRoot(getSuiteDirectory()); should be written more defensively as File d = getSuiteDirectory(); if (d != null) { ModuleList.refreshModuleListForRoot(d); } since getSuiteDirectory returns null under a variety of conditions and there is no obvious reason why in this situation it should never be null. Well, I'd personally rather leave the assertion there than turn it into a condition, this assertion has already revealed a bug in the code before and situations when it shouldn't hold seem quite obscure to me. But since it is not reproducible and there is allegedly a rule (which I was not aware of) that it cannot be closed as WORKSFORME, I'll just write it more defensively. core-main #51cd17877049 Integrated into 'main-golden', will be available in build *200908080201* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress) Changeset: http://hg.netbeans.org/main-golden/rev/51cd17877049 User: Richard Michalsky <rmichalsky@netbeans.org> Log: #169529: fixing AssertionError |