Use 'default' instead of '' as default actions parameters

hswong3i - November 19, 2008 - 18:27
Project:Drupal
Version:7.x-dev
Component:base system
Category:feature request
Priority:normal
Assigned:hswong3i
Status:duplicate
Description

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.

AttachmentSizeStatusTest resultOperations
actions-parameters-default-1227118569.patch4.18 KBIdlePassed: 7252 passes, 0 fails, 0 exceptionsView details | Re-test

#1

Damien Tournoud - November 19, 2008 - 19:15
Status:needs review» won't fix

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

#2

hswong3i - November 19, 2008 - 20:40
Status: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.

#3

catch - November 19, 2008 - 21:20
Status:needs review» 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.

 
 

Drupal is a registered trademark of Dries Buytaert.