Improve MessageFilter logging and refactor search element styles

[tonygermano]

commit d19499d

Improve MessageFilter logging and refactor search element styles

Refactors the ToStringStyle implementations for MessageFilter and related
search elements to significantly improve debuggability and log readability.

Key Changes:
- Refactored MetaDataSearchElementToStringStyle into a generic
  SearchElementToStringStyle to eliminate code duplication.
- Added ContentSearchElementToStringStyle:
  - Converts raw content codes into human-readable ContentType names
    (e.g., "1(Raw)") instead of opaque integers.
- Updated MessageFilterToStringStyle:
  - Replaced legacy Calendar formatting with java.time (ZonedDateTime)
    and included time and timezone id.
  - Added recursive indentation logic to render nested collections
    in a hierarchical format.
- Applied new styles to ContentSearchElement and MetaDataSearchElement
  toString() methods.
- Previously ContentSearchElement was inheriting the Object.toString
  method.

This change primarily adds detail for ContentSearchElements when viewing
the MessageFilter in the server event log. The addition of the time and
timezone id fill in some missing details for the start and end times.
The additonal reformats improve readability.

commit 9ad3055

Add JUnit 4 tests for search element model and style classes

Added 100% test coverage for

* `ContentSearchElement`
* `MetaDataSearchElement`
* `MessageFilterToStringStyle`
* `SearchElementToStringStyle`
* `ContentSearchElementToStringStyle`

[gibson9583]

can you add a screenshot of the new styling?

[tonygermano]

For the following search options:

before:

MessageFilter[
attachment=false,
contentSearch=[com.mirth.connect.model.filters.elements.ContentSearchElement@3b0e8c4, com.mirth.connect.model.filters.elements.ContentSearchElement@46b276d6],
endDate=2026-02-02,
error=false,
excludedMetaDataIds=[1],
importIdLower=6,
importIdUpper=7,
includedMetaDataIds=<null>,
maxMessageId=6,
metaDataSearch=[
MetaDataSearchElement[columnName=SOURCE,ignoreCase=true,operator=CONTAINS,value=XYZ],
MetaDataSearchElement[columnName=TYPE,ignoreCase=false,operator=EQUAL,value=FOO]
],
minMessageId=5,
originalIdLower=1,
originalIdUpper=2,
sendAttemptsLower=1,
sendAttemptsUpper=5,
serverId=792e3c00-6906-4444-9642-02603403335e,
startDate=2026-02-01,
statuses=[RECEIVED, SENT, TRANSFORMED, QUEUED],
textSearch=M0000123,
textSearchMetaDataColumns=[SOURCE, TYPE],
textSearchRegex=<null>
]

after:

MessageFilter[
  attachment=false,
  contentSearch=[
    ContentSearchElement[contentCode=1(Raw), searches=["raw 1", "raw 2"]],
    ContentSearchElement[contentCode=5(Sent), searches=["sent 1"]]
  ],
  endDate=2026-02-02 17:16-05:00[America/New_York],
  error=false,
  excludedMetaDataIds=[1],
  importIdLower=6,
  importIdUpper=7,
  includedMetaDataIds=<null>,
  maxMessageId=6,
  metaDataSearch=[
    MetaDataSearchElement[columnName="SOURCE",ignoreCase=true,operator="CONTAINS",value="XYZ"],
    MetaDataSearchElement[columnName="TYPE",ignoreCase=false,operator="EQUAL",value="FOO"]
  ],
  minMessageId=5,
  originalIdLower=1,
  originalIdUpper=2,
  sendAttemptsLower=1,
  sendAttemptsUpper=5,
  serverId=792e3c00-6906-4444-9642-02603403335e,
  startDate=2026-02-01 08:01-05:00[America/New_York],
  statuses=[TRANSFORMED, QUEUED, SENT, RECEIVED],
  textSearch=M0000123,
  textSearchMetaDataColumns=[SOURCE, TYPE],
  textSearchRegex=<null>
]

[github-actions]

Test Results

  110 files  + 5    212 suites  +10   7m 19s ⏱️ + 1m 27s
  650 tests +17    650 ✅ +17  0 💤 ±0  0 ❌ ±0 
1 300 runs  +34  1 300 ✅ +34  0 💤 ±0  0 ❌ ±0 

Results for commit 2e9d422. ± Comparison against base commit 9349375.

♻️ This comment has been updated with latest results.

[gibson9583]

looks good again. appreciate the well documented PR

[jonbartels]

This is a good improvement. It should be specifically called out in the release notes since someone could be doing something obscure parsing this data via an API call. I would not consider that likely.

Presenting this as a UI/UX improvement with a note that API usages may be affected would be ideal.

[tonygermano]

Please see the latest commit's message for details. With the changes, the output now looks like this.

MessageFilter[
  attachment=false,
  contentSearch=[
    ContentSearchElement[contentCode=1(Raw), searches=[raw 1, raw 2]],
    ContentSearchElement[contentCode=5(Sent), searches=[sent 1]]
  ],
  endDate=2026-02-02 17:16-05:00[America/New_York],
  error=false,
  excludedMetaDataIds=[1],
  importIdLower=6,
  importIdUpper=7,
  includedMetaDataIds=<null>,
  maxMessageId=6,
  metaDataSearch=[
    MetaDataSearchElement[columnName=SOURCE, ignoreCase=true, operator=CONTAINS, value=XYZ],
    MetaDataSearchElement[columnName=TYPE, ignoreCase=false, operator=EQUAL, value=FOO]
  ],
  minMessageId=5,
  originalIdLower=1,
  originalIdUpper=2,
  sendAttemptsLower=1,
  sendAttemptsUpper=5,
  serverId=792e3c00-6906-4444-9642-02603403335e,
  startDate=2026-02-01 08:01-05:00[America/New_York],
  statuses=[SENT, QUEUED, TRANSFORMED, RECEIVED],
  textSearch=M0000123,
  textSearchMetaDataColumns=[SOURCE, TYPE],
  textSearchRegex=<null>
]