aboutsummaryrefslogtreecommitdiff
path: root/src/iso19111/crs.cpp
diff options
context:
space:
mode:
Diffstat (limited to 'src/iso19111/crs.cpp')
-rw-r--r--src/iso19111/crs.cpp42
1 files changed, 33 insertions, 9 deletions
diff --git a/src/iso19111/crs.cpp b/src/iso19111/crs.cpp
index b9694ba2..60b316f1 100644
--- a/src/iso19111/crs.cpp
+++ b/src/iso19111/crs.cpp
@@ -1308,7 +1308,7 @@ void GeodeticCRS::addDatumInfoToPROJString(
const auto &nadgrids = formatter->getHDatumExtension();
const auto &l_datum = datum();
if (formatter->getCRSExport() && l_datum && TOWGS84Params.empty() &&
- nadgrids.empty() && !formatter->getDropEarlyBindingsTerms()) {
+ nadgrids.empty()) {
if (l_datum->_isEquivalentTo(
datum::GeodeticReferenceFrame::EPSG_6326.get(),
util::IComparable::Criterion::EQUIVALENT)) {
@@ -1323,7 +1323,12 @@ void GeodeticCRS::addDatumInfoToPROJString(
datum::GeodeticReferenceFrame::EPSG_6269.get(),
util::IComparable::Criterion::EQUIVALENT)) {
datumWritten = true;
- formatter->addParam("datum", "NAD83");
+ if (formatter->getLegacyCRSToCRSContext()) {
+ // We do not want datum=NAD83 to cause a useless towgs84=0,0,0
+ formatter->addParam("ellps", "GRS80");
+ } else {
+ formatter->addParam("datum", "NAD83");
+ }
}
}
if (!datumWritten) {
@@ -2083,8 +2088,31 @@ void GeographicCRS::_exportToPROJString(
primeMeridian()->longitude().getSIValue() != 0.0 ||
!formatter->getTOWGS84Parameters().empty() ||
!formatter->getHDatumExtension().empty()) {
+
formatter->addStep("longlat");
- addDatumInfoToPROJString(formatter);
+ bool done = false;
+ if (formatter->getLegacyCRSToCRSContext() &&
+ formatter->getHDatumExtension().empty() &&
+ formatter->getTOWGS84Parameters().empty()) {
+ const auto &l_datum = datum();
+ if (l_datum &&
+ l_datum->_isEquivalentTo(
+ datum::GeodeticReferenceFrame::EPSG_6326.get(),
+ util::IComparable::Criterion::EQUIVALENT)) {
+ done = true;
+ formatter->addParam("ellps", "WGS84");
+ } else if (l_datum &&
+ l_datum->_isEquivalentTo(
+ datum::GeodeticReferenceFrame::EPSG_6269.get(),
+ util::IComparable::Criterion::EQUIVALENT)) {
+ done = true;
+ // We do not want datum=NAD83 to cause a useless towgs84=0,0,0
+ formatter->addParam("ellps", "GRS80");
+ }
+ }
+ if (!done) {
+ addDatumInfoToPROJString(formatter);
+ }
}
if (!formatter->getCRSExport()) {
addAngularUnitConvertAndAxisSwap(formatter);
@@ -3132,7 +3160,8 @@ void ProjectedCRS::addUnitConvertAndAxisSwap(io::PROJStringFormatter *formatter,
formatter->addParam("units", projUnit);
}
}
- } else if (formatter->getCRSExport()) {
+ } else if (formatter->getCRSExport() &&
+ !formatter->getLegacyCRSToCRSContext()) {
formatter->addParam("units", "m");
}
@@ -4143,11 +4172,6 @@ void BoundCRS::_exportToPROJString(
"baseCRS of BoundCRS cannot be exported as a PROJ string");
}
- if (formatter->getDropEarlyBindingsTerms()) {
- crs_exportable->_exportToPROJString(formatter);
- return;
- }
-
auto vdatumProj4GridName = getVDatumPROJ4GRIDS();
if (!vdatumProj4GridName.empty()) {
formatter->setVDatumExtension(vdatumProj4GridName);