feat: update payment_id display

[CjS77]

Description

Update PaymentId Display implementation

Motivation and Context

The main current use case for PaymentId::to_string(), besides logging, is passing the payment ID to the notification script.

The current implentation makes it awkward to deal with:

• spaces in the formatting (bad for command line arguments)
• The payment id type is not self describing.
• emojis are awkward to deal with on some terminals and OSs
• A slash may have to be escaped on some OSs (for empty => N/A)

This update addresses these issues, while still maintaining clear readibility in logs:

• Change N/A to None and eliminate any edge cases that may creep in when dealing with a slash
• Other formats are <type>(<data>,...) e.g. u64(1235678) or address_and_data(f3S7XTiyKQauZpDUjdR8NbcQ33MYJigiWiS44ccZCxwAAjk,48656c6c6f20576f726c64)
• No spaces

How Has This Been Tested?

A single test for displaying each PaymentId type is also included.

Breaking Changes

• [x] None
• [ ] Requires data directory on base node to be deleted
• [ ] Requires hard fork
• [ ] Other - Please specify

[github-actions[bot]]

Test Results (CI)

    3 files    129 suites   36m 17s :stopwatch: 1 325 tests 1 325 :white_check_mark: 0 :zzz: 0 :x: 3 973 runs  3 973 :white_check_mark: 0 :zzz: 0 :x:

Results for commit f5a5de6e.

:recycle: This comment has been updated with latest results.

[github-actions[bot]]

Test Results (Integration tests)

 2 files  11 suites   15m 40s :stopwatch: 36 tests 35 :white_check_mark: 0 :zzz: 1 :x: 37 runs  36 :white_check_mark: 0 :zzz: 1 :x:

For more details on these failures, see this check.

Results for commit f5a5de6e.

:recycle: This comment has been updated with latest results.