Help: Connect to the online help manual

This fix add's ability to connect to the online user manual if
the local help file is missing.
The patch also fixes other minor issues related to data types in
the Qt API function calls - see document_viewer file for those
changes.

Author: pgathogo

stdm/data/configuration/columns.py

@@ -268,7 +268,7 @@ def on_delete(self):
         """
         pass
 
-    def dependencies(self):
+    def dependencies(self) ->dict:
         """
         Gets the tables and views that are related to this column.
         :return: A dictionary containing a list of related entity names and

stdm/plugin.py

@@ -31,10 +31,12 @@
     QCoreApplication,
     Qt,
     QStandardPaths,
+    QUrl,
     QDir
 )
 from qgis.PyQt.QtGui import (
-    QKeySequence
+    QKeySequence,
+    QDesktopServices
 )
 from qgis.PyQt.QtWidgets import (
     QApplication,
@@ -52,6 +54,7 @@
     QgsApplication,
     QgsTask
 )
+
 from qgis.gui import (
     QgsLayoutDesignerInterface
 )
@@ -1244,7 +1247,10 @@ def loadModules(self):
         username = globals.APP_DBCONN.User.UserName
 
         if username == 'postgres':
-            self.grant_privilege_base_tables(username)
+            if self.grant_privilege_base_tables(username):
+                LOGGER.info('Privileges granted to base tables.')
+            else:
+                LOGGER.error('Failed to grant privilege to base tables.')
 
         self.moduleCntGroup = None
         self.moduleContentGroups = []
@@ -1492,14 +1498,29 @@ def _doc_temp_exist(self, doc_temp, profile_templates):
                 break
         return doc_exist
 
-    def grant_privilege_base_tables(self, username):
+    def grant_privilege_base_tables(self, username) ->bool:
         roles = []
-        roleProvider = RoleProvider()
-        roles = roleProvider.GetSysRoles()
+        try:
+            roleProvider = RoleProvider()
+            roles = roleProvider.GetSysRoles()
+
+            privilege_provider = SinglePrivilegeProvider('', current_profile())
+            for role in roles:
+                privilege_provider.grant_privilege_base_table(role)
+        except ConfigurationException as c_ex:
+            QMessageBox.information(
+                self.iface.mainWindow(),
+                QApplication.translate(
+                    'STDM',
+                    'Grant privilege base tables'
+                ),
+                str(c_ex))
+            return False
+
+        return True
+
+
 
-        privilege_provider = SinglePrivilegeProvider('', current_profile())
-        for role in roles:
-            privilege_provider.grant_privilege_base_table(role)
 
     def load_profiles_combobox(self):
         """
@@ -2220,21 +2241,45 @@ def user_entities(self):
 
     def help_contents(self):
         """
-        Load and open documentation manual
+        Open documentation manual
         """
+        HELP_URL = "https://www.stdm.gltn.net/docs/1_8/STDM_1.8.1_User_Manual.pdf"
+
         help_manual = '{0}/stdm.chm'.format(self.plugin_dir)
 
         try:
             os.startfile(
                 help_manual, 'open'
             )
         except FileNotFoundError as ex:
-            QMessageBox.critical(
-                self.iface.mainWindow(),
-                QApplication.translate(
-                    "STDMQGISLoader",
-                    'Open File Error'
-                ), str("Error trying to open help file (stdm.chm), file is missing."))
+            if self.view_online_user_manual():
+                ds = QDesktopServices()
+                ds.openUrl(QUrl(HELP_URL))
+
+
+    def view_online_user_manual(self) -> bool:
+        msg_box = QMessageBox()
+        msg_box.setIcon(QMessageBox.Critical)
+        msg_box.setWindowTitle(QApplication.translate(
+            "STDMQGISLoader",
+            "STDM Help File Error"))
+        msg_box.setText(QApplication.translate(
+            "STDMQGISLoader",
+            "Local version of the help file (stdm.chm) file is missing."))
+        msg_box.setInformativeText(QApplication.translate(
+            "STDMQGISLoader",
+            'Would you like to open the online help?'))
+        msg_box.setStandardButtons(QMessageBox.Yes | QMessageBox.No)
+        msg_box.setDefaultButton(QMessageBox.Yes)
+
+        yes_btn = msg_box.button(QMessageBox.Yes)
+
+        msg_box.exec_()
+        if msg_box.clickedButton() == yes_btn:
+            return True
+        else:
+            return False
+
 
     def reset_content_modules_id(self, title, message_text):
         return QMessageBox.critical(

stdm/ui/document_viewer.py

@@ -352,7 +352,7 @@ def update_size(self, doc_width, doc_height):
         if par_height > doc_height and par_width > doc_width:
             self.resize(doc_width, doc_height)
         else:
-            self.resize(doc_width * ratio, doc_height * ratio)
+            self.resize(doc_width * int(ratio), doc_height * int(ratio))
 
     def closeEvent(self, event):
         """
@@ -416,7 +416,7 @@ def center(self):
         # vertical position
         vpos = (screen.height() - mdi_area_size.height()) / 2
         # repositions the window
-        self.move(hpos, vpos)
+        self.move(int(hpos), int(vpos))
 
     def _create_menu_actions(self):
         self._window_menu = self.menuBar().addMenu(

stdm/ui/geoodk_profile_importer.py

@@ -651,6 +651,7 @@ def save_instance_data_to_db(self, entities : List[str], xml_files: List[str]) -
                 # Check if the file exists in the log file
                 if xml_filename in log_data:
                     msg = f"File `{xml_filename}` already imported. Skipping..."
+                    self.txt_feedback.append(msg)
                     self.importlogger.log_info(msg)
                     continue
 

stdm/ui/sourcedocument.py

@@ -946,7 +946,7 @@ def onBlockWritten(self, size):
         """
         progress = (size * 100) / self._docSize
 
-        self.pgBar.setValue(progress)
+        self.pgBar.setValue(int(progress))
         #QgsApplication.processEvents()
 
     def onCompleteTransfer(self, fileid):

stdm/ui/wizard/column_editor.py

@@ -804,7 +804,6 @@ def create_column(self) -> BaseColumn:
             self.entity, **self.form_fields
         )
 
-
     def property_set(self):
         self.prop_set = True
         self.type_attribs[self.current_type_info()]['prop_set'] = True

stdm/ui/wizard/create_lookup.py

@@ -180,7 +180,7 @@ def accept(self):
 
         if self.lookup is None:  # new lookup
             if self.duplicate_check(short_name):
-                self.show_message(
+                self.error_message(
                     self.tr(
                         "Lookup with the same name already "
                         "exist in the current profile!"

stdm/ui/wizard/wizard.py

@@ -2538,7 +2538,7 @@ def process_privilege(self, entity, column):
         if new_content is None: return
         self.update_privilege_cache(entity, column, new_content)
 
-    def column_has_entity_relation(self, column):
+    def column_has_entity_relation(self, column: 'Column') ->bool:
         answer = False
         if (isinstance(column, ForeignKeyColumn) or
                 isinstance(column, AdministrativeSpatialUnitColumn)):
@@ -2810,6 +2810,11 @@ def delete_lookup(self):
         # the current profile
         dependencies = self.all_column_dependencies(profile)
 
+        print('------------------')
+        print(dependencies)
+        print(lookup.name)
+        print('------------------')
+
         if self.find_lookup(lookup.name, dependencies):
             self.show_message(self.tr("Cannot delete '{0}' lookup!\n "
                                       "Lookup is been used by existing columns."
@@ -2829,7 +2834,7 @@ def delete_lookup(self):
             self.lookup_view_model.removeRow(row_id)
             self.refresh_lookup_view()
 
-    def all_column_dependencies(self, profile):
+    def all_column_dependencies(self, profile:Profile) -> list:
         """
         Returns a list of all column dependencies, for all
         enitites in the current profile
@@ -2849,7 +2854,7 @@ def all_column_dependencies(self, profile):
                     depends.append(column.dependencies())
         return depends
 
-    def find_lookup(self, name, dependencies):
+    def find_lookup(self, name, dependencies:list) -> list:
         """
         Returns True if name is in the list of dependencies
         else False
@@ -2859,6 +2864,7 @@ def find_lookup(self, name, dependencies):
         :type dependencies: list
         :rtype: boolean
         """
+
         for depend in dependencies:
             if name in depend['entities']:
                 return True