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

Zusätzl. Datenquellen :: Diverse Tabellen erst gar nicht anzeigen #163

Closed tbaddade closed 4 years ago

tbaddade commented 6 years ago

Man könnte die Liste der Tabellen minimieren. Folgende Tabellen bräuchten man sicherlich nicht indexieren

Dann alle

alxndr-w commented 6 years ago

unsichtbare Tabellen würde ich drin behalten, die Szenarien dafür gäbe es. Aber sonst stimme ich vorbehaltlos zu.

tbaddade commented 6 years ago

unsichtbare Tabellen würde ich drin behalten

stimmt. Können Datentabellen in popups sein. Habe ich oben korrigiert

xong commented 6 years ago

Ich mach mal ´n Veto. Ihr wisst nicht, wofür Search it verwendet wird. Am Ende ist das nur ein Index, der aufgebaut wird. Ich indexiere für Spezialfälle auch Systemtabellen von Redaxo. Die Volltextsuche ist ja nur ein Einsatzgebiet für das Addon.

tyrant88 commented 6 years ago

Ich hatte da eh keinen sinnvollen Ansatz gesehen, wer wie bestimmen soll welche Tabellen sichtbar sein sollen. Jemand installiert ein neues AddOn mit 50 Tabellen, das search it nicht kennt, und es erscheinen trotzdem 50 Tabellen. Wenn es eine Eigenschaft gäbe wie "Systemtabelle" oder so ähnlich könnte man die ausblenden und dann mit ner checkbox "auch Systemtabellen anzeigen" wieder einblenden.

xong commented 6 years ago

Eventuell könnte man auch einfach den Auswahlprozess in den Einstellungen so umstellen, dass man über eine Selectbox die Tabelle auswählen und dann auf "hinzufügen" klicken kann. Dann ist das nicht mehr so unübersichtlich.

alxndr-w commented 4 years ago
alxndr-w commented 4 years ago

Ich war erst zuversichtlich, dass das entfernen die Ansicht übersichtlicher macht - muss aber sagen, dass das Ergebnis ernüchternd ist: image

alxndr-w commented 4 years ago

in diesem Zuge könnte man höchstens sagen, dass die Tabellen von Search it selbst noch ausgeblendet gehören. (und yform_rest_token vermutlich auch)

Aber wie gesagt, richtig hilfreich war's jetzt nicht bei so vielen verbleibenden Tabellen. Ich würde daher gerne lieber dieses Issue schließen.

tyrant88 commented 4 years ago

mal über xongs Vorschlag nachdenken.. bzw. so ähnlich... Ein Multiselect was alle Tabellen und alle Spalten enthält mit dem man Spalten auswählt. Und eine eigene Anzeige für alle bereits ausgewählten...

tyrant88 commented 4 years ago

Den Dialog optimieren finde ich gut, aber das issue unter diesem Titel kannst du gerne schließen, weil ich keine Tabellen ausgeblendet haben möchte.

alxndr-w commented 4 years ago

Ziel muss sein, möglichst einfach Felder an/abwählbar zu machen. Ich denke, wir finden da eine bessere Eingabemöglichkeit, die ein Ausblenden überflüssig machen.