Update app.py

app.py

@@ -221,7 +221,7 @@ def get_font_path(font_family):
 # =============================================
 
 def create_text_overlay(input_video, output_video, text_style):
-    """Add text overlay using ASS subtitles - FIXED for subfolders"""
+    """Add text overlay using ASS subtitles - FIXED with full font path"""
     font_path = get_font_path(text_style.font_family)
     if not font_path:
         print(f"⚠️ Font not found: {text_style.font_family}")
@@ -248,7 +248,7 @@ def create_text_overlay(input_video, output_video, text_style):
     bg_color_name = bg_parts[0]
     bg_opacity = float(bg_parts[1]) if len(bg_parts) > 1 else 0.5
     bg_color_hex = color_map.get(bg_color_name.lower(), "000000")
-    bg_alpha = int((1 - bg_opacity) * 255)
+    bg_alpha = int((1 - bg_opacity) * 255)  # ASS opacity
     
     # Map position to ASS alignment
     pos_map = {
@@ -263,10 +263,37 @@ def create_text_overlay(input_video, output_video, text_style):
     margin_r = text_style.margin if alignment in [3,6,9] else 0
     margin_v = text_style.margin
     
-    # Get just the filename without extension and path
-    font_name = os.path.basename(font_path).split('.')[0]
+    # IMPORTANT: For ASS, we need to use the actual font filename
+    # But FFmpeg needs the font to be accessible via fontconfig
+    # Let's create a fontconfig configuration file
     
-    # Create ASS file content with proper font path
+    font_dir = os.path.dirname(font_path)
+    font_file = os.path.basename(font_path)
+    font_name = os.path.splitext(font_file)[0]
+    
+    # Create a fontconfig configuration file
+    fc_config = f"""<?xml version="1.0"?>
+<!DOCTYPE fontconfig SYSTEM "fonts.dtd">
+<fontconfig>
+    <dir>{font_dir}</dir>
+    <match target="pattern">
+        <test qual="any" name="family">
+            <string>{font_name}</string>
+        </test>
+        <edit name="family" mode="assign" binding="same">
+            <string>{font_name}</string>
+        </edit>
+    </match>
+</fontconfig>"""
+    
+    fc_file = os.path.join(work_dir, "fonts.conf")
+    with open(fc_file, 'w') as f:
+        f.write(fc_config)
+    
+    # Set environment variable for fontconfig
+    os.environ['FONTCONFIG_FILE'] = fc_file
+    
+    # Create ASS file content
     ass_content = f"""[Script Info]
 ; Script generated by Video Styling Space
 ScriptType: v4.00+
@@ -287,8 +314,13 @@ Dialogue: 0,0:00:00.00,0:00:10.00,Default,,0,0,0,,{text_style.text}"""
         f.write(ass_content)
     
     print(f"📝 Created ASS subtitle file")
+    print(f"📝 Fontconfig file created at: {fc_file}")
     
-    # Use FFmpeg to burn subtitles
+    # First, test if the font is recognized by fontconfig
+    fc_list = subprocess.run(['fc-list', ':' + font_name], capture_output=True, text=True)
+    print(f"📋 Fontconfig lookup: {fc_list.stdout}")
+    
+    # Use FFmpeg with the ASS file
     cmd = [
         'ffmpeg', '-y',
         '-i', input_video,
@@ -302,7 +334,37 @@ Dialogue: 0,0:00:00.00,0:00:10.00,Default,,0,0,0,,{text_style.text}"""
     
     if result.returncode != 0:
         print(f"❌ FFmpeg error: {result.stderr}")
-        return False
+        
+        # Try alternative: use drawtext with direct font path
+        print("🔄 Trying alternative drawtext method...")
+        
+        # Position mapping for drawtext
+        drawtext_pos = {
+            "bottom-left": "x=20:y=h-th-20",
+            "bottom-center": "x=(w-tw)/2:y=h-th-20",
+            "bottom-right": "x=w-tw-20:y=h-th-20",
+            "center": "x=(w-tw)/2:y=(h-th)/2",
+            "top-left": "x=20:y=20",
+            "top-center": "x=(w-tw)/2:y=20",
+            "top-right": "x=w-tw-20:y=20"
+        }
+        position = drawtext_pos.get(text_style.position, "x=(w-tw)/2:y=(h-th)/2")
+        
+        drawtext_cmd = [
+            'ffmpeg', '-y',
+            '-i', input_video,
+            '-vf', f"drawtext=text='{text_style.text}':fontfile={font_path}:fontsize={text_style.font_size}:fontcolor={text_style.color}:{position}:box=1:boxcolor={bg_color_name}@{bg_opacity}:boxborderw={text_style.padding}",
+            '-c:a', 'copy',
+            output_video
+        ]
+        
+        result2 = subprocess.run(drawtext_cmd, capture_output=True, text=True)
+        if result2.returncode == 0:
+            print(f"✅ Alternative drawtext method succeeded")
+            return True
+        else:
+            print(f"❌ Alternative drawtext also failed: {result2.stderr}")
+            return False
     
     print(f"✅ Video styled successfully with ASS subtitles")
     return True