fix optimisation with super session that may lead to integrity loss at some point I've decided to stop ensuring ?1 cardinality was respected when adding a new relation using a super session, to avoid the cost of the delete query. That was yet discussable because it introduced unexpected difference between execute and unsafe_execute, which is imo not worth it. Also, now that rql() in migration script default to unsafe_execute, we definitly don't want that implicit behaviour change (which already cause bug when for instance adding another default workflow for an entity type: without that fix we end up with *two* default workflows while the schema tells we can have only one. IMO we should go to the direction that super session skip all security check, but nothing else, unless explicitly asked.

     1

# type attribute might already be there if migrating from

     2

# version < 3.5 to version >= 3.5.3, BaseTransition being added

     3

# in bootstrap_migration

     4

if versions_map['cubicweb'][0] >= (3, 5, 0):

     5

    add_attribute('BaseTransition', 'type')

     6

    sync_schema_props_perms('state_of')

     7

    sync_schema_props_perms('transition_of')