Obtained Endings Screen Iteration Not Working [Solved]

[chucklabs]

Hello! I'm having trouble with my obtained endings screen. I used this viewtopic.php?t=29242 post as reference, with some modified code.

My current code is:

Code: Select all

screen endings:
    tag menu
    use navigation

    $ all_endings = [(1, "firedOne", "Bad End 1 - Fired"), (2, "firedTwo", "Bad End 2 - Fired")]


    vbox:
        python:
            for i in (all_endings):
                ending_number = i[0]
                ending_name = i[1]
                ending_title = i[2]
                ending_string = "% 2d. ---------------------------------" % (ending_number)
                if ending_name in persistent.endings:
                    ending_string = "% 2d. %s" % (ending_number, ending_title)
                print ending_string

However, I'm getting the error that 'arument type of 'NoneType' is not iterable.' referring to the line "

Code: Select all

if ending_name in persistent.endings:

I'm only receiving this error when I navigate to the endings menu.

[hell_oh_world]

In your shared link there's a line that declares this persistent.endings.

Code: Select all

init python:
    
    # If persistent.endings is None (the first pass through the game), then make it a set.
    if persistent.endings is None:
        persistent.endings = set()

Also I would prefer, persistent.endings to be a list (persistent.endings = []) instead of a set (persistent.endings = set()) I guess. so you would use .append instead .add when adding an ending name and use .remove instead of .discard when removing an ending name. But that depends, if you don't want any duplicate endings then you're right about using a set, list doesn't care for duplicate items.

[chucklabs]

In your shared link there's a line that declares this persistent.endings.

Code: Select all

init python:
    
    # If persistent.endings is None (the first pass through the game), then make it a set.
    if persistent.endings is None:
        persistent.endings = set()

Also I would prefer, persistent.endings to be a list (persistent.endings = []) instead of a set (persistent.endings = set()) I guess. so you would use .append instead .add when adding an ending name and use .remove instead of .discard when removing an ending name. But that depends, if you don't want any duplicate endings then you're right about using a set, list doesn't care for duplicate items.

No more error now, thanks!