ukdtom / WebTools.bundle

WebTools is a collection of tools for Plex Media Server. Like the Unsupported AppStore (UAS)
https://forums.plex.tv/discussion/288191
Mozilla Public License 2.0
1.3k stars 170 forks source link

SUB: Error deleting a sub on Mac and part of Linux #280

Closed ukdtom closed 5 years ago

ukdtom commented 7 years ago

CRITICAL (logs:149) - Fatal error happened in Logs show: bad argument type for built-in operation (most recent call last): File "/Users/thepanders/Library/Application Support/Plex Media Server/Plug-ins/WebTools.bundle/Contents/Code/logs.py", line 137, in show with io.open(file, 'r', errors='ignore') as content_file: TypeError: bad argument type for built-in operation

Sadly has to wait until I can get my hands on a Mac, since it works nicely on Linux, so suspect a diff in Plex framework here on the different platforms

jorgeserna commented 7 years ago

Fixed this on Mac by adding the b type to the open action:

with io.open(fileName, 'rb') as f:

I don't know why this is needed since a XML file is a text file by definition, but apparently python in Mac sees it as binary.

ukdtom commented 7 years ago

Thanks for this input.

ukdtom commented 7 years ago

And fixed on WT V3 as well

triantos commented 6 years ago

It looks like you fixed this in August, but the version of WT 3.0.0 I have appears to still have this bug. I get the same error, and the error goes away if I edit DelFromXML()'s Non-OSX (else) clause to with io.open(fileName, 'rb') as f:. Should this fix be in the 3.0.0 release? Or do I need to install some other branch?

ukdtom commented 6 years ago

Currently, due to the way that the Framework works on Mac, I had to disable it on that platform, since I also had issues with editing the XML files :(

Ref: https://github.com/ukdtom/WebTools.bundle/issues/420

triantos commented 6 years ago

Hi Tommy, this was in the else (non-Mac) clause. I run PMS and WebTools on FreeBSD.

Thanks, -Nick

On Jan 26, 2018, at 11:47 AM, Tommy Mikkelsen notifications@github.com wrote:

Currently, due to the way that the Framework works on Mac, I had to disable it on that platform, since I also had issues with editing the XML files :(

Ref: #420

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

ukdtom commented 6 years ago

Ups....missed that part ;)

And you are correct here....

I "fixed" two out of tree :(

https://github.com/ukdtom/WebTools.bundle/blob/master/Contents/Code/pmsV3.py#L1190

ukdtom commented 5 years ago

Sadly, dev work has been put on hold, due to plugins been discontinued by Plex

Ref: https://www.plex.tv/blog/subtitles-and-sunsets-big-improvements-little-housekeeping/

No new versions will be release, and this project has been put on hold

UAS will still be maintained for as long as it lasts