Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
It is possible to access the add backend form after installing only search_api without any backend providers. In which case an error is displayed on the screen to inform the user that no backends are available. There is however still a "Save" button present, which will cause an internal server error when clicked.
This button should not be present at all when no backends are available. I've added a patch that removes it.
Comment | File | Size | Author |
---|---|---|---|
#3 | 3052877-3--no_backends_add_server_form.patch | 2.1 KB | drunken monkey |
| |||
#3 | 3052877-3--no_backends_add_server_form--tests_only.patch | 1.55 KB | drunken monkey |
Comments
Comment #2
borisson_Yes this makes sense. It also makes sens to do this in a seperate test because otherwise we'd have to uninstall things in the test method.
Comment #3
drunken monkeyOoh, Joris missing multiple nit-picks (unnecessary double quotes, using translation in tests, wrong case in
New
keyword)? You’re getting old … ;PIn general I agree, though: Nice patch, and sensible test – thanks a lot!
Comment #5
borisson_I don't think it is the age, it's probably more how busy things are :)
Comment #7
drunken monkeyYes, I know, sorry. ;) Can’t help my snide comments sometimes.
Also, to make you feel better: I also missed one (wrong case in
@inheritDoc
) which only my pre-commit hook caught.Fixed that on-the-fly and committed.
(Oooh, and forgot to correct the order of names in the commit message! Sorry, Legolasbo! Anyways, I at least got the author right, so its a detail.)
Thanks again, everyone!