-
Notifications
You must be signed in to change notification settings - Fork 63
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature request regarding selling commands #105
Comments
Hm, so I do have a possible solution in mind, but it requires some effort from your side.
The shopitem with the condition will be hidden if the condition is not met (e.g. inventory full). In that case, the second item is shown. Now the tricky part: BSP does NOT come with a "inventory is full" condition. However, it DOES support placeholder conditions (also explained on the page linked above). You just need to find some PlaceholderAPI extension, which provides an "inventory is full" placeholder or "count of filled inventory slots" or "first index of empty inventory slot" or something of that kind. I am sure there are placeholders of that kind available. The CheckItem extension might be suitable (https://github.com/PlaceholderAPI/PlaceholderAPI/wiki/Placeholders). |
Reopening the issue: |
Still very much in need of this if possible <3 |
So I'm attempting to use bossshop to sell 'stat trackers' to players using this plugin https://www.spigotmc.org/resources/%E2%9A%A1-1-16-1-16-5-stattrackers-%E2%9C%A8-track-stats-on-your-items-%E2%9C%85-eula-friendly-perk.88247/
My config looks like this for BSP
1:
RewardType: item
PriceType: money
Price: 100
RewardType: command
Reward:
- stgive %player% damage_dealt
Message: '&aYou''ve purchased a damage dealt tracker for $100'
InventoryLocation: 1
ExtraPermission: ''
The issue I have is, It lets you purchase even if your inventory is full, Now, I know why it does this, because BSP sees it as a command, not an item, But is it possible to add something that STOPS the sale if the inventory is full, Even if it's a command reward. aka PreventFullInv: True
The text was updated successfully, but these errors were encountered: