Remove the required \Drupal\ai\AiVdbProviderInterface::getConfig() and use ::isSetup() instead in base class
>>> [!note] Migrated issue <!-- Drupal.org comment --> <!-- Migrated from issue #3539212. --> Reported by: [kim.pepper](https://www.drupal.org/user/370574) Related to !928 !810 >>> <h3 id="summary-problem-motivation">Problem/Motivation</h3> <p><code>\Drupal\ai\AiVdbProviderInterface::getConfig()</code> is only used in <code>\Drupal\ai\Base\AiVdbProviderClientBase::validateSettingsForm()</code> to check that the Vector DB is configured. We already have a method for that.</p> <p>Some VDB providers don't store config, and requiring ImmutableConfig requires a lot of workarounds.</p> <h4 id="summary-steps-reproduce">Steps to reproduce (required for bugs, but not feature requests)</h4> <p>Please provide information like AI modules enabled, which AI provider, browser, etc.</p> <h3 id="summary-proposed-resolution">Proposed resolution</h3> <ol> <li>Remove usage from base class</li> <li>Remove method from the interface</li> <li>Add a note to the change record that its now optional</li> </ol> <h3 id="summary-remaining-tasks">Remaining tasks</h3> <h3>Optional: Other details as applicable (e.g., User interface changes, API changes, Data model changes)</h3>
issue