← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

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

  • Committer: Jay Pipes
  • Date: 2009-01-30 15:31:56 UTC
  • mto: This revision was merged to the branch mainline in revision 830.
  • Revision ID: jpipes@serialcoder-20090130153156-kq2bis6mwh48qi1w
Fixes the WEEKDAY() function use the new temporal system and enable proper error throwing on bad datetimes.  Adds test case for the WEEKDAY() function.

Show diffs side-by-side

added added

removed removed

Lines of Context:
drizzled/function/time/weekday.cc
17
17
 *  Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
18
18
 */
19
19
 
20
 
#include <drizzled/server_includes.h>
 
20
#include "drizzled/server_includes.h"
21
21
#include CSTDINT_H
22
 
#include <drizzled/function/time/weekday.h>
 
22
#include "drizzled/temporal.h"
 
23
#include "drizzled/error.h"
 
24
#include "drizzled/function/time/weekday.h"
23
25
 
24
26
int64_t Item_func_weekday::val_int()
25
27
{
26
 
  assert(fixed == 1);
27
 
  DRIZZLE_TIME ltime;
 
28
  assert(fixed);
28
29
 
29
 
  if (get_arg0_date(&ltime, TIME_NO_ZERO_DATE))
 
30
  if (args[0]->is_null())
 
31
  {
 
32
    /* For NULL argument, we return a NULL result */
 
33
    null_value= true;
30
34
    return 0;
31
 
 
32
 
  return (int64_t) calc_weekday(calc_daynr(ltime.year, ltime.month,
33
 
                                            ltime.day),
34
 
                                 odbc_type) + test(odbc_type);
 
35
  }
 
36
 
 
37
  /* Grab the first argument as a DateTime object */
 
38
  drizzled::DateTime temporal;
 
39
  Item_result arg0_result_type= args[0]->result_type();
 
40
  
 
41
  switch (arg0_result_type)
 
42
  {
 
43
    case DECIMAL_RESULT: 
 
44
      /* 
 
45
       * For doubles supplied, interpret the arg as a string, 
 
46
       * so intentionally fall-through here...
 
47
       * This allows us to accept double parameters like 
 
48
       * 19971231235959.01 and interpret it the way MySQL does:
 
49
       * as a TIMESTAMP-like thing with a microsecond component.
 
50
       * Ugh, but need to keep backwards-compat.
 
51
       */
 
52
    case STRING_RESULT:
 
53
      {
 
54
        char buff[DRIZZLE_MAX_LENGTH_DATETIME_AS_STRING];
 
55
        String tmp(buff,sizeof(buff), &my_charset_utf8_bin);
 
56
        String *res= args[0]->val_str(&tmp);
 
57
        if (! temporal.from_string(res->c_ptr(), res->length()))
 
58
        {
 
59
          /* 
 
60
          * Could not interpret the function argument as a temporal value, 
 
61
          * so throw an error and return 0
 
62
          */
 
63
          my_error(ER_INVALID_DATETIME_VALUE, MYF(0), res->c_ptr());
 
64
          return 0;
 
65
        }
 
66
      }
 
67
      break;
 
68
    case INT_RESULT:
 
69
      if (temporal.from_int64_t(args[0]->val_int()))
 
70
        break;
 
71
      /* Intentionally fall-through on invalid conversion from integer */
 
72
    default:
 
73
      {
 
74
        /* 
 
75
        * Could not interpret the function argument as a temporal value, 
 
76
        * so throw an error and return 0
 
77
        */
 
78
        null_value= true;
 
79
        char buff[DRIZZLE_MAX_LENGTH_DATETIME_AS_STRING];
 
80
        String tmp(buff,sizeof(buff), &my_charset_utf8_bin);
 
81
        String *res;
 
82
 
 
83
        res= args[0]->val_str(&tmp);
 
84
 
 
85
        my_error(ER_INVALID_DATETIME_VALUE, MYF(0), res->c_ptr());
 
86
        return 0;
 
87
      }
 
88
  }
 
89
  /* @TODO This ODBC thing is a complete hack. Fix it. */
 
90
  bool sunday_first= odbc_type;
 
91
  int64_t julian_day= julian_day_number_from_gregorian_date(temporal.years(), temporal.months(), temporal.days());
 
92
  return (int64_t) day_of_week(julian_day, sunday_first) + test(odbc_type);
35
93
}
36
 
 
37