Sample deletion as before

Want something added? Ask for it here.
Post Reply
ErikBrown
Release Testers
Release Testers
Posts: 135
Joined: December 20th, 2009, 1:25 am
Location: Home

Sample deletion as before

Post by ErikBrown »

Hi,

The deletion of samples has been changed in version 2.3.9 as per the change log. But now I get lots of sample files. Would it be possible to make it optional to delete the files again as before so that the user can decide to automatically remove all the files with sample in the name or not?

Thanks,
Erik

pxr5
Newbie
Newbie
Posts: 22
Joined: July 22nd, 2008, 5:10 am

Re: Sample deletion as before

Post by pxr5 »

I've started to get samples coming back too. The above would be useful.

User avatar
safihre
Administrator
Administrator
Posts: 3645
Joined: April 30th, 2015, 7:35 am
Location: Switzerland
Contact:

Re: Sample deletion as before

Post by safihre »

Could you send me example NZBs where you are getting too many samples?
It should only not delete them if all files contain the word "sample" or "proof" or in case there's only 1 file.

ErikBrown
Release Testers
Release Testers
Posts: 135
Joined: December 20th, 2009, 1:25 am
Location: Home

Re: Sample deletion as before

Post by ErikBrown »

Hi Safihre,

I have emailed you at your personal email address at sabnzbd dot org some NZBs that cause sample files not to be deleted.

Regards,
Erik

pxr5
Newbie
Newbie
Posts: 22
Joined: July 22nd, 2008, 5:10 am

Re: Sample deletion as before

Post by pxr5 »

Hi safihre,

I've emailed you one as well.

User avatar
safihre
Administrator
Administrator
Posts: 3645
Joined: April 30th, 2015, 7:35 am
Location: Switzerland
Contact:

Re: Sample deletion as before

Post by safihre »

Found the issue with your NZB's.
Bad testing on my part, didn't test the case with just 1 sample file.

ErikBrown
Release Testers
Release Testers
Posts: 135
Joined: December 20th, 2009, 1:25 am
Location: Home

Re: Sample deletion as before

Post by ErikBrown »

Thanks for fixing it!

pxr5
Newbie
Newbie
Posts: 22
Joined: July 22nd, 2008, 5:10 am

Re: Sample deletion as before

Post by pxr5 »

Thanks from me too :-)

osen
Newbie
Newbie
Posts: 4
Joined: November 4th, 2010, 6:01 pm

Re: Sample deletion as before

Post by osen »

Sorry for bumping this thread, but what was the fix for this? I'm on latest release (2.3.9) and I'm experiencing this problem. Do I have to build from source to get the fix?

User avatar
jcfp
Release Testers
Release Testers
Posts: 929
Joined: February 7th, 2008, 12:45 pm

Re: Sample deletion as before

Post by jcfp »

osen wrote:
August 17th, 2019, 1:02 pm
Sorry for bumping this thread, but what was the fix for this? I'm on latest release (2.3.9) and I'm experiencing this problem. Do I have to build from source to get the fix?
https://github.com/sabnzbd/sabnzbd/comm ... 8083b36548

There's no release yet that includes that commit though.

osen
Newbie
Newbie
Posts: 4
Joined: November 4th, 2010, 6:01 pm

Re: Sample deletion as before

Post by osen »

jcfp wrote:
August 18th, 2019, 2:34 am
osen wrote:
August 17th, 2019, 1:02 pm
Sorry for bumping this thread, but what was the fix for this? I'm on latest release (2.3.9) and I'm experiencing this problem. Do I have to build from source to get the fix?
https://github.com/sabnzbd/sabnzbd/comm ... 8083b36548

There's no release yet that includes that commit though.
I see. Successfully installed from source and sample deletion does indeed seem to be working again. I read the first post too fast, didn't notice this "bug" was introduced in 2.3.9 so I guess simply downgrading to 2.3.8 for the time being would be a lot more convenient.

Thank you very much.

Xenius
Newbie
Newbie
Posts: 1
Joined: January 11th, 2020, 10:07 am

Re: Sample deletion as before

Post by Xenius »

hey can someone tell me how i can install this fix?

User avatar
safihre
Administrator
Administrator
Posts: 3645
Joined: April 30th, 2015, 7:35 am
Location: Switzerland
Contact:

Re: Sample deletion as before

Post by safihre »

It will be in the next release of SABnzbd!

Post Reply