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.In Drupal 8.3.x this is a fatal when submitting devel config form.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | devel-config_form_test-2833517-4.patch | 2.86 KB | AaronBauman |
| |||
| #2 | devel-missing_config_for_api_key-2833517.patch | 405 bytes | AaronBauman |
| |||
Comments
Comment #2
AaronBaumanI didn't bother to figure out where where api_url is coming from, or if it's even necessary, but this patch fixes the fatal.
Comment #3
AaronBaumanComment #4
AaronBaumanHere's a test for the devel config form which totally disputes my original claim.
(See the final assertion.)
This test might be worth rolling into the module, even if the original patch isn't.
Very frustrating - still haven't figured out where the additional junk key "api_url" is coming from.
Comment #5
willzyx CreditAttribution: willzyx commented@aaronbauman thanks for reporting!
api_urlwas removed some time ago when devel was only in dev version. I think that the properly solution for your problem is reinstall the devel module and start from scratch with the new configuration.As regards the test I really appreciate the fact that you submit it :) but IMHO every config settings should be tested contextually with the related functionality and I think that we already have test coverage for the config elements that you have tested (obviously not for api_url that is not more present in the code base).
Feel free to reopen this issue if you think that there is a better solution.
Comment #6
AaronBaumanOK, thank you for explaining.
I was pulling my hair out trying to find that config key!