Deleting a default store leaves it active

未关闭 适合新手
  • 哪里不对？
#70 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
78/100
Issue 类型
缺陷
描述清晰度
描述清楚
活跃度
活跃
技术栈
java
领域
backend, databases, testing

调研方向

先阅读 DeleteCommand 以及维护 Store.STORES、Store.DEFAULT_STORES 和 config.yml 的代码；追踪默认商店如何按世界以及通过全局 null 条目建立键。添加或扩展一个测试，先删除默认商店，再尝试从该商店购买。完成标准是删除操作清除内存和配置中的默认值，并且之后的购买不再成功。

由索引模型根据 Issue 内容生成。

描述

bug

Problem

Deleting a store that's a default store doesn't clear it as the default. DeleteCommand only removes it from Store.STORES, so it stays in Store.DEFAULT_STORES and in config.yml.

Until the next restart, players in that world are still served by the deleted store. Reproduced in a test: after /shop delete Def -f, /shop buy stone 1 still succeeded, and the player's money went to a store that no longer exists.

Done when

  • Deleting a store removes it from the default stores in memory and in the config, for its world and for the global null entry.
  • A test covers buying after the default store is deleted.
主要语言
Java
星标
1
派生
0
平均合并
2 小时 17 分钟
30 天内合并 PR
13

环境准备

  • 没有 Dockerfile 或 Docker Compose 文件
  • 没有 Pull Request 模板
  • 阅读贡献指南

从这里开始

  1. 先读完整个 Issue，再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库，在一个分支上完成修改。
  4. 提交 Pull Request，并在描述里引用这个 Issue 编号。

BrendonButler/Shops 的其他 Issue

查看 BrendonButler/Shops 的全部 Issue

相似的 Issue

更多 Java Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。