Merge pull request '1.11.2124' (#30) from 1.11.2124 into main

Reviewed-on: #30

Changelog

@@ -8,51 +8,81 @@ My standard System: Linux Mint 22 Cinnamon
  - for loops with lists replaced by List Comprehensions
  - Update search after start of Wire-Py
 
-  ### Added
-08-11-2024
-
-- Preparation for language translation part 1
-- separate folder for general icons
-- install edit for new dir lx-icons
-
-  ### Added
-08-11-2024
-
-- methods from class MainWindow move to class FrameWidgets for active color_label when theme change
-- optimize columnconfigure, rowconfigure in class MainWindow and FrameWidgets
-- add new Frame for Widgets on Bottom
-- optimize from tkinter * to from tkinter import filedialog, ttk, TclError
-
   ### Added
 07-11-2024
 
 - remove classes and add methods to class FrameWidgets (removed self errors)
 
 
+  ### Added
+10-11-2024
+
+ - Fix Checkbutton Autostart when first install Wire-Py
+ - Update Translate Files
+
+  ### Added
+10-11-2024
+
+ - Translate file de_De complete
+ - new FrameWidget for rename Entry and rename Button
+
+
+  ### Added
+09-11-2024
+
+ - Move Tips Method in separate class for Tooltips in another Apps
+ - Move Version Variable in main script
+ - Edit Class GiteaUpdate for requests in api_down and download
+ - Description on Class GiteaUpdate
+
+
+  ### Added
+08-11-2024
+
+ - Preparation for language translation part 1
+ - separate folder for general icons
+ - install edit for new dir lx-icons
+
+
+  ### Added
+08-11-2024
+
+ - methods from class MainWindow move to class FrameWidgets for active color_label when theme change
+ - optimize columnconfigure, rowconfigure in class MainWindow and FrameWidgets
+ - add new Frame for Widgets on Bottom
+ - optimize from tkinter * to from tkinter import filedialog, ttk, TclError
+
+
+  ### Added
+07-11-2024
+
+ - remove classes and add methods to class FrameWidgets (removed self errors)
+
+
   ### Added
 27-10-2024
 
-- Add Autoconnect settings to settings.conf
+ - Add Autoconnect settings to settings.conf
 
 
   ### Added
 26-10-2024
 
-- Add run_as Bash script and open_gitea.py python script
-- Add Tooltip disable/enable
-- Rename settings to settings.conf for theme, updates and tooltip enable in one file
+ - Add run_as Bash script and open_gitea.py python script
+ - Add Tooltip disable/enable
+ - Rename settings to settings.conf for theme, updates and tooltip enable in one file
 
 
   ### Added
 25-10-2024
 
-- Optimize Class and Tooltip
+ - Optimize Class and Tooltip
 
 
   ### Added
 21-10-2024
 
-- Optimize Class. Move to wg_main Import Start/StopBTN and Tooltip
+ - Optimize Class. Move to wg_main Import Start/StopBTN and Tooltip
 
 
  ### Added

cls_mth_fc.py

@@ -1,62 +1,65 @@
-""" Wireguard Classes and Method for Wire-Py """
+""" Classes Method and functions for lx apps """
 
+import gettext
+import locale
 import os
 import shutil
 import subprocess
-import time
 import tkinter as tk
 import zipfile
-import gettext
-import locale
 from datetime import datetime
 from pathlib import Path
-from subprocess import check_call
 from tkinter import ttk
 import requests
 
-APP = 'cls_mth_fc'
+APP = 'wirepy'
 LOCALE_DIR = "/usr/share/locale/"
 locale.bindtextdomain(APP, LOCALE_DIR)
 gettext.bindtextdomain(APP, LOCALE_DIR)
 gettext.textdomain(APP)
 _ = gettext.gettext
-''' 1 = 1. Year, 09 = Month of the Year, 2924 = Day and Year of the Year '''
-version = 'v. 1.11.0824'
 
-path_to_file2 = Path('/etc/wire_py/settings.conf')
+wg_set = Path('/etc/wire_py/settings.conf')
 _u = Path.read_text(Path('/tmp/_u'))
 
 
-
 class GiteaUpdate:
+    """
+    Calling api_down requests the URL and the version of the running script.
+    Example: version = 'v. 1.1.1.1' GiteaUpdate.api_down(http://example.de, version)
+
+    Calling download requests the download URL of the running script,
+    the taskbar image for the “Download OK” window, the taskbar image for the
+    “Download error” window and the variable res
+    """
     @staticmethod
-    def api_down(UPDATE_API_URL):
+    def api_down(update_api_url, version):
         try:
-            response = requests.get(UPDATE_API_URL)
+            response = requests.get(update_api_url)
             response_dict = response.json()
             response_dict = response_dict[0]
-            with open(path_to_file2, 'r') as set_file:
+            with open(wg_set, 'r') as set_file:
                 set_file = set_file.read()
                 if 'on\n' in set_file:
                     if version[3:] != response_dict['tag_name']:
                         return response_dict['tag_name']
                     else:
-                        return _('No Updates')
+                        return 'No Updates'
                 else:
-                    return _('False')
+                    return 'False'
         except requests.exceptions.ConnectionError:
             return 'No Internet Connection!'
 
     @staticmethod
-    def download(URLD):
+    def download(urld, down_ok_image, down_not_ok_image, res):
         try:
-            to_down = 'wget -qP ' + str(_u) + ' ' + URLD
+            to_down = 'wget -qP ' + str(_u) + ' ' + urld
             result = subprocess.call(to_down, shell=True)
             if result == 0:
                 shutil.chown(str(_u) + f'/{res}.zip', 1000, 1000)
                 """img_w, img_i, w_title, w_txt hand over"""
                 iw = r'/usr/share/icons/lx-icons/64/info.png'
-                ii = r'/usr/share/icons/wp-icons/48/wg_vpn.png'
+                ii = down_ok_image
                 wt = _('Download Successful')
                 msg_t = _('Your zip file is in home directory')
                 msg_window(iw, ii, wt, msg_t)
@@ -64,23 +67,19 @@ class GiteaUpdate:
             else:
                 """img_w, img_i, w_title, w_txt hand over"""
                 iw = r'/usr/share/icons/lx-icons/64/error.png'
-                ii = r'/usr/share/icons/wp-icons/48/wg_msg.png'
+                ii = down_not_ok_image
                 wt = _('Download error')
                 msg_t = _('Download failed! Please try again')
                 msg_window(iw, ii, wt, msg_t)
         except subprocess.CalledProcessError:
             """img_w, img_i, w_title, w_txt hand over"""
             iw = r'/usr/share/icons/lx-icons/64/error.png'
-            ii = r'/usr/share/icons/wg-icons/48/wg_msg.png'
+            ii = down_not_ok_image
             wt = _('Download error')
             msg_t = _('Download failed! No internet connection!')
             msg_window(iw, ii, wt, msg_t)
 
 
-res = GiteaUpdate.api_down('https://git.ilunix.de/api/v1/repos/punix/Wire-Py/releases')
-
-
-
 def msg_window(img_w, img_i, w_title, w_txt, txt2=None, com=None):
     """
     Function for different message windows for the user. with 4 arguments to be passed.
@@ -90,8 +89,9 @@ def msg_window(img_w, img_i, w_title, w_txt, txt2=None, com=None):
     img_i = Image for Icon
     w_title = Windows Title
     w_txt = Text for Tk Window
+    txt2 = Text for Button two
+    com = function for Button command
     """
-
     msg = tk.Toplevel()
     msg.resizable(width=False, height=False)
     msg.title(w_title)
@@ -103,7 +103,7 @@ def msg_window(img_w, img_i, w_title, w_txt, txt2=None, com=None):
 
     label.grid(column=1, row=0)
 
-    if txt2 != None and com != None:
+    if txt2 is not None and com is not None:
         label.config(font=('Ubuntu', 11), padx=15, justify='left')
         msg.i_window.grid(column=0, row=0, sticky='nw')
         button2 = ttk.Button(msg, text=f'{txt2}', command=com, padding=4)
@@ -125,7 +125,6 @@ def msg_window(img_w, img_i, w_title, w_txt, txt2=None, com=None):
 
 
 class Tunnel:
-
     """
     Class of Methods for Wire-Py
     """
@@ -134,7 +133,6 @@ class Tunnel:
     The config file is packed into a dictionary,
     to display the values Address , DNS and Peer in the labels
     """
-
     @classmethod
     def con_to_dict(cls, file):
 
@@ -176,11 +174,9 @@ class Tunnel:
             pre_key = final_dict['PreSharedKey']
         return address, dns, endpoint, pre_key
 
-
     """
     Shows the Active Tunnel
     """
-
     @staticmethod
     def active():
 
@@ -195,7 +191,6 @@ class Tunnel:
     """
     Shows all existing Wireguard tunnels
     """
-
     @staticmethod
     def list():
         wg_s = os.popen('nmcli con show | grep -iPo "(.*)(wireguard)"').read().split()
@@ -204,13 +199,11 @@ class Tunnel:
         tl = wg_s[::3]
         return tl
 
-
     """
     This will export the tunnels.
     A zipfile with current date and time is created
     in the user's home directory with correct right
     """
-
     @staticmethod
     def export():
         _u1 = str(_u[6:])
@@ -254,19 +247,26 @@ class Tunnel:
                 iw = r'/usr/share/icons/lx-icons/64/info.png'
                 ii = r'/usr/share/icons/wp-icons/48/wg_msg.png'
                 wt = _('Select tunnel')
-                msg_t = _('Please first import tunnel.')
+                msg_t = _('Please first import tunnel')
                 msg_window(iw, ii, wt, msg_t)
 
         except TypeError:
             pass
 
-    @staticmethod
-    def if_tip():
-        with open(path_to_file2, 'r') as set_file2:
-                lines2 = set_file2.readlines()
-                if 'False\n' in lines2:
-                    return False
-                else:
-                    return True
 
-tips = Tunnel.if_tip()
+class Tipi:
+    """
+    Class for Tooltip setting write in File
+    Calling request path to file
+    """
+    @staticmethod
+    def if_tip(path):
+        with open(path, 'r') as set_file2:
+            lines2 = set_file2.readlines()
+            if 'False\n' in lines2:
+                return False
+            else:
+                return True
+
+
+wg_tips = Tipi.if_tip(wg_set)

install

@@ -11,6 +11,7 @@ install_file_with(){
   sudo mkdir -p /etc/wire_py && sudo touch /etc/wire_py/.keys && sudo cp -u settings.conf /etc/wire_py/ && \
   sudo cp -uR wp-icons lx-icons /usr/share/icons/ &&  sudo cp -uR TK-Themes /usr/share/ && \
   sudo chown -R root:root /etc/wire_py && sudo chmod 755 /etc/wire_py && \
+  sudo cp -u languages/de/*.mo /usr/share/locale/de/LC_MESSAGES/ && \
   sudo ln -sf /usr/bin/wirepy.py /usr/local/bin/wirepy && \
   sudo cp -u org.wirepy.policy /usr/share/polkit-1/actions/ && \
   sudo cp -u Wire-Py.desktop /usr/share/applications/ && \
@@ -25,6 +26,7 @@ install_arch_d(){
   sudo mkdir -p /etc/wire_py && sudo touch /etc/wire_py/.keys && sudo cp -u settings.conf  /etc/wire_py/ && \
   sudo cp -uR wp-icons lx-icons /usr/share/icons/ && sudo cp -uR TK-Themes /usr/share/ && \
   sudo chown -R root:root /etc/wire_py && sudo chmod 755 /etc/wire_py && \
+  sudo cp -u languages/de/*.mo /usr/share/locale/de/LC_MESSAGES/ && \
   sudo ln -sf /usr/bin/wirepy.py /usr/local/bin/wirepy && \
   sudo cp -u org.wirepy.policy /usr/share/polkit-1/actions/ && \
   sudo cp -u Wire-Py.desktop /usr/share/applications/ && \
@@ -92,6 +94,7 @@ elif grep -i 'fedora' /etc/os-release > /dev/null 2>&1
                 sudo cp -u wg_main.py start_wg.py cls_mth_fc.py wirepy.py run_as open_gitea.py /usr/bin/ && \
                 sudo mkdir -p /etc/wire_py && sudo touch /etc/wire_py/.keys && \
                 sudo cp -u settings.conf /etc/wire_py/ && \
+                sudo cp -u languages/de/*.mo /usr/share/locale/de/LC_MESSAGES/ && \
                 sudo cp -uR wp-icons lx-icons /usr/share/icons/ && sudo cp -uR TK-Themes /usr/share/ && \
                 sudo chown -R root:root /etc/wire_py && sudo chmod 755 /etc/wire_py && \
                 sudo ln -sf /usr/bin/wirepy.py /usr/local/bin/wirepy && \
@@ -108,6 +111,7 @@ elif grep -i 'suse' /etc/os-release > /dev/null 2>&1
                  sudo cp -u wg_main.py start_wg.py cls_mth_fc.py wirepy.py run_as open_gitea.py /usr/bin/ && \
                  sudo mkdir -p /etc/wire_py && sudo touch /etc/wire_py/.keys && \
                  sudo cp -u settings.conf /etc/wire_py/ && \
+                 sudo cp -u languages/de/*.mo /usr/share/locale/de/LC_MESSAGES/ && \
                  sudo cp -uR wp-icons lx-icons /usr/share/icons/ && sudo cp -uR TK-Themes /usr/share/ && \
                  sudo chown -R root:root /etc/wire_py && sudo chmod 755 /etc/wire_py && \
                  sudo ln -sf /usr/bin/wirepy.py /usr/local/bin/wirepy && \

wg_main.py

@@ -1,4 +1,5 @@
 #!/usr/bin/python3
+
 import gettext
 import locale
 import os
@@ -8,12 +9,17 @@ import tkinter as tk
 from pathlib import Path
 from subprocess import check_call
 from tkinter import filedialog, ttk, TclError
-
-from cls_mth_fc import (Tunnel, msg_window, GiteaUpdate, res, _u, version, path_to_file2, tips)
+from cls_mth_fc import (Tunnel, msg_window, GiteaUpdate, _u, wg_tips, wg_set)
 
 tcl_path = Path('/usr/share/TK-Themes')
 
-APP = 'wg_main'
+''' 1 = 1. Year, 09 = Month of the Year, 2924 = Day and Year of the Year '''
+version = 'v. 1.11.1824'
+
+res = GiteaUpdate.api_down('https://git.ilunix.de/api/v1/repos/punix/Wire-Py/releases', version)
+
+''' Translate '''
+APP = 'wirepy'
 LOCALE_DIR = "/usr/share/locale/"
 locale.bindtextdomain(APP, LOCALE_DIR)
 gettext.bindtextdomain(APP, LOCALE_DIR)
@@ -40,7 +46,7 @@ class MainWindow(tk.Tk):
         self.style = ttk.Style(self)
         self.tk.call('source', str(tcl_path) + '/water.tcl')
         ''' self.tk.call('source', 'TK-Themes/water.tcl') '''
-        with open(path_to_file2, 'r') as read_file:
+        with open(wg_set, 'r') as read_file:
             lines = read_file.readlines()
             if 'light\n' in lines:
                 self.tk.call('set_theme', 'light')
@@ -79,34 +85,34 @@ class FrameWidgets(ttk.Frame):
 
         def update():
             if set_update.get() == 1:
-                with open(path_to_file2, 'r') as set_file2:
+                with open(wg_set, 'r') as set_file2:
                     lines2 = set_file2.readlines()
                     lines2[1] = 'off\n'
-                with open(path_to_file2, 'w') as set_file2:
+                with open(wg_set, 'w') as set_file2:
                     set_file2.writelines(lines2)
 
             if set_update.get() == 0:
-                with open(path_to_file2, 'r') as set_file2:
+                with open(wg_set, 'r') as set_file2:
                     lines2 = set_file2.readlines()
                     lines2[1] = 'on\n'
-                with open(path_to_file2, 'w') as set_file2:
+                with open(wg_set, 'w') as set_file2:
                     set_file2.writelines(lines2)
 
         ''' Set True or False in file '''
 
         def tooltip():
             if set_tip.get():
-                with open(path_to_file2, 'r') as set_file2:
+                with open(wg_set, 'r') as set_file2:
                     lines2 = set_file2.readlines()
                     lines2[5] = 'False\n'
-                with open(path_to_file2, 'w') as set_file2:
+                with open(wg_set, 'w') as set_file2:
                     set_file2.writelines(lines2)
 
             else:
-                with open(path_to_file2, 'r') as set_file2:
+                with open(wg_set, 'r') as set_file2:
                     lines2 = set_file2.readlines()
                     lines2[5] = 'True\n'
-                with open(path_to_file2, 'w') as set_file2:
+                with open(wg_set, 'w') as set_file2:
                     set_file2.writelines(lines2)
 
         ''' Set dark or light '''
@@ -115,10 +121,10 @@ class FrameWidgets(ttk.Frame):
             if self.tk.call("ttk::style", "theme", "use") == "water-dark":
                 ''' Set light theme '''
                 self.tk.call('set_theme', 'light')
-                with open(path_to_file2, 'r') as theme_set2:
+                with open(wg_set, 'r') as theme_set2:
                     lines3 = theme_set2.readlines()
                     lines3[3] = 'light\n'
-                with open(path_to_file2, 'w') as theme_set2:
+                with open(wg_set, 'w') as theme_set2:
                     theme_set2.writelines(lines3)
                 self.color_label()
 
@@ -126,10 +132,10 @@ class FrameWidgets(ttk.Frame):
             if not self.tk.call("ttk::style", "theme", "use") == "water-dark":
                 ''' Set dark theme '''
                 self.tk.call('set_theme', 'dark')
-                with open(path_to_file2, 'r') as theme_set2:
+                with open(wg_set, 'r') as theme_set2:
                     lines4 = theme_set2.readlines()
                     lines4[3] = 'dark\n'
-                with open(path_to_file2, 'w') as theme_set2:
+                with open(wg_set, 'w') as theme_set2:
                     theme_set2.writelines(lines4)
                 self.color_label()
 
@@ -172,7 +178,7 @@ class FrameWidgets(ttk.Frame):
             window.my_tool_tip.destroy()
 
         self.options_btn = ttk.Menubutton(self.menu_frame, text=_('Options'))
-        self.options_btn.grid(column=1, row=0)
+        self.options_btn.grid(column=1, columnspan=1, row=0)
 
         def sets_enter(event):
             """ The mouse moves into the entry widget """
@@ -199,14 +205,14 @@ class FrameWidgets(ttk.Frame):
 
         ''' About BTN Menu / Label '''
         self.about_btn = ttk.Button(self.menu_frame, text=_('About'), style='Toolbutton', command=info)
-        self.about_btn.grid(column=2, row=0)
+        self.about_btn.grid(column=2, columnspan=2, row=0)
         self.readme = tk.Menu(self)
 
         ''' Update and Tooltip Label '''
         self.updates_lb = ttk.Label(self.menu_frame)
-        self.updates_lb.grid(column=3, row=0, padx=10)
+        self.updates_lb.grid(column=4, columnspan=3, row=0, padx=10)
         '''View Checkbox for enable or disable Tooltip '''
-        if tips:
+        if wg_tips:
             set_tip.set(value=False)
         else:
             set_tip.set(value=True)
@@ -245,11 +251,11 @@ class FrameWidgets(ttk.Frame):
             self.updates_lb.bind('<Leave>', congratulations_leave)
         else:
             set_update.set(value=0)
-            text = _(f'Update {res} available!')
+            text = f'Update {res} ' + _('available!')
 
             ''' Update BTN Menu'''
             self.update_btn = ttk.Menubutton(self.menu_frame, text=text)
-            self.update_btn.grid(column=4, row=0, padx=0)
+            self.update_btn.grid(column=4, columnspan=3, row=0, padx=0)
 
             def download_enter(event):
                 """ The mouse moves into the entry widget """
@@ -266,8 +272,15 @@ class FrameWidgets(ttk.Frame):
             self.download = tk.Menu(self, relief='flat')
 
             self.update_btn.configure(menu=self.download, style='Toolbutton')
-            self.download.add_command(label=_('Download'), command=GiteaUpdate.download(f'https://git.ilunix.de/punix'
-                                                                                        f'/Wire-Py/archive/{res}.zip'))
+            self.download.add_command(label=_('Download'),
+                                      command=lambda: GiteaUpdate.download(f'https://git.ilunix.de/punix'
+                                                                           f'/Wire-Py/archive/'
+                                                                           f'{res}.zip', r'/usr/'
+                                                                                         r'share/icons/wp-icons'
+                                                                                         r'/48/wg_vpn.png',
+                                                                           r'/usr/share/icons/wp-icons'
+                                                                           r'/48/wg_msg.png', res))
+
         ''' Show active Tunnel '''
         self.a = Tunnel.active()
 
@@ -291,6 +304,11 @@ class FrameWidgets(ttk.Frame):
         self.lb_frame3.configure(relief='flat')
         self.lb_frame3.grid(column=0, row=5, columnspan=4, sticky='snew', padx=2, pady=2)
 
+        ''' Bottom Frame 5 '''
+        self.lb_frame4 = ttk.Frame(self)
+        self.lb_frame4.configure(relief='flat')
+        self.lb_frame4.grid(column=2, row=5, columnspan=3, sticky='e', padx=15)
+
         ''' Show active Label '''
         self.select_tunnel = None
         self.lb = ttk.Label(self, text=_('Active: '))
@@ -407,11 +425,11 @@ class FrameWidgets(ttk.Frame):
                     pre_key = key[3]
                 check_call(['nmcli', 'connection', 'delete', select_tl])
                 self.l_box.delete(self.select_tunnel[0])
-                with open(path_to_file2, 'r') as set_file6:
+                with open(wg_set, 'r') as set_file6:
                     lines6 = set_file6.readlines()
-                    if select_tl == lines6[7].strip() and 'off' not in lines6[7].strip():
-                        lines6[7] = 'off'
-                        with open(path_to_file2, 'w') as set_file7:
+                    if select_tl == lines6[7].strip() and 'off\n' not in lines6[7].strip():
+                        lines6[7] = 'off\n'
+                        with open(wg_set, 'w') as set_file7:
                             set_file7.writelines(lines6)
                         self.selected_option.set(0)
                         self.autoconnect_var.set(_('no Autoconnect'))
@@ -474,7 +492,7 @@ class FrameWidgets(ttk.Frame):
                     iw = r'/usr/share/icons/lx-icons/64/info.png'
                     ii = r'/usr/share/icons/wp-icons/48/wg_msg.png'
                     wt = _('Select tunnel')
-                    msg_t = _('Please select a tunnel from the list.')
+                    msg_t = _('Please select a tunnel from the list')
                     msg_window(iw, ii, wt, msg_t)
 
                 else:
@@ -483,7 +501,7 @@ class FrameWidgets(ttk.Frame):
                     iw = r'/usr/share/icons/lx-icons/64/info.png'
                     ii = r'/usr/share/icons/wp-icons/48/wg_msg.png'
                     wt = _('Select tunnel')
-                    msg_t = _('Please first import tunnel.')
+                    msg_t = _('Please first import tunnel')
                     msg_window(iw, ii, wt, msg_t)
 
         ''' Button Trash '''
@@ -548,7 +566,7 @@ class FrameWidgets(ttk.Frame):
             self.btn_exp.bind('<Leave>', exp_leave)
 
         ''' Label Entry '''
-        self.lb_rename = ttk.Entry(self.lb_frame3, width=20)
+        self.lb_rename = ttk.Entry(self.lb_frame4, width=20)
         self.lb_rename.grid(column=2, row=0, padx=8, pady=10, sticky='ne')
         self.lb_rename.insert(0, _('Max. 12 characters!'))
         self.lb_rename.config(state='disable')
@@ -589,7 +607,7 @@ class FrameWidgets(ttk.Frame):
                 iw = r'/usr/share/icons/lx-icons/64/info.png'
                 ii = r'/usr/share/icons/wp-icons/48/wg_msg.png'
                 wt = _('Renaming not possible')
-                msg_t = _('The new name may contain only 12 characters.')
+                msg_t = _('The new name may contain only 12 characters')
                 msg_window(iw, ii, wt, msg_t)
 
             elif len(self.lb_rename.get()) == 0:
@@ -598,7 +616,7 @@ class FrameWidgets(ttk.Frame):
                 iw = r'/usr/share/icons/lx-icons/64/info.png'
                 ii = r'/usr/share/icons/wp-icons/48/wg_msg.png'
                 wt = _('Renaming not possible')
-                msg_t = _('At least one character must be entered.')
+                msg_t = _('At least one character must be entered')
                 msg_window(iw, ii, wt, msg_t)
 
             elif any(ch in special_characters for ch in self.lb_rename.get()):
@@ -629,11 +647,11 @@ class FrameWidgets(ttk.Frame):
                     if self.a != '' and self.a == select_tl:
                         self.a = Tunnel.active()
                         self.StrVar.set(value=self.a)
-                        with open(path_to_file2, 'r') as set_file5:
+                        with open(wg_set, 'r') as set_file5:
                             lines5 = set_file5.readlines()
-                        if select_tl == lines5[7].strip() and 'off' not in lines5[7].strip():
+                        if select_tl == lines5[7].strip() and 'off\n' not in lines5[7].strip():
                             lines5[7] = new_a_connect
-                            with open(path_to_file2, 'w') as theme_set5:
+                            with open(wg_set, 'w') as theme_set5:
                                 theme_set5.writelines(lines5)
                             self.autoconnect_var.set(value=new_a_connect)
 
@@ -645,11 +663,11 @@ class FrameWidgets(ttk.Frame):
                     iw = r'/usr/share/icons/lx-icons/64/info.png'
                     ii = r'/usr/share/icons/wp-icons/48/wg_msg.png'
                     wt = _('Renaming not possible')
-                    msg_t = _('Please select a tunnel from the list.')
+                    msg_t = _('Please select a tunnel from the list')
                     msg_window(iw, ii, wt, msg_t)
 
         ''' Button Rename '''
-        self.btn_rename = ttk.Button(self.lb_frame3, text=_('Rename'), state='disable', command=tl_rename, padding=4,
+        self.btn_rename = ttk.Button(self.lb_frame4, text=_('Rename'), state='disable', command=tl_rename, padding=4,
                                      style='RnButton.TButton')
         self.btn_rename.grid(column=3, row=0, padx=5, pady=10, sticky='ne')
 
@@ -886,10 +904,10 @@ class FrameWidgets(ttk.Frame):
             select_tl = self.l_box.get(select_tunnel[0])
 
             if self.selected_option.get() == 0:
-                with open(path_to_file2, 'r') as set_file3:
+                with open(wg_set, 'r') as set_file3:
                     lines3 = set_file3.readlines()
-                    lines3[7] = 'off'
-                with open(path_to_file2, 'w') as set_file3:
+                    lines3[7] = 'off\n'
+                with open(wg_set, 'w') as set_file3:
                     set_file3.writelines(lines3)
 
                 tl = Tunnel.list()
@@ -898,10 +916,10 @@ class FrameWidgets(ttk.Frame):
                     self.wg_autostart.configure(state='disabled')
 
             if self.selected_option.get() >= 1:
-                with open(path_to_file2, 'r') as set_file3:
+                with open(wg_set, 'r') as set_file3:
                     lines3 = set_file3.readlines()
                     lines3[7] = select_tl
-                with open(path_to_file2, 'w') as set_file3:
+                with open(wg_set, 'w') as set_file3:
                     set_file3.writelines(lines3)
 
         except IndexError:
@@ -915,18 +933,19 @@ class FrameWidgets(ttk.Frame):
         Set (on), the selected tunnel is displayed in the label.
         At (off) the label is first emptied then filled with No Autoconnect
         """
-        with open(path_to_file2, 'r') as set_file4:
+        with open(wg_set, 'r') as set_file4:
             lines4 = set_file4.readlines()
 
-        if lines4[7] != 'off':
+        if lines4[7] != 'off\n':
+            print(lines4[7])
             self.selected_option.set(1)
             self.autoconnect_var.set('')
             self.auto_con = lines4[7]
 
         else:
-
-            self.wg_autostart.configure(state='disabled')
+            self.selected_option.set(0)
             self.auto_con = _('no Autoconnect')
+
         self.autoconnect_var.set('')
         self.autoconnect_var = tk.StringVar()
         self.autoconnect_var.set(self.auto_con)
@@ -1021,7 +1040,7 @@ class FrameWidgets(ttk.Frame):
     def color_label(self):
         """ View activ Tunnel in color green or yellow """
 
-        with open(path_to_file2, 'r') as read_file:
+        with open(wg_set, 'r') as read_file:
             lines = read_file.readlines()
             if 'light\n' in lines:
                 self.lb_tunnel = ttk.Label(self, textvariable=self.StrVar, foreground='green')
@@ -1083,7 +1102,7 @@ class FrameWidgets(ttk.Frame):
                 iw = r'/usr/share/icons/lx-icons/64/info.png'
                 ii = r'/usr/share/icons/wp-icons/48/wg_msg.png'
                 wt = _('Select tunnel')
-                msg_t = _('Please select a tunnel from the list.')
+                msg_t = _('Please select a tunnel from the list')
                 msg_window(iw, ii, wt, msg_t)
 
             else:
@@ -1092,14 +1111,14 @@ class FrameWidgets(ttk.Frame):
                 iw = r'/usr/share/icons/lx-icons/64/info.png'
                 ii = r'/usr/share/icons/wp-icons/48/wg_msg.png'
                 wt = _('Select tunnel')
-                msg_t = _('Please first import tunnel.')
+                msg_t = _('Please first import tunnel')
                 msg_window(iw, ii, wt, msg_t)
 
 
 class MyToolTip(tk.Toplevel):
     TIP_X_OFFSET = 20
     TIP_Y_OFFSET = 20
-    if not tips:
+    if not wg_tips:
         AUTO_CLEAR_TIME = 0
     else:
         AUTO_CLEAR_TIME = 2000  # Millisecond. (1/200 sec.)