Cliffhanger should gracefully hide fields which do not have data

Bug #1253039 reported by Nekhelesh Ramananthan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Flashback
Fix Released
High
Arash Badie Modiri

Bug Description

Currently it seems that when the API does not a return a particular field, the UI sometimes displays an invalid string which is not pretty. We have to lose at these cases.

Steps to reproduce this:

1. Go to Movies Tab
2. Search for the movie "The snow queen"
3. Press the first result
4. In the console, you will notice errors such as "Unable to assign void* to QString".

1. Go to Tv Tab
2. In the Popular carousel, look the tv show "Doctor Who" and click it.
3. In the Tv details page, you will notice the minutes appearing as "undefined minutes", seasons "Qvariant(void*) Seasons" etc etc.

All these should gracefully fall back properly so that the user does not notice this.

Related branches

Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

Hey arash, I assigned this bug to you since you are more familiar with the json functions which requires a little bit more tweaking to address this bug.

Revision history for this message
Arash Badie Modiri (arashbm) wrote :

I don't have slightest idea about why JSON.parse is acting so viewred... it returns QT pointers (void*) instead of undefined.

js:
```
a = JSON.parse("{}")
a.notKey // => undefined
```

Qt:
```
a = JSON.parse("{}")
a.notKey // => Qvariant(void*)
```

Changed in cliffhanger:
status: Triaged → In Progress
Revision history for this message
Arash Badie Modiri (arashbm) wrote :

viewred => weird (how did I type that?!)

Changed in cliffhanger:
status: In Progress → Fix Committed
Changed in cliffhanger:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.