sij/mapnik
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

postgis/oracle: use mapnik::boolean type for 'estimate_extent' rather than std::string, and strip double quotes as the estimated_extent function operates on single quotes - improves upon #393

Browse source
This commit is contained in:
Dane Springmeyer 2010-03-17 16:06:39 +00:00
parent e680c3c473
commit d6ef3014a3
2 changed files with 8 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
plugins/input/occi/occi_datasource.cpp
View file

@ -297,9 +297,9 @@ Envelope<double> occi_datasource::envelope() const
double lox, loy, hix, hiy;
occi_connection_ptr conn (pool_);
boost::optional<std::string> estimate_extent = params_.get<std::string>("estimate_extent");
boost::optional<mapnik::boolean> estimate_extent = params_.get<mapnik::boolean>("estimate_extent",false);
if (estimate_extent && *estimate_extent == "true")
if (estimate_extent && *estimate_extent)
{
std::ostringstream s;
s << "select min(c.x), min(c.y), max(c.x), max(c.y) from ";

12
plugins/input/postgis/postgis.cpp
View file

@ -561,7 +561,7 @@ Envelope<double> postgis_datasource::envelope() const
{
PoolGuard<shared_ptr<Connection>,shared_ptr<Pool<Connection,ConnectionCreator> > > guard(conn,pool);
std::ostringstream s;
boost::optional<std::string> estimate_extent = params_.get<std::string>("estimate_extent");
boost::optional<mapnik::boolean> estimate_extent = params_.get<mapnik::boolean>("estimate_extent",false);
if (!geometryColumn_.length() > 0)
{
@ -574,18 +574,18 @@ Envelope<double> postgis_datasource::envelope() const
throw mapnik::datasource_exception(s_error.str());
}
if (estimate_extent && *estimate_extent == "true")
if (estimate_extent && *estimate_extent)
{
s << "select xmin(ext),ymin(ext),xmax(ext),ymax(ext)"
<< " from (select estimated_extent('";
if (schema_.length() > 0)
{
s << schema_ << "','";
s << unquote(schema_) << "','";
}
s << geometry_table_ << "','"
<< geometryColumn_ << "') as ext) as tmp";
s << unquote(geometry_table_) << "','"
<< unquote(geometryColumn_) << "') as ext) as tmp";
}
else
{
@ -614,7 +614,7 @@ Envelope<double> postgis_datasource::envelope() const
{
clog << boost::format("PostGIS: sending query: %s\n") % s.str();
}*/
shared_ptr<ResultSet> rs=conn->executeQuery(s.str());
if (rs->next())
{