-
Notifications
You must be signed in to change notification settings - Fork 21
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
PF2E - Move item to "held" state, help handle toggle "Torch is Lit" #67
Comments
I try to avoid letting Torch get involved with the specific mechanics of any system. RationaleThis policy keeps me from having to make changes to Torch every time the wind blows across one system or another. Even the very narrow exception we made for GURPS was based on providing a cheap generic mechanism to support navigating any other custom item topology that came down the line. Since you can't provide a customization mechanism without at least two samples of use, we used the standard topology and the 40 lines of GURPS topology code as working examples that others could crib from, but since the first release of our new design last July, nothing else has needed it so far. The Counter-proposalThe challenge here is to find a way where Torch can provide a generic interface that can then be used externally by any system or module for integration.
Assuming we had such an event, could a rule element or some workbench-type module for pf2e register for notification of the change? If so, such an event would seem to me a broadly usable alternative. The pf2e code, which will need to track the progress of pf2e anyway, could then make the appropriate Let me know if this would give you a starting place you could work with. |
hmmm,
However, GMs and Players can really just go and set their torch or lantern to "Held in One Hand", by
So any work for you and for someone else to integrate the event you raise so as to cause a light emitting object to be held in one hand, would probably greatly out weigh the value. I was talking to a GM friend about my feature request and my first counter point against myself was system agnostic... however, as unfair as it certainly is, it's PROBABLY more reasonable for a mod like Torch to tell PF2e a light emitting object has just changed from worn to held in 1 hand than to find a way for pf2e system or a Mod to look for "User is requesting a light source [named X]" (and of course, I haven't looked at how hard it would be to set an Item to "is being held", so maybe i'm full of it). And again, I very much get the value of system agnostic, and I want to make sure that you know that I am not requesting or beggin', etc. I am tickled that you responded to the ticket at all, and I can see how the design of an event that could be available in any system, would allow integration, but keep your code streamlined. TLDR, i think that this would probably be better left as a pondering idea, but just know that pf2e has some extra gears related to "torches" and "lanterns" |
For now I've labeled this as enhancement, as I should at least provide the event that will make it possible for a third party to marry the module and the system. Perhaps, once the base hook is in place, someone with the appropriate js/ts experience and deep familiarity with That will certainly never happen if I don't provide some way for the communication to occur, so while I may not be implementing the solution here, I can at least "get out of the way" of the community providing satisfaction. The Foundry community is just amazing when they've got something to grab onto. |
Foundry: 11.313
PF2e: 5.6.2
without torch module, one of the most normal ways to cause a light aura is via an equipable item being set to "held"
in the PF2e "Rules Engine" a torch object has several "Rules", and like most PF2e the rules are only "in play" when the item is "equipped" versus exists, stowed, or if it has a "Worn Slot" then it must be in that slot (so a boring Torch has a Worn Slot of "Held in 1 Hand" (see first screenshot),
Because of the rules in a boring ol' torch, equipping the torch in at least 1 hand (see attached):
the point of all of this is...
I love the Torch Module for 2 reasons...
I would like to see the Torch module better integrate into PF2e's "Worn State", and instead of determining if a light is available and then creating a Token Light aura, and instead I'd like to see the HUD interface puts the light source item into the characters hand.
I see lots of complications...
Extra: In theory you can find any item that is capable of being a light source by looking for if an item in a token's inventory has a TokenLight rule on it (and maybe has a TokenLight rule, where the type is 'torch', which i found to be true on torches and lanterns, but the PF2E system programmers could tell you more about what the purpose of that property is). Then you could support any item right away that existed in someone inventory, as far as "equipping" to a hand and unequipping, even if you dont do anything with the torch complications of the toggle (which again, in my opinion, i would be absolutely ok with when your HUD is used that the toggle is just forced on right away and then when used to turn off a light, the entire torch is just stowed... let the player use the inventory interface if they want an extinguished torch in their hand)
The text was updated successfully, but these errors were encountered: