[chirp_devel] [PATCH] [FT8900] Deleted inherited get_bank_model function, fixes #6293

[Dan Smith]

Hi Kalle,

> # HG changeset patch
> # User Kalle Tuulos <oh1hqz at gmail.com>
> # Date 1544563646 -7200
> #      Tue Dec 11 23:27:26 2018 +0200
> # Node ID a7e1b67290c9775f5e7be286d2a05855ac4066f2
> # Parent  62b4cbefa86738c37317a133f903500f669baf23
> [FT8900] Deleted inherited get_bank_model function, fixes #6293
> 
> The FT-8900 inherits its class from FT-8800, which defines also the
> get_bank_model function. The existence of this function is checked
> in order to decide, whether Banks tab is shown or not. As banks are
> not supported in FT-8900, the banks tab must not be shown.

I had applied this, reading your convincing description and quickly looking at the code where we check for the existence of that. I should have looked closer, but the style checker caught it and made me :)

I was able to replicate the issue and verify that just overriding the method in the 8900 class to return None will yield the correct behavior. Let me know if this doesn't work for you and we'll figure out what to do about it.

Thanks!

--Dan