when intializing from WKT favourize EPSG instead of PROJ.4

qgis · Nov 13, 2011 · fe67683 · fe67683
1 parent ccd8487
commit fe67683
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 41 deletions.
diff --git a/src/core/qgscoordinatereferencesystem.cpp b/src/core/qgscoordinatereferencesystem.cpp
@@ -310,6 +310,15 @@ bool QgsCoordinateReferenceSystem::createFromWkt( QString theWkt )
     return mIsValidFlag;
   }
 
+  if ( OSRAutoIdentifyEPSG( mCRS ) == OGRERR_NONE )
+  {
+    QString authid = QString( "%1:%2" )
+                     .arg( OSRGetAuthorityName( mCRS, NULL ) )
+                     .arg( OSRGetAuthorityCode( mCRS, NULL ) );
+    QgsDebugMsg( "authid recognized as " + authid );
+    return createFromOgcWmsCrs( authid );
+  }
+
   // always morph from esri as it doesn't hurt anything
   // FW: Hey, that's not right!  It can screw stuff up! Disable
   //myOgrSpatialRef.morphFromESRI();

diff --git a/src/providers/ogr/qgsogrprovider.cpp b/src/providers/ogr/qgsogrprovider.cpp
@@ -1807,44 +1807,6 @@ QGISEXTERN bool createEmptyDataSource( const QString &uri,
   return true;
 }
 
-bool QgsOgrProvider::crsFromWkt( QgsCoordinateReferenceSystem &srs, const char *wkt )
-{
-  void *hCRS = OSRNewSpatialReference( NULL );
-
-  if ( OSRImportFromWkt( hCRS, ( char ** ) &wkt ) == OGRERR_NONE )
-  {
-    if ( OSRAutoIdentifyEPSG( hCRS ) == OGRERR_NONE )
-    {
-      QString authid = QString( "%1:%2" )
-                       .arg( OSRGetAuthorityName( hCRS, NULL ) )
-                       .arg( OSRGetAuthorityCode( hCRS, NULL ) );
-      QgsDebugMsg( "authid recognized as " + authid );
-      srs.createFromOgcWmsCrs( authid );
-    }
-    else
-    {
-      // get the proj4 text
-      char *pszProj4;
-      OSRExportToProj4( hCRS, &pszProj4 );
-      QgsDebugMsg( pszProj4 );
-      OGRFree( pszProj4 );
-
-      char *pszWkt = NULL;
-      OSRExportToWkt( hCRS, &pszWkt );
-      QString myWktString = QString( pszWkt );
-      OGRFree( pszWkt );
-
-      // create CRS from Wkt
-      srs.createFromWkt( myWktString );
-    }
-  }
-
-  OSRRelease( hCRS );
-
-  return srs.isValid();
-}
-
-
 QgsCoordinateReferenceSystem QgsOgrProvider::crs()
 {
   QgsDebugMsg( "entering." );
@@ -1865,7 +1827,7 @@ QgsCoordinateReferenceSystem QgsOgrProvider::crs()
         QString myWktString = prjStream.readLine();
         prjFile.close();
 
-        if ( crsFromWkt( srs, myWktString.toUtf8().constData() ) )
+        if ( srs.createFromWkt( myWktString.toUtf8().constData() ) )
           return srs;
       }
     }
@@ -1882,7 +1844,8 @@ QgsCoordinateReferenceSystem QgsOgrProvider::crs()
 
     char *pszWkt = NULL;
     OSRExportToWkt( mySpatialRefSys, &pszWkt );
-    crsFromWkt( srs, pszWkt );
+
+    srs.createFromWkt( pszWkt );
     OGRFree( pszWkt );
   }
   else

diff --git a/src/providers/ogr/qgsogrprovider.h b/src/providers/ogr/qgsogrprovider.h
@@ -255,7 +255,6 @@ class QgsOgrProvider : public QgsVectorDataProvider
     void setRelevantFields( bool fetchGeometry, const QgsAttributeList& fetchAttributes );
 
   private:
-    bool crsFromWkt( QgsCoordinateReferenceSystem &srs, const char *wkt );
     unsigned char *getGeometryPointer( OGRFeatureH fet );
     QgsFieldMap mAttributeFields;
     OGRDataSourceH ogrDataSource;