missing example for the department scope

[XiaoGuo, Liu]

       // Open weather map API error code can either be a string or int
        QVariant cod = root.toVariant().toMap()["cod"];
        if ((cod.canConvert<QString>() && cod.toString() != "200")
                || (cod.canConvert<unsigned int>() && cod.toUInt() != 200)) {
            throw domain_error(root.toVariant().toMap()["message"].toString().toStdSt
ring());
        }

In fact, there is no "cod" field at all.
Reported from: http://developer.ubuntu.com/scopes/tutorials/adding-departments-to-your-scope/

[David Callé (davidc3)]

Hmm, the template has probably changed since the tutorial was made. I guess it's crashing the example if the cod field is missing, right?

[XiaoGuo, Liu (liu-xiao-guo)]

In fact, it does not crash since it is not convertible. The thing is that the code should not be there which makes the code not so good :)

[David Callé (davidc3)]

Just checked, and there is a "cod" field in the response : http://api.openweathermap.org/data/2.5/forecast/daily?q=London%2Cuk&units=metric&cnt=7

Or maybe you are talking about a different scope or tutorial?

[XiaoGuo, Liu (liu-xiao-guo)]

Hi David,

Yes, you are right. Maybe this is the one:

http://bazaar.launchpad.net/~davidc3/ubuntu-sdk-tutorials/scope-tutorial-ifixit-departments/view/head:/src/api/client.cpp

In fact, a lot of developers may just simply copy the code, and they do to really check the part of the code. We probably need to review rest of the tutorials as well.

best regards,
Xiaoguo

[David Callé (davidc3)]

Unless this is needed before next week, this will be fixed once we have moved out of Wordpress. Changing code snippets (even line-numbering, like in this case) can break the whole page in unexpected ways. I'd prefer to avoid that during these last days of the scopes contest.