Merge lp:~phablet-team/dialer-app/fix_prefix_settings_saving into lp:dialer-app/staging

Proposed by Gustavo Pichorim Boiko
Status: Merged
Approved by: Tiago Salem Herrmann
Approved revision: 561
Merged at revision: 561
Proposed branch: lp:~phablet-team/dialer-app/fix_prefix_settings_saving
Merge into: lp:dialer-app/staging
Diff against target: 19 lines (+5/-5)
1 file modified
src/qml/SettingsPage/AccountSettings/SipNumberRewrite.qml (+5/-5)
To merge this branch: bzr merge lp:~phablet-team/dialer-app/fix_prefix_settings_saving
Reviewer Review Type Date Requested Status
Tiago Salem Herrmann (community) Approve
Review via email: mp+321995@code.launchpad.net

Commit message

Only try to assing previously set values if they are available.

Description of the change

Only try to assing previously set values if they are available.

To post a comment you must log in.
Revision history for this message
Tiago Salem Herrmann (tiagosh) wrote :

Looks good. thanks.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/qml/SettingsPage/AccountSettings/SipNumberRewrite.qml'
--- src/qml/SettingsPage/AccountSettings/SipNumberRewrite.qml 2017-03-22 17:08:12 +0000
+++ src/qml/SettingsPage/AccountSettings/SipNumberRewrite.qml 2017-04-05 13:45:43 +0000
@@ -48,11 +48,11 @@
48 }48 }
4949
50 updating = true50 updating = true
51 numberRewriteSwitch.checked = props.numberRewrite51 numberRewriteSwitch.checked = props.numberRewrite ? props.numberRewrite : false
52 countryCodeField.text = props.defaultCountryCode52 countryCodeField.text = props.defaultCountryCode ? props.defaultCountryCode : ""
53 areaCodeField.text = props.defaultAreaCode53 areaCodeField.text = props.defaultAreaCode ? props.defaultAreaCode : ""
54 removeInputField.text = props.removeCharacters54 removeInputField.text = props.removeCharacters ? props.removeCharacters : ""
55 prefixInputField.text = props.prefix55 prefixInputField.text = props.prefix ? props.prefix : ""
56 updating = false56 updating = false
57 }57 }
5858

Subscribers

People subscribed via source and target branches

to all changes: