diff options
author | Joerg Bornemann <joerg.bornemann@nokia.com> | 2012-03-22 11:07:23 +0100 |
---|---|---|
committer | Joerg Bornemann <joerg.bornemann@nokia.com> | 2012-03-22 11:50:30 +0100 |
commit | 8fc9ac73ef4d14bbb83f72fbe0dea939b5c0a9c3 (patch) | |
tree | e912489dae9cb0cd9f9ad795ba895d0028a69069 | |
parent | d69c9fd1898b7ce8916e91a75970dad910053e85 (diff) |
fix default parameter of configurationValue(.)
Change-Id: I47cceb835d3f6dbd0b59d56814e98f0dbebd879b
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@nokia.com>
-rw-r--r-- | src/lib/language/loader.cpp | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/src/lib/language/loader.cpp b/src/lib/language/loader.cpp index 19df5a53a..3264680f2 100644 --- a/src/lib/language/loader.cpp +++ b/src/lib/language/loader.cpp @@ -2539,7 +2539,12 @@ QScriptValue Loader::js_configurationValue(QScriptContext *context, QScriptEngin Settings::Ptr settings = Loader::get(engine)->m_settings; const bool defaultValueProvided = context->argumentCount() > 1; const QString key = context->argument(0).toString(); - const QString defaultValue = (defaultValueProvided ? QString() : (context->argument(1).isUndefined() ? QString() : context->argument(1).toString())); + QString defaultValue; + if (defaultValueProvided) { + const QScriptValue arg = context->argument(1); + if (!arg.isUndefined()) + defaultValue = arg.toString(); + } QVariant v = settings->value(key, defaultValue); if (!defaultValueProvided && v.isNull()) return context->throwError(QScriptContext::SyntaxError, |