-
Notifications
You must be signed in to change notification settings - Fork 24
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
Admin panel/blueprint breaks indexer #132
Comments
Confirm that, using Grav v1.7.43, Admin v1.10.43, TNT Search v3.4.0. After having tried the German stemmer, I wanted to switch it off again and got a My quick-n-dirty solution was to look up the original in I also tried your solution (quote the |
Glad to hear that you could solve your problem. Unfortunately there's no response from the package owner @trilby-bot |
@trilby-bot is just a bot, he's not going to be much help :) I will look at this issue when i'm back next week. |
Hi @rhukster ! Testing GRAV+TNT Search on two different VMs.
No apparent differences in PHP config... Any ideas on why these two VMs behave differently? Can the OS difference be a factor here? |
That's strange, do you happen to know what the error was when the stemmer was disabled? Was it a permissions issue? |
I may have found a bug. The value for the stemmer setting "Default" should be
no
. (At least as far as I can see from code.)For the dropdown in the admin panel the value for stemmer->default is defined here:
grav-plugin-tntsearch/blueprints.yaml
Line 184 in 9299fec
grav-plugin-tntsearch/blueprints.yaml
Line 186 in 9299fec
Unfortunately the 'no' gets rendered as
0
in the admin ui (check "data-value"):If you click on save in admin ui, the value
0
will be stored inuser/config/plugins/tntsearch.yaml
which ultimately lead to the following exeception when you start the indexer:To get an actual
no
into the config file via the admin ui, I propose to change the blueprint.yaml fromto
The text was updated successfully, but these errors were encountered: