Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 317174

Summary:	Space in icon filename generate a bug in EMPC
Product:	Community	Reporter:	Etienne Juliot <etienne.juliot>
Component:	Marketplace	Assignee:	Darrell Armstrong <darrell.armstrong>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3	CC:	chris.guindon, leftylynn, sbouchet
Version:	unspecified
Target Milestone:	---
Hardware:	PC
OS:	All
Whiteboard:	stalebug

Description Etienne Juliot

2010-06-17 09:50:17 EDT

I upload an icon with a space in its name.
Upload works fine, but in the MP client, it fails :

!ENTRY org.eclipse.epp.mpc.ui 4 4 2010-06-16 12:15:26.859
!MESSAGE Bad URI: http://marketplace.eclipse.org/sites/default/files/logo_agility_professional - Copie.png
!STACK 0
java.net.URISyntaxException: Illegal character in path at index 76: http://marketplace.eclipse.org/sites/default/files/logo_agility_professional - Copie.png
	at java.net.URI$Parser.fail(URI.java:2809)
	at java.net.URI$Parser.checkChars(URI.java:2982)
	at java.net.URI$Parser.parseHierarchical(URI.java:3066)
	at java.net.URI$Parser.parse(URI.java:3014)
	at java.net.URI.<init>(URI.java:578)
	at java.net.URL.toURI(URL.java:918)
	at org.eclipse.epp.internal.mpc.ui.catalog.AbstractResourceRunnable.run(AbstractResourceRunnable.java:63)
	at org.eclipse.epp.internal.mpc.ui.catalog.AbstractResourceRunnable.call(AbstractResourceRunnable.java:50)
	at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:303)
	at java.util.concurrent.FutureTask.run(FutureTask.java:138)
	at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
	at java.lang.Thread.run(Thread.java:619)

	
Stéphane Bouchet <stephane.bouchet@obeo.fr>

Comment 1 Eclipse Genie

2014-05-17 05:24:35 EDT

This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.

Comment 2 Eclipse Genie

2016-05-07 13:01:47 EDT

This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.

Comment 3 Christopher Guindon

2016-11-07 19:12:40 EST

Darrell,

this bug was posted in 2010, can you confirm if this is still a problem?

Comment 4 Darrell Armstrong

2016-11-08 16:58:45 EST

Tested with a .png icon containing a space, and a few spaces.  

Spaces were properly encoded and the graphic appeared in MP client.

Comment 5 Christopher Guindon

2016-11-09 11:40:47 EST

(In reply to Darrell Armstrong from comment #4)
> Tested with a .png icon containing a space, and a few spaces.  
> 
> Spaces were properly encoded and the graphic appeared in MP client.

Thanks for looking into this. Closing this to fix. I presume this was fixed with the drupal 7 migration.