(PW-3151) Changes in Status trail API

[fernando-prowide]

No description provided.

[zubri]

Code review

Found 1 issue:

1. SwiftMessageNote.setCreationDate() accepts null without guard, inconsistent with SwiftMessageStatusInfo

The PR adds null guards to SwiftMessageStatusInfo.setCreationDate() (line 197-200) to prevent null creationDate, since @OrderBy("creationDate ASC, id ASC") relies on it being non-null. However, SwiftMessageNote also uses @OrderBy("creationDate ASC, id ASC") and its setCreationDate() still accepts null without guard — calling setCreationDate(null) would override the field-level default and break ordering.

prowide-core/src/main/java/com/prowidesoftware/swift/model/SwiftMessageNote.java

Lines 77 to 80 in dedbc37

	*/
	public void setCreationDate(Calendar creationDate) {
	this.creationDate = creationDate;
	}

For reference, the guarded version in SwiftMessageStatusInfo:

prowide-core/src/main/java/com/prowidesoftware/swift/model/SwiftMessageStatusInfo.java

Lines 196 to 202 in dedbc37

	public SwiftMessageStatusInfo setCreationDate(Calendar creationDate) {
	if (creationDate != null) {
	this.creationDate = creationDate;
	}
	return this;
	}

🤖 Generated with Claude Code

_{- If this code review was useful, please react with 👍. Otherwise, react with 👎.}