// Тут ещё добавил проверку на пустоту строки на всякий случай - не знаю,
// как поведёт себя layerTable.Has, если ему сунуть пустую строку... Может и зря паникую
if (!string.IsNullOrEmpty(sLayerName) && sLayerName != "0")
{
Document doc = Application.DocumentManager.MdiActiveDocument;
Database db = doc.Database;

using (Transaction tr = db.TransactionManager.StartTransaction())
{
LayerTable layerTable = (LayerTable)tr.GetObject(db.LayerTableId, OpenMode.ForRead);
if (layerTable.Has(sLayerName))
{
ObjectId idLayerDelete = layerTable[sLayerName];
ObjectId idCurrentLayer = db.Clayer;

if (idCurrentLayer == idLayerDelete)
{
//db.Clayer = layerTable["0"];
db.Clayer = db.LayerZero;
}

// Здесь слой открывать не нужно, сделаем это перед удалением
//LayerTableRecord layer = (LayerTableRecord)tr.GetObject(idLayerDelete, OpenMode.ForRead);
BlockTable blockTable = (BlockTable)tr.GetObject(db.BlockTableId, OpenMode.ForRead);

//layer.IsLocked = false; // < - Вот тут, по идее, должно упасть. Объект layer открыт на чтение.
//layer.IsFrozen = false; // <- Вот тут, по идее, должно упасть. Объект layer открыт на чтение.
// Вообще, в .NET API об этом можно не париться. Достаточно использовать перегрузку метода открытия объекта:
// tr.GetObject(id, OpenMode.ForWrite, false, true) <- последняя true - это разрешить оперировать объектами на блокированных слоях
// а то что слой выключен или заморожен, вроде как, .NETу фиолетово (хотя, могу ошибаться - лучше перепроверить)

foreach (ObjectId btrId in blockTable)
{
BlockTableRecord blockTableRecord = (BlockTableRecord)tr.GetObject(btrId, OpenMode.ForRead);
if (!blockTableRecord.IsFromExternalReference)
{
foreach (ObjectId entId in blockTableRecord)
{
//Entity ent = (Entity)tr.GetObject(entId, OpenMode.ForRead);
// Использовать преобразование через as безопаснее. Если объект не того типа - то будет просто null.
// А если использовать преобразование типа через скобочки спереди (Entity), то если объект вдруг окажется не Entity,
// то будет исключение и сбой. В этом случае такое маловероятно, но в других - вполне возможно
Entity ent = tr.GetObject(entId, OpenMode.ForRead, false, true) as Entity;

//if (ent.Layer.ToUpper() == sLayerName.ToUpper())
if (ent.LayerId.Equals(idLayerDelete))
{
// У транзакций есть один косяк - не любят этот метод
// Я не ловил с этим пролем, но вроде как они есть
// Исправлять не буду, если интересно - почитай: https://adn-cis.org/v-autocad-2018.1-metod-upgradeopen-privodit-k-fatalnoj-oshibke-vnutri-tranzakczii.html
ent.UpgradeOpen();
ent.Erase();
}
//else

// Можно одним выражением "убить двух зайцев" - и проверить что это блок и сохранить его в переменной
else if (ent is BlockReference blkRef)
{
//BlockReference blkRef = ent as BlockReference;
//if (blkRef != null)
//{
// Что-то мне кажется это неправильно - удалять атрибуты блока, оставляя при этом сам блок...
// Может, лучше их на другой слой кидать?
foreach (ObjectId attId in blkRef.AttributeCollection)
{
//AttributeReference att = (AttributeReference)tr.GetObject(attId, OpenMode.ForRead);
AttributeReference att = tr.GetObject(attId, OpenMode.ForRead, false, true) as AttributeReference;
//if (att.Layer.ToUpper() == sLayerName.ToUpper())
// Добавил проверку на null на всякий случай
if (att != null && att.LayerId.Equals(idLayerDelete))
{
att.UpgradeOpen();
att.Erase();
}
}
//}
}
}
}
}

LayerTableRecord layer = (LayerTableRecord)tr.GetObject(idLayerDelete, OpenMode.ForWrite);
//layer.UpgradeOpen();
layer.Erase();