← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

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

  • Committer: Jay Pipes
  • Date: 2009-01-30 04:38:21 UTC
  • mto: This revision was merged to the branch mainline in revision 830.
  • Revision ID: jpipes@serialcoder-20090130043821-4d7jg2ftabefamxb
Fixes for the QUARTER() function to use new Temporal system and throw
errors on bad datetime values.

Added test case for QUARTER() function and modified func_time.test existing
test to correctly throw errors and report NULL, not 0 on NULL input.

Show diffs side-by-side

added added

removed removed

Lines of Context:
drizzled/function/time/quarter.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/quarter.h>
23
 
 
24
 
/**
25
 
  Returns the quarter of the year.
26
 
*/
 
22
#include "drizzled/temporal.h"
 
23
#include "drizzled/error.h"
 
24
#include "drizzled/function/time/quarter.h"
27
25
 
28
26
int64_t Item_func_quarter::val_int()
29
27
{
30
 
  assert(fixed == 1);
31
 
  DRIZZLE_TIME ltime;
32
 
  if (get_arg0_date(&ltime, TIME_FUZZY_DATE))
 
28
  assert(fixed);
 
29
 
 
30
  if (args[0]->is_null())
 
31
  {
 
32
    /* For NULL argument, we return a NULL result */
 
33
    null_value= true;
33
34
    return 0;
34
 
  return (int64_t) ((ltime.month+2)/3);
 
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
  return (int64_t) ((temporal.months() + 2) / 3);
35
90
}
36