show file type for each layer based on file it was in

- use GDAL to detect type of every uploaded file
- make a field on UploadFile to store the detected file type
- make a property on UploadLayer to expose the type of containing file
- expose this property from the HTTP API as readonly
- use same from the angular template to correct display of layer items

also don't mess up template formatting when fields are blank, as in rasters.
also don't try to show feature_count if it's None, as in rasters:
either we have an uninformative, unnecessarily alarming "UNKNOWN" or
we have whitespace which bootstrap can't handle, breaking page formatting.
Rather just not have it when it's not actually meaningful to show.

osgeo_importer/api.py

@@ -33,6 +33,8 @@ class UploadedLayerResource(ModelResource):
     configuration_options = DictField(attribute='configuration_options', null=True)
     fields = ListField(attribute='fields')
     status = CharField(attribute='status', readonly=True, null=True)
+    file_type = CharField(attribute='file_type', readonly=True)
+    file_name = CharField(attribute='file_name', readonly=True)
 
     class Meta:
         queryset = UploadLayer.objects.all()

osgeo_importer/models.py

@@ -172,6 +172,7 @@ def __unicode__(self):
 class UploadFile(models.Model):
     upload = models.ForeignKey(UploadedData, null=True, blank=True)
     file = models.FileField(upload_to="uploads", validators=[validate_file_extension, validate_inspector_can_read])
+    file_type = models.CharField(max_length=50, null=True, blank=True)
     slug = models.SlugField(max_length=250, blank=True)
 
     def __unicode__(self):
@@ -205,6 +206,20 @@ class UploadLayer(models.Model):
     task_id = models.CharField(max_length=36, blank=True, null=True)
     feature_count = models.IntegerField(null=True, blank=True)
 
+    @property
+    def file_name(self):
+        if not self.upload_file:
+            return None
+        return self.upload_file.name
+
+    @property
+    def file_type(self):
+        """A layer's 'file type' - really the file type of the file it is in.
+        """
+        if not self.upload_file:
+            return None
+        return self.upload_file.file_type
+
     @property
     def layer_data(self):
         """

osgeo_importer/static/osgeo_importer/partials/upload.html

@@ -23,23 +23,25 @@
                                 <!--<div class="light-bold">Layers</div>-->
 
                                 <div class="col-xs-10 col-sm-10 col-md-10 col-lg-10 col-xs-offset-1 col-sm-offset-1 col-md-offset-1 col-lg-offset-1 layer-in-upload" ng-repeat="layer in upload.layers">
-                                    <div class="layer-upload-name"> {{layer.name}}
+                                    <div class="layer-upload-name">
+                                        <span>{{layer.name || layer.file_name}}</span>
+                                        <span ng-show="layer.name && layer.name != layer.file_name"><br>(in {{layer.file_name}})</span>
                                         <span class="pull-right" ng-show="layer.geonode_layer.title"><span style="font-weight: 600">Layer:</span> <a href="{{layer.geonode_layer.url}}" target="_self">{{layer.geonode_layer.title}}</a></span>
                                     </div>
                                     <hr/>
                                     <div class="row">
                                         <div class="layer-upload-details col-md-12">
                                             <div>
                                                 <span class="col-md-3 layer-upload-field-name">File Type</span>
-                                                <span class="col-md-9">{{upload.file_type}}</span>
+                                                <span class="col-md-9">{{layer.file_type || upload.file_type || "UNKNOWN" }}</span>
                                             </div>
-                                            <div>
+                                            <div ng-show="layer.feature_count">
                                                 <span class="col-md-3 layer-upload-field-name">Features</span>
                                                 <span class="col-md-9">{{layer.feature_count | number : 0}}</span>
                                             </div>
                                             <div>
                                                 <span class="col-md-3 layer-upload-field-name">Status</span>
-                                                <span class="col-md-9">{{layer.status}}</span>
+                                                <span class="col-md-9">{{layer.status || "UNKNOWN" }}</span>
                                             </div>
                                         </div>
 

osgeo_importer/views.py

@@ -11,7 +11,7 @@
 from .models import UploadedData, UploadLayer, UploadFile, DEFAULT_LAYER_CONFIGURATION
 from .importers import OSGEO_IMPORTER
 from .inspectors import OSGEO_INSPECTOR
-from .utils import import_string
+from .utils import import_string, NoDataSourceFound
 from django.core.files.storage import FileSystemStorage
 
 OSGEO_INSPECTOR = import_string(OSGEO_INSPECTOR)
@@ -183,6 +183,12 @@ def form_valid(self, form):
             upfile = UploadFile.objects.create(upload=upload)
             upfiles.append(upfile)
             upfile.file.name = each
+            # Detect and store file type for later reporting, since it is no
+            # longer true that every upload has only one file type.
+            try:
+                upfile.file_type = self.get_file_type(each)
+            except NoDataSourceFound:
+                upfile.file_type = None
             upfile.save()
             upfile_basename = os.path.basename(each)
             _, upfile_ext = os.path.splitext(upfile_basename)