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

uBO 1.11.5rc0 Webextensions under Firefox : No settings save dialog is displayed. #2477

Closed
streetwolf opened this issue Mar 24, 2017 · 15 comments

Comments

@streetwolf
Copy link

Using uBO 1.11.5rc0 Webextensions version under Fx55 Nightly 64 bit.

After clicking on the 'Backup to file' button to save my settings I do not get a dialog asking me where to save the file. The backup temp file that Fx creates is in my Windows Temp folder. This is an interim file Fx creates named something like xxxxxxxxx.fileextension.part. If the dialog was presented to me and I saved it it would have been renamed to uBO's proper backup settings naming convention.

@gorhill
Copy link
Owner

gorhill commented Mar 24, 2017

I do not get a dialog asking me where to save the file

There has never been a dialog, the backup file is saved into whatever folder your browser is set to save downloaded files.

@gorhill
Copy link
Owner

gorhill commented Mar 24, 2017

Although the dialog not showing is not the issue, after trying to reproduce I confirm there is indeed an issue with the webext version of uBO, which is that the backup file is not being saved into the ~/Downloads/ directory on my side, which is where it should have been saved. Need to investigate.

@streetwolf
Copy link
Author

People were telling me there was a dialog, The file is being saved in my temp folder as I mentioned. Fx creates this temporary file in case the download is interrupted like exiting Fx. When Fx is run again the download continues. When the download is done the file should go to it's final destination with the proper name. In my case it stops at the Temp folder. uBO does display the file was saved but not to my download folder. Perhaps Webextensions is causing this to happen.

@gorhill
Copy link
Owner

gorhill commented Mar 24, 2017

I get this error in the browser console when trying to back up settings:

> TypeError: this.mDialog is null[Learn More]  nsHelperAppDlg.js:187:5

@streetwolf
Copy link
Author

Same message on my system.

@gorhill
Copy link
Owner

gorhill commented Mar 24, 2017

"Export to file" for all other cases (My filters, My rules, Whitelist) works as expected.

@streetwolf
Copy link
Author

streetwolf commented Mar 24, 2017

The Export button does produce a dialog in all cases unlike the backup settings button and does save the file in my download folder.

@gorhill
Copy link
Owner

gorhill commented Mar 24, 2017

The Export button does produce a dialog in all cases

I did not know that, it's not that way on my side. What is "WE"?

@streetwolf
Copy link
Author

streetwolf commented Mar 24, 2017

Webextensions.

@gorhill
Copy link
Owner

gorhill commented Mar 24, 2017

Did you get the dialogs with the legacy version of uBO?

@streetwolf
Copy link
Author

Let me setup a new test version of Fx55. I'll try both versions. brb.

@streetwolf
Copy link
Author

Created a clean test version of Fx55. Installed uBlock Origin v1.11.4. Backup Settings button gave me a dialog. Save worked as expected. The Export buttons also gave me dialogs.

I then removed 1.11.4 and installed uBO 1.11.5rc0 Webextensions. Backup Settings gave me no dialog and the backup didn't make it to my download folder. Exports worked as expected with dialogs.

@gorhill
Copy link
Owner

gorhill commented Mar 24, 2017

Ok, thanks for investigating. I don't know why I don't get the dialog, personally I prefer this way anwyays.

So I will just re-work a bit the code to ensure that vAPI.download is called from within the dashboard when backing up setting, that is the key difference with other backup buttons.

@streetwolf
Copy link
Author

streetwolf commented Mar 24, 2017

Always glad to help. Let me know if you want me test out the new version when it's ready.

@gwarser
Copy link
Contributor

gwarser commented Apr 4, 2017

I cannot create backup on Firefox 54, UBO 1.11.5rc7
"Last backup" date is updated, but no dialog, no file in default download directory, nothing in dev console.
Browser console shows:
21:56:02.152 TypeError: this.mDialog is null[Learn More] nsHelperAppDlg.js:187:5

@gorhill gorhill closed this as completed in 510eba6 Apr 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants