feature/events #19

Merged
Jonathan merged 10 commits from feature/events into develop 3 months ago
Owner

Created a new resource class called EventResource that can be used to signal events and store values outside of scenes.
The Payload is stored as a Godot Variant and can be used for all kinds of typed functions.
I also added active event raising passive onvaluechange-events with old and new payloads.
You can check out a sample scene under scenes/testing/EventTest.

Created a new resource class called EventResource that can be used to signal events and store values outside of scenes. The Payload is stored as a Godot Variant and can be used for all kinds of typed functions. I also added active event raising passive onvaluechange-events with old and new payloads. You can check out a sample scene under scenes/testing/EventTest.
Jonathan was assigned by kziolkowski 3 months ago
kziolkowski added 39 commits 3 months ago
kziolkowski added 1 commit 3 months ago
kziolkowski self-assigned this 3 months ago
kziolkowski requested review from Jonathan 3 months ago
Jonathan was unassigned by kziolkowski 3 months ago
Jonathan requested changes 3 months ago
Jonathan left a comment
Owner

Ich verstehe nicht, wie Payload verwendet werden soll, bzw. was die Idee hinter der Implementierung ist. @kziolkowski Können wir da noch einmal drüber sprechen?

Ich verstehe nicht, wie Payload verwendet werden soll, bzw. was die Idee hinter der Implementierung ist. @kziolkowski Können wir da noch einmal drüber sprechen?
public void EventInvoked(Variant payload)
{
if(_showLog)
GD.Print("Event Raised on: " + Name);
Owner

Debug logs bitte noch entfernen

Debug logs bitte noch entfernen
Poster
Owner

An sich stimme ich dir zu, aber diese habe ich mit Absicht drin gelassen und hinter einem bool versteckt, weil ich die Erfahrung gemacht habe, dass man diese Debugs relativ häufig braucht, wenn man mit Events und Variablen arbeitet.

An sich stimme ich dir zu, aber diese habe ich mit Absicht drin gelassen und hinter einem bool versteckt, weil ich die Erfahrung gemacht habe, dass man diese Debugs relativ häufig braucht, wenn man mit Events und Variablen arbeitet.
public void EventPayloadChanged(Variant payload, Variant oldPayload)
{
if(_showLog)
GD.Print($"Calling Event Payload Changed Signals on: " + Name);
Owner

Debug logs bitte noch entfernen

Debug logs bitte noch entfernen
kziolkowski marked this conversation as resolved
/// Raises <see cref="EventResource"/> at index.
/// </summary>
/// <param name="index"></param>
public void RaiseEvent(int index)
Owner

Sollte nicht existieren. Es gibt keinen guten Grund, warum man das aufrufen sollte. Wenn man zwei verschiedene Events separat voneinander raisen will, sollte man ohnehin zwei Event Raiser benutzen.

Sollte nicht existieren. Es gibt keinen guten Grund, warum man das aufrufen sollte. Wenn man zwei verschiedene Events separat voneinander raisen will, sollte man ohnehin zwei Event Raiser benutzen.
kziolkowski marked this conversation as resolved
kziolkowski added 3 commits 3 months ago
Jonathan approved these changes 3 months ago
Jonathan merged commit afce4ad0b9 into develop 3 months ago

Reviewers

Jonathan approved these changes 3 months ago
The pull request has been merged as afce4ad0b9.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Jonathan/Babushka#19
Loading…
There is no content yet.