[herniqeu]

Problem

create_group.py has two handlers. The second one binds the exception as serverError:

except ServerResponseError as serverError:
    if serverError.code == "409011":
        print(f"user {user.name} is already a member of group {group.name}")
    else:
        raise rError                      # <- name from the first handler

rError is the name bound by the first handler, earlier in the function. Python deletes the exception name at the end of an except block, so at this point rError does not exist and every failure other than 409011 becomes:

NameError: name 'rError' is not defined

which is precisely the diagnostic the sample should be printing.

Fix

raise serverError — re-raise the error actually caught here. The first handler keeps raise rError (valid: it is inside its own except block).

Verification

python3 -m pyflakes samples/create_group.py

• before: undefined name 'rError' (line 83)
• after: gone, and no new undefined name

---

Achado com analise estatica (pyflakes) e leitura do chamador; verificado antes/depois do patch. Commit com Signed-off-by.

[salesforce-cla]

Thanks for the contribution! Before we can merge this, we need @herniqeu to sign the Salesforce Inc. Contributor License Agreement.