<p><b>@battleofwizards</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/3134#discussion_r317373305">desktop/src/main/java/bisq/desktop/util/validation/PhoneNumberValidator.java</a>:</p>
<pre style='color:#555'>> +    @Getter
+    private final String callingCode;
+    /**
+     * The normalized (digits only) representation of an international calling code.
+     */
+    private final String normalizedCallingCode;
+    /**
+     * Phone number in E.164 format.
+     */
+    @Nullable
+    @Getter
+    private String normalizedPhoneNumber;
+
+    // Hide no-arg constructor
+    private PhoneNumberValidator() {
+        this.isoCountryCode = null;
</pre>
<p>Sorry! I didn't mean to imply removing private constructor. I do agree it is important here.</p>
<p>So please either revert back to assigning nulls <strong>or</strong> simply get rid off the problematic final. In the context of clean code (small methods and classes) Java finals are overrated. They only protect the reference anyway and not the object's content.</p>
<p>Whatever you pick it will be OK with me.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/bisq-network/bisq/pull/3134?email_source=notifications&email_token=AJFFTNWXRNBCVS6Z5R2GCHTQGGJ3TA5CNFSM4IPCU6SKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCCS63ZA#discussion_r317373305">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNQ2ALVVZPCS5S5VWTTQGGJ3TANCNFSM4IPCU6SA">mute the thread</a>.<img src="https://github.com/notifications/beacon/AJFFTNUHTBTLMYBH4T62L5DQGGJ3TA5CNFSM4IPCU6SKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCCS63ZA.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/bisq-network/bisq/pull/3134?email_source=notifications\u0026email_token=AJFFTNWXRNBCVS6Z5R2GCHTQGGJ3TA5CNFSM4IPCU6SKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCCS63ZA#discussion_r317373305",
"url": "https://github.com/bisq-network/bisq/pull/3134?email_source=notifications\u0026email_token=AJFFTNWXRNBCVS6Z5R2GCHTQGGJ3TA5CNFSM4IPCU6SKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCCS63ZA#discussion_r317373305",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>