Closed Keith94 closed 10 years ago
Interesting. Thanks for this report man.
Can you please tell me how to replicate this error.
By moved/deleted in explorer, do you mean if the profile is deleted by something other than Profilist that it should be able to detect that it's gone and make the necessary adjustment to proflies.ini
?
Can you please tell me how to replicate this error.
- Create new profile via Profilist button.
- Close the Firefox window that launched from it.
- Navigate to
%APPDATA%\Mozilla\Firefox\Profiles
and delete the profile folder that was created.- Try to delete the profile from the Panel UI, like how you explained in the add-on description.
By moved/deleted in explorer, do you mean if the profile is deleted by something other than Profilist that it should be able to detect that it's gone and make the necessary adjustment to proflies.ini?
Yes that does sound like it should be the expected behavior.
Ah ok. I think this is between bug/enhancement because I intended for all profile creating/deleting/renaming to happen from Profilist. I didn't foresee people using multiple tools (and direct OS methods) to do this. Let's aim for this feature sometime after v1.1 please. So maybe v1.2.
Sounds good.
Hey @Keith94, I was thinking about handling corrupt deletions etc, and there are just so many possibilities/ways a person can corrupt it. I'm thinking that when 3rd party tools do delete a profile, they will do so properly, as in they will also edit profiles.ini. If the 3rd party tool works right, then Profilist doesn't have the issue of left over button, in fact, if it properly deletes it, then the button automatically disappears.
This is good for error handling/corrupt stuff, but I don't think this is a bug.
If you want to delete manually on OS, you should do it correctly:
C:\Documents and Settings\noitidart\Application Data\Mozilla\Firefox\Profiles
C:\Documents and Settings\noitidart\Local Settings\Application Data\Mozilla\Firefox\Profiles
So I'm thinking this is not a bug or enhancement anymore? What do you think I need your help on this.
Well I thought if maybe the profile folder was missing and you try to delete it with Profilist, it could instead remove that entry from profiles.ini. Much like how the Delete Profile button works when launching Firefox with -p
command-line argument:
But if you don't like that idea, then I won't mind if you close this issue.
Ah you're right, if it tries to delete and it finds its already deleted, then it shouldn't say error deleting, it should realize it's already gone. You're right, I'll fix that. :) I just got intimidated by all the possible ways people can corrupt profiles and having to try to catch them all haha.
@Keith94 hey man, please install and test out pacth-5
from my yajd account.
Please test the deleting, I tested it but you're the deleting profile bug catching expert! Please share your findings. :)
Hey,
Well I ain't no expert, but I can tell you that it successfully removes them without any error now, so that's working good :)
Btw with that patch if I try to delete a profile that is in use, Profilist tells me it was deleted instead of warning me that it's still in use.
Perfect testing man thanks for the detail on the if in use, I'll look into that. You're better test than me because I missed that! :) Please let me know if anything else comes up.
Ok man updated. Please install and test, let me know if anything else comes up. repo: https://github.com/yajd/Profilist/tree/patch-5
If a root folder is missing and I attempt to load the profile, it shouldn't display a "launching profile" notification.
Man you are so good! Thanks for that find I'll try to address that. Having some lunch right now. :)
Hey man for the issue where it shouldn't say "launching profile", it actually is launching it, but in process of launch it finds its corrupt and throws error message. We can make a seperate issue for that though, can you please make that topic. I'll close this topic as soon as i merge patch-5 as we addressed the issue of deletin when folder is missing.
If a profile's folder is moved/deleted in Windows Explorer, Profilist should be able to remove its button from the Panel UI, instead of just failing like this:
Otherwise I would have to edit profiles.ini file to prevent the non-existant profiles from displaying in the Panel UI.