← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to drizzled/function/time/month.cc

  • Committer: Jay Pipes
  • Date: 2009-01-30 04:01:12 UTC
  • mto: This revision was merged to the branch mainline in revision 830.
  • Revision ID: jpipes@serialcoder-20090130040112-svbn774guj98pwi4
To remain in compatibility with MySQL, added ability to interpret
decimal arguments as datetime strings for temporal functions.

Fixed YEAR(), MONTH(), DAYOFMONTH(), DAYOFYEAR(), HOUR(), MINUTE(), SECOND(), and MICROSECOND()
to accept decimal parameters and interpret them the same way as MySQL.

Fixed an issue with the TemporalFormat::matches() method which was 
incorrectly assuming all microsecond arguments were specified as 6 digits.
Added power of 10 multiplier to usecond calculation. This fixes issues with
failures in type_date and func_sapdb test cases.

Show diffs side-by-side

added added

removed removed

Lines of Context:
drizzled/function/time/month.cc
41
41
  
42
42
  switch (arg0_result_type)
43
43
  {
 
44
    case DECIMAL_RESULT: 
 
45
      /* 
 
46
       * For doubles supplied, interpret the arg as a string, 
 
47
       * so intentionally fall-through here...
 
48
       * This allows us to accept double parameters like 
 
49
       * 19971231235959.01 and interpret it the way MySQL does:
 
50
       * as a TIMESTAMP-like thing with a microsecond component.
 
51
       * Ugh, but need to keep backwards-compat.
 
52
       */
44
53
    case STRING_RESULT:
45
54
      {
46
55
        char buff[DRIZZLE_MAX_LENGTH_DATETIME_AS_STRING];