Time type for Forms API?

How did you do this? I'm interested in this code as I need it for a data export module i'm writing...

Nice... just tried it and it looks like it's working. :) Thanks!

-Mike Cantelon
http://mikecantelon.com

If you use the code posted above in Drupal 5.1, you won't able to see title and description. I did minor modification and it works like charm.

Change,

function theme_time($element) {
  $output = '<div class="container-inline">'. $element['#children'] .'</div>';
  return theme('form_element', $element['#title'], $output, $element['#description'], $element['#id'], $element['#required'], form_get_error($element));
}

To,

function theme_time($element) {
  $output = '<div class="container-inline">'. $element['#children'] .'</div>';
  return theme('form_element', $element, $output);
}

Regards,
Dhaval

The above code uses the 'h' format for Hour and 'i' format for Minutes, which return values including leading zeros. However, to act as a proper default value for the HTML SELECT tag, leading zeros do not work. In the SELECT tag you'll notice that the actual *value* of each option doesn't have any leading zero...just the display string has the leading zeros to keep the values properly sorted.

So the above code only properly pre-selects the correct default hour and minute if the hour is 10,11,12 and the minute is >= 10. Here is the modified code I used to fix this. I've included the full code to make it easier for people to use it, but the change is just in the first few lines. Also added the $parents code taken from the existing expand_date code in form.inc and removed the unneeded time_value function.

/**
 * Implementation of hook_elements().
 */
function modulename_elements() {
  $type["time"] = array(
    "#input"       => true,
    "#process"     => array("expand_time"),
  );

  return $type;
}

/**
 * Define a Time form element.  Adapted from Date element type in form.inc
 * 
 * @param mixed $element
 */
function expand_time($element) {
  // Default to current time
  if (empty($element['#value'])) {
    $element['#value'] = array(
    // note we need the defaults WITHOUT leading zeros
    // so we use a trick for minutes since PHP doesn't have a format for it
      'hour' => format_date(time(), 'custom', 'g'),
      'minute' => intval( format_date(time(), 'custom', 'i')),
      'meridiem' => format_date(time(), 'custom', 'a')
    );
  }

  $element['#tree'] = TRUE;

  foreach ($element['#value'] as $type => $value) {
    switch ($type) {
      case 'hour':
        $options = range(0, 12);
        break;
      case 'minute':
        $options = range(0, 59);
        break;
      case 'meridiem':
        $options = drupal_map_assoc(array('am', 'pm'));
        break;
    }
   
   // pad with zeros
    if ($type != 'meridiem') {
      foreach ($options as $option) {
        strlen($option) <= 1 ? $options[$option] = 0 . $option : '';
      }
    }
   
    $parents = $element['#parents'];
    $parents[] = $type;
    $element[$type] = array(
      '#type' => 'select',
      '#value' => $element['#value'][$type],
      '#attributes' => $element['#attributes'],
      '#options' => $options,
    );
  }

  return $element;
}

function theme_time($element) {
  $output = '<div class="container-inline">'. $element['#children'] .'</div>';
  return theme('form_element', $element, $output);
}