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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

legolasbo created an issue. See original summary.

borisson_’s picture

Status: Needs review » Reviewed & tested by the community

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.

drunken monkey’s picture

Ooh, Joris missing multiple nit-picks (unnecessary double quotes, using translation in tests, wrong case in New keyword)? You’re getting old … ;P
In general I agree, though: Nice patch, and sensible test – thanks a lot!

borisson_’s picture

Status: Needs review » Reviewed & tested by the community

I don't think it is the age, it's probably more how busy things are :)

drunken monkey’s picture

Status: Reviewed & tested by the community » Fixed

Yes, 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!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.