Oqtane框架中ModuleRepository.GetModules方法的IDeletable属性问题解析

在Oqtane框架开发过程中，ModuleRepository.GetModules方法存在一个值得开发者注意的问题：该方法未能正确填充IDeletable接口定义的属性（DeletedBy、DeletedOn和IsDeleted），这可能导致在查询模块时获取到不准确的数据信息。

问题本质

Oqtane框架中的Module实体实现了IDeletable接口，该接口定义了三个关键属性用于软删除功能：

• DeletedBy：记录删除操作执行者
• DeletedOn：记录删除时间
• IsDeleted：标记是否已被删除

然而，当开发者使用ModuleRepository.GetModules方法查询站点模块时，这些属性并未被正确填充，导致查询结果中包含了所有模块记录，包括那些已被标记为删除的模块。这违背了软删除机制的设计初衷，可能引发数据一致性问题。

影响范围

这个问题会影响到所有依赖IDeletable属性进行数据筛选的业务逻辑。例如，当开发者尝试获取特定类型的未删除模块时，使用如下代码：

var allModules = moduleRepository.GetModules(site.SiteId);
return allModules.Where(m => m.ModuleDefinitionName == "SomeModule" && !m.IsDeleted)
    .ToList();

这段代码实际上会返回所有匹配模块定义名称的模块，无论它们是否已被删除，因为IsDeleted属性始终为false。

解决方案

针对这个问题，Oqtane框架提供了更合适的替代方法。开发者应当使用：

IEnumerable<PageModule> GetPageModules(int siteId)

这个方法会返回PageModule对象集合，其中包含了完整的领域属性，包括IDeletable接口定义的所有属性。通过这种方式，开发者可以准确获取到模块的删除状态信息，实现正确的数据筛选。

最佳实践建议

1. 当需要查询模块信息并考虑删除状态时，优先使用GetPageModules方法而非GetModules方法。

2. 如果确实需要使用GetModules方法，开发者应当意识到该方法不会提供删除状态信息，需要自行处理可能的数据一致性风险。

3. 在自定义模块开发中，如果需要实现类似的软删除功能，建议参考Oqtane框架中PageModule的实现方式，确保IDeletable属性被正确维护和使用。

理解这个问题有助于开发者在Oqtane框架中正确处理模块数据，避免因数据状态不明确而导致的业务逻辑错误。