Neu Fehler bei Preisspannenfilter(?)

Tomas

Sehr aktives Mitglied
8. Januar 2018
330
70
Lübeck
Hallo Leute 🙂

Die SQL-Abfrage bei den Preisspannenfiltern scheint nicht ganz ok zu sein. Wenn ich z.B. die Spanne zwischen 99 und 300 filtern möchte, werden mir die Artikel für 300€ nicht angezeigt.

In der filter_inc.php Zeile 2305 steht:

Code:
$oFilter->cWhere .= " < " . $NaviFilter->PreisspannenFilter->fBis . " AND ";

jedoch würde

Code:
$oFilter->cWhere .= " <= " . $NaviFilter->PreisspannenFilter->fBis . " AND ";

mehr Sinn machen, oder sehe ich das falsch? In Zeile 2328 wird es ja quasi genau richtig gemacht.

Hier ein kleines Beispiel am JTL-Demo-Shop

Bei folgendem FIlter wird mir der "Fitness Tracker mit GPS" für 110€ noch angezeigt.
https://demo.jtl-shop.blackbike-forest.de/navi.php?k=7&pf=0_111

Jedoch bei https://demo.jtl-shop.blackbike-forest.de/navi.php?k=7&pf=0_110
nicht mehr.



VG
Tomas
 

waldemar.p

Moderator
Mitarbeiter
6. Januar 2016
111
17
Hallo Tomas,
danke für deinen Hinweis.

Das Problem habe ich an unsere Entwicklungsabteilung weitergeleitet.
 
  • Gefällt mir
Reaktionen: Tomas

FMoche

Moderator
Mitarbeiter
15. Dezember 2014
1.369
347
Halle (Saale)
Eigentlich ist das so Absicht.
Die untere Schwelle ist immer inklusiv, die obere nicht.
Ansonsten wäre der Artikel für 110€ ja sowohl in [0-110] als auch [110-999] vorhanden-
 

Tomas

Sehr aktives Mitglied
8. Januar 2018
330
70
Lübeck
Das habe ich mir bereits gedacht, dass das der Hintergrund dafür ist. Für die Preis-Filter macht das auch durchaus Sinn.

Die Spanne von [0€ - 99,99€] würde vermutlich auch einfach ein wenig unsexy aussehen.

Ich war da nur drauf gestoßen, weil ich eine Range an Artikeln mit gewissem Preis anzeigen wollte.

Nehmen wir das Beispiel eines Links für Geschenk-Ideen bis 50€.
Dann müsstest du ja den Parameter ?pf=0_51 statt ?pf=0_50 mit schicken damit Artikel die genau 50€ teuer sind angezeigt werden. Dann werden aber wiederum Artikel angezeigt die z.B. 50,99€ kosten.

In meinem konkretem Fall habe ich die Preis-Filter so umgebaut, dass sie nun einen Range-Slider haben. Da muss ich natürlich die obere Schwelle mit berücksichtigen. Das hab ich im Plugin schon direkt mit gelöst. Ich dachte aber ich spreche das einfach mal hier an. 😉
 

Ähnliche Themen