search

FriendsOfREDAXO / search_it

Umfangreiche Volltextsuche für REDAXO 5 CMS. Durchsucht Artikel, Medien, Dateien, PDF-Inhalte und Datenbank-Einträge.
https://github.com/FriendsOfREDAXO/search_it
MIT License
65 stars 18 forks source link

Search_it vs. access denied #383

Closed skerbis closed 10 months ago

skerbis commented 10 months ago

Search_it versucht Artikel die mit accessdenied gesperrt wurden zu indexieren. Das wirft Fehler im Log. (404). Ich denke es liegt an der Vererbung der Rechte von accessdenied. Könnte access denied eine Info vorab an search_it übergeben diesen Artikel zu überspringen?

Beispiel: https://github.com/FriendsOfREDAXO/accessdenied/blob/133d97e317498b1eb46cdb4ef1a5126860e50d41/boot.php#L23

tyrant88 commented 10 months ago

Ich nehme mal an, du hast "Indexiere offline Artikel" angehakt? Und access_denied setzt den Status auf "2"?

skerbis commented 10 months ago

Nein. Offline Artikel werden nicht indexiert. Accessdenied hat aber eine Vererbung auf Unterkategorien (Siehe Beispiel oben) dort wird der Status nicht verändert. Somit versucht search_it diese zu indexieren? Ich habe nicht geprüft ob die nur mit Status 2 nicht indexiert werden.

tyrant88 commented 10 months ago

Ah, okay. Puh, das macht die Sache schwierig. Da muss ich noch mal überlegen...

Bisher habe ich als Problem gefunden: Im Moment wird nur geprüft, ob eventuell die Option "indexoffline" gesetzt ist. Das könnte ich erweitern zu ("indexoffline" UND status == 0). Also "nur wenn old school offline"...

skerbis commented 10 months ago

Wo findet der check beim indexieren statt? ... würde mir das gerne mal anschauen. Vielleicht finden wir gemeinsam eine Lösung.

tyrant88 commented 10 months ago

Nun ja... unmittelbar... hier: https://github.com/FriendsOfREDAXO/search_it/blob/master/lib/search_it.php#L258

skerbis commented 10 months ago

ok. Dann wäre dort bzw davor ein EP ganz gut wo das Access-Denied vielleicht reingrätschen kann und im IF dann eine weitere Bedingung reinmogeln könnte für den aktuellen Artikel. ?

tyrant88 commented 10 months ago

Das wäre eine Idee.

skerbis commented 10 months ago

Mache ich fertig

skerbis commented 10 months ago

So könnten auch andere AddOns darauf Einfluss nehmen :-)

tyrant88 commented 10 months ago

Ja, sonst fände ich die Idee auch nicht gut.

skerbis commented 10 months ago

https://github.com/FriendsOfREDAXO/search_it/pull/384 https://github.com/FriendsOfREDAXO/accessdenied/pull/43

skerbis commented 10 months ago

Merged in: https://github.com/FriendsOfREDAXO/search_it/pull/384 Danke @tyrant88

tyrant88 commented 10 months ago

So bald ich Zeit habe mach ich auch ein release

skerbis commented 10 months ago

👍🏻