yahoo does not contain art, mascot or emblem for News agg

Bug #1600026 reported by Kyle Nitzsche on 2016-07-07
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
News Scope
Undecided
Unassigned
Ubuntu Rest Scopes
Undecided
Alejandro J. Cura

Bug Description

Yahoo is aggregated by the News scope. It expects a result to have an "art" attribute.

https://developer.ubuntu.com/en/phone/scopes/guides/integration-aggregator-scopes/

(I think it used to have art...)

There is no art attribute currently. Here's a yahoo result output directly during aggregation without modification:
{
    "attrs": {
        "attributes": [
            {
                "value": "44.34"
            },
            {
                "value": "-4.79"
            },
            {
                "value": "-9.75%"
            }
        ],
        "change_p": "-9.75%",
        "change_v": "-4.79",
        "dnd_uri": "http://finance.yahoo.com/q?s=FSLR",
        "graph": "http://chart.finance.yahoo.com/z?s=FSLR&z=s&t=7d",
        "result_json": "{\"symbol\":\"FSLR\",\"attributes\":[{\"value\":\"44.34\"},{\"value\":\"-4.79\"},{\"value\":\"-9.75%\"}],\"dnd_uri\":\"http://finance.yahoo.com/q?s=FSLR\",\"title\":\"First Solar, Inc.\",\"graph\":\"http://chart.finance.yahoo.com/z?s=FSLR&z=s&t=7d\",\"cat_id\":\"trending\",\"change_v\":\"-4.79\",\"uri\":\"http://finance.yahoo.com/q?s=FSLR\",\"value\":\"44.34\",\"change_p\":\"-9.75%\"}\n",
        "symbol": "FSLR",
        "title": "First Solar, Inc.",
        "uri": "http://finance.yahoo.com/q?s=FSLR",
        "value": "44.34"
    },
    "internal": {
        "cat_id": "trending",
        "origin": "ipc:///run/user/32011/zmq/priv/SSRegistry#com.canonical.scopes.yahoofinance!c=Scope!t=500"
    }
}

I suppose you could add art and let it be graph attribute's value. This should be innocuous if yahoo scope does not use art attribute itself.

Currently, the lack of art attribute causes News scope exceptions, but that is bug being fixed: https://bugs.launchpad.net/scope-aggregator/+bug/1599948

News scope could easily look in another field (graph) for fallback art, but that would be an exception for this one child scope.

Changed in news-scope:
status: New → Confirmed
Changed in ubuntu-rest-scopes:
assignee: nobody → James Henstridge (jamesh)
Changed in news-scope:
status: Confirmed → Invalid
Changed in ubuntu-rest-scopes:
assignee: James Henstridge (jamesh) → Alejandro J. Cura (alecu)
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers