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: | The wizard doesn't check, whether the created folder in document path already exists | ||
---|---|---|---|
Product: | php | Reporter: | Petr Pisl <ppisl> |
Component: | Code | Assignee: | Erno Mononen <emononen> |
Status: | RESOLVED FIXED | ||
Severity: | blocker | CC: | akorostelev |
Priority: | P2 | ||
Version: | 6.x | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | DEFECT | Exception Reporter: |
Description
Petr Pisl
2007-11-16 11:55:04 UTC
will notify that folder already exists, if possible to check this (can be not possible for remote server). I think it is correct to allow to use path to existing folder. The fix is clear: when server is local then check and warn user if the folder exist. It is a candidate for the external help. I'm asking Erno to take a look. Taking over. Fixed in trunk, please review. Checking in LocalServerProjectWizardPanel.java; /cvs/scripting/php/rtexplorer/src/org/netbeans/modules/php/rt/providers/impl/local/ LocalServerProjectWizardPanel.java,v <-- LocalServerProjectWizardPanel.java new revision: 1.3; previous revision: 1.2 done Checking in Bundle.properties; /cvs/scripting/php/rtexplorer/src/org/netbeans/modules/php/rt/providers/impl/local/Bundle.properties,v <-- Bundle.properties new revision: 1.15; previous revision: 1.14 done Thank you. just one correction. Common user's behavior is to download already existing files into project and modify. In this case document path (document root from server settings + context path from project) points to existing web site files inside server's http root. Therefore it should be allowed to finish wizard. I think validation method should return true, to show warning message, not error. I've updated sources. I see, thanks a lot for reviewing the fix. |