The VG Resource

Did we make ourselves look stupid? Something amiss?
Point it out, and we'll look into it next time we go on a mass fixing fest

Cookies should always be loaded before the page content is printed on screen, or even any HTML is processed. Cookies will not work if you already have printed something in HTML on screen, and it will leave a nasty header error.

The main problem you guys have is this. You directly print the content the moment it gets read. Instead, the content should be put in a separate string first before being output to the actual page. This will require a bit more CPU, but in the end a lot of functions perform better.

on the main site Pokemon customs, some Pokemon have their japanese names on the tags instead of the english, mainly Drifloon, Starly and Cherinbo

In the Final Fantasy Tactics Advance section, Black Mage is in the Friendly Characters category, when he should be in the Playable Characters category like all the other classes. Also, as far as I know, Doned Radijiu and Nono are not playable characters, and should be put into the Friendly Characters category.

I have a few minor suggestions.

In the Jump! Ultimate Stars section, the anime which have two or more characters have been given their own subsection. I did a bit of research and found that three of the sheets on the site can be sorted into their own subsections. These sheets are Ikki, Shiryu and Hyoga. All three of these characters come from the anime Saint Seiya, which I personally haven't watched but I am familiar the character designs.

Also, Bleach already has its own subsection, but the sheet of Yasutora Sado has been placed in the Unsorted section, when he should've (at least in my opinion) been placed in the Bleach section.

If you go back to the "various styles" section of Super Smash Bros. when you're looking at a sprite by using the navigation thingy you get a 404 because it brings you to http://www.spriters-resource.com/nintend...0original/ when the real url is http://www.spriters-resource.com/nintend..._original/.

http://www.spriters-resource.com/other/other/jump/jus/
Yasutora Sado should be with the other Bleach characters.

A small error and a suggestion regarding the Lunar section.

http://www.spriters-resource.com/other/o...lthena.gif
http://www.spriters-resource.com/other/o...illuna.gif

These two sheets are both of the same character(s), however the former of the two has a lot more poses ripped. Additionally, the former is actually put in the Friendly Characters section, when she is an enemy. I believe that the "Althena" sheet should simply replace the "Evil Luna" sheet in the Enemies section.

Omega Wrote:on the main site Pokemon customs, some Pokemon have their japanese names on the tags instead of the english, mainly Drifloon, Starly and Cherinbo

I'll fix that later on, once I've split up that section.

Sol Wrote:In the Final Fantasy Tactics Advance section, Black Mage is in the Friendly Characters category, when he should be in the Playable Characters category like all the other classes. Also, as far as I know, Doned Radijiu and Nono are not playable characters, and should be put into the Friendly Characters category.

Fixed.

Cascade Wrote:I have a few minor suggestions.

In the Jump! Ultimate Stars section, the anime which have two or more characters have been given their own subsection. I did a bit of research and found that three of the sheets on the site can be sorted into their own subsections. These sheets are Ikki, Shiryu and Hyoga. All three of these characters come from the anime Saint Seiya, which I personally haven't watched but I am familiar the character designs.

Also, Bleach already has its own subsection, but the sheet of Yasutora Sado has been placed in the Unsorted section, when he should've (at least in my opinion) been placed in the Bleach section.

Lunyka Wrote:http://www.spriters-resource.com/other/other/jump/jus/
Yasutora Sado should be with the other Bleach characters.

Done (sorted out the whole section), and fixed.

Pigyman Wrote:If you go back to the "various styles" section of Super Smash Bros. when you're looking at a sprite by using the navigation thingy you get a 404 because it brings you to http://www.spriters-resource.com/nintend...0original/ when the real url is http://www.spriters-resource.com/nintend..._original/.

Fixed. The folder name is now "ssboriginal", and it doesn't give you a 404 when you go back anymore.

Cascade Wrote:A small error and a suggestion regarding the Lunar section.

http://www.spriters-resource.com/other/o...lthena.gif
http://www.spriters-resource.com/other/o...illuna.gif

These two sheets are both of the same character(s), however the former of the two has a lot more poses ripped. Additionally, the former is actually put in the Friendly Characters section, when she is an enemy. I believe that the "Althena" sheet should simply replace the "Evil Luna" sheet in the Enemies section.

Done.

1) When you get the 404 error page, the forum link there points to Pixeltendo.

2)

This can be deleted from the Metroid Fusion section. The ship has been fully ripped already by others.

3)

Move to Enemy Characters in the Metroid Fusion section please. Also change the icon to:


That's all.

Ngamer01 Wrote:1) When you get the 404 error page, the forum link there points to Pixeltendo.

2)

This can be deleted from the Metroid Fusion section. The ship has been fully ripped already by others.

3)

Move to Enemy Characters in the Metroid Fusion section please. Also change the icon to:


That's all.

1) Not anymore.

2) Deleted.

3) Done.

This sheet doesn't work: http://www.spriters-resource.com/nintend...iglett.png

JoshR691 Wrote:This sheet doesn't work: http://www.spriters-resource.com/nintend...iglett.png

Fixed. The file name was missing a "t" at the end. :P

The icon next to Dr. Toad has an error. I think it was suppose to be a Koopa Troop.

display.php doesnt validate properly, send it this and you expose an include();
I think it may be a good idea to strengthen the input validation on the file variable?

display.php?file=file_name.png" /><!--

perhaps use this:
<?php
$file = preg_replace('/[^a-z0-9_\.]/', '', $_GET['file']);
?>
that way only valid characters get through

Even better would be to associate images with an image ID stored in a database and send that ID instead of a filename, its a few more lines of code, but worth it...