make enabling optional

Author: christophe195

cms/settings.py

@@ -43,6 +43,9 @@
 REPORTED_TIMES_THRESHOLD = 10
 ALLOW_ANONYMOUS_ACTIONS = ["report", "like", "dislike", "watch"]  # need be a list
 
+# whether media attachments functionality is enabled
+ENABLE_MEDIA_ATTACHMENTS = False  # Set to True to enable attachments feature
+
 # experimental functionality for user ratings - does not work
 ALLOW_RATINGS = False
 ALLOW_RATINGS_CONFIRMED_EMAIL_ONLY = True

files/admin.py

@@ -283,6 +283,9 @@ class AttachmentAdmin(admin.ModelAdmin):
 admin.site.register(Language, LanguageAdmin)
 admin.site.register(VideoTrimRequest, VideoTrimRequestAdmin)
 admin.site.register(TranscriptionRequest, TranscriptionRequestAdmin)
-admin.site.register(Attachment, AttachmentAdmin)
+
+# Only register Attachment admin if the feature is enabled
+if settings.ENABLE_MEDIA_ATTACHMENTS:
+    admin.site.register(Attachment, AttachmentAdmin)
 
 Media._meta.app_config.verbose_name = "Media"

files/context_processors.py

@@ -29,6 +29,7 @@ def stuff(request):
     ret["CAN_DISLIKE_MEDIA"] = settings.CAN_DISLIKE_MEDIA
     ret["CAN_REPORT_MEDIA"] = settings.CAN_REPORT_MEDIA
     ret["CAN_SHARE_MEDIA"] = settings.CAN_SHARE_MEDIA
+    ret["ENABLE_MEDIA_ATTACHMENTS"] = settings.ENABLE_MEDIA_ATTACHMENTS
     ret["UPLOAD_MAX_SIZE"] = settings.UPLOAD_MAX_SIZE
     ret["UPLOAD_MAX_FILES_NUMBER"] = settings.UPLOAD_MAX_FILES_NUMBER
     ret["PRE_UPLOAD_MEDIA_MESSAGE"] = settings.PRE_UPLOAD_MEDIA_MESSAGE

files/models/attachment.py

@@ -1,15 +1,23 @@
 from django.db import models
 from django.db.models.signals import post_delete
 from django.dispatch import receiver
+import os
 from .media import Media
 
 class Attachment(models.Model):
     """Attachment model for files linked to a video (Media)"""
     media = models.ForeignKey(Media, on_delete=models.CASCADE, related_name="attachments")
-    name = models.CharField(max_length=255, help_text="Naam van het bestand")
+    name = models.CharField(max_length=255, blank=True, help_text="Name of the file (optional, uses filename if empty)")
     file = models.FileField(upload_to="attachments/%Y/%m/%d/")
     uploaded_at = models.DateTimeField(auto_now_add=True)
 
+    def save(self, *args, **kwargs):
+        # If name is empty, use the filename
+        if not self.name and self.file:
+            # Get the filename without the path
+            self.name = os.path.basename(self.file.name)
+        super().save(*args, **kwargs)
+
     def __str__(self):
         return f"{self.name} ({self.file.name})"
 

files/serializers.py

@@ -107,6 +107,9 @@ def get_url(self, obj):
         return self.context["request"].build_absolute_uri(obj.get_absolute_url())
 
     def get_attachments(self, obj):
+        # Only return attachments if the feature is enabled
+        if not settings.ENABLE_MEDIA_ATTACHMENTS:
+            return []
         attachments = obj.attachments.all()
         return AttachmentSerializer(attachments, many=True, context=self.context).data
 
@@ -274,6 +277,7 @@ class Meta:
 class AttachmentSerializer(serializers.ModelSerializer):
     file_url = serializers.SerializerMethodField()
     file_size = serializers.SerializerMethodField()
+    name = serializers.CharField(required=False, allow_blank=True)
 
     def get_file_url(self, obj):
         request = self.context.get('request')

files/views/attachment.py

@@ -3,7 +3,9 @@
 from ..models import Attachment, Media
 from ..serializers import AttachmentSerializer
 from django import forms
+from django.conf import settings
 from django.core.paginator import Paginator
+from django.http import Http404
 from django.views.decorators.http import require_http_methods
 from django.shortcuts import render, get_object_or_404, redirect
 
@@ -18,6 +20,12 @@ class AttachmentViewSet(viewsets.ModelViewSet):
     parser_classes = [MultiPartParser, FormParser]
     permission_classes = [permissions.IsAuthenticatedOrReadOnly]
 
+    def initial(self, request, *args, **kwargs):
+        # Check if attachments feature is enabled
+        if not settings.ENABLE_MEDIA_ATTACHMENTS:
+            raise Http404("Attachments feature is not enabled")
+        super().initial(request, *args, **kwargs)
+
     def get_queryset(self):
         media_id = self.request.query_params.get('media')
         if media_id:
@@ -43,6 +51,10 @@ def perform_destroy(self, instance):
 
 @require_http_methods(["GET", "POST"])
 def edit_attachments(request):
+    # Check if attachments feature is enabled
+    if not settings.ENABLE_MEDIA_ATTACHMENTS:
+        raise Http404("Attachments feature is not enabled")
+    
     media_token = request.GET.get('m')
     media_object = get_object_or_404(Media, friendly_token=media_token)
     attachments_qs = Attachment.objects.filter(media=media_object).order_by('-uploaded_at')

frontend/src/static/js/components/media-actions/AttachmentDownloadLink.jsx

@@ -1,7 +1,7 @@
 import React, { useState, useEffect } from 'react';
 import { usePopup } from '../../utils/hooks/';
 import { SiteContext } from '../../utils/contexts/';
-import { MediaPageStore } from '../../utils/stores/';
+import { MediaPageStore, PageStore } from '../../utils/stores/';
 import { formatInnerLink } from '../../utils/helpers/';
 import { CircleIconButton, MaterialIcon, NavigationContentApp, NavigationMenuList, PopupMain } from '../_shared/';
 import { translateString } from '../../utils/helpers/';
@@ -60,7 +60,12 @@ export function AttachmentDownloadLink(props) {
     };
   }, []);
 
-  if (!hasAttachments) {
+  // Check if attachments feature is enabled via MediaCMS config
+  const features = PageStore.get('config-options')?.features;
+  const attachmentsEnabled = features?.media?.actions?.attachments !== false;
+  
+  // Don't render if feature is disabled or no attachments
+  if (!attachmentsEnabled || !hasAttachments) {
     return null;
   }