Sample deletion as before

Want something added? Ask for it here.
Post Reply
ErikBrown
Release Testers
Release Testers
Posts: 118
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: 26
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: 5338
Joined: April 30th, 2015, 7:35 am
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.
If you like our support, check our special newsserver deal or donate at: https://sabnzbd.org/donate
ErikBrown
Release Testers
Release Testers
Posts: 118
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: 26
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: 5338
Joined: April 30th, 2015, 7:35 am
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.
If you like our support, check our special newsserver deal or donate at: https://sabnzbd.org/donate
ErikBrown
Release Testers
Release Testers
Posts: 118
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: 26
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: 986
Joined: February 7th, 2008, 12:45 pm

Re: Sample deletion as before

Post by jcfp »

osen wrote: August 17th, 2019, 1:02 pmSorry 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 pmSorry 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: 2
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: 5338
Joined: April 30th, 2015, 7:35 am
Contact:

Re: Sample deletion as before

Post by safihre »

It will be in the next release of SABnzbd!
If you like our support, check our special newsserver deal or donate at: https://sabnzbd.org/donate
Post Reply