on xenial help sign in
refactor the modem handling up to allow for language and country detection from other pages
commit0ffae8d8a2fc851f7ff8f0e9e826d5fdde66c9d9 authorLukáš Tinkl <lukas.tinkl@canonical.com> dateFri 3 Jul 2015, 15:14 (11 years ago) parent3c627b7
2 files changed+32−22
qml/Wizard/Page.qml+25 qml/Wizard/Pages/20-sim.qml+7−22
qml/Wizard/Page.qml+25
@@ -16,6 +16,7 @@
1616
1717import QtQuick 2.3
1818import Ubuntu.Components 1.2
19+import MeeGo.QOfono 0.2
1920
2021Item {
2122 readonly property real buttonMargin: units.gu(3)
@@ -52,6 +53,30 @@ Item {
5253 visible: false
5354 anchors.fill: parent
5455
56+ property alias modemManager: modemManager
57+ property alias simManager0: simManager0
58+ property alias simManager1: simManager1
59+
60+ OfonoManager { // need it here for the language and country detection
61+ id: modemManager
62+ property bool ready: false
63+ onModemsChanged: {
64+ ready = true
65+ }
66+ }
67+
68+ // Ideally we would query the system more cleverly than hardcoding two
69+ // sims. But we don't yet have a more clever way. :(
70+ OfonoSimManager {
71+ id: simManager0
72+ modemPath: modemManager.modems.length >= 1 ? modemManager.modems[0] : ""
73+ }
74+
75+ OfonoSimManager {
76+ id: simManager1
77+ modemPath: modemManager.modems.length >= 2 ? modemManager.modems[1] : ""
78+ }
79+
5580 // title
5681 Image {
5782 id: titleRect
qml/Wizard/Pages/20-sim.qml+7−22
@@ -29,10 +29,10 @@ LocalComponents.Page {
2929 customTitle: true
3030 hasBackButton: false
3131
32− skipValid: !manager.available ||
33− (manager.ready && (manager.modems.length < 1 || simManager0.ready)
34− && (manager.modems.length < 2 || simManager1.ready))
35− skip: !manager.available || manager.modems.length === 0 || simManager0.present || simManager1.present
32+ skipValid: !modemManager.available ||
33+ (modemManager.ready && (modemManager.modems.length < 1 || simManager0.ready)
34+ && (modemManager.modems.length < 2 || simManager1.ready))
35+ skip: !modemManager.available || modemManager.modems.length === 0 || simManager0.present || simManager1.present
3636
3737 property bool hadModem: false
3838
@@ -57,30 +57,15 @@ LocalComponents.Page {
5757 }
5858 }
5959
60− OfonoManager {
61− id: manager
62− property bool ready: false
60+ Connections {
61+ target: modemManager
6362 onModemsChanged: {
64− print("Modems changed " + modems)
65− ready = true
6663 if (!hadModem && (simManager0.present || simManager1.present)) { // show the restart dialog in case a SIM gets inserted
67− PopupUtils.open(restartDialog)
64+ restartDialog.visible = true
6865 }
6966 }
7067 }
7168
72− // Ideally we would query the system more cleverly than hardcoding two
73− // sims. But we don't yet have a more clever way. :(
74− OfonoSimManager {
75− id: simManager0
76− modemPath: manager.modems.length >= 1 ? manager.modems[0] : ""
77− }
78−
79− OfonoSimManager {
80− id: simManager1
81− modemPath: manager.modems.length >= 2 ? manager.modems[1] : ""
82− }
83−
8469 Column {
8570 anchors.fill: content
8671 spacing: units.gu(4)