Skip to content

Small change to primary script#66

Open
kryptoxide wants to merge 2 commits intol3uddz:masterfrom
kryptoxide:master
Open

Small change to primary script#66
kryptoxide wants to merge 2 commits intol3uddz:masterfrom
kryptoxide:master

Conversation

@kryptoxide
Copy link

@kryptoxide kryptoxide commented Nov 21, 2022

Fixed an issue on line #224

--Replaced code--
with open(decision_filename, 'a') as fp:
fp.writelines(lines)
return

--replacement code--
with open(decision_filename, 'a', encoding="utf-8") as fp:
fp.writelines(lines)
return

Traceback (most recent call last):
File "C:\Users\Admin\Desktop\DupeFinder\venv\Scripts\plex_dupefinder.py", line 448, in
write_decision(keeping=part_info)
File "C:\Users\Admin\Desktop\DupeFinder\venv\Scripts\plex_dupefinder.py", line 225, in write_decision
fp.writelines(lines)
File "C:\Users\Admin\AppData\Local\Programs\Python\Python310\lib\encodings\cp1252.py", line 19, in encode
return codecs.charmap_encode(input,self.errors,encoding_table)[0]
UnicodeEncodeError: 'charmap' codec can't encode characters in position 234-245: character maps to

Fixed an issue on line #224

--Replaced code--
with open(decision_filename, 'a') as fp:
        fp.writelines(lines)
    return

--replacement code--
with open(decision_filename, 'a' encoding="utf-8") as fp:
        fp.writelines(lines)
    return


Traceback (most recent call last):
  File "C:\Users\Admin\Desktop\DupeFinder\venv\Scripts\plex_dupefinder.py", line 448, in <module>
    write_decision(keeping=part_info)
  File "C:\Users\Admin\Desktop\DupeFinder\venv\Scripts\plex_dupefinder.py", line 225, in write_decision
    fp.writelines(lines)
  File "C:\Users\Admin\AppData\Local\Programs\Python\Python310\lib\encodings\cp1252.py", line 19, in encode
    return codecs.charmap_encode(input,self.errors,encoding_table)[0]
UnicodeEncodeError: 'charmap' codec can't encode characters in position 234-245: character maps to <undefined>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant