Merge pull request #24 from jeanslack/sanitize_error

fix UnboundLocalError: local variable newstr referenced before assign…

AUTHORS

@@ -1,3 +1,4 @@
 Author:
 jeanslack <[email protected]>
+Copyright: (C) 2023 Gianluca Pernigotto <[email protected]>
 

CHANGELOG

@@ -1,3 +1,10 @@
++--------------------------------+
+January 09 2023  Version 1.0.17 Beta
++--------------------------------+
+
+- Fixes utils.sanitize error #23
+- Update year.
+
 +--------------------------------+
 December 28 2022  Version 1.0.16 Beta
 +--------------------------------+

README.md

@@ -101,7 +101,7 @@ For arguments meaning and more details, type `help(FFCueSplitter)`
 
 ## License and Copyright
 
-Copyright © 2022 Gianluca Pernigotto   
+Copyright (C) 2023 Gianluca Pernigotto   
 Author and Developer: Gianluca Pernigotto   
 Mail: <[email protected]>   
 License: GPL3 (see LICENSE file in the docs folder)   

TODO

@@ -1,6 +1,6 @@
 
 - Consider using `logger` for messages (?)
-- Create log file and list all processed file.cue.
+
 
 
 

ffcuesplitter.py

@@ -8,6 +8,7 @@
 Platform: all platforms
 Writer: jeanslack <[email protected]>
 license: GPL3
+Copyright: (C) 2023 Gianluca Pernigotto <[email protected]>
 Rev: December 24 2022
 """
 

ffcuesplitter/cuesplitter.py

@@ -6,6 +6,7 @@
 Platform: all
 Writer: jeanslack <[email protected]>
 license: GPL3
+Copyright: (C) 2023 Gianluca Pernigotto <[email protected]>
 Rev: Dec 22 2022
 Code checker: flake8 and pylint
 ####################################################################

ffcuesplitter/exceptions.py

@@ -4,6 +4,7 @@
 Platform: all
 Writer: jeanslack <[email protected]>
 license: GPL3
+Copyright: (C) 2023 Gianluca Pernigotto <[email protected]>
 Rev: February 03 2022
 Code checker: flake8 and pylint
 ####################################################################

ffcuesplitter/ffmpeg.py

@@ -5,7 +5,7 @@
 Compatibility: Python3
 Platform: all platforms
 Author: Gianluca Pernigotto <[email protected]>
-Copyright: (c) 2022/2023 Gianluca Pernigotto <[email protected]>
+Copyright: (C) 2023 Gianluca Pernigotto <[email protected]>
 license: GPL3
 Rev: Dec.27.2022
 Code checker: flake8, pylint

ffcuesplitter/ffprobe.py

@@ -5,7 +5,7 @@
 Compatibility: Python3
 Platform: all
 Author: Gianluca Pernigotto <[email protected]>
-Copyright: (c) 2022/2023 Gianluca Pernigotto <[email protected]>
+Copyright: (C) 2023 Gianluca Pernigotto <[email protected]>
 license: GPL3
 Rev: Dec.14.2022
 Code checker: flake8, pylint

ffcuesplitter/info.py

@@ -3,7 +3,8 @@
 Porpose: useful information strings
 Writer: jeanslack <[email protected]>
 license: GPL3
-Rev: February 03 2022
+Copyright: (C) 2023 Gianluca Pernigotto <[email protected]>
+Rev: January 09 2023
 ####################################################################
 
 This file is part of FFcuesplitter.
@@ -25,9 +26,9 @@
 __contact__ = '<[email protected]>'
 __maintainer__ = "Gianluca Pernigotto - Jeanslack"
 __maintainer_contact__ = "[email protected]"
-__copyleft__ = '2022'
-__version__ = '1.0.16'
-__release__ = 'December 28 2022'
+__copyleft__ = '2023'
+__version__ = '1.0.17'
+__release__ = 'January 09 2023'
 __appname__ = "FFcuesplitter"
 __packagename__ = "ffcuesplitter"
 __license__ = "GPL3 (Gnu Public License)"

ffcuesplitter/main.py

@@ -6,6 +6,7 @@
 Platform: all
 Writer: jeanslack <[email protected]>
 license: GPL3
+Copyright: (C) 2023 Gianluca Pernigotto <[email protected]>
 Rev: Dec 27 2022
 Code checker: flake8, pylint
 ####################################################################

ffcuesplitter/str_utils.py

@@ -1,11 +1,11 @@
 # -*- coding: utf-8 -*-
 """
-Name:         str_utils.py (module)
-Porpose:      module for cosmetic output console in ANSI sequences
-Writer:       Gianluca Pernigoto <[email protected]>
-Copyright:    (c) 2022 Gianluca Pernigoto <[email protected]>
-license:      GPL3
-Rev:          Jan 10 2022
+Name: str_utils.py (module)
+Porpose: module for cosmetic output console in ANSI sequences
+Writer: Gianluca Pernigoto <[email protected]>
+Copyright: (C) 2023 Gianluca Pernigotto <[email protected]>
+license: GPL3
+Rev: Jan 10 2022
 Code checker: flake8, pylint
 """
 

ffcuesplitter/utils.py

@@ -4,6 +4,7 @@
 Platform: all
 Writer: jeanslack <[email protected]>
 license: GPL3
+Copyright: (C) 2023 Gianluca Pernigotto <[email protected]>
 Rev: Dec 27 2022
 Code checker: flake8 and pylint
 ####################################################################
@@ -30,28 +31,31 @@
 import datetime
 
 
-def sanitize(string: str = 'stringa') -> str:
+def sanitize(string: str = 'string') -> str:
     r"""
     Makes the passed string consistent and compatible
-    with file systems of some operating systems.
+    with the OS file system.
 
-    All OS:
-    Remove all leading/trailing spaces and dots.
+    - On Windows, removes the following illegal chars: " * : < > ? / \ |
+      and replaces slash (/) character with hyphen (-).
+    - On other operating systems, it replaces slash (/) character
+      with hyphen (-).
+    - On all operating systems, removes leading/trailing spaces
+      and dots (.).
 
-    On Windows it removes the following illegal chars: " * : < > ? / \ |
-    On Unix it remove slash char: /
-    Returns the new sanitized string
+    Returns the new sanitized string.
 
     """
     if not isinstance(string, str):
         raise TypeError("Expects Type string only")
 
     if platform.system() == 'Windows':
-        newstr = re.sub(r"[\"\*\:\<\>\?\/\|\\]", '', newstr)
+        string = re.sub(r"[\"\*\:\<\>\?\|\\]", '', string)
+        string = string.replace('/', '-')
     else:
-        newstr.replace('/', '')
+        string = string.replace('/', '-')
 
-    return newstr.strip().strip('.')  # spaces and dots
+    return string.strip().strip('.')  # removes spaces and dots
 # ------------------------------------------------------------------------
 
 

setup.py

@@ -9,6 +9,7 @@
 Platform: all
 Writer: jeanslack <[email protected]>
 license: GPL3
+Copyright: (C) 2023 Gianluca Pernigotto <[email protected]>
 Rev: Feb 02 2022
 Code checker: flake8, pylint
 ####################################################################