TBR Map

[Knabbelaar]

Bugs of verbeteringen in de code van TBR Map mogen wat mij betreft
hier gesuggereerd worden, of in een PM naar mij op
knabb...@twscripters.wyk.nl.

Zoiets vind ik op het AF niet echt gepast, omdat het voor de meesten
daar om de werking gaat, niet om de programmeer stijl.

Joost

[Lekensteyn]

In map.php zit mogelijk een gevaar voor SQL injection.
http://tbrmap.twproject.nl/map.php?v=3.1&tribe=1&x=1&y=1&world=2'
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 ''2'')' at line 1

Mvg,
Lekensteyn

On 30 jul, 11:14, Knabbelaar <knabbel...@twscripters.wyk.nl> wrote:
> Bugs of verbeteringen in de code van TBR Map mogen wat mij betreft
> hier gesuggereerd worden, of in een PM naar mij op

> knabbel...@twscripters.wyk.nl.

[Knabbelaar]

On 28 jul, 15:49, Lekensteyn <uwre...@gmail.com> schreef op het AF:
> @developers
> Misschien is het handig om een dev channel te maken.
> Hier miss: http://groups.google.com/group/tt-dev

Bij deze de vragen even doorgekopieerd

> 10:Ik wil even opmerken dat je de namespace altijd hetzelfde moet houden om te voorkomen dat nieuwe en oude scripts door elkaar werken.
Ik zal hier naar kijken, dat is meer voor firefox van belang dan
Opera, dus had ik er niet veel aandacht aan besteed.

> 101:error_re -> moeten die haakjes niet escaped worden?
Nee, die haken is omdat er (op termijn) meer fouten in de RE gaan
komen: /(fout1|fout2|fout3)/

> 216-225:sprintf zal falen op argumenten met %s erin. Als je dit niet erg vind kun je het sneller maken door '%s' te gebruiken i.p.v. /%s/.
> Je kunt beter splitten door '%s' en de rest opvullen met de args.

Voor het doel is dit afdoende, ik ken de beperkingen. Het is niet mijn
bedoeling een library te gaan schrijven.

[Lekensteyn]

Nog een extra opmerking over browsercompatibiliteit,
om optimale compatibiliteit te garanderen tussen verschillende
browsers die de standaarden ondersteunen (FF, Opera, Chrome)
is het aan te raden dat je de scripts echt als "script" in de
pagina behandelt.
Gebruik geen unsafeWindow en andere hacks voor Firefox, maar
wrap het script:
(function(f){var
d=document,s=d.createElement('script');s.setAttribute('type','application/
javascript');s.textContent = '('+f.toString()+')()';(d.body||d.head||
d.documentElement).appendChild(s);s.parentNode.removeChild(s)})
(function(){
//hier de code
})

Een voorbeeld kun je in TW BB++ zien:
http://www.tribetool.nl/gm_download.php/1.user.js

Een alternatieve oplossing zou het hosten van het script op
de website, en gebruik maken van de HTTP headers voor caching:
Cache-Control: cache, max-age=2592000

Dan nog E-tag / Last-Modified meesturen en caching is gedaan.