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

<hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/2413#discussion_r256985761">core/src/main/java/bisq/core/payment/payload/PaymentMethod.java</a>:</p>
<pre style='color:#555'>>              return Coin.parseCoin("4");
+
+        // We use the class field maxTradeLimit only for mapping the risk factor.
+        long riskFactor;
+        if (maxTradeLimit == DEFAULT_TRADE_LIMIT_VERY_LOW_RISK.value)
</pre>
<p>Ups, the PaymentMethod constructor was called from the Offer, that caused the issue. I will remove that. the fields are all final now so it has to be guaranteed anyway that the data from the offer payload matches the default values in the code. The difference is that we supported with that that an offer could have different limits and durations as the actual code base (in case a new version has changed those). I think that is not much an issue as we must not change those default values anyway and if so they only may increase otherwise we could render an offer invalid (if amount becomes lower as limit). The maker gets a "promis" when he creates the offer to those data (most relevant is trade duration here) so we have to consider that if we would decide to change default values. So far there have been no need to change those but for altcoins it might become interesting to support very short durations (live trades) in some future releases. But lets discuss once we get there, the duration should be probably refactored then so that there is only the default value in and the real value will be delivered by some service, or we add a new payment methods reflecting that new properties (easier probably).</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/2413#discussion_r256985761">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AkpZtqj_juHSjWhz00EBJYKHWktLSRRqks5vNb2EgaJpZM4a3ntC">mute the thread</a>.<img src="https://github.com/notifications/beacon/AkpZtjIa5W-MFGaPL9WeZBjrJioE7ebfks5vNb2EgaJpZM4a3ntC.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/bisq-network/bisq","title":"bisq-network/bisq","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/bisq-network/bisq"}},"updates":{"snippets":[{"icon":"PERSON","message":"@ManfredKarrer commented on #2413"}],"action":{"name":"View Pull Request","url":"https://github.com/bisq-network/bisq/pull/2413#discussion_r256985761"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/bisq-network/bisq/pull/2413#discussion_r256985761",
"url": "https://github.com/bisq-network/bisq/pull/2413#discussion_r256985761",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>