Respect cleanup = never on disableEntities

I'm hitting this as a bug on a cleanup=never entity which has a v4 api and not a v3 api so it could be
solved using more failure tolerance in the disable function or checking more values

However, I actually think the intuitive meaning of cleanup = never is 'never try to clean this up
' and I don't see why that would apply to uninstall & not disable

CRM/Core/ManagedEntities.php

@@ -195,7 +195,7 @@ public function reconcileDisabledModules() {
 
     $in = CRM_Core_DAO::escapeStrings(array_keys($this->moduleIndex[FALSE]));
     $dao = new CRM_Core_DAO_Managed();
-    $dao->whereAdd("module in ($in)");
+    $dao->whereAdd("module in ($in) AND cleanup != 'never'");
     $dao->orderBy('id DESC');
     $dao->find();
     while ($dao->fetch()) {