Properly cast `authenticated` field for subscription_attribute table

[Eygene Ryabinkin]

authenticated is an integer in SQL schema, so it should be properly cast: for example, PostgreSQL is picky about this.

[Ryan J Ollos]

This patch looks correct to me. authenticated is an INT in the database schema (​t:TracDev/DatabaseSchema) and is a bool in the ​Session class, where it is ​cast to int before writing to the database.

[Ryan J Ollos]

#10191 closed as a duplicate. This ticket was kept open since it has a patch. #10191 will be useful for confirming that the same traceback is seen when reproducing the issue.

[Ryan J Ollos]

#9247 closed as a duplicate.

[Bas van der Vlies]

bas: Are you also running PostgreSQL?

Yes

[anonymous]

See also #8065

[Ryan J Ollos]

#8065 closed as a duplicate, and it has a patch which we may want to review.

[Steffen Hoffmann]

(In [12302]) TracAnnouncer: Part 7 of 7 - Finally: Go from present to future, refs #5774, #7975, #8065, #9742 and #10384.

Now we've bridged the gap and provide an upgrade path for each historic schema revision of this plugin, while data migration is incomplete yet, especially regarding subscription attributes stored in session_attribute (before v3). Due to component name changes the conversion will be rather complicated, and therefore corresponding research and development is postponed to a future date and will largely depend on explicite demand as well.

[Steffen Hoffmann]

(In [12303]) TracAnnouncer: Convert type to match db table definitions, refs #7975, #8065 and #10384.

These changes are based on work by olistudent, Stephen Anderson and rea. I made sure, that we respect PEP8 as well, at least as far as Trac core does.

Thanks to all of you for testing, reports and suggestions towards a portable fix, and - ultimately - patience to get it finally resolved.