Changeset 64898

Timestamp:

Mar 24, 2015, 2:23:17 AM (9 years ago)

Author:

lucadelu

Message:

r.unpack: added -o flag to overwrite project system; thanks to Nikos Alexandris

File:

• grass/trunk/scripts/r.unpack/r.unpack.py (modified) (5 diffs)

grass/trunk/scripts/r.unpack/r.unpack.py

@@ -31 +31 @@
 #%flag
 #% key: o
-#% description: Override projection check (use current location's projection)
+#% description: Override projection check (use location's projection)
 #% guisection: Output settings
 #%end
@@ -54 +54 @@
     global tmp_dir
     tmp_dir = grass.tempdir()
-    grass.debug('tmp_dir = %s' % tmp_dir)
+    grass.debug('tmp_dir = {tmpdir}'.format(tmpdir=tmp_dir))
 
     if not os.path.exists(infile):
-        grass.fatal(_("File <%s> not found") % infile)
+        grass.fatal(_('File {name} not found.'.format(name=infile)))
 
     gisenv = grass.gisenv()
@@ -80 +80 @@
     if gfile['file']:
         if os.environ.get('GRASS_OVERWRITE', '0') != '1':
-            grass.fatal(_("Raster map <%s> already exists") % map_name)
+            grass.fatal(_('Raster map <{name}> already exists'.format(name=map_name)))
         else:
-            grass.warning(_("Raster map <%s> already exists and will be overwritten") % map_name)
+            grass.warning(_('Raster map <{name}> already exists and will be overwritten'.format(name=map_name)))
 
     # extract data
@@ -91 +91 @@
         pass
     elif os.path.exists('coor'):
-        grass.fatal(_("This GRASS GIS pack file contains vector data. Use "
-                      "v.unpack to unpack <%s>" % map_name))
+        grass.fatal(_('This GRASS GIS pack file contains vector data. Use '
+                      'v.unpack to unpack <{name}>'.format(name=map_name)))
     else:
         grass.fatal(_("Pack file unreadable"))
 
     # check projection compatibility in a rather crappy way
-    diff_result_1 = diff_result_2 = None
-    proj_info_file_1 = 'PROJ_INFO'
-    proj_info_file_2 = os.path.join(mset_dir, '..', 'PERMANENT', 'PROJ_INFO')
-    if not grass.compare_key_value_text_files(filename_a=proj_info_file_1,
-                                              filename_b=proj_info_file_2,
-                                              proj=True):
-        diff_result_1 = diff_files(proj_info_file_1, proj_info_file_2)
 
-    proj_units_file_1 = 'PROJ_UNITS'
-    proj_units_file_2 = os.path.join(mset_dir, '..', 'PERMANENT', 'PROJ_UNITS')
-    if not grass.compare_key_value_text_files(filename_a=proj_units_file_1,
-                                              filename_b=proj_units_file_2,
-                                              units=True):
-        diff_result_2 = diff_files(proj_units_file_1, proj_units_file_2)
-
-    if diff_result_1 or diff_result_2:
-        if flags['o']:
-            grass.warning(_("Projection information does not match. Proceeding..."))
-        else:
+    if flags['o']:
+        grass.warning(_("Overriding projection check (using current location's projection)."))
+        
+    else:
+        
+        diff_result_1 = diff_result_2 = None
+        
+        proj_info_file_1 = 'PROJ_INFO'
+        proj_info_file_2 = os.path.join(mset_dir, '..', 'PERMANENT', 'PROJ_INFO')    
+    
+        if not grass.compare_key_value_text_files(filename_a=proj_info_file_1,
+                                                  filename_b=proj_info_file_2,
+                                                  proj=True):                                                      
+            diff_result_1 = diff_files(proj_info_file_1, proj_info_file_2)
+    
+        proj_units_file_1 = 'PROJ_UNITS'
+        proj_units_file_2 = os.path.join(mset_dir, '..', 'PERMANENT', 'PROJ_UNITS')
+    
+        if not grass.compare_key_value_text_files(filename_a=proj_units_file_1,
+                                                  filename_b=proj_units_file_2,
+                                                  units=True):                                                      
+            diff_result_2 = diff_files(proj_units_file_1, proj_units_file_2)
+    
+        if diff_result_1 or diff_result_2:
+            
             if diff_result_1:
                 grass.warning(_("Difference between PROJ_INFO file of packed map "
@@ -139 +146 @@
             shutil.copyfile(element, os.path.join(mset_dir, element, map_name))
 
-    grass.message(_("Raster map <%s> unpacked") % map_name)
+    grass.message(_('Raster map <{name}> unpacked'.format(name=map_name)))
 
 if __name__ == "__main__":