patchinfo_uninstall() has redundant config->delete() call
>>> [!note] Migrated issue <!-- Drupal.org comment --> <!-- Migrated from issue #3008869. --> Reported by: [idebr](https://www.drupal.org/user/1879760) >>> <h3 id="summary-problem-motivation">Problem/Motivation</h3> <p>The <code>patchinfo_uninstall()</code> hook removes the configuration related to the patchinfo module. However, this configuration is removed automatically in <code>\Drupal\Core\Extension\ModuleInstaller::uninstall()</code>:</p> <div class="codeblock"> <pre><span style="color: #000000"><span style="color: #0000BB">&lt;?php<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span><span style="color: #FF8000">// Remove all configuration belonging to the module.<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span><span style="color: #007700">\</span><span style="color: #0000BB">Drupal</span><span style="color: #007700">::</span><span style="color: #0000BB">service</span><span style="color: #007700">(</span><span style="color: #DD0000">'config.manager'</span><span style="color: #007700">)-&gt;</span><span style="color: #0000BB">uninstall</span><span style="color: #007700">(</span><span style="color: #DD0000">'module'</span><span style="color: #007700">, </span><span style="color: #0000BB">$module</span><span style="color: #007700">);<br></span><span style="color: #0000BB">?&gt;</span></span></pre></div> <h3 id="summary-proposed-resolution">Proposed resolution</h3> <p>Remove the redundant call to <code>$config-&gt;delete()</code> in <code>patchinfo_uninstall()</code></p> <h3 id="summary-remaining-tasks">Remaining tasks</h3> <p>None.</p> <h3 id="summary-ui-changes">User interface changes</h3> <p>None.</p> <h3 id="summary-api-changes">API changes</h3> <p>None.</p> <h3 id="summary-data-model-changes">Data model changes</h3> <p>None.</p>
issue