From 2e342b50c88722fb5d9511a180486bc0fe231519 Mon Sep 17 00:00:00 2001 From: Alvaro Neira Ayuso Date: Mon, 13 Jan 2020 11:37:14 +0100 Subject: Modify methods to use less arguments Now, all the arguments are received from httpparser. Those arguments convert the function in long lines of codes. Passing directly the httpparser, all the function will have less arguments and will be more clear the code. --- src/linux/ogOperations.py | 30 +++++++++++++++++++++++++----- 1 file changed, 25 insertions(+), 5 deletions(-) (limited to 'src/linux') diff --git a/src/linux/ogOperations.py b/src/linux/ogOperations.py index bbdfa3d..5c136fa 100644 --- a/src/linux/ogOperations.py +++ b/src/linux/ogOperations.py @@ -15,7 +15,8 @@ def reboot(): else: subprocess.call(['/sbin/reboot']) -def execCMD(cmd): +def execCMD(httpparser): + cmd = httpparser.getCMD() cmds = cmd.split(" ") try: result = subprocess.check_output(cmds) @@ -24,11 +25,17 @@ def execCMD(cmd): return result.decode('utf-8') -def procsession(disk, partition): +def procsession(httpparser): + disk = httpparser.getDisk() + partition = httpparser.getPartition() + result = subprocess.check_output([OG_PATH + 'interfaceAdm/IniciarSesion', disk, partition], shell=True) return result.decode('utf-8') -def procsoftware(disk, partition, path): +def procsoftware(httpparser, path): + disk = httpparser.getDisk() + partition = httpparser.getPartition() + result = subprocess.check_output([OG_PATH + 'interfaceAdm/InventarioSoftware', disk, partition, path], shell=True) return result.decode('utf-8') @@ -36,11 +43,24 @@ def prochardware(path): result = subprocess.check_output([OG_PATH + 'interfaceAdm/InventarioHardware', path], shell=True) return result.decode('utf-8') -def procsetup(disk, cache, cachesize, partlist): +def procsetup(httpparser): + disk = httpparser.getDisk() + cache = httpparser.getCache() + cachesize = httpparser.getCacheSize() + partlist = httpparser.getPartitionSetup() + for part in partlist: cfg = 'dis=' + disk + '*che=' + cache + '*tch=' + cachesize + '!par=' + part["partition"] + '*cpt='+part["code"] + '*sfi=' + part['filesystem'] + '*tam=' + part['size'] + '*ope=' + part['format'] + '%' subprocess.check_output([OG_PATH + 'interfaceAdm/Configurar', disk, cfg], shell=True) -def procirestore(disk, partition, name, repo, ctype, profile, cid): +def procirestore(httpparser): + disk = httpparser.getDisk() + partition = httpparser.getPartition() + name = httpparser.getName() + repo = httpparser.getRepo() + ctype = httpparser.getType() + profile = httpparser.getProfile() + cid = httpparser.getId() + result = subprocess.check_output([OG_PATH + 'interfaceAdm/RestaurarImagen', disk, partition, name, repo, ctype], shell=True) return result.decode('utf-8') -- cgit v1.2.3-18-g5258