search

tollwerk / TYPO3-ext-tw_lucenesearch

Simple and lightweight implementation of the Apache Lucene Index as frontend search solution for TYPO3. Created by Joschi Kuphal (@jkphl), licensed under the terms of the GPL v2 license.
http://typo3.org/extensions/repository/view/tw_lucenesearch/
GNU General Public License v2.0
4 stars 3 forks source link

Bugfixes: L-Param (0 != '') -> duplicates, languageFlag in BE Module #11

Closed cdfre closed 7 years ago

cdfre commented 10 years ago

Hallo @jkphl,

ich hab mir lange Zeit gelassen, aber jetzt habe ich den PR einmal komplett überarbeitet. Besser gesagt: Von Grund auf neu entwickelt.

Denn deine Hinweise auf die config.index_reference Konfiguraton hatte ich zuvor wirklich nicht bedacht.

Da in "linkParams" allerdings nicht nur der Sprach-Parameter vorhanden sein kann, sondern auch weitere Parameter wie z.B. "print" und es auch keine default-Einstellung für diesen Parameter gibt, habe ich eine Standardlösung geschaffen, die somit nicht verlangt, dass man in index_reference den Sprachparameter mit einer Default-Konfiguration einfügt um duplikate zu verhindern.

Bei mir trat dieses Problem bei einem Kunden auf, bei dem immer der Sprachparameter (auch bei der Standardsprache) in der URL (Realurl-Segment) vorhanden sein sollte, die URL aber natürlich auch ohne aufgerufen werden kann. Ohne eine Konfiguration wie "config.index_reference = id,L(0-1)=0" würde es somit für die Standardsprache duplikate geben.

Deswegen habe ich nun einen zusätzlichen Konfigurationparameter "languageReference" eingefügt, der nur angibt, welcher Parameter für die Sprache zuständig ist. Wird kein default für die Sprache definiert wird 0 als default gesetzt.

Zudem ist mir aufgefallen dass die Constraints [(0-1)-Angabe] teilweise einen anderen Typ haben können, als die Parameter (String != Integer) und dafür eine Abhilfe geschaffen.

Für die Flaggen-Anzeige im Backend musste ich ein Utility einfügen, dass im Backend das TypoScript parsed und ebenfalls diesen languageReference-Parameter ausliest.

Hoffe dir gefallen meine Ergänzungen.

Beste Grüße David

jkphl commented 10 years ago

Hi @cdfre,

danke! Hier ist derzeit so sehr Land unter, dass ich noch nicht weiß, wann ich einen Blick auf den PR werfen kann, aber in jedem Fall vielen Dank für die viele Mühe!

Grüße, Joschi

cdfre commented 10 years ago

push Mal ein kleiner Reminder ;)

jkphl commented 10 years ago

Hallo @cdfre,

danke für den Reminder. Die Arbeitslast bei mir war die letzten Monate derart hoch, dass ich keinerlei Zeit für Nebenprojekte hatte, und das wird — auch wenn es sich nun langsam bessert — wohl noch ein bißchen so bleiben. Ich hoffe, spätestens gegen Jahresende etwas Zeit für solche Dinge zu finden. Ich bitte um noch etwas Geduld. :)