Small correction needed in ThemeInstallerInterface::uninstall() docblock.

Kindly review patch.
I have updated docblock.

Adding event tag.

The next step is to review the language in the uploaded patch.

This look like a good change to me.

The update is not quite correct. The error is thrown for any base theme that has currently installed dependent theme is uninstalled. See the code:

      // Base themes cannot be uninstalled if sub themes are installed, and if
      // they are not uninstalled at the same time.
      if (!empty($list[$key]->sub_themes)) {
        foreach ($list[$key]->sub_themes as $sub_key => $sub_label) {
          if (isset($list[$sub_key]) && !in_array($sub_key, $theme_list, TRUE)) {
            throw new \InvalidArgumentException("The base theme $key cannot be uninstalled, because theme $sub_key depends on it.");
          }
        }
      }

+++ b/core/lib/Drupal/Core/Extension/ThemeInstallerInterface.php
@@ -44,7 +44,7 @@ public function install(array $theme_list, $install_dependencies = TRUE);
-   *   Thrown when trying to uninstall the default theme or the admin theme.

Well this bit is true as well as the base theme thing. You can't uninstall the admin theme, the default theme or a theme that another theme depends on.

-   *   Thrown when trying to uninstall a theme from which other themes depend.
+   *   Thrown when trying to uninstall the admin theme, the default theme or
+   *   a theme that another theme depends on.

Re-upping @jungle's version in #10 since I like that wording better, to make clear that is the version I'm RTBCing

Saving issue credit.

Committed and pushed 76aa51fdf9 to 10.1.x and 8295e5de71 to 10.0.x and 93ea80732b to 9.5.x and 3e872baf91 to 9.4.x. Thanks!

Backported to 9.4.x since this is a docs bug fix.

The fail was a random one.