[bisq-network/bisq] Add missing @Named annotations for CoinFormatter injection (#3658)

Justin Carter notifications at github.com
Fri Nov 22 09:04:42 UTC 2019


bodymindarts commented on this pull request.



> @@ -139,7 +142,7 @@
     public DisputeView(DisputeManager<? extends DisputeList<? extends DisputeList>> disputeManager,
                        KeyRing keyRing,
                        TradeManager tradeManager,
-                       CoinFormatter formatter,
+                       @Named(FormattingUtils.BTC_FORMATTER_KEY) CoinFormatter formatter,

That is what I thought too, which is why I left these out originally. But it turns out `@inject` is not infact mandatory and the program crashes on these views without this annotation. Thanks to @stejbac for discovering this.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/3658#discussion_r349492042
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20191122/d5fead11/attachment-0001.html>


More information about the bisq-github mailing list