javidpack / bannerbonanza Goto Github PK

I've been having issues where:

1. I couldn't add more banners to a rack after the initial addition
2. Adding to a banner rack doesn't add all the banners in my inventory. I've already built more than 10 and we struggle to figure out how to add all the banners in a single rack.

Is there a limit to the number of banners? If none, how?

Additionally, does the banner rack accept multiple banners of the same type?

Silently Caught Exception: Attempted to divide by zero. at BannerBonanza.Tiles.BannerRackTile.SpecialDraw(Int32 i, Int32 j, SpriteBatch spriteBatch) in D:\Documents\My Games\Terraria/ModLoader\Mod Sources\BannerBonanza\Tiles\BannerRackTile.cs:line 124

There isn't much more that I can explain about this.

When clicking on the banner bonanza for what to kill for the next banner it won't tell the person clicking but instead everyone else on the server.

While in multiplayer, having your map open, and hovering your mouse over a banner rack, a crash occurs 100% of the time. However, if the chunk that contains the banner rack is already loaded for the player, it will not occur.

client.log

A request has been made to alphabetize banners.

This is a nice mod. It can be very useful for a lot of people. Though it's not my cup of tea since I like placing my banners all around the base.

I want to report some bugs I've found:

1. The Banner Rack accepts favourited banners. I think it shouldn't be the case.

2. The number of banners from the base game that the Banner Rack shows is wrong. It says that there are only 249 banners, yet there are 283 of them.

3. The Banner Rack accepts a White Cultist Archer Banner, making the full completion impossible since this banner is unobtainable.

It'd also be nice if it was possible to attach a Banner Rack to platforms, but I guess it will be hard to implement.

I hope I was able to help. Good luck!

-- From https://forums.terraria.org/index.php?threads/banner-bonanza-collect-and-consolidate-banners.60797/page-5#post-2995315

Need to support this new feature, as banners do this now and this should follow.

Multiple reports.

client (74).log
client (73).log