[jbosstools-issues] [JBoss JIRA] (JBIDE-18411) DownloadRuntimesWizard shouldn't need to parse the d/l url to infer the file name
Rob Stryker (JIRA)
issues at jboss.org
Tue Sep 23 16:10:02 EDT 2014
[ https://issues.jboss.org/browse/JBIDE-18411?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13005515#comment-13005515 ]
Rob Stryker commented on JBIDE-18411:
-------------------------------------
Yes, rejected. I'll leave it open though in case we do decide to add API for this issue, though I sincerely doubt it would be clean to accomplish. But it's worth looking into.
> DownloadRuntimesWizard shouldn't need to parse the d/l url to infer the file name
> ---------------------------------------------------------------------------------
>
> Key: JBIDE-18411
> URL: https://issues.jboss.org/browse/JBIDE-18411
> Project: Tools (JBoss Tools)
> Issue Type: Bug
> Components: runtime-detection
> Affects Versions: 4.2.0.CR1
> Environment: JBT 4.2.0.CR1, JBDS 8.0.0.CR1
> Reporter: Radim Hopp
> Assignee: Rob Stryker
> Fix For: 4.3.0.Alpha1
>
>
> JBIDE-18393 showed that parsing d/l urls was fragile :
> {noformat:title=An internal error occurred during: "Download 'JBoss 6.0.0".}java.lang.StringIndexOutOfBoundsException: String index out of range: -1
> at java.lang.String.substring(String.java:1954)
> at org.jboss.tools.runtime.ui.internal.wizard.DownloadRuntimeOperationUtility.getNextUnusedFilename(DownloadRuntimeOperationUtility.java:56)
> at org.jboss.tools.runtime.ui.internal.wizard.DownloadRuntimeOperationUtility.getDestinationFile(DownloadRuntimeOperationUtility.java:89)
> at org.jboss.tools.runtime.ui.internal.wizard.DownloadRuntimeOperationUtility.downloadRemoteRuntime(DownloadRuntimeOperationUtility.java:194)
> at org.jboss.tools.runtime.ui.internal.wizard.DownloadRuntimeOperationUtility.downloadAndInstall(DownloadRuntimeOperationUtility.java:155)
> at org.jboss.tools.runtime.ui.internal.wizard.FinalizeRuntimeDownloadFragment$9.run(FinalizeRuntimeDownloadFragment.java:588)
> at org.eclipse.core.internal.jobs.Worker.run(Worker.java:54){noformat}
> [~maxandersen] suggested to get the filename from the d/l response header instead
--
This message was sent by Atlassian JIRA
(v6.3.1#6329)
More information about the jbosstools-issues
mailing list