Xion Audio Player

Skins submission/display issue

Discuss potential bugs with other users or try to find out a solution to an existing bug

Postby SLoB » July 18th, 2008, 2:04 am

Cliff
you need to add in htmlspecialchars for skin/pic names to prevent people abusing/misusing/accidentally boshing ampersands in their skins titles/pictures etc..

for example new skin just added
http://xion.r2.com.au/skins/paqra/roll_&_rock_full.jpg

should be

http://xion.r2.com.au/skins/paqra/roll_ ... k_full.jpg
User avatar
SLoB
Xion Junkie
 
Posts: 1340
Joined: September 11th, 2006, 9:21 pm
Location: UK

Postby Cliff Cawley » July 18th, 2008, 8:10 am

You only need to do that when storing in a database. Storing on a filesystem using the characters is fine. In the database that skin is currently stored as roll_&_rock :)

Cliff :)
Cliff Cawley
Creator of Xion
r2 Studios
http://www.r2.com.au
http://xion.r2.com.au
User avatar
Cliff Cawley
Creator of Xion
 
Posts: 1955
Joined: September 3rd, 2006, 11:33 am
Location: Brisbane, Australia

Postby SLoB » July 18th, 2008, 8:16 am

its broken on the site, check it out on the skins page, the recent update also barfs, the skin preview barfs and the main skin pic barfs

The requested URL /skins/paqra/roll_&_rock_full.jpg was not found on this server.

Additionally, a 404 Not Found error was encountered while trying to use an ErrorDocument to handle the request.
User avatar
SLoB
Xion Junkie
 
Posts: 1340
Joined: September 11th, 2006, 9:21 pm
Location: UK

Return to Bugs (Release Build)

Who is online

Users browsing this forum: No registered users and 2 guests

cron