Points of Required Attention™
Please chime in on a proposed restructuring of the ROM hacking sections.
Views: 88,536,458
 Main | FAQ | Uploader | IRC chat | Radio | Memberlist | Active users | Latest posts | Calendar | Stats | Online users | Search 05-05-24 02:51 AM
Guest: Register | Login

0 users currently in Help/Suggestions/Bug Reports | 1 guest

Main - Help/Suggestions/Bug Reports - Shop.php is broken. New thread | New reply

Pages: 1 2

joe
Posted on 06-24-07 05:20 AM Link | Quote | ID: 49042


Red Paratroopa
Level: 31

Posts: 102/172
EXP: 178147
Next: 7216

Since: 04-08-07
From: Justus League 2

Last post: 6125 days
Last view: 6093 days 		Someone here should have been working on shop.php rather than performing psychology tests on the board members.

For one, I can't preview items with an apostrophe in the name because there's no \ in front of the name in the preview script.

For another, I was given the option to buy an item that I couldn't afford, and when I clicked, it said "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1Unknown column 'eq99' in 'field list'" and that I had purchased the item. (I didn't get the item, unfortunately.)

____________________
"I'm sick of it and I'm out. See you around." ~Cpu's last post here

Skreeny
Posted on 06-24-07 05:36 AM Link | Quote | ID: 49047


Cobrat
Level: 56

Posts: 268/653
EXP: 1322660
Next: 75516

Since: 02-21-07

Last post: 5349 days
Last view: 4851 days 		Don't try to cheat and get the admin-only stuff. I know you aren't messing with Xkeeper's Goggles, because I both previewed and bought them without problem, and the only other items with quotation marks are *cough* illicit ones.

That, and the 'eq99' kinda suggests you're trying to get something in category 99, which, amazingly enough, doesn't have a matching field in the character data.

joe
Posted on 06-24-07 05:56 AM Link | Quote | ID: 49052


Red Paratroopa
Level: 31

Posts: 104/172
EXP: 178147
Next: 7216

Since: 04-08-07
From: Justus League 2

Last post: 6125 days
Last view: 6093 days 		You aren't using IE7, are you? It complains about an unterminated string when the page loads. I don't even have to try to preview, although when I do nothing happens. It gives the same error when I go to Weapons (Fairy's Dagger) but nowhere else. You are able to preview Xkeeper's Goggles and I am not. I can preview the goggles and the Valkyrie Pendant, but not Xkeeper's goggles or the Fairy's Dagger.

As for the category 99 item, it said I could buy it even though I didn't have enough green coins. This is more of a forewarning on the off-chance Xk or someone decides to add items that cost green coins than anything else.

____________________
"I'm sick of it and I'm out. See you around." ~Cpu's last post here

Xkeeper
Posted on 06-24-07 06:49 AM (rev. 2 of 06-24-07 06:50 AM) Link | Quote | ID: 49066


Level: 105

Posts: 2303/2846
EXP: 12036541
Next: 225719

Since: 02-19-07

Last post: 6068 days
Last view: 2806 days
Posted by joe
[I'm] using IE7


____________________
I dealt with it.

NightKev
Posted on 06-24-07 07:08 AM (rev. 2 of 06-24-07 07:19 AM) Link | Quote | ID: 49075


Cape Luigi
Level: 131

Posts: 1327/4792
EXP: 26248198
Next: 176422

Since: 03-15-07

Last post: 3743 days
Last view: 3655 days
Posted by Xkeeper
Posted by joe
[I'm] using IE7
That image is made of pure "Win".

No problems with shop.php for me in Firefox (last time I checked). So Xkeeper is right, it must be IE .

____________________

Skreeny
Posted on 06-24-07 07:17 AM Link | Quote | ID: 49076


Cobrat
Level: 56

Posts: 270/653
EXP: 1322660
Next: 75516

Since: 02-21-07

Last post: 5349 days
Last view: 4851 days 		Actually, trying for myself in Firefox, "Fairy's Dagger" and "Thief's Dagger" refuse to Preview properly.

Somehow forgot those two existed for my last post

stag019
Posted on 06-24-07 07:18 AM Link | Quote | ID: 49078


Snifit
Level: 40

Posts: 141/299
EXP: 409878
Next: 31431

Since: 02-19-07

Last post: 4289 days
Last view: 2175 days 		Actually, previewing the goggles errored for me too.
onclick='preview(27,240,6,"Xkeeper's goggles")'
That's why, obviously.
Why not change it to this:
onclick="preview(27,240,6,"Xkeeper's goggles")"
But yeah, I also checked this which spits me that SQL error too.
And neither of these only break in IE, NightKev, Xkeeper was just making fun of it because it sucks.

joe
Posted on 06-24-07 08:07 AM Link | Quote | ID: 49093


Red Paratroopa
Level: 31

Posts: 108/172
EXP: 178147
Next: 7216

Since: 04-08-07
From: Justus League 2

Last post: 6125 days
Last view: 6093 days
Posted by stag019
Why not change it to this:
onclick="preview(27,240,6,"Xkeeper's goggles")"

...I would assume because the script expects a string, and a string is in either single or double quotes, nothing else. (I may be wrong, I haven't done this in a while.)

If I remember correctly, it would work if it was changed to this:
onclick='preview(27,240,6"Xkeeper\'s goggles")'

____________________
"I'm sick of it and I'm out. See you around." ~Cpu's last post here

stag019
Posted on 06-24-07 08:14 AM Link | Quote | ID: 49097


Snifit
Level: 40

Posts: 142/299
EXP: 409878
Next: 31431

Since: 02-19-07

Last post: 4289 days
Last view: 2175 days
Posted by joe
If I remember correctly, it would work if it was changed to this:
onclick='preview(27,240,6"Xkeeper\'s goggles")'
Adding slashes doesn't fix HTML. Encoding characters do. He could also use ' or '

HyperHacker
Posted on 06-24-07 09:33 AM (rev. 2 of 06-24-07 09:33 AM) Link | Quote | ID: 49124

...
Level: 73

Posts: 978/1220
EXP: 3369121
Next: 116747

Since: 03-25-07
From: no

Last post: 6100 days
Last view: 6083 days 		Preview doesn't work at all for me. Either nothing happens or I get the same image with less coins.

NightKev
Posted on 06-25-07 12:39 AM (rev. 2 of 06-25-07 12:51 AM) Link | Quote | ID: 49196


Cape Luigi
Level: 131

Posts: 1337/4792
EXP: 26248198
Next: 176422

Since: 03-15-07

Last post: 3743 days
Last view: 3655 days
Posted by HyperHacker
Preview doesn't work at all for me. Either nothing happens or I get the same image with less coins.
I think Xk disabled preview until he fixes it.


____________________

OoBurns
Posted on 06-25-07 12:47 AM (rev. 2 of 06-25-07 12:47 AM) Link | Quote | ID: 49198


Boo
Level: 68

Posts: 773/1036
EXP: 2643391
Next: 85409

Since: 02-19-07
From: It is a mystery.

Last post: 5731 days
Last view: 5731 days
I think you just wasted your 1337 post there, Nightkev.

____________________

NightKev
Posted on 06-25-07 12:49 AM Link | Quote | ID: 49199


Cape Luigi
Level: 131

Posts: 1339/4792
EXP: 26248198
Next: 176422

Since: 03-15-07

Last post: 3743 days
Last view: 3655 days
Posted by BooUrns
I think you just wasted your 1337 post there, Nightkev.
Damn you boo why did you point that out

____________________

joe
Posted on 06-25-07 01:00 AM Link | Quote | ID: 49205


Red Paratroopa
Level: 31

Posts: 110/172
EXP: 178147
Next: 7216

Since: 04-08-07
From: Justus League 2

Last post: 6125 days
Last view: 6093 days
Posted by stag019
Adding slashes doesn't fix HTML.
I added a slash because it's Javascript, not HTML. Javascript will return the string without the slash. In fact, try this script.

<script language="JavaScript">
document.write("Hey, look. \"\"\" It works.")
</script>

____________________
"I'm sick of it and I'm out. See you around." ~Cpu's last post here

stag019
Posted on 06-25-07 01:20 AM Link | Quote | ID: 49210


Snifit
Level: 40

Posts: 146/299
EXP: 409878
Next: 31431

Since: 02-19-07

Last post: 4289 days
Last view: 2175 days 		Yes, but you're escaping the HTML quote, not the javascript quote.

<script type="text/javascript">preview(27,240,6'Xkeeper\'s goggles');</script>

Something like that would be a valid escape.

However, because the ' is around the HTML attribute of "onclick", that wouldn't fix it.
onclick='preview(27,240,6"Xkeeper\'s goggles")'
There would still be an error by the s.

Still don't believe me? Try downloading the HTML, and trying the two methods (escaping via HTML encoding and adding slashes). Also, add <base href="http://acmlm.no-ip.org/board/"> if you hate broken images/broken stylesheets.

Also, NightKev, you were wrong. Previewing still works for me. HyperHacker, what did you preview, the goggles? Because they do nothing. Whenever I preview an item that does something to your stats.... it does something to my stats, and shows me with less coins.

Try previewing with the Valkyrie Staff. Do you notice a change in your stats and coins?

UnsurpassedDarkness
Posted on 06-25-07 02:52 AM Link | Quote | ID: 49221


Goomba
Level: 13

Posts: 19/25
EXP: 9891
Next: 376

Since: 03-14-07
From: Kentucky

Last post: 6059 days
Last view: 5867 days
It appears that my CSS broke...
Posted by stag019
Because they do nothing.
Is that the regular goggles or Xkeeper’s goggles?

____________________
SMW Central
YGO:TAS
Text-Image
Hex Conv.
I4 Posts
I3 Profile
I2 Profile
Send a PM

NightKev
Posted on 06-25-07 02:53 AM Link | Quote | ID: 49222


Cape Luigi
Level: 131

Posts: 1342/4792
EXP: 26248198
Next: 176422

Since: 03-15-07

Last post: 3743 days
Last view: 3655 days
Posted by UnsurpassedDarkness
Posted by stag019
Because they do nothing.
Is that the regular goggles or Xkeeper’s goggles?
The goggles, they do nothing!

____________________

HyperHacker
Posted on 06-25-07 04:52 AM Link | Quote | ID: 49260

...
Level: 73

Posts: 982/1220
EXP: 3369121
Next: 116747

Since: 03-25-07
From: no

Last post: 6100 days
Last view: 6083 days 		Yeah, it's working now. Yesterday it wasn't doing anything.

joe
Posted on 06-25-07 05:34 AM Link | Quote | ID: 49276


Red Paratroopa
Level: 31

Posts: 113/172
EXP: 178147
Next: 7216

Since: 04-08-07
From: Justus League 2

Last post: 6125 days
Last view: 6093 days
Posted by stag019
Yes, but you're escaping the HTML quote, not the javascript quote.
Ah, I see. Then it would be this:

<a href=#status onclick="preview(367,240,6,'Xkeeper\'s goggles')">

I tried it, it works.

____________________
"I'm sick of it and I'm out. See you around." ~Cpu's last post here

HyperHacker
Posted on 06-25-07 06:25 AM Link | Quote | ID: 49290

...
Level: 73

Posts: 985/1220
EXP: 3369121
Next: 116747

Since: 03-25-07
From: no

Last post: 6100 days
Last view: 6083 days
Posted by joe
Posted by stag019
Yes, but you're escaping the HTML quote, not the javascript quote.
Ah, I see. Then it would be this:

<a href="#status" onclick="preview(367,240,6,'Xkeeper\'s goggles')">

I tried it, it works.
Pages: 1 2


Main - Help/Suggestions/Bug Reports - Shop.php is broken. New thread | New reply

Acmlmboard 2.1+4δ (2023-01-15)
© 2005-2023 Acmlm, blackhole89, Xkeeper et al.

Page rendered in 0.028 seconds. (322KB of memory used)
MySQL - queries: 107, rows: 135/136, time: 0.018 seconds.