Code review comment for lp:~samuel-buffet/entertainer/bug_380745

Revision history for this message
Samuel Buffet (samuel-buffet) wrote :

Devs,

Here is a proposed fixed for the linked bug.

In a few words, the bug was due to the fact that the movie screen was not handling the possibility to receive empty list for genres, actors, directors or writters.

Those cases are now handled and "Unknown" will be displayed instead of a nice traceback.

I've also taken the occasion to improve variable names. No more dlist or wri_title ...

Samuel-

« Back to merge proposal