Add check for index and constraint name length
Extracted from old TODO
- Show closed items
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- Cédric Krier added trytond type::behavior + 1 deleted label
added trytond type::behavior + 1 deleted label
- Author Owner
Indeed for PostgreSQL the limit is 64.
But probably the name of the constraint should be generated per the database backend. - Cédric Krier added 1 deleted label and removed 1 deleted label
added 1 deleted label and removed 1 deleted label
- Author Owner
Indeed the limit is 63. I propose to use the hexdigest from sha256 and remove the last char.
The main issue is how to manage __raise_integrity_error. At some point the code is already not very safe by testing constraint name in the exception. - Cédric Krier assigned to @ced
assigned to @ced
- Cédric Krier added 1 deleted label and removed 1 deleted label
added 1 deleted label and removed 1 deleted label
- Author Owner
Here is review37381003
I did not changed the table name, primary key nor sequence because they can be fixed if needed in the configuration file (and also it is better to keep their meaning). - Cédric Krier added 1 deleted label and removed 1 deleted label
added 1 deleted label and removed 1 deleted label
review37381003 updated at https://codereview.tryton.org/37381003/#ps1
review37381003 updated at https://codereview.tryton.org/37381003/#ps20001
New changeset ada310e4e733 by C?dric Krier in branch 'default':
Convert into hash too long data name
http://hg.tryton.org/trytond/rev/ada310e4e733- Roundup Robot added 1 deleted label and removed 1 deleted label
added 1 deleted label and removed 1 deleted label
- Roundup Robot closed
closed
New changeset 4c039aa7e8c9 by C?dric Krier in branch '4.4':
Convert into hash too long data name
http://hg.tryton.org/trytond/rev/4c039aa7e8c9- Cédric Krier mentioned in issue #6026 (closed)
mentioned in issue #6026 (closed)
- Cédric Krier mentioned in issue #6494 (closed)
mentioned in issue #6494 (closed)
- Cédric Krier mentioned in issue #6965 (closed)
mentioned in issue #6965 (closed)