Geotag Plugin Bugfix

Hier können Probleme und alles andere in Deutscher Sprache gelöst werden.
Post Reply
WMK
Regular
Posts: 32
Joined: Mon Jun 22, 2009 11:14 am
Contact:

Geotag Plugin Bugfix

Post 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
Viele Grüße,

Werner
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: Geotag Plugin Bugfix

Post 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
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
WMK
Regular
Posts: 32
Joined: Mon Jun 22, 2009 11:14 am
Contact:

Re: Geotag Plugin Bugfix

Post by WMK »

Here we are ;)
Attachments
serendipity_plugin_geotag.php.txt
kleiner Bugfix: index.php? optional in Links
(12.35 KiB) Downloaded 189 times
Viele Grüße,

Werner
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: Geotag Plugin Bugfix

Post by garvinhicking »

Hi!

Super, danke. Ist committed!

Grüße,
Garvin
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
WMK
Regular
Posts: 32
Joined: Mon Jun 22, 2009 11:14 am
Contact:

Re: Geotag Plugin Bugfix

Post 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
Attachments
s9y_event_geotag_1.23.tar.gz
(13.2 KiB) Downloaded 171 times
Viele Grüße,

Werner
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: Geotag Plugin Bugfix

Post 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
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
onli
Regular
Posts: 2836
Joined: Tue Sep 09, 2008 10:04 pm
Contact:

Re: Geotag Plugin Bugfix

Post 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ß
WMK
Regular
Posts: 32
Joined: Mon Jun 22, 2009 11:14 am
Contact:

Re: Geotag Plugin Bugfix

Post 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
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: Geotag Plugin Bugfix

Post by garvinhicking »

Hi!

Committed :)

GRüße,
Garvin
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
Post Reply