Fix dead code

desktop/cl_desktop.py

@@ -37,47 +37,13 @@ __ = getLazyLocalTranslate(_)
 class DesktopError(Exception):
     """Desktop Error"""
 
-class share:
-    """Общие методы"""
-    # Объект хранения переменных
-    clVars = False
-
-    def isRoot(self, printError=True):
-        """Определяет является ли пользователь root"""
-        if os.getuid() == 0 and os.getgid() == 0:
-            return True
-        else:
-            if printError:
-                self.printERROR(_("The user is not root"))
-            return False
-
-    def createClVars(self, clVars=False):
-        """Создает объект Vars"""
-        if not clVars:
-            clVars = DataVarsDesktop()
-            # Импортируем переменные
-            clVars.importDesktop()
-            # Заменяем значения переменных переменными из env файлов
-            clVars.flIniFile()
-            # Устанавливаем у объекта атрибут объект переменных
-            self.clVars = clVars
-        return True
-
-    def applyTemplatesFromSystem(self):
-        """Применяем шаблоны для cистемы"""
-        # Cоздаем объект обработки шаблонов
-        clTempl = template(self.clVars)
-        # Объединяем шаблоны
-        dirsFiles = clTempl.applyTemplates()
-        if clTempl.getError():
-            self.printERROR(clTempl.getError().strip())
-            return False
-        else:
-            return dirsFiles
-
 class Desktop(share):
-    """Методы работы с профилем пользователя"""
+    """
-    # Имя пользователя
+    Desktop logic object
+
+    Has fundamental method createHome for configure user profile
+    """
+    # username
     userName = ""
     verbose = False
     
@@ -86,43 +52,6 @@ class Desktop(share):
         self.clTempl = None
         self.clVars = None
 
-#    def installProg(self,dv):
-#        """Наложение шаблонов на систему при инсталяции"""
-#        # Проверяем на root
-#        if not self.isRoot():
-#            return False
-#        self.clVars.AppendToList("cl_merges", __app__, force=True)
-#        # Действие инсталяция
-#        self.clVars.Set("cl_action", "install", True)
-#        if not self.applyTemplatesFromSystem():
-#            raise DesktopError(_("Failed to apply install templates"))
-#        # Добавление программы в инсталяционную переменную
-#        if not appendProgramToEnvFile(__app__, self.clVars):
-#            self.printERROR(_("Failed to save '%s'") %__app__ + " " +\
-#                            _("to %s") %self.clVars.Get("cl_env_path")[0])
-#            return False
-#        self.printOK(_("Install templates applied"))
-#        return True
-#
-#    def uninstallProg(self):
-#        """Наложение шаблонов на систему при деинсталяции"""
-#        # Проверяем на root
-#        if not self.isRoot():
-#            return False
-#        # Действие деинсталяция
-#        self.clVars.Set("cl_action", "uninstall", True)
-#        if not self.applyTemplatesFromSystem():
-#            self.printERROR(_("Failed to apply uninstall templates"))
-#            return False
-#        # Удаление программы из инсталяционной переменной
-#        if not removeProgramToEnvFile(__app__, self.clVars):
-#            self.printERROR(_("Failed to remove '%(app)s' from %(path)s")%
-#                {'app':__app__,
-#                 'path': self.clVars.Get("cl_env_path")[0]})
-#            return False
-#        self.printOK(_("Uninstall templates applied"))
-#        return True
-
     def createUserDir(self, uid, gid, userDir, mode=0700):
         """
         Create user directory with need uid and gid
@@ -148,7 +77,7 @@ class Desktop(share):
             self.printWARNING(" "*5 + nameFile)
 
     def applyTemplatesFromUser(self):
-        """Применяем шаблоны для пользователя"""
+        """Apply templates for user"""
         if self.clTempl:
             self.closeClTemplate()
         self.clTempl = ProgressTemplate(self.setProgress,self.clVars,
@@ -179,7 +108,9 @@ class Desktop(share):
             self.clTempl = None
 
     def umountUserRes(self, error):
-        """Отмонтируем пользовательские директории если они есть"""
+        """
+        Umount user directory
+        """
         self.closeClTemplate()
         if error and self.homeDir:
             umountPaths = self.getMountUserPaths(self.homeDir)
@@ -205,8 +136,6 @@ class Desktop(share):
         #except:
         #    realUserName = ""
         userName = self.clVars.Get("ur_login")
-        #if userName != realUserName and not self.isRoot():
-        #    return False
         uidGid = False
         if self.clVars.isModuleInstalled("client"):
             # domain host
@@ -230,21 +159,21 @@ class Desktop(share):
             self.startTask(_("Creating the home directory for %s")%self.homeDir)
             self.createUserDir(uid,gid,self.homeDir)
             self.endTask()
-        # Действие -  шаблоны пользователя
+        # action - "user profile configuration"
         self.clVars.Set("cl_action", "desktop", True)
-        # Применяем профили для пользователя
+        # apply user profiles
         self.startTask(_("Setting up the user profile"),progress=True)
         dirsAndFiles = self.applyTemplatesFromUser()
         self.endTask()
         if not dirsAndFiles:
-            # Отмонтируем пользовательские ресурсы в случае ошибки
             raise DesktopError(_("Failed to apply user profile templates"))
         self.printSUCCESS(_("User account %s is configured")%userName + " ...")
         return True
 
     def getMountUserPaths(self, homeDir=False):
-        """Находит пользовательские примонтированные пути"""
+        """
-        # Имя пользователя
+        Found user resources
+        """
         if not homeDir:
             userName = self.clVars.Get("ur_login")
             homeDir = self.clVars.Get("ur_home_path")
@@ -260,13 +189,8 @@ class Desktop(share):
                           open("/proc/mounts").readlines()))
 
     def execProg(self, cmdStrProg, inStr=False, envProg={}):
-        """Выполняет внешнюю программу
+        """
-
+        Exec external program
-        Параметры:
-        cmdStrProg     внешняя программа
-        inStr          данные передаваемые программе на страндартный вход.
-        Возвращаемые параметры:
-        строки которые выведет внешняя программа или False в случае ошибки
         """
         env_path = {"PATH":getpathenv()}
         env = {}
@@ -276,29 +200,28 @@ class Desktop(share):
             return programOut
         return False
 
-
+    def umountSleepPath(self, rpath):
-    def umountSleepPath(self, umountpath):
+        """
-        """Отмонтирует путь при неудаче задержка потом повтор"""
+        Unmount path, sleep by failed and repeat
-        # Задержки при отмонтированиии директории
+        """
-        sleeps = [0.5, 2, 5]
+        # check for mount
-        # Проверяем на монтирование директорию
+        if isMount(rpath):
-        if path.ismount(umountpath):
+            for waittime in [0,0.5,1,2]:
-            textLine = self.execProg("umount %s"%umountpath)
+                time.sleep(waittime)
-            if textLine is False:
+                if not self.execProg("umount %s"%rpath) is False \
-                i = 0
+                    or not isMount(rpath):
-                flagError = False
+                    if not isMount(rpath):
-                while (i<len(sleeps) and textLine is False):
+                        return True
-                    # Задержка перед следующей попыткой
+            self.execProg("fuser -km %s"%rpath)
-                    time.sleep(sleeps[i])
+            for waittime in [0.5, 0.5, 0.5, 0.5, 0.5, 0.5]:
-                    # Отмонтируем Samba ресурс
+                time.sleep(waittime)
-                    if path.ismount(umountpath):
+                if not self.execProg("umount %s"%rpath) is False \
-                        textLine = self.execProg("umount %s"%umountpath)
+                    or not isMount(rpath):
+                    if not isMount(rpath):
+                        return True
             else:
-                        textLine = True
+                if isMount(rpath):
-                        break
+                    self.printERROR(_("Failed to unmount path %s")%rpath)
-                    i += 1
-            if textLine is False:
-                self.printERROR(_("Failed to unmount path %s")%
-                                umountpath + " ...")
                     return False
         return True
+