hschottm / tags

Contao tags extension
11 stars 16 forks source link

leerzeichen #2

Closed epsreps closed 10 years ago

epsreps commented 10 years ago

tags_events 3.0.0 / tags 3.1.1 tags mit leerzeichen, die bisher für die url-ausgabe mit einem plus getrennt wurden, führen zu einem null-ergebniss in der tagcloud

hschottm commented 10 years ago

Ich kann das leider nicht nachvollziehen. Ich habe zu einem Event drei tags angelegt:

abc,def,tag mit leerzeichen

Wenn ich eine TagCloud (Events) erstelle, werden diese Tags auch entsprechend angezeigt, also z.B. http://domain.tld/contao31/index.php/index/tag/tag+mit+leerzeichen.html

Ich verwende Contao 3.1.5. Unter welchen Bedingungen tritt das Problem denn auf? (Mit welchem Modul, etc.)

epsreps commented 10 years ago

ich verwende derzeit contao 3.1.4 hier ein link bsp: http://leipzigschoolofdesign.de/kursangebote/tag/Final+Cut.html ich habe die tagcloud als ein auswahlmenü dargestellt, über welche ich die events (kurse) filtere bisher hatte ich auch keine probleme damit.

epsreps commented 10 years ago

wenn ich in der url ausgabe der tagcloud ($tag['tag_url']) das "+" durch "%20" ersetze klappts

epsreps commented 10 years ago

der fehler passiert auch, wenn ich alle erweiterungen bis auf tags,tags_events ausschalte

epsreps commented 10 years ago

wenn ich im template oder in der ModuleEventlistTags.php ein "$tag = str_replace("+"," ",$tag);" einsetze (bspw in zeile 59) funktioniert es wieder. an welcher stelle werden denn eigentlich die "+" der url mit dem leerzeichen zum korrekten vergleich für die ausgabefilterung ersetzt. oder versteh ich da was falsch?

epsreps commented 10 years ago

sorry, das problem war nicht korrekt beschrieben. es ist also nicht die tagcloud sondern die ausgabefilterung, die ich meinte.

hschottm commented 10 years ago

OK. Ich habe jetzt einfach mal sämtliche +-Ersetzungen durch die offizielle \System::urlencode Funktion ersetzt, die nimmt eigentlich immer %20, was auch kein Problem ist. + darf offiziell in den "query"-Teilen von URL's (alles, was hinter dem ? in der URL ist) für Leerzeichen verwendet werden. Sicherer ist aber %20, denn das darf sowohl als auch verwendet werden. \System::urlencode macht das, deswegen sollte das jetzt funktionieren.

hschottm commented 10 years ago

Fixed in f9721e10b754e61ece2a1204e484ae7b3f622ca2