fix .cpg file contents

minorua · minorua · commit 1113c6099a41 · 2013-04-29T12:22:37.000+09:00
diff --git a/src/core/qgsvectorfilewriter.cpp b/src/core/qgsvectorfilewriter.cpp
@@ -113,7 +113,7 @@ QgsVectorFileWriter::QgsVectorFileWriter(
   {
     if ( layOptions.join( "" ).toUpper().indexOf( "ENCODING=" ) == -1 )
     {
-      layOptions.append( "ENCODING=" + fileEncoding );
+      layOptions.append( "ENCODING=" + convertCodecNameForEncodingOption( fileEncoding ) );
     }
 
     CPLSetConfigOption( "SHAPE_ENCODING", "" );
@@ -1023,6 +1023,23 @@ QString QgsVectorFileWriter::filterForDriver( const QString& driverName )
   return trLongName + " [OGR] (" + glob.toLower() + " " + glob.toUpper() + ")";
 }
 
+QString QgsVectorFileWriter::convertCodecNameForEncodingOption( const QString &codecName )
+{
+  if ( codecName == "System" )
+    return QString( "LDID/0" );
+
+  QRegExp re = QRegExp( QString( "(CP|windows-|ISO[ -])(.+)" ), Qt::CaseInsensitive );
+  if ( re.exactMatch( codecName ) )
+  {
+    QString c = re.cap( 2 ).replace( "-" , "" );
+    bool isNumber;
+    c.toInt( &isNumber );
+    if ( isNumber )
+      return c;
+  }
+  return codecName;
+}
+
 bool QgsVectorFileWriter::driverMetadata( QString driverName, QString &longName, QString &trLongName, QString &glob, QString &ext )
 {
   if ( driverName.startsWith( "AVCE00" ) )
diff --git a/src/core/qgsvectorfilewriter.h b/src/core/qgsvectorfilewriter.h
@@ -123,6 +123,9 @@ class CORE_EXPORT QgsVectorFileWriter
     /**Creates a filter for an OGR driver key*/
     static QString filterForDriver( const QString& driverName );
 
+    /**Converts codec name to string passed to ENCODING layer creation option of OGR Shapefile*/
+    static QString convertCodecNameForEncodingOption( const QString &codecName );
+
     /** checks whether there were any errors in constructor */
     WriterError hasError();
 
diff --git a/src/providers/ogr/qgsogrprovider.cpp b/src/providers/ogr/qgsogrprovider.cpp
@@ -1850,7 +1850,7 @@ QGISEXTERN bool createEmptyDataSource( const QString &uri,
   char **papszOptions = NULL;
   if ( driverName == "ESRI Shapefile" )
   {
-    papszOptions = CSLSetNameValue( papszOptions, "ENCODING", encoding.toLocal8Bit().data() );
+    papszOptions = CSLSetNameValue( papszOptions, "ENCODING", QgsVectorFileWriter::convertCodecNameForEncodingOption( encoding ).toLocal8Bit().data() );
     // OGR Shapefile fails to create fields if given encoding is not supported by its side
     // so disable encoding conversion of OGR Shapefile layer
     CPLSetConfigOption( "SHAPE_ENCODING", "" );

Original file line number	Diff line number	Diff line change
`@@ -1850,7 +1850,7 @@ QGISEXTERN bool createEmptyDataSource( const QString &uri,`
`1850`	`1850`	`char **papszOptions = NULL;`
`1851`	`1851`	`if ( driverName == "ESRI Shapefile" )`
`1852`	`1852`	`{`
`1853`		`- papszOptions = CSLSetNameValue( papszOptions, "ENCODING", encoding.toLocal8Bit().data() );`
	`1853`	`+ papszOptions = CSLSetNameValue( papszOptions, "ENCODING", QgsVectorFileWriter::convertCodecNameForEncodingOption( encoding ).toLocal8Bit().data() );`
`1854`	`1854`	`// OGR Shapefile fails to create fields if given encoding is not supported by its side`
`1855`	`1855`	`// so disable encoding conversion of OGR Shapefile layer`
`1856`	`1856`	`CPLSetConfigOption( "SHAPE_ENCODING", "" );`