Duplicate Entries in Recent Wallet Menu

[jackgbct]

I have been switching between two wallets and in the version (3.0.5) my wallet had duplicate entries in the Recent Wallet Menu such that it edited the config file in the electrum data folder to be:
"recently_open": [
"C:\users\\downloads\electrum_data\wallets\default_wallet",
"C:\Users\\Downloads\electrum_data\wallets\Default_Wallet",
"C:\Users\\Downloads\electrum_data\wallets\Bech32",
"C:\Users\\Downloads\electrum_data\wallets\default_wallet"
],

Instead of, as it should be
"recently_open": [
"C:\Users\\Downloads\electrum_data\wallets\Default_Wallet",
"C:\Users\\Downloads\electrum_data\wallets\Bech32",
},,

I looked through the recent update log and it says nothing about fixing this so I'm not sure if it was fixed or not with the version 3.0.6, though this error took a few weeks to arise so I imagine updating will temporarily fix the error but it will inevitably come back if it wasn't fixed between the two versions.

[SomberNight]

The underlying cause is that we assume paths to be case sensitive.

In Windows, paths are case insensitive.
In general this depends on the file system or maybe the OS, so I am not sure if we should "fix" this.

[jackgbct]

I'm using NTFS filesystems (it's the default). The non case sensitive pathways come from Windows (it's Windows 7 as I hate all the later ones).

It'd be great if it could be fixed though (other than me having to manually edit my config log each time).

Thanks though!

[AbdussamadA]

Slightly related. On linux I have this:

"/home/abdussamad/temp/electrum/testnet/../testnet1",

    "/home/abdussamad/temp/electrum/testnet1"
],

I suggest running it through os.path.abspath first.

[SomberNight]

fixed in eccb8ec

I suggest running it through os.path.abspath first.

Thanks, indeed, good idea. :)

We are not going to clean existing configs of duplicates.