r/redditdev

[SOLVED] I've been using this code to update user flairs:

def update_flair(username, new_line):
    flair = next(subreddit.flair(redditor=username), None)

    if flair and flair.get("flair_text") is not None:
        base_text = flair.get("flair_text") or ""
        template_id = flair.get("flair_template_id")
    else:
        base_text = ""
        template_id = DEFAULT_FLAIR_TEMPLATE_ID

    lines = [
        l for l in base_text.split("\n")
        if not l.lower().startswith("streak -")
    ]

    lines.append(new_line)
    final_text = "\n".join(lines)

    subreddit.flair.set(
        username,
        text=final_text,
        flair_template_id=template_id
    )

It's part of a larger code for a bot that I'm working on. I'm attaching the expectations vs outcome in the comments. Please let me know what went wrong. Thank you :)

Edit - I've finally managed to solve the issue. The problem is with subreddit.flair(redditor=username). It returns an object like {'flair_css_class': None, 'user': Redditor(name='username'), 'flair_text': 'text'}. It does NOT contain flair_template_id. So, the solution is to get the flair text from the flair first. Then, get all the user flairs. These include both flair text & template ID. Then, go through the list until you find the flair which has the same flair text as yours & get its ID. This is the work-around that I've found. The complete code is given below:

def get_template_text(subreddit, template_id):
    for tpl in subreddit.flair.templates:
        if tpl["id"] == template_id:
            return tpl.get("text") or ""
    return ""

def get_user_flair(subreddit, username):
    try:
        redditor = reddit.redditor(username)
        flair = next(subreddit.flair(redditor=redditor), None)
        return flair
    except Exception as e:
        print("Error fetching flair:", e)
        return None
    
def get_template_id_from_flair_text(subreddit, flair_text):
    if not flair_text:
        return None

    for tpl in subreddit.flair.templates:
        tpl_text = (tpl.get("text") or "").strip()
        if tpl_text == flair_text:
            return tpl["id"]
    return None

def update_flair(username, new_line):
    flair = get_user_flair(subreddit, username)

    if flair:
        base_text = flair.get("flair_text") or ""
        template_id = flair.get("flair_template_id")
        if template_id is None:
            template_id = get_template_id_from_flair_text(subreddit, base_text)

        if template_id is None:
            template_id = DEFAULT_FLAIR_TEMPLATE_ID
            base_text = get_template_text(subreddit, template_id)
        subreddit.flair.delete(username)
    else:
        template_id = DEFAULT_FLAIR_TEMPLATE_ID
        base_text = get_template_text(subreddit, template_id)

    lines = [
        l for l in base_text.split("\n")
        if not l.lower().startswith("streak -")
    ]
    lines.append(new_line)
    final_text = "\n".join(lines)

    subreddit.flair.set(
        username,
        flair_template_id=template_id
    )

    subreddit.flair.set(
        username,
        flair_template_id=template_id,
        text=final_text
    )