Skip to content
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

Global Loot Tables #33

Open
nazwadi opened this issue Jan 15, 2023 · 1 comment
Open

Global Loot Tables #33

nazwadi opened this issue Jan 15, 2023 · 1 comment

Comments

@nazwadi
Copy link
Contributor

nazwadi commented Jan 15, 2023

Issue:
Currently there is no way to add pre-existing lootdrop tables to the loottable_entries of a global loottable. It only allows you to create new ones. This feature request is to provide the options for searching by name or entering an existing lootdrop ID

Discussion:
When we want to add a lootdrop to a loottable for a single NPC, we do the following:

  1. Click the yellow + icon next to the label that reads, Lootdrops associated with this loottable:
    image
  2. Decide an option from one of the following:
  • Search for a Lootdrop by Name
  • Enter a Lootdrop ID
  • Create a New Lootdrop
    image.

For Global Loot, when we want to add a lootdrop to a loottable, we do the following:

  1. Click the yellow + icon next to the label that reads, Lootdrops associated with this loottable:
    image

  2. It assumes "Create a New Lootdrop", and takes us straight to the template for "Add New Lootdrop".
    image


Doing this, allows me to run a database query to add existing lootdrop tables:

INSERT INTO loottable_entries (loottable_id, lootdrop_id, multiplier, droplimit, mindrop, probability)
VALUES ('110562', '7638633', '1', '1', '0', '10');

using the peqphpeditor interface instead of switching to a SQL client.

I intend to work on this feature myself, but adding this issue here for transparency/feedback. Am I not considering a limitation in how global drops are supposed to work?

@joligario
Copy link
Contributor

Bringing in @mackal for discussion just in case.

I don't see any issues. Risk of orphans across loottables is my only immediate thought.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants