From 1dccf5f0ee88b8b62b2d1cca8fb60d85082be86d Mon Sep 17 00:00:00 2001
From: catch <catch@35733.no-reply.drupal.org>
Date: Fri, 20 Mar 2020 12:30:11 +0000
Subject: [PATCH] Issue #3121024 by jungle:  Remove todo in
 ThemeInstaller::uninstall() or refactor to share logic with module installer

---
 core/lib/Drupal/Core/Extension/ThemeInstaller.php | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/core/lib/Drupal/Core/Extension/ThemeInstaller.php b/core/lib/Drupal/Core/Extension/ThemeInstaller.php
index fdcb6ed931ce..860d79fe8d4e 100644
--- a/core/lib/Drupal/Core/Extension/ThemeInstaller.php
+++ b/core/lib/Drupal/Core/Extension/ThemeInstaller.php
@@ -266,9 +266,6 @@ public function uninstall(array $theme_list) {
       }
       // Base themes cannot be uninstalled if sub themes are installed, and if
       // they are not uninstalled at the same time.
-      // @todo https://www.drupal.org/node/474684 and
-      //   https://www.drupal.org/node/1297856 themes should leverage the module
-      //   dependency system.
       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)) {
-- 
GitLab