Ghostfolio项目自定义货币删除功能的技术分析与实现

背景介绍

Ghostfolio是一款开源的投资组合管理工具，在2.110.0版本中出现了一个关于自定义货币删除功能的bug。该功能允许管理员添加和管理自定义货币，但在删除操作时出现了异常情况。

问题现象

当管理员尝试删除一个自定义货币时，系统会抛出Prisma相关的数据库错误，提示"Record to delete does not exist"。具体表现为：

1. 管理员在控制面板的"市场数据"部分添加了一个自定义货币(如PLN)
2. 尝试通过操作菜单删除该货币时
3. 前端发起DELETE请求到api/v1/admin/profile-data/YAHOO/USDPLN端点
4. 后端抛出PrismaClientKnownRequestError异常

技术分析

问题根源

经过分析，这个问题源于系统对自定义货币的存储和管理方式存在不一致性：

1. 自定义货币信息实际上存储在Property表的PROPERTY_CURRENCIES字段中，而不是SymbolProfile表
2. 删除操作错误地尝试从SymbolProfile表中删除记录
3. 前端展示的市场数据可能缓存了已删除的货币信息

系统架构理解

Ghostfolio采用NestJS框架构建，数据访问层使用Prisma ORM。自定义货币的管理涉及多个组件：

1. 前端Admin组件负责展示和触发操作
2. 后端AdminController处理API请求
3. PropertyService负责货币属性的持久化
4. SymbolProfileService处理符号配置(错误地被用于货币删除)

解决方案

正确实现方式

要实现自定义货币的完整删除功能，需要：

1. 从Property[PROPERTY_CURRENCIES]中移除目标货币
2. 更新前端缓存和展示
3. 确保不涉及SymbolProfile表的操作

具体步骤应为：

1. 通过propertyService.getByKey(PROPERTY_CURRENCIES)获取当前货币列表
2. 从数组中过滤掉要删除的货币
3. 使用propertyService.put()更新属性值
4. 前端同步更新展示

代码实现要点

后端控制器应修改为：

async deleteProfileData(
  @Param('dataSource') dataSource: string,
  @Param('symbol') symbol: string
) {
  // 获取当前货币列表
  const currenciesProperty = await this.propertyService.getByKey(
    PROPERTY_CURRENCIES
  );
  
  // 解析并过滤
  const currencies = JSON.parse(currenciesProperty.value);
  const updatedCurrencies = currencies.filter(c => c !== symbol);
  
  // 更新存储
  await this.propertyService.put({
    key: PROPERTY_CURRENCIES,
    value: JSON.stringify(updatedCurrencies)
  });
  
  return true;
}

前端组件需要相应调整，确保：

1. 正确调用删除API
2. 及时更新本地状态
3. 重新获取货币列表以确保一致性

技术启示

这个问题展示了在系统设计中保持数据模型一致性的重要性。开发者需要注意：

1. 明确不同业务实体的存储位置和方式
2. 避免将不同概念的数据混用同一套CRUD操作
3. 前后端对业务逻辑的理解必须一致
4. 错误处理应提供足够的信息帮助诊断

总结

Ghostfolio的自定义货币删除功能问题是一个典型的数据模型与业务操作不匹配案例。通过正确理解系统存储机制，将操作限制在Property表而非SymbolProfile表，可以解决这个问题。这也提醒开发者在设计类似功能时，需要仔细规划数据存储策略和操作边界。