inventory glitch

[TDMidkwhatisxd]

KitUI Issue

• This issue isn't duplicated - you can check if it is by using the search bar located at the top left hand corner and select "Issues" on the left.
• This issue includes appropriate markdown for sections - e.g. code blocks for crash dumps.
• This issue is understandable in English.

Issue description

...
If u have 9 slots in inventory, and try to buy kit, it say ur inventory full and some items didn't send,
but you get no items from kit and your money get took.

Reproducing the issue

1. ... have 9 slots free
2. ... buy kit

Client information

PocketMine-MP Version: (insert commit hash here)
MCPE Version: (insert game version here)
Plugin Version: (insert commit hash here)
PHP version: 7.2

Optional information

...

Replace this text with console / error logs or crash dumps. (Do not remove the triple backticks above and below this.)

Config

...

[TDMidkwhatisxd]

Can you add a check? Like if u cant send all items , it just wont send any? And it will say clear inventory before selecting a kit?

[TDMidkwhatisxd]

proof
https://youtu.be/QvKth8eNe8U

[TDMidkwhatisxd]

kit give 6 items, but I had 8 slots left.... it didn't give anything and took my money

[Infernus101]

You're using the old version, i removed adding them till inventory gets full feature and added counting the slots before adding the kit, use the latest version

[TDMidkwhatisxd]

ok, thx, ill try

[TDMidkwhatisxd]

is really old version? I updated 4 days ago it seems + items didn't get added , I could claim the kit without cooldown but money got took.

edit: ill update to see

[Infernus101]

Update lang.properties file and config.yml too, and try editing config.yml

[TDMidkwhatisxd]

all is latest, config, lang, even plugin, what now? same old glitch

[Infernus101]

The inventory glitch is fixed though, right?

[TDMidkwhatisxd]

same error, I already had latest plugin ^^ , idk why I updated it

[Infernus101]

But there is no message saying "Some of the items weren't added..." in the plugin anymore...

[TDMidkwhatisxd]

i didn't update the lang, but config and plugin were latest

[TDMidkwhatisxd]

there is no reason to lie about that ... I really don't have time to troll.

[Infernus101]

Check this issue with latest commit

[TDMidkwhatisxd]

[Server thread/CRITICAL]: Could not pass event 'pocketmine\event\server\DataPacketReceiveEvent' to 'KitUI v1.1.8': Undefined variable: money on Infernus101\KitUI\PlayerEvents
[18:48:54] [Server thread/CRITICAL]: ErrorException: "Undefined variable: money" (EXCEPTION) in "KitUI_dev-50.phar/src/Infernus101/KitUI/UI/windows/KitInfo" at line 67
[18:48:54] [Server thread/DEBUG]: #0 KitUI_dev-50.phar/src/Infernus101/KitUI/UI/windows/KitInfo(67): pocketmine\error_handler(integer 8, string Undefined variable: money, string phar:///root/12/plugins/KitUI_dev-50.phar/src/Infernus101/KitUI/UI/windows/KitInfo.php, integer 67, array Array())
[18:48:54] [Server thread/DEBUG]: #1 KitUI_dev-50.phar/src/Infernus101/KitUI/UI/windows/KitInfo(96): Infernus101\KitUI\UI\windows\KitInfo->select(string true.)
[18:48:54] [Server thread/DEBUG]: #2 KitUI_dev-50.phar/src/Infernus101/KitUI/PlayerEvents(80): Infernus101\KitUI\UI\windows\KitInfo->handle(pocketmine\network\mcpe\protocol\ModalFormResponsePacket object)

[TDMidkwhatisxd]

happens wen I claim a kit and not enough money

[Infernus101]

Fixed

[TDMidkwhatisxd]

ok , ill try

[TDMidkwhatisxd]

fixed