Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Speichern eines YForm Datensatzes, indexiert alle Datensätze #364

Closed
tbaddade opened this issue Jan 27, 2023 · 7 comments · Fixed by #365
Closed

Speichern eines YForm Datensatzes, indexiert alle Datensätze #364

tbaddade opened this issue Jan 27, 2023 · 7 comments · Fixed by #365

Comments

@tbaddade
Copy link
Member

tbaddade commented Jan 27, 2023

Description / Beschreibung
Beim debuggen für #361 ist mir aufgefallen, dass nach dem Speichern eines YForm-Datensatzes alle Datensätze der Tabelle indexiert werden.

Würde es nicht ausreichen, nur diesen bearbeiteten Datensatz neu zu indexieren?

@tyrant88
Copy link
Member

tyrant88 commented Feb 2, 2023

Ja, das ist super. Da ist tasächlich ein Bug drin. Ich bau die IndexColumn()-Methode um. Das könnte ein echter Performance-Gewinn werden.

@palber
Copy link

palber commented Feb 3, 2023

Bin heute auf die selbe Problematik gestoßen.
Wir haben ein Projekt wo wir yrewrite_title und yrewrite_description bei den rex_article zusätzliche mit indexieren lassen wollen.

Sobald nun aber z.B. ein User bei einem Artikel die yrewrite URL umstellt (z.B: eine Umleitung auf internen Artikel hinterlegt) werden auch über die indexColumn()-Methode alle Artikel neu indexiert (in dem Fall sogar mehrmals, da er für jede einzelne column alles neu indexiert wird,)

Kann bei einem Umbau gerne behilflich sein

@tyrant88
Copy link
Member

tyrant88 commented Feb 3, 2023

@palber war schon mitten dabei, aber danke für das Angebot!!!

@tyrant88
Copy link
Member

tyrant88 commented Feb 3, 2023

wäre cool wenn du es noch nmal testen kannst

@tyrant88
Copy link
Member

tyrant88 commented Feb 3, 2023

mist doch noch ein bug

@palber
Copy link

palber commented Feb 3, 2023

Habe es nun getestet. Mein Fall scheint ebenfalls behoben zu sein! Danke dir!

@tyrant88
Copy link
Member

tyrant88 commented Feb 3, 2023

Das freut mich!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants