For D7 we are now using '' (empty string) as default actions parameters. This issue hope to redefine this empty string as 'default'.

As mentioned in #334687: [DBTNG + XDB] NOT NULL fields using a DEFAULT '' clause , '' (empty string) is not always functioning for all database engine, e.g. Oracle will translate empty string as NULL automatically, which is not ANSI standard. It is also not too meaningful if using empty string as undefined default value. Use 'default' seems to be a possible replacement.

CommentFileSizeAuthor
actions-parameters-default-1227118569.patch4.18 KBhswong3i

Comments

damien tournoud’s picture

damien tournoud commented
Status: Needs review » Closed (won't fix)

As per chx' analysis in #334687: [DBTNG + XDB] NOT NULL fields using a DEFAULT '' clause , which I totally agree with.

hswong3i’s picture

hswong3i commented
Status: Closed (won't fix) » Needs review

I just give a reply for original issue (http://drupal.org/node/334687#comment-1116672) and hopefully may trigger more open end discussion.

catch’s picture

catch
he/him
Primary language English
commented
Status: Needs review » Closed (duplicate)

Effectively a duplicate of #336491: Use 'default' instead of '' as default user name and password since this is a matter of code style/policy rather than the specific implementation. Marking duplicate of that won't fix issue.