/shop add duplicates damaged, enchanted or renamed items

Đang mở
  • Có gì không ổn?
#64 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
63/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
java
Lĩnh vực
backend

Hướng nghiên cứu

Read InventoryManagementSystem.canRemove and its countQuantity logic, then trace how AddCommand removes items and how Transaction.process handles leftovers from addItem. Check the existing inventory tests and add coverage for damaged and renamed items during stock addition and purchases into partial stacks. Done when only eligible items are counted and removed, and inventory changes match the quantities recorded by the store.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

bug

Problem

/shop add counts items by material, but removes them by exact match. A damaged, enchanted or renamed item passes the check, isn't removed from the player, and is still added to the store. This duplicates items.

InventoryManagementSystem.canRemove uses countQuantity, which only compares the material. AddCommand then calls removeItem(new ItemStack(material, quantity)), which only removes plain items, and ignores the leftovers it returns.

Steps to reproduce

  1. Have a store selling diamond_sword, and hold one damaged diamond sword.
  2. Run /shop add diamond_sword 1. The store's quantity goes up and the player keeps the damaged sword.
  3. Run /shop remove diamond_sword 1. The player now has two swords, one of them undamaged.

Reproduced in a test: one sword in the inventory became two.

Related

getAvailableSpace also counts partial stacks by material only, so space in a stack of renamed items is treated as free. On a real server addItem can't merge into those stacks, so a purchase could take the player's money and lose the items that didn't fit. Transaction.process ignores the leftovers from addItem. MockBukkit merged those stacks in testing, so this part comes from reading the code and should be confirmed on a test server.

Scope

  • Decide how non-plain items are treated. The simplest rule is that only plain items (matching a new ItemStack of that material) can be added.
  • Use the same rule when counting, checking space and removing items.

Done when

  • A damaged, enchanted or renamed item can't be added as plain stock, and the player's inventory always changes by exactly what the store records.
  • Tests cover damaged and renamed items for add, and for buying into partial stacks.
Ngôn ngữ chính
Java
Star
1
Fork
0
Merge trung bình
2 giờ 35 phút
Pull request đã merge (30 ngày)
14

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của BrendonButler/Shops

Tất cả issue của BrendonButler/Shops

Issue tương tự

Thêm issue về Java

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.