release v1.0.28

Author: jeanslack

CHANGELOG

@@ -1,3 +1,10 @@
++--------------------------------+
+June 14 2025  Version 1.0.28
++--------------------------------+
+
+- Improving functionalities
+- code refactor
+
 +--------------------------------+
 June 12 2025  Version 1.0.27
 +--------------------------------+

ffcuesplitter/about.py

@@ -26,8 +26,8 @@
 CONTACT = '<jeanlucperni@gmail.com>'
 MAINTAINER = "Gianluca Pernigotto - Jeanslack"
 COPYLEFT = '2025'
-VERSION = '1.0.27'
-RELEASE = 'June 12 2025'
+VERSION = '1.0.28'
+RELEASE = 'June 14 2025'
 APPNAME = "FFcuesplitter"
 PKGNAME = "ffcuesplitter"
 LICENSE_NAME = "GPL3 (Gnu Public License)"

ffcuesplitter/cuesplitter.py

@@ -7,7 +7,7 @@
 Writer: jeanslack <jeanlucperni@gmail.com>
 license: GPL3
 Copyleft: (C) 2025 Gianluca Pernigotto <jeanlucperni@gmail.com>
-Rev: June 10 2025
+Rev: June 14 2025
 Code checker: flake8 and pylint
 ####################################################################
 
@@ -238,13 +238,14 @@ def deflacue_object_handler(self):
             track_file = track[1].file.path
 
             if not track_file.exists():
-                logging.warning('Not found: `%s`. Track is skipped.',
+                logging.warning('Not found: "%s"',
                                 os.path.abspath(track_file))
 
                 if str(track_file) in sourcenames:
                     sourcenames.pop(str(track_file))
                     if not sourcenames:
-                        raise FFCueSplitterError('No audio files found!')
+                        raise FFCueSplitterError(f'No audio track found: '
+                                                 f'«{track_file}»')
                 continue
 
             data = {'FILE': str(track_file), **cd_info, **track[1].data}

ffcuesplitter/user_service.py

@@ -32,6 +32,7 @@
 import logging
 from ffcuesplitter.cuesplitter import FFCueSplitter
 from ffcuesplitter.exceptions import FFCueSplitterError
+from ffcuesplitter.utils import remove_source_file
 
 
 class FileSystemOperations(FFCueSplitter):
@@ -61,7 +62,7 @@ class FileSystemOperations(FFCueSplitter):
     """
     # ----------------------------------------------------------------#
 
-    def remove_source_file(self):
+    def del_orig_files(self):
         """
         Remove the CUE file and the original audio file
         from the filesystem. If either one is missing, no
@@ -70,18 +71,12 @@ def remove_source_file(self):
         """
         cuef = self.kwargs['filename']
         audf = self.audiosource
-        exist = False
-        if os.path.exists(cuef) and os.path.exists(audf):
-            if os.path.isfile(cuef) and os.path.isfile(audf):
-                exist = True
-                logging.info("Deleting CUE file: '%s'", cuef)
-                os.remove(cuef)
-                logging.info("Deleting audio source file: '%s'", audf)
-                os.remove(audf)
-        if not exist:
+        ret = remove_source_file(cuef, audf)
+        if ret is True:
+            logging.info("Deleting CUE file: '%s'", cuef)
+            logging.info("Deleting audio source file: '%s'", audf)
+        else:
             logging.warning("File deletion failed, source files are missing")
-            return exist
-        return exist
     # ----------------------------------------------------------------#
 
     def dry_run_mode(self):
@@ -205,7 +200,8 @@ def work_on_temporary_directory(self):
             self.move_files_to_outputdir()
             # remove source file if `del_orig_files` argument is given.
             if self.kwargs['del_orig_files']:
-                self.remove_source_file()
+                #self.remove_source_file()
+                self.del_orig_files()
 # ------------------------------------------------------------------------
 
 

ffcuesplitter/utils.py

@@ -5,7 +5,7 @@
 Writer: jeanslack <jeanlucperni@gmail.com>
 license: GPL3
 Copyleft: (C) 2025 Gianluca Pernigotto <jeanlucperni@gmail.com>
-Rev: June 10 2025
+Rev: June 14 2025
 Code checker: flake8 and pylint
 ####################################################################
 
@@ -108,6 +108,24 @@ def makeoutputdirs(outputdir):
 # ------------------------------------------------------------------------
 
 
+def remove_source_file(cuef, audf):
+    """
+    Remove the CUE file and the original audio file
+    from the filesystem. If either one is missing, no
+    deletion will be applied.
+    Return True if deletion was successfull, False otherwise.
+    """
+    if os.path.exists(cuef) and os.path.exists(audf):
+        if os.path.isfile(cuef) and os.path.isfile(audf):
+            os.remove(cuef)
+            os.remove(audf)
+            return True
+        else:
+            return False
+    return False
+# ------------------------------------------------------------------------
+
+
 class Popen(subprocess.Popen):
     """
     Inherit `subprocess.Popen` class to set `_startupinfo`.