Approved Question about deleting movie files

[Sek0n 7]

I was just curious if it was intentional to not allow movies to be deleted the same way shows can. If so, what is the reasoning? I added  " or "movie" " to the monitor file, and have had no issues so far. Is there a risk with doing this that I've not thought of?

 

 

Thanks for the great work, btw. The latest stuff has been fantastic.

Edited May 22, 2015 by Sek0n

[xnappo 1608]

The reasoning was that the episode delete was added for people who use Kodi like a DVR, but that most people want a little more protection for their movie collection.

 

There is no risk to your mod.. We can make it an option if more people want it. 

Edited May 22, 2015 by xnappo

[Sek0n 7]

Ah, ok. Makes sense.

 

Would be nice as an option, but I'm content with just doing it myself. Thanks.

[xnappo 1608]

It is a reasonable and very easy request.  I'll get it in there at some point.

[marcelveldt 736]

I was just curious if it was intentional to not allow movies to be deleted the same way shows can. If so, what is the reasoning? I added  " or "movie" " to the monitor file, and have had no issues so far. Is there a risk with doing this that I've not thought of?

 

 

Thanks for the great work, btw. The latest stuff has been fantastic.

Feel free to commit your change as a PR to us on Github, that way other users can benefit from this too :-)

[xnappo 1608]

Well - his is a hard-coded offer delete for movies - definitely don't want that. 

[Sek0n 7]

Yea, I'll leave that be and let you add the option when it's ready.

 

In-case anyone does actually wan't to do it at their own risk;

 

I changed line 76 in KodiMonitor.py from

if jsondata.get("type") == "episode":

to

if jsondata.get("type") == "episode" or "movie":

(Let me know if you don't want this here, or just edit it out)

Edited May 22, 2015 by Sek0n

[xnappo 1608]

Naw, totally fine to post that