issues: 221393225
This data as json
id | node_id | number | title | user | state | locked | assignee | milestone | comments | created_at | updated_at | closed_at | author_association | active_lock_reason | draft | pull_request | body | reactions | performed_via_github_app | state_reason | repo | type |
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
221393225 | MDExOlB1bGxSZXF1ZXN0MTE1NjM3ODc2 | 236 | Concept review: Broadcast Generic Messages for Effects and Sounds | 37453 | closed | 0 | 2 | 2017-04-12T21:16:50Z | 2017-06-28T19:46:54Z | 2017-04-18T14:26:25Z | CONTRIBUTOR | 0 | ACEmulator/ACE/pulls/236 | Please comment and discuss the code below - The code length was getting quite large for landblock action queue broadcasting, so I came up with a generic solution that packs any `GameMessage` inside of an `OutboundMessageArgs` object. This allows for any `GameMessage` to potentially be sent over the `Landblock` `Broadcast` function. I also implemented changes in landblock to broadcast the effects and sounds, with generic functions. PLEASE NOTE: One obstacle that this method does not address, is per session sequences. If a Game Message has been extended into another message type (like `GameEventPlayerKilled`), then the initial sequence for the source session will be used. This does not matter for game messages that do not use sequences, like effects or sounds, but a solution will need to created to address this issue. This also means that the PlayerKilled message is currently bugged in this PR. The functionality that was worked on to produce the proof of concept, are effects and sounds, so when reviewing you may find it useful to start your breakpoints at `ActionApplyVisualEffect`. All discussion is welcome for this PR - Thanks! | {"url": "https://api.github.com/repos/ACEmulator/ACE/issues/236/reactions", "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | 79078680 | pull |