Status bar, some refactoring

Author: teschmitt

mainwindow.ui

@@ -763,8 +763,8 @@
             </layout>
             <child>
               <object class="ttk.Label" id="StatusText">
-                <property name="text" translatable="yes">PyPDF v0.1</property>
-                <property name="textvariable">string:status_text</property>
+                <property name="text" translatable="yes">PyPDF v1.0</property>
+                <property name="textvariable">string:application_status_text</property>
                 <layout>
                   <property name="column">0</property>
                   <property name="propagate">True</property>

pypdfbuilder.py

@@ -102,12 +102,6 @@ def __save_user_data(self):
         except FileNotFoundError:
             print('Something went horribly wrong while trying to save your current user data.')
 
-    def save_success(self):
-        '''Gets called when a PDF file was processed successfully. Currently only
-        increases the `number_of_processed_files´-counter by 1
-        '''
-        self.number_of_processed_files += 1
-
 
 class PDFInfo:
     '''File info class for PDF files.
@@ -238,7 +232,7 @@ def save_as(self):
                     out_pdf.addPage(bottom_page)
                 with open(save_filepath, "wb") as out_pdf_stream:
                     out_pdf.write(out_pdf_stream)
-            self.parent.user_data.save_success()
+            self.parent.save_success(status_text=BG_FILE_SUCCESS.format(os.path.basename(save_filepath)))
 
 
 class SplitTabManager:
@@ -289,7 +283,7 @@ def save_as(self):
                 out_pdf.addPage(in_pdf.getPage(p))
                 with open(output_path, "wb") as out_pdf_stream:
                     out_pdf.write(out_pdf_stream)
-            self.parent.user_data.save_success()
+            self.parent.save_success(status_text=SPLIT_FILE_SUCCESS.format(os.path.dirname(self.__split_filepath)))
 
 
 class RotateTabManager:
@@ -345,7 +339,7 @@ def save_as(self):
                     out_pdf.addPage(in_pdf.getPage(p))
             with open(save_filepath, "wb") as out_pdf_stream:
                 out_pdf.write(out_pdf_stream)
-            self.parent.user_data.save_success()
+            self.parent.save_success(status_text=ROTATE_FILE_SUCCESS.format(os.path.basename(save_filepath)))
 
 
 class JoinTabManager:
@@ -432,7 +426,7 @@ def save_as(self):
                             merger.append(fileobj=open(f[PDF_FILEPATH], 'rb'), pages=page_range)
                 with open(save_filepath, 'wb') as out_pdf:
                     merger.write(out_pdf)
-                self.parent.user_data.save_success()
+                self.parent.save_success(status_text=JOIN_FILE_SUCCESS.format(os.path.basename(save_filepath)))
 
     def move_up(self):
         selected_files = self.__selected_files
@@ -462,6 +456,8 @@ def remove_file(self):
 
 
 class PyPDFBuilderApplication:
+    '''Main application class. Handles setup and running of all application parts.'''
+
     def __init__(self):
         self.builder = pgBuilder()
         self.builder.add_from_file(os.path.join(CURRENT_DIR, 'mainwindow.ui'))
@@ -475,10 +471,10 @@ def __init__(self):
             'bg': self.builder.get_object('BgFrame'),
             'rotate': self.builder.get_object('RotateFrame'),
         }
-
         self.mainmenu = self.builder.get_object('MainMenu')
         self.mainwindow.config(menu=self.mainmenu)
-
+        self.__status_text_variable = self.builder.get_variable('application_status_text')
+        self.status_text = None
         self.builder.connect_callbacks(self)
 
         self.user_data = UserData()
@@ -489,6 +485,14 @@ def __init__(self):
         self.bgtab = BgTabManager(self)
         self.rotatetab = RotateTabManager(self)
 
+    @property
+    def status_text(self):
+        return self.__status_text_variable.get()
+
+    @status_text.setter
+    def status_text(self, val):
+        self.__status_text_variable.set(val)
+
     # boy oh boy if there's anyway to do these callsbacks more elegantly, please let me gain that knowledge!
     def select_tab_join(self, *args, **kwargs):
         '''Gets called when menu item "View > Join Files" is selected.
@@ -565,6 +569,13 @@ def rotatetab_open_file(self):
     def rotatetab_save_as(self):
         self.rotatetab.save_as()
 
+    def save_success(self, status_text=DEFAULT_STATUS):
+        '''Gets called when a PDF file was processed successfully. Currently only
+        increases the `number_of_processed_files`-counter by 1
+        '''
+        self.user_data.number_of_processed_files += 1
+        self.status_text = status_text
+
     def show_settings(self, *args, **kwargs):
         '''Shows the settings dialog. The close event is handled by `self.close_settings()`
         and all the settings management is handled there. Args and kwargs are included in

settings.py

@@ -6,3 +6,10 @@
 PDF_PAGES = 3
 
 ROTATE_DEGREES = {'LEFT': 270, 'RIGHT': 90, 'ONE_EIGHTY': 180}
+
+
+SPLIT_FILE_SUCCESS = 'Files saved successfully to {}!'
+JOIN_FILE_SUCCESS = 'Files joined successfully to {}!'
+ROTATE_FILE_SUCCESS = 'Pages in {} rotated successfully!'
+BG_FILE_SUCCESS = 'File saved successfully to {}!'
+DEFAULT_STATUS = 'PyPDF Builder v1.0'