Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.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
Newkeyword)? 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!