Bug 999554
Summary: | No Quickstarts installation path validation | ||
---|---|---|---|
Product: | [JBoss] JBoss Enterprise Application Platform 6 | Reporter: | Petr Kremensky <pkremens> |
Component: | Installer | Assignee: | Francisco Canas <fcanas> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Petr Kremensky <pkremens> |
Severity: | medium | Docs Contact: | Russell Dickenson <rdickens> |
Priority: | unspecified | ||
Version: | 6.1.1 | CC: | fcanas, thauser |
Target Milestone: | ER3 | ||
Target Release: | EAP 6.2.0 | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2013-12-15 16:12:37 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Petr Kremensky
2013-08-21 14:38:32 UTC
Fixed the logic error during validation of the quickstarts directory. The quickstarts dir panel now prompts you to create a dir if it doesn't already exist. It also checks for write permissions and emits an appropriate error msg if the directory cannot be created or written to. If it does not pass these validation checks, the user has to change his directory before continuing to the next installation panel. See this commit for details: http://git.app.eng.bos.redhat.com/?p=izpack.git;a=commit;h=1156da42f195442b9e6d875e5ec429a9e3a0248d There will be no ER8. Changing target release to EAP 6.2. Hi, please move BZs to ON_QA once the issue is fixed and target release is available so I can verify it. Thanks Verified on EAP 6.2.0 ER1 I found some issues regarding this one. - select non-existing directory as a quickstarts target: Message window is thrown, but it uses string from path validation: <str id="TargetPanel.createdir" txt="A fresh Red Hat JBoss Enterprise Application Platform installation will be placed here: " />. It made sense in 6.1.1 where TargetPanel.createdir text was "The target directory will be created: ", but not in 6.2. We should create new string more related to quickstarts, or make TargetPanel.createdir string more general. Once I click on cancel of first message dialog, another message window is thrown saying "The directory you have chosen either does not exist or is not valid". Second message dialog shouldn't be there. For the issue with the incorrect string: I have made the string for the directory creation more general: "A new directory will be created here:" See commit for details: http://git.app.eng.bos.redhat.com/?p=jbossas-installer.git;a=commit;h=d3747201bf67d85a4c0ba7d740924f8f7c973a3b For the issue with the unnecessary dialog window: Removing this requires modifying izpack and will be done next time izpack is built. I will update this BZ when this occurs. I have opened a new BZ for this issue, since it's separate from the incorrect string and will likely not be fixed for the upcoming milestone: https://bugzilla.redhat.com/show_bug.cgi?id=1011654 Verified on EAP 6.2.0.ER3 |