Revert "Renames File.file -> File.source to prevent issues in Django 1.9"

This reverts commit b1399d5.

That change is no longer necessary, since removing the related_name has
the same effect without possibly breaking existing code relying on the
'file' field. Those changes have not been release yet, so it is ok to
remove the migrations.

Author: stefanfoulis

djangocms_file/cms_plugins.py

@@ -15,7 +15,7 @@ class FilePlugin(CMSPluginBase):
 
     fieldsets = [
         (None, {
-            'fields': ('title', 'source', 'tooltip', 'target', )
+            'fields': ('title', 'file', 'tooltip', 'target', )
         })
     ]
 
@@ -30,7 +30,6 @@ def icon_src(self, instance):
         file_icon = instance.get_icon_url()
         if file_icon:
             return file_icon
-        return settings.STATIC_URL + u"cms/img/icons/plugins/source.png"
         return settings.STATIC_URL + u"cms/img/icons/plugins/file.png"
 
 plugin_pool.register_plugin(FilePlugin)

djangocms_file/migrations/0003_rename_file_field_20160115_1904.py

@@ -39,7 +39,7 @@ class File(CMSPlugin):
     The icon search is currently performed within get_icon_url; this is
     probably a performance concern.
     """
-    source = models.FileField(_("source"), upload_to=get_plugin_media_path)
+    file = models.FileField(_("file"), upload_to=get_plugin_media_path)
     title = models.CharField(
         _("title"), max_length=255, null=True, blank=True,
         help_text=_("Optional title to display. If not supplied, the filename "
@@ -70,9 +70,9 @@ class File(CMSPlugin):
     def __str__(self):
         if self.title:
             return self.title
-        elif self.source:
+        elif self.file:
             # added if, because it raised attribute error when
-            # source wasn't defined
+            # file wasn't defined
             return self.get_file_name()
         return "<empty>"
 
@@ -85,10 +85,10 @@ def get_icon_url(self):
         return None
 
     def file_exists(self):
-        return default_storage.exists(self.source.name)
+        return default_storage.exists(self.file.name)
 
     def get_file_name(self):
-        return os.path.basename(self.source.name)
+        return os.path.basename(self.file.name)
 
     def get_ext(self):
         return os.path.splitext(self.get_file_name())[1][1:].lower()

djangocms_file/models.py

@@ -1,9 +1,9 @@
 {% load i18n %}
-{% if object.source.url %}
+{% if object.file.url %}
 <p class="plugin_file">
-	<a href="{{ object.source.url }}"{% if object.target %} target="{{ object.target }}"{% endif %}{% if object.tooltip %} title="{{ object.tooltip }}"{% endif %}>
+	<a href="{{ object.file.url }}"{% if object.target %} target="{{ object.target }}"{% endif %}{% if object.tooltip %} title="{{ object.tooltip }}"{% endif %}>
 		{% if object.get_icon_url %}<img src="{{ object.get_icon_url }}" alt="" />{% endif %}
-		{% if object.title %}<span class="filetitle">{{ object.title }}{% else %}{{ object.get_file_name }}{% endif %} {% if object.file_exists %}<span class="filesize">({{ object.source.size|filesizeformat }})</span>{% else %}({% trans "file missing!" %}){% endif %}{% if object.title %}</span>{% endif %}
+		{% if object.title %}<span class="filetitle">{{ object.title }}{% else %}{{ object.get_file_name }}{% endif %} {% if object.file_exists %}<span class="filesize">({{ object.file.size|filesizeformat }})</span>{% else %}({% trans "file missing!" %}){% endif %}{% if object.title %}</span>{% endif %}
 	</a>
 </p>
 {% endif %}