Add altitude value

Shouldn't this patch implement hook_update_N?

What if you use 7.x-1.6 and want to use the "alt" value after an update? The the database field defaults to 0 but its not there in 7.x-1.6+? My understanding was that we need hook_update_N to create the altitude in all geolocation tables for existing installations.

Btw. maybe its better to name the column 'altitude' instead of 'alt' since it could be confused with image 'alt'?

<?php
/**
 * Add altitude field to geolocation tables.
 */
function mymodule_update_7103() {
  $fields = geolocation_get_geolocation_fields();

  foreach ($fields as $field) {
    $tables = array(_field_sql_storage_tablename($field), _field_sql_storage_revision_tablename($field));
    foreach ($tables as $table) {
      $spec = array(
        'description' => 'Stores the altitude value',
        'type' => 'float',
        'size' => 'big',
        'not null' => TRUE,
        'default' => 0,
      ); 
      db_add_field($table, 'altitude', $spec);
    }
  }
  return t(Added altitude field to geolocation tables.');
}
?>

Re-rolled patch.

Grr...

Grrr...

Sorry, ignore my comment, I need this for D8

Closing all 7.x issues. It's time.