Move update bounding box to table model

pull/11922/head
Javier Torres 8 years ago
parent d438e23312
commit 2d740da433

@ -2,7 +2,6 @@
require 'uuidtools'
require_relative '../models/visualization/support_tables'
require_dependency 'carto/bounding_box_utils'
require_dependency 'carto/db/user_schema'
module CartoDB
@ -232,7 +231,7 @@ module CartoDB
table_registrar.register(name, data_import_id)
@table = table_registrar.table
@imported_table_visualization_ids << @table.table_visualization.id
BoundingBoxHelper.update_visualizations_bbox(table)
table.update_bounding_box
self
end

@ -1,7 +1,6 @@
# coding: UTF-8
require_relative '../../../models/visualization/presenter'
require_dependency '../../../helpers/bounding_box_helper'
require_dependency 'carto/db/user_schema'
require_dependency 'carto/tracking/events'
@ -97,7 +96,7 @@ class Api::Json::TablesController < Api::ApplicationController
@stats_aggregator.timing('georeference') do
@table.georeference_from!(:latitude_column => latitude_column, :longitude_column => longitude_column)
end
BoundingBoxHelper.update_visualizations_bbox(@table)
@table.update_bounding_box
render_jsonp(@table.public_values({request:request}).merge(warnings: warnings)) and return
end

@ -5,7 +5,6 @@ require_relative '../../services/table-geocoder/lib/exceptions'
require_relative '../../services/table-geocoder/lib/mail_geocoder'
require_relative '../../services/geocoder/lib/geocoder_config'
require_relative '../../lib/cartodb/metrics'
require_relative '../../app/helpers/bounding_box_helper'
require_relative 'log'
require_relative '../../lib/cartodb/stats/geocoding'
@ -351,7 +350,7 @@ class Geocoding < Sequel::Model
# In the import table_service could be nil
if !table_service.nil?
# To store the bbox in visualizations
BoundingBoxHelper.update_visualizations_bbox(table_service)
table_service.update_bounding_box
end
self.report
end

@ -1246,6 +1246,12 @@ class Table
@user_table.table_visualization
end
def update_bounding_box
update_table_geom_pg_stats
bounds = Carto::BoundingBoxService.new(owner).get_table_bounds(name) || Carto::BoundingBoxUtils::DEFAULT_BOUNDS
# TODO
end
private
def related_visualizations

Loading…
Cancel
Save