Bugzilla – Full Text Bug Listing
| Summary: | EAR Library Directory field should not have preceding slash | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | [WebTools] WTP Java EE Tools | Reporter: | Aidyl Kareh <amkareh> | ||||||
| Component: | jst.j2ee | Assignee: | Aidyl Kareh <amkareh> | ||||||
| Status: | RESOLVED FIXED | QA Contact: | Chuck Bridgham <cbridgha> | ||||||
| Severity: | normal | ||||||||
| Priority: | P3 | CC: | amkareh, ccc, jsholl | ||||||
| Version: | unspecified | Flags: | cbridgha:
review+
|
||||||
| Target Milestone: | 3.2.2 P | ||||||||
| Hardware: | PC | ||||||||
| OS: | Windows XP | ||||||||
| Whiteboard: | |||||||||
| Attachments: |
|
||||||||
Created attachment 180527 [details]
Proposed Patch
This patch fixes the default behavior of adding a preceding slash to the library directory field in the 'Deployment Assembly' properties page and also adds validation to avoid having the user add a preceding slash. This patch also fixes the issue where the 'Revert' button wasn't reverting the value of the library directory field.
I approve, but the hover message should give more details re: the issue, not redirect to another place. The hover can be several sentences long Created attachment 180617 [details]
Updated Proposed Patch
This patch takes Chuck's comments into account and adds more description to the error messages.
Committed to R3_2_2_patches checked into 32M and HEAD streams for WTP 3.2.3 and WTP 3.3. |
Build Identifier: WTP 3.2.2 The library directory field's default behavior in the EAR's 'Deployment Assembly' page is to have a preceding slash ('/') character. This character should not be added to the beginning of the library directory path. Also, using the 'Revert' button in the EAR's 'Deployment Assembly' page will not revert the library directory value. Reproducible: Always