Posted by hswong3i on November 19, 2008 at 6:27pm
Jump to:
| Project: | Drupal core |
| Version: | 7.x-dev |
| Component: | base system |
| Category: | feature request |
| Priority: | normal |
| Assigned: | hswong3i |
| Status: | closed (duplicate) |
Issue Summary
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.
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| actions-parameters-default-1227118569.patch | 4.18 KB | Idle | Passed: 7252 passes, 0 fails, 0 exceptions | View details | Re-test |
Comments
#1
As per chx' analysis in #334687: [DBTNG + XDB] NOT NULL fields using a DEFAULT '' clause , which I totally agree with.
#2
I just give a reply for original issue (http://drupal.org/node/334687#comment-1116672) and hopefully may trigger more open end discussion.
#3
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.