Page 1 of 1
Geotag Plugin Bugfix
Posted: Wed Jun 24, 2009 3:48 pm
by WMK
Hallo,
hab heute festgestellt, dass das Geotag-Plugin in der Sidebar die Links zu den Artikeln immer mit "index.php?" anzeigt. Ich hab einen kleinen Bugfix erstellt, das die Funktion serendipity_archiveURL() benutzt. Der Programmierer, der im Sourcecode steht, sagt, er betreut das Plugin nicht mehr.
Daher meine Frage: Wie kann ich die Änderung an euch zurückgeben? Was benögigt ihr? Ganzes File oder diff -u ?
Ist es besser, die Funktion serendipity_archiveURL() zu verwenden oder direkt $serendipity['rewrite'] zu prüfen?
Viele Grüße,
WMK
Re: Geotag Plugin Bugfix
Posted: Wed Jun 24, 2009 4:27 pm
by garvinhicking
Hi!
Ein ganzes file ist für mich mittlerweile einfacher zu committen, wäre also toll wenn du es hier attachen kannst (ggf. in .txt umbenennen oder so)
Grüße,
Garvin
Re: Geotag Plugin Bugfix
Posted: Wed Jun 24, 2009 4:34 pm
by WMK
Here we are

Re: Geotag Plugin Bugfix
Posted: Wed Jun 24, 2009 4:56 pm
by garvinhicking
Hi!
Super, danke. Ist committed!
Grüße,
Garvin
Re: Geotag Plugin Bugfix
Posted: Thu Jun 25, 2009 12:28 pm
by WMK
Hi Garvin,
danke.
Ich war noch weiter kreativ und hab ein paar Fehler gefixt und Geocoding implementiert. Hat Spaß gemacht und war hoffentlich nützlich...
Aus dem Changelog:
* added ChangeLog File

* added ability to use mod_rewrite urls without "index.php?"
* fixed error when no config values for default coordinates have been set
Ich hab die Felder zuerst leer gelassen und nur die Inputs für Länge und Breite gesehen.
* added geocode: just type in an address and the plugin tries to get the correct coordinates
* map gets updated on manual change of the coordinates (hit enter)
Getestet im Firefox. Im IE7 hab ich schon beim original Plugin einen Darstellungsfehler festgestellt. Vielleicht hilft dieser Tipp weiter:
http://groups.google.com/group/Google-M ... 3875?pli=1. Mein Template ist eine Abwandlung von "bulletproof".
Bitte überprüfe und teste meine Änderungen und schick mir Verbesserungsvorschläge, falls ich irgendwas "falsch" gemacht habe.
Hat s9y eigentlich eine Javascript Bibliothek? Sonst benutze ich gerne jQuery, diesmal hab ich (leider) direkt im html die Funktionalität eingebaut.
Viele Grüße,
Werner
Re: Geotag Plugin Bugfix
Posted: Thu Jun 25, 2009 3:04 pm
by garvinhicking
Hi!
Super, das klingt doch sehr vielversprechend. Vielen DAnk für deine INitiative und Mühe! Ich werde mir das am WE angucken und dann committen.
Noch bindet s9y von sich aus kein framework ein da es bisher noch nicht notwendig war. Einige plugins bundlen jquery, mittlerweile gibt es dank onli auch ein eigenes jquery event plugin, was man für seine plugins dann als dependency benutzen kann. Mittelfristig wäre es aber sicherlich ganz geschickt, jquery direkt in s9y zu integrieren. Da müsste ich mal gucken ob das lizenztechnisch unproblematisch ist.
Viele GRüße,
Garvin
Re: Geotag Plugin Bugfix
Posted: Thu Jun 25, 2009 3:15 pm
by onli
Um das mit dem Plugin auszuführen: setze das jQuery-Plugin mit
Code: Select all
$this->dependencies = array('serendipity_event_jquery' => 'keep');
als erforderlich und schon wird es mitinstalliert.
Gruß
Re: Geotag Plugin Bugfix
Posted: Thu Jun 25, 2009 3:36 pm
by WMK
Gut, die Tendenz geht also zu jQuery, was ich 1) gut finde und mich 2) sogar sehr wohl dabei fühle
Dann werd ich den Code bei zeiten jQuerisieren
viele Grüße,
Werner
Re: Geotag Plugin Bugfix
Posted: Fri Jun 26, 2009 9:35 am
by garvinhicking
Hi!
Committed
GRüße,
Garvin