Popcorn time wont close

[Arizona-dev]

Operating System Version: Windows 10 latest update

Popcorn Time Version: 0.4.3-0

Download date: 07/04/2020

Expected Behaviour

when clicking close button or alt-f4, it won't close, so I have to close it from taskmanager.

Actual Behaviour

Not closing

Steps to reproduce the behaviour

1. Installed new update from the app
2. launch the app
3. downloaded a movie
4. stopped the download
5. Click red cross button

[reddo]

This is also happening for me

[edik06]

yes me to

[VladimiroPaschali]

You can "fix" it by disabling the "Clear tmp folder after closing app" in the Cache directory settings, but you will have to manually delete the downloaded files every now and then

[Simsan]

same to me.

[Persei08]

Can you all confirm that you use windows too ?

Also if some of you can copy-paste console output (F12 key > "console" tab) when the issue happen, it can be useful

[JavierLopezBravo]

This is the console output

[Jany-M]

Also on 0.4.4

[Persei08]

Yep issue is confirmed. It happen because PT encounter issue when trying to delete the cache, on exit.

Like @vlod98 pointed, you can disable "Clear Tmp folder when app exit" in settings as a workaround, but you will have to delete it yourself.. if you really want

[elitonfilho]

Same problem on 0.4.7, flushing the cache in settings solves it

[JavierLopezBravo]

This is the console output

As you can see, the issue is caused by the rmdir command not being able to delete a non-empty directory. I think you should use rmdir "directory name" /s /q instead

[Mitezuss]

This problem is because the download/upload do not stop when you cancel (press X) a movie/serie/... so, on the temporary directory, the file keep "sharing" and blocked.

One solution is do not delete if the file is opened/in use, and just close (end the app) and delete after new start.

---

Temporary solution, open and edit:

src\app\app.js

var deleteFolder = function (path) { try { rimraf.sync(path); } catch (e) { } };

[zephuros-droid]

Same problem on Windows 10, version 1909.

[PollutionMAN]

same problem for me windows 10 1909 latest on popcorn time 0.4.4

[dutchy1001]

This should be fixed in build 415, get it from here

[Persei08]

dev build 415 don't fix it. Check my comment here: https://github.com/popcorn-official/popcorn-desktop/pull/1369#issuecomment-616189121

[dutchy1001]

I looked at the changes of build 415 "Update DeleteFolder / Fix bug when changing TMPfolder" I presumed it was fixed this way, sorry about the misunderstanding.

[PollutionMAN]

This should be fixed in build 415, get it from here

not fixed in this version too , problem is still there

[Persei08]

415, 416 ?

[amberphoenix]

same here, I can reproduce it everytime, also exists in version 0.4.4

[ze0ss]

same for me, windows 10, 0.4.4-beta

[Persei08]

I pin this issue in hope it will reduce the number of duplicate reports

[afonsosantos]

Hi, any ETA for this issue to be solved? Thank you!

[Arizona-dev]

Issue still not fixed.

Le jeu. 30 avr. 2020 à 01:03, Afonso Santos notifications@github.com a écrit :

Hi, any ETA for this issue to be solved? Thank you!

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/popcorn-official/popcorn-desktop/issues/1280#issuecomment-621513447, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKURL2VABUT5MJURM5ZJ64LRPCW3FANCNFSM4MDH5S4A .

[Persei08]

I just tried (windows 10 - build 423), for me it works.

Can you guys tell us which build number you use, because it should be fixed since build ~417~ 416.

[linchiyu]

I noticed that if you go to the "Seedbox" and delete all the entries there, it will exit the program normally. So maybe this might be an approach to solve this issue temporarily. On exit > delete all seedbox entries > exit

[Persei08]

@Arizona-dev @linchiyu This issue should be already fixed since dev build 416.

Have you tried the latest dev builds ? https://ci.popcorntime.sh/job/Popcorn-Time-Desktop/lastStableBuild/.

[dsnvwlmnt]

I just tried (windows 10 - build 423), for me it works.

Can you guys tell us which build number you use, because it should be fixed since build ~417~ 416.

2007344

[Jany-M]

Yep latest stable build works for me too.

[Persei08]

@Jany-M You mean dev build, right ? Because latest stable (project still in beta in reality) build is 0.4.4 (=dev build 412)

[Jany-M]

Yes, sorry didn't specify that. I've always updated by checking manually the successful dev builds, since the app actually gets updated quite rarely, while the dev releases are relatively frequent. Latest one here.

[Persei08]

Ok, thx

[linchiyu]

Hi, reopening this bug. I installed the latest version. Now what happens is: -if you are using the software and then closes it, it closes normally -but if finished watching something, then leave it open for a while, then tries to close it, it doesn't close

Then I tried to delete the entries in the seedbox, but nothing happens (it doesn't delete).

Edit: by the way, I'm a Windows user.

[Persei08]

-but if finished whatching something, then leave it open for a while, then tries to close it, it doesn't close

Then I tried to delete the entries in the seedbox, but nothing happens (it doesn't delete).

Like it totally freeze ?

[linchiyu]

No, just nothing happens when you click on Close or on delete in the seedbox. I can still navigate and the rest just still works normally, even after clicking.

[sheepdestroyer]

I'm on last windows build from https://ci.popcorntime.app/job/Popcorn-Time-Desktop/ Currently 424, Revision: 5eb53fc0

I see the same behaviour : nothing happens when you click on Close after watching something. if manually deleting temp files, then I can close

[ze0ss]

@sheepdestroyer expected behaviour... it's a dev build (and not the stable release) after all...

[sheepdestroyer]

The stable release exposes the same bug... It was suggested here that dev builds have it corrected, so I just chimed in to say that wasn't the case

[AsmodusLustro]

One problem I have is when I watch a movie and then I can't close the program from x .The program is 0.4.4 and my o.s is win10 pro 1909

[soeirotm]

I'd got same problem with 0.4.4 x32... version x64 is working properly on my win10 1909.
I'd downloaded x64 from https://popcorntime.app/windows

[dutchy1001]

@soeirotm please download the version you need from this link here, https://ci.popcorntime.sh/job/Popcorn-Time-Desktop/lastStableBuild/ . This way you will have the latest build which might fix your the issue

[soeirotm]

@soeirotm please download the version you need from this link here, https://ci.popcorntime.sh/job/Popcorn-Time-Desktop/lastStableBuild/ . This way you will have the latest build which might fix your the issue

TY!

[ArcticZeroo]

I'm still getting this issue, even after installing the latest build. Dev console is telling me that it's the same "directory is not empty" error.

[ze0ss]

I'm still getting this issue, even after installing the latest build. Dev console is telling me that it's the same "directory is not empty" error.

Try installing a CI build.

[ArcticZeroo]

@abh-oss01 I already have whatever was linked above from https://ci.popcorntime.app/job/Popcorn-Time-Desktop/lastStableBuild/ , which seems to be 12 commits behind development. That is also apparently the latest successful build

[dutchy1001]

@ArcticZeroo Have you tried a clean install? ("export database" from the settings page if you dont want to loose "seen" movies/series)

[ArcticZeroo]

@dutchy1001 I have not yet tried a clean install, will do later tonight.

Regardless, there are clear issues that would lead to this happening (even on a clean install) in app.js. If I can actually get this thing to build (node v0.10.0 is quite finnicky) I plan to submit a PR to "fix" it tonight. By "fix", I mean I will actually properly catch the errors associated with directory deletion and close the app regardless of error status. Right now, errors from deleteDirectory are uncaught. Still unsure why those errors appear to begin with

[ArcticZeroo]

Should be fixed now that #1557 has landed. I can no longer reproduce. We should figure out why the directory errors are appearing to begin with, but in the mean time this mitigation will ensure that users no longer feel the consequences of those errors.

If anyone else was having this issue, please install a CI build at https://ci.popcorntime.app/job/Popcorn-Time-Desktop/lastStableBuild/ and let me know if it's still happening for you

[ArcticZeroo]

I don't really agree with the (now deleted) request to close this issue, my fix is only a workaround. There are still large (multi-GB or more) temp files being left in users' %Temp% directories when removal of the folder fails

[Persei08]

arf sorry, closed issues in batch and forget to unselect this one.. and maybe some other :/ .

[ze0ss]

@Persei08 can the close button and the other two follow windows title bar guidelines? this was seen in some older popcorn time builds iirc.