sync_schema_simulate when used with triggers about sqlite_orm HOT 11 CLOSED

PR is here #1281

from sqlite_orm.

Sure:

            auto storage = sqlite_orm::make_storage(
                path,
                sqlite_orm::make_virtual_table(
                "test", 
                sqlite_orm::using_fts5(
                    sqlite_orm::make_column("X", &Test::X))
            )
            );

            auto simulation = storage.sync_schema_simulate(false);

from sqlite_orm.

PR is here #1283

from sqlite_orm.

@trevornagy please check dev branch

from sqlite_orm.

That fixes it, will close this issue now. Thanks!

from sqlite_orm.

@trevornagy what kind of error you get?

from sqlite_orm.

Updated the original bug with the error, sorry meant to include that to begin with.

from sqlite_orm.

oh kurwa, it is a bug! Looks like we never using triggers with sync_schema_simulate. It turned out that schema_status override for trigger_t doesn't exist. Let me fix it fast

from sqlite_orm.

merged. Please check

from sqlite_orm.

@fnc12 Works for triggers, but just noticed this same issue happens with virtual tables (created with FTS5)

Should I create a new issue or should we reuse this?

from sqlite_orm.

@trevornagy this issue is enough. I reopened it. feel free to reopen issues by yourself if you are sure something more has to be committed to the code base to fix the issue. Could you share repro code please?

from sqlite_orm.