mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2024-11-28 05:00:20 +00:00
Code Issues Projects Releases Packages Wiki Activity

mobile-widgets/qml: remove use of eval()

Browse source 
eval() cannot be used in combination with the Qt5 qml compiler,

replace eval() with switch statements.

Signed-off-by: Jan Iversen <jani@apache.org>
This commit is contained in:
jan Iversen 2019-12-22 18:43:47 +01:00 committed by Dirk Hohndel
parent 636b26ff4b
commit 9d74b4dc82
1 changed files with 51 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

59
mobile-widgets/qml/DownloadFromDiveComputer.qml
View file

@ -162,21 +162,47 @@ Kirigami.Page {
elide: Text.ElideRight
}
onCurrentTextChanged: {
var curVendor
var curProduct
var curDevice
dc1.enabled = dc2.enabled = dc3.enabled = dc4.enabled = true
for (var i = 1; i < 5; i++) {
switch (i) {
case 1:
curVendor = PrefDiveComputer.vendor1
curProduct = PrefDiveComputer.product1
curDevice = PrefDiveComputer.device1
break
case 2:
curVendor = PrefDiveComputer.vendor2
curProduct = PrefDiveComputer.product2
curDevice = PrefDiveComputer.device2
break
case 3:
curVendor = PrefDiveComputer.vendor3
curProduct = PrefDiveComputer.product3
curDevice = PrefDiveComputer.device3
break
case 4:
curVendor = PrefDiveComputer.vendor4
curProduct = PrefDiveComputer.product4
curDevice = PrefDiveComputer.device4
break
}
if (comboProduct.currentIndex === -1 && currentText === "FTDI"){
if ( eval("PrefDiveComputer.vendor" + i) === comboVendor.currentText && eval("PrefDiveComputer.device" + i).toUpperCase() === currentText)
rememberedDCsGrid.setDC(eval("PrefDiveComputer.vendor" + i), eval("PrefDiveComputer.product" + i), ("PrefDiveComputer.device" + i))
if ( curVendor === comboVendor.currentText && curDevice.toUpperCase() === currentText)
rememberedDCsGrid.setDC(curVendor, curProduct, curDevice)
}else if (comboProduct.currentIndex !== -1 && currentText === "FTDI") {
if ( eval("PrefDiveComputer.vendor" + i) === comboVendor.currentText && eval("PrefDiveComputer.product" + i) === comboProduct.currentText && eval("PrefDiveComputer.device" + i).toUpperCase() === currentText) {
eval("dc"+ i + ".enabled = false")
if ( curVendor === comboVendor.currentText && cyrProduct === comboProduct.currentText && curDevice.toUpperCase() === currentText) {
disableDC(i)
break
}
}else if ( eval("PrefDiveComputer.vendor" + i) === comboVendor.currentText && eval("PrefDiveComputer.product" + i) === comboProduct.currentText && eval("PrefDiveComputer.product" + i) +" " + eval("PrefDiveComputer.device" + i) === currentText) {
eval("dc"+ i + ".enabled = false")
}else if ( curVendor === comboVendor.currentText && curProduct === comboProduct.currentText && curProduct +" " + curDevice === currentText) {
disableDC(i)
break
}else if ( eval("PrefDiveComputer.vendor" + i) === comboVendor.currentText && eval("PrefDiveComputer.product" + i) === comboProduct.currentText && eval("PrefDiveComputer.device" + i) === currentText) {
eval("dc"+ i + ".enabled = false")
}else if ( curVendor === comboVendor.currentText && curProduct === comboProduct.currentText && curDevice === currentText) {
disableDC(i)
break
}
}
@ -202,6 +228,23 @@ Kirigami.Page {
comboProduct.currentIndex = comboProduct.find(product);
comboConnection.currentIndex = manager.getConnectionIndex(device);
}
function disableDC(inx) {
switch (inx) {
case 1:
dc1.enabled = false
break;
case 2:
dc2.enabled = false
break;
case 3:
dc3.enabled = false
break;
case 4:
dc4.enabled = false
break;
}
}
SsrfButton {
id: dc1
visible: PrefDiveComputer.vendor1 !== ""