Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Icon not permanent #449

Open
RealRaven2000 opened this issue Jan 8, 2024 · 14 comments
Open

Remove Icon not permanent #449

RealRaven2000 opened this issue Jan 8, 2024 · 14 comments
Assignees
Labels

Comments

@RealRaven2000
Copy link
Owner

After removing a custom icon from folder tree using the context menu - the icon returns again after restart

image

I couldn't reproduce this issue on my test accounts so far.

@RealRaven2000 RealRaven2000 self-assigned this Jan 8, 2024
@blechma
Copy link

blechma commented Jan 8, 2024

Thank you very much!

@RealRaven2000
Copy link
Owner Author

Thank you very much!

Thanks for sending the test data (off github) - I will investigate over the week.

@blechma
Copy link

blechma commented Jan 9, 2024

😊

@RealRaven2000
Copy link
Owner Author

Here is the portable installation (account name redacted)

image

Is the test folder the one with the custom icon? I guess I cannot see it because it references an icon on your hard drive....

@RealRaven2000
Copy link
Owner Author

I tried to restore the Test folder but the (invisible) custom icon returns after restart, hopefully I can debug the problem.

@RealRaven2000
Copy link
Owner Author

While debugging, this is the code during removal of the icon:

image

values before:

folder.getStringProperty("folderIcon")
"folderIcon_server1_******_******_test_imap_web_de_Test"
folder.getStringProperty("iconURL")
"url(file:///C:/Users/Public/Pictures/Icons/Action.ico)" 

values after execution:

folder.getStringProperty("folderIcon")
"noIcon"
folder.getStringProperty("iconURL")
"" 

so it looks like these properties are set in the session, as expected but for some reason they are not stored to the actual folder data by Thunderbird. I need to investigate whether there is a way to force submitting them or whether it needs an actual change in the contents (e.g. adding or removing an email) to trigger this behavior. As far as I know is no explicite, scriptable "store()" or "persist()" function to make folder state permanent. It could also be a special behavior of Thunderbird portable?

@RealRaven2000
Copy link
Owner Author

I made some more tests and found out it behaved the same with my own icons, but only on the test folder. I was able to remove the icon from the Inbox folder permanently. So there must be a fundamental difference between both (probably above and beyond that folder flags). I will try with some subfolder under test next. I will also try what happens if I move Test to be a child folder of inbox.

@RealRaven2000
Copy link
Owner Author

I couldn't create subfolders within inbox (maybe this is because I have no access to the original IMAP server) but I was able to generate a new Test 2 folder on the same level as Test (root). With that I was able to add and remove icons permanently.

@RealRaven2000
Copy link
Owner Author

Unfortunately I cannot move folders with in the tree (may also be a problem of Imap access). Still it is weird to me that I am able to modify my own folder and the inbox and persist this but not with the original Test folder.... does it exist on the original server?

@RealRaven2000
Copy link
Owner Author

CAn you test again what happens if you create a new folder alongside one of the "misbehaving" folders, does it also allow removal of the icon or does it behave the same way as the "Test" folder you sent in the portable profile?

@blechma
Copy link

blechma commented Jan 9, 2024

Unfortunately I cannot move folders with in the tree (may also be a problem of Imap access). Still it is weird to me that I am able to modify my own folder and the inbox and persist this but not with the original Test folder.... does it exist on the original server?

Yes, the "Test" and the "Test 2" folder exist on the original server. I will send you the datas to access to the original folder per mail.

@RealRaven2000
Copy link
Owner Author

Yes, the "Test" and the "Test 2" folder exist on the original server. I will send you the datas to access to the original folder per mail.

no thanks - no need to do that right now, I think I can manage with the profile as is. I would rather not have access to your data - I can find out with the colleagues from Thunderbird on the development chat on Thursday

@blechma
Copy link

blechma commented Jan 9, 2024

CAn you test again what happens if you create a new folder alongside one of the "misbehaving" folders, does it also allow removal of the icon or does it behave the same way as the "Test" folder you sent in the portable profile?

The new folder "Test3" works normal. It is not a "misbehaving" folder

@blechma
Copy link

blechma commented Jan 9, 2024

Yes, the "Test" and the "Test 2" folder exist on the original server. I will send you the datas to access to the original folder per mail.

no thanks - no need to do that right now, I think I can manage with the profile as is. I would rather not have access to your data - I can find out with the colleagues from Thunderbird on the development chat on Thursday

I have send the datas before I have read your statement. But it is no problem. It is a "Test"-account

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants