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.When entering the config_inspector form page (/admin/reports/config-inspector/.../form) I get this error:
Fatal error: Call to undefined method Drupal\Core\TypedData\Plugin\DataType\Boolean::getType() in .../modules/config_inspector/config_inspector.module on line 280
Am I missing something or is it broken?
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | get-type-undefined-method-error-2019091-8.patch | 1.3 KB | juanolalla |
| #2 | get-type-undefined-method-error-2019091-2.patch | 468 bytes | juanolalla |
Comments
Comment #1
Gábor HojtsyLol, we just found this same bug with @swentel as we are about to talk about config and would demo this in an hour or so :) Thankfully @swentel has a slightly older D8 copy, so that will work for the demo. Core dropped this I guess in http://drupalcode.org/project/drupal.git/commit/a18a231ea46c011369ca72a2... (#2056721: Remove TypedDataInterface::getType()). We should figure out how to do something useful without that.
Comment #2
juanolalla CreditAttribution: juanolalla commentedI've tried to fix it.
Comment #3
Gábor HojtsyReopened the issue for change notice update at #2056721-45: Remove TypedDataInterface::getType(). I don't have a good idea as to how to proceed here ATM. I agree this is a critical bug report, almost nothing of the module works ATM.
Comment #4
Gábor HojtsyWait, is that due to how the config schema provides this data anyway? :D That is not the only place we use getType() in the code though.
Comment #5
Gábor HojtsyAll right, based on Berdir in #2056721: Remove TypedDataInterface::getType(), using definition's type is the best we can do for now. Can you fix in all occurrances? :)
Comment #6
Gábor HojtsyComment #7
juanolalla CreditAttribution: juanolalla commentedOf course, I'll do it at soon as my kid go to bed ;)
Comment #8
juanolalla CreditAttribution: juanolalla commentedReplaced all occurrences of getType() for type element from getDefinition() array.
Comment #9
Gábor HojtsyOk, this module does not have automated tests yet, so we need someone to manually try these out. Did you click through the different variants yourself?
Comment #10
juanolalla CreditAttribution: juanolalla commentedYes, I just clicked the four options of Config Inspector: List, Tree, Form, and Raw data for a couple of items and it seems to be ok.
Comment #11
Gábor HojtsyYay, thanks, committed, pushed!