Shop.php is broken.

Points of Required Attention™

Please chime in on a proposed restructuring of the ROM hacking sections.

05-05-24 10:01 AM

Guest: Register | Login

0 users currently in Help/Suggestions/Bug Reports | 2 guests

Main - Help/Suggestions/Bug Reports - Shop.php is broken.

New thread | New reply

Pages: 1 2

stag019

Posted on 06-25-07 06:44 AM

Link | Quote | ID: 49307

Snifit
Level: 40

Posts: 147/299
EXP: 409888
Next: 31421

Since: 02-19-07

Last post: 4289 days
Last view: 2175 days

Posted by HyperHacker
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.

You should know by now this board uses horrible HTML coding.
The quotes around that aren't needed for it to run correctly, and therefore, Acmlm didn't put them in when coding shop.php.

Tyty

Posted on 06-29-07 08:02 AM

Link | Quote | ID: 50608

Vire
Jizz Danmaku
Level: 77

Posts: 245/1381
EXP: 4048934
Next: 142195

Since: 04-20-07
From: Canada

Last post: 4456 days
Last view: 3728 days

I know this has already been said but...

please put some stats on em....

____________________

Xkeeper

Posted on 06-29-07 08:03 AM

Link | Quote | ID: 50609

Level: 105

Posts: 2383/2846
EXP: 12036827
Next: 225433

Since: 02-19-07

Last post: 6069 days
Last view: 2806 days

That's the whole point, dumbass.

____________________
I dealt with it.

joe

Posted on 06-29-07 08:06 AM

Link | Quote | ID: 50613

Red Paratroopa
Level: 31

Posts: 125/172
EXP: 178151
Next: 7212

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

Last post: 6126 days
Last view: 6094 days

Now that you're here, Xkeeper, please go fix it.

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

Xkeeper

Posted on 06-29-07 08:06 AM

Link | Quote | ID: 50614

Level: 105

Posts: 2385/2846
EXP: 12036827
Next: 225433

Since: 02-19-07

Last post: 6069 days
Last view: 2806 days

I'm not fixing it because it's not broken.

____________________
I dealt with it.

joe

Posted on 06-29-07 08:10 AM

Link | Quote | ID: 50616

Red Paratroopa
Level: 31

Posts: 126/172
EXP: 178151
Next: 7212

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

Last post: 6126 days
Last view: 6094 days

Yes, it is.

While this may be valid to some browsers, those that use spaces to determine the attributes within a tag,...

onclick='preview(367,240,6,"Xkeeper's goggles")'

...those that use the apostrophes to find the string will see this instead.

onclick='preview(367,240,6,"Xkeeper'

Therefore, it is broken.

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

Xkeeper

Posted on 06-29-07 08:11 AM

Link | Quote | ID: 50617

Level: 105

Posts: 2386/2846
EXP: 12036827
Next: 225433

Since: 02-19-07

Last post: 6069 days
Last view: 2806 days

Okay, then I'm not fixing it.

I don't deal with Javascript, I'm not willing to waste my night (what's left of it) debugging some stupid bug.

____________________
I dealt with it.

joe

Posted on 06-29-07 08:13 AM

Link | Quote | ID: 50618

Red Paratroopa
Level: 31

Posts: 127/172
EXP: 178151
Next: 7212

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

Last post: 6126 days
Last view: 6094 days

You don't need to, we already did.

You just need to change this:

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

To this:

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

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

Cellar Dweller

Posted on 06-30-07 04:42 AM

Link | Quote | ID: 50865

Snifit
Level: 39

Posts: 38/287
EXP: 385462
Next: 19309

Since: 02-19-07
From: Arkansas

Last post: 4061 days
Last view: 3228 days

Posted by joe
You don't need to, we already did.

That doesn't matter much:

Apparently sent by Xkeeper
Sent by Boom.dk
I can't seem to get an answer on this simple yes or no question:

Do you need/want any help with coding this board? It seems like it's coming along kinda slowly, so I thought, I could help speed things up a bit.

So what'll it be?

*235926» Xkeeper> ~8ball should I get help coding the board
*235927» Tina> absolutely not

Xkeeper

Posted on 06-30-07 04:43 AM

Link | Quote | ID: 50866

Level: 105

Posts: 2399/2846
EXP: 12036827
Next: 225433

Since: 02-19-07

Last post: 6069 days
Last view: 2806 days

the board code is annoying enough to code with two other people working on it, adding a third would just make me give up completely

____________________
I dealt with it.

Cellar Dweller

Posted on 06-30-07 06:58 AM

Link | Quote | ID: 50893

Snifit
Level: 39

Posts: 39/287
EXP: 385462
Next: 19309

Since: 02-19-07
From: Arkansas

Last post: 4061 days
Last view: 3228 days

I doubt that there are really two other people actively working on the board, and, by your own admission, you are often too lazy to work on it yourself. Therefore, there should be little difficulty working with other programmers.

Also, there are some features from the old AcmlmBoard (eg. the calendar) that can be reimplemented without changing shared files.

Pages: 1 2

Main - Help/Suggestions/Bug Reports - Shop.php is broken.

New thread | New reply

Page rendered in 0.023 seconds. (322KB of memory used)
MySQL - queries: 67, rows: 86/87, time: 0.016 seconds.