Column definition not parsed in class Table about piccolo

[dantownsend]

From the DDL for creating the table that you posted:

['CREATE TABLE IF NOT EXISTS "users" ("id" SERIAL PRIMARY KEY NOT NULL, "username" VARCHAR(255) UNIQUE NOT NULL DEFAULT \'\', "email" VARCHAR(255) UNIQUE NOT NULL DEFAULT \'\')']

It looks like it just created the id, username and email columns - not the secret and bio columns.

Maybe try deleting the table from the database, and try it again?

Also, be careful with this User.create_table(if_not_exists=True) it doesn't do anything unless you do the following:

await User.create_table(if_not_exists=True)
# OR
User.create_table(if_not_exists=True).run_sync()

[bedi7]

After changed names from User to Users on fresh database, the same issue ;(.

class Users(Table, db=DB):
    # id = BigSerial(primary_key=True, unique=True, auto_update=True),
    secret = Varchar(length=255, secret=True),
    bio = Varchar(length=255),
    username = Varchar(length=255, unique=True)
    email = Email(length=255, unique=True)

[bedi7]

Debug status from piccolo/table.py in Table.init_subclass

for column bio:

for column username:

I haven't idea why bio object is not recognized as Column class instance if Varchar has the same instance id in both checks.

[dantownsend]

I think I see your problem - there's commas after some of the columns:

class Users(Table, db=DB):
    # id = BigSerial(primary_key=True, unique=True, auto_update=True),
    secret = Varchar(length=255, secret=True),   # <- here
    bio = Varchar(length=255),   # <- here
    username = Varchar(length=255, unique=True)
    email = Email(length=255, unique=True)

So Python is treating secret and bio as tuples, instead of just columns.

[bedi7]

Thank you so much for finding such an ugly bug
I continue to play with piccolo-orm