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

[Bug]: Keep not played for X days setting not respsected #54

Open
xiNeRoar opened this issue Aug 15, 2024 · 10 comments
Open

[Bug]: Keep not played for X days setting not respsected #54

xiNeRoar opened this issue Aug 15, 2024 · 10 comments
Labels
bug Something isn't working

Comments

@xiNeRoar
Copy link

xiNeRoar commented Aug 15, 2024

What happened?

I have tried to configure keep not played for X days from 1 to 30, the plugin will keep everything since someone has watched it no matter how long it was

Troubleshooting info

This is the only occurrence of the show watch which is 2 months ago;
image
This is my setting;
image

image
image
image
image
This is the original log;
log.txt

@xiNeRoar xiNeRoar added the bug Something isn't working label Aug 15, 2024
@shemanaev
Copy link
Owner

Hi, will setting like "don't count item as played after X days of last play" solve this problem?

For example: the value is set to 10, the last time the TV show episode was watched on January 10th, so after January 20th this play will not be counted.

@xiNeRoar
Copy link
Author

xiNeRoar commented Aug 25, 2024

Hi, will setting like "don't count item as played after X days of last play" solve this problem?

For example: the value is set to 10, the last time the TV show episode was watched on January 10th, so after January 20th this play will not be counted.

yes I think so, but I think how this setting interact with "keep not played episode for x days" need to be treated or documented carefully?
Also, as the original purpose of this plugin is to "delete played media files after specified amount of time", I guess re-group the options into two main category will help user better understand how they should setup the plugin?

e.g.;

  1. Delete media after played
    Time - [ ] Days
    Trigger Scope - [ ] Single Episode / Whole Season / Whole Series
    Delete Scope - [ ] Single Episode / Whole Season / Whole Series
    Monitor - [ ] All User / Any User / Any User(Latest Playback)

  2. Delete media after not being played
    Time - [ ] Days
    Trigger Scope - [ ] Single Episode / Whole Season / Whole Series
    Delete Scope - [ ] Single Episode / Whole Season / Whole Series
    Monitor - [ ] All User / Any User / Any User(Latest Playback)

@shemanaev
Copy link
Owner

yes I think so, but I think how this setting interact with "keep not played episode for x days" need to be treated or documented carefully?

Could you try this build?

Also, as the original purpose of this plugin is to "delete played media files after specified amount of time", I guess re-group the options into two main category will help user better understand how they should setup the plugin?

e.g.;

1. Delete media after played
   Time - [ ] Days
   Trigger Scope - [ ] Single Episode / Whole Season / Whole Series
   Delete Scope - [ ] Single Episode / Whole Season / Whole Series
   Monitor - [ ] All User / Any User / Any User(Latest Playback)

2. Delete media after not being played
   Time - [ ] Days
   Trigger Scope - [ ] Single Episode / Whole Season / Whole Series
   Delete Scope - [ ] Single Episode / Whole Season / Whole Series
   Monitor - [ ] All User / Any User / Any User(Latest Playback)

That's a good suggestion, I will look into it. Current UX isn't great for sure, it started as simple plugin for my needs and grows with features.

@xiNeRoar
Copy link
Author

xiNeRoar commented Aug 25, 2024

yes I think so, but I think how this setting interact with "keep not played episode for x days" need to be treated or documented carefully?

Could you try this build?

Also, as the original purpose of this plugin is to "delete played media files after specified amount of time", I guess re-group the options into two main category will help user better understand how they should setup the plugin?
e.g.;

1. Delete media after played
   Time - [ ] Days
   Trigger Scope - [ ] Single Episode / Whole Season / Whole Series
   Delete Scope - [ ] Single Episode / Whole Season / Whole Series
   Monitor - [ ] All User / Any User / Any User(Latest Playback)

2. Delete media after not being played
   Time - [ ] Days
   Trigger Scope - [ ] Single Episode / Whole Season / Whole Series
   Delete Scope - [ ] Single Episode / Whole Season / Whole Series
   Monitor - [ ] All User / Any User / Any User(Latest Playback)

That's a good suggestion, I will look into it. Current UX isn't great for sure, it started as simple plugin for my needs and grows with features.

I am more than willing to try it out but I am not so familiar with the file, how can I use it?

@shemanaev
Copy link
Owner

I am more than willing to try it out but I am not so familiar with the file, how can I use it?

Just unzip it, replace original dll (<Jellyfin config folder>/plugins/Media Cleaner_2.17.0.0) and restart server.

@xiNeRoar
Copy link
Author

xiNeRoar commented Aug 25, 2024

I tried and here is the log;
log.txt
the item being tested is "鬼灭之刃" season 1 and 4, they only played once on 07/23/2023 but still endup the cleanup task doesn't list out this season to delete
image

@shemanaev
Copy link
Owner

I tried and here is the log; log.txt the item being tested is "鬼灭之刃" season 1 and 4, they only played once on 07/23/2023 but still endup the cleanup task doesn't list out this season to delete image

You need to configure the new option first
изображение

@xiNeRoar
Copy link
Author

thanks it seems working! :) but i still need to observe it for couple days to ensure it works as intended
image
image
I was trying to configure it to only acknowledge current day's watch state. If someone watch it today, extend it for another 30 days. If no one watch it, it will count down 30 days until being deleted. I will try to shorten the period from 30 days to 2 or 3 days and observe how it operate and report here :D

@shemanaev
Copy link
Owner

Value "30" should do it. Like "if no one watched in last 30 days".

observe how it operate and report here :D

I will keep an eye on it 😉

@NicolaSmaniotto
Copy link
Contributor

Hi, just want to say that I'm testing this new build and everything seems to work as expected.

Also, it fixes the thing I mentioned in #39 (comment), because if someone drops a series its partial plays are now ignored after X days, and it is then removed because nobody watched it.

With these changes the plugin does pretty much everything I wanted it to, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants