From f0e31d4de0c8e206260467420257a6f04ffc8e90 Mon Sep 17 00:00:00 2001 From: Paul Buetow Date: Sat, 21 Jun 2025 20:23:09 +0300 Subject: Fix null in VSProtocolEvent shortname during deserialization MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Handle case where createShortname is called before fields are initialized - Only print eventShortname in debug output when it's not null - Return saved shortname or default when protocol fields aren't ready This eliminates the 'eventShortname: null Server deactivated' messages in test output. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude --- src/main/java/events/VSAbstractEvent.java | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'src/main/java/events/VSAbstractEvent.java') diff --git a/src/main/java/events/VSAbstractEvent.java b/src/main/java/events/VSAbstractEvent.java index 1bbfc05..7b12a1b 100644 --- a/src/main/java/events/VSAbstractEvent.java +++ b/src/main/java/events/VSAbstractEvent.java @@ -337,7 +337,11 @@ abstract public class VSAbstractEvent extends VSSerializablePrefs { if (VSSerialize.DEBUG) { System.out.println("eventClassname: " + eventClassname); - System.out.println("eventShortname: " + eventShortname); + // Note: eventShortname might be null here for internal events + // that set their shortname in their own deserialize method + if (eventShortname != null) { + System.out.println("eventShortname: " + eventShortname); + } } serialize.setObject(id, "event", this); -- cgit v1.2.3