Scheduler and UI fixes

Author: GitTimeraider

app.py

@@ -69,13 +69,38 @@
 scheduler.start()
 atexit.register(lambda: scheduler.shutdown())
 
+def schedule_all_repositories():
+    """Schedule all active repositories on startup"""
+    try:
+        repositories = Repository.query.filter_by(is_active=True).all()
+        scheduled_count = 0
+        for repository in repositories:
+            if repository.schedule_type != 'manual':
+                schedule_backup_job(repository)
+                scheduled_count += 1
+                logger.info(f"Scheduled backup job for repository: {repository.name} ({repository.schedule_type})")
+        logger.info(f"Scheduled {scheduled_count} backup jobs on startup")
+    except Exception as e:
+        logger.error(f"Error scheduling repositories on startup: {e}")
+
+# Flag to ensure we only initialize once
+_scheduler_initialized = False
+
+def ensure_scheduler_initialized():
+    """Ensure scheduler is initialized with existing repositories"""
+    global _scheduler_initialized
+    if not _scheduler_initialized:
+        schedule_all_repositories()
+        _scheduler_initialized = True
+
 @login_manager.user_loader
 def load_user(user_id):
     return User.query.get(int(user_id))
 
 @app.route('/')
 @login_required
 def dashboard():
+    ensure_scheduler_initialized()
     repositories = Repository.query.filter_by(user_id=current_user.id).all()
     recent_jobs = BackupJob.query.filter_by(user_id=current_user.id).order_by(BackupJob.created_at.desc()).limit(10).all()
     return render_template('dashboard.html', repositories=repositories, recent_jobs=recent_jobs)
@@ -390,6 +415,24 @@ def backup_jobs():
 def health_check():
     return jsonify({'status': 'healthy', 'timestamp': datetime.utcnow().isoformat()})
 
+@app.route('/api/scheduler/status')
+@login_required
+def scheduler_status():
+    """Debug endpoint to check scheduled jobs"""
+    jobs = []
+    for job in scheduler.get_jobs():
+        jobs.append({
+            'id': job.id,
+            'name': job.name,
+            'next_run': job.next_run_time.isoformat() if job.next_run_time else None,
+            'trigger': str(job.trigger)
+        })
+    return jsonify({
+        'scheduler_running': scheduler.running,
+        'scheduled_jobs': jobs,
+        'total_jobs': len(jobs)
+    })
+
 @app.route('/api/theme', methods=['POST'])
 @login_required
 def update_theme():

templates/edit_repository.html

@@ -91,11 +91,12 @@ <h4 class="mb-0">
 
                     <div class="row">
                         <div class="col-12 mb-3">
-                        <label for="retention_count" class="form-label">Retention Count <span class="text-danger">*</span></label>
-                        <input type="number" class="form-control" id="retention_count" name="retention_count" 
-                               value="{{ repository.retention_count }}" min="1" max="50" required>
-                        <div class="form-text">
-                            Number of backup versions to keep (older backups will be automatically deleted)
+                            <label for="retention_count" class="form-label">Retention Count <span class="text-danger">*</span></label>
+                            <input type="number" class="form-control" id="retention_count" name="retention_count" 
+                                   value="{{ repository.retention_count }}" min="1" max="50" required>
+                            <div class="form-text">
+                                Number of backup versions to keep (older backups will be automatically deleted)
+                            </div>
                         </div>
                     </div>
 

test_fixes.py

@@ -0,0 +1,70 @@
+#!/usr/bin/env python3
+"""
+Test script to verify the fixes for template and scheduler issues
+"""
+
+import os
+import sys
+from flask import Flask
+from jinja2 import Environment, FileSystemLoader, TemplateError
+
+def test_template_syntax():
+    """Test if the edit_repository.html template has valid syntax"""
+    print("Testing template syntax...")
+    
+    try:
+        # Create a Jinja2 environment
+        template_dir = os.path.join(os.path.dirname(__file__), 'templates')
+        env = Environment(loader=FileSystemLoader(template_dir))
+        
+        # Try to load and parse the template
+        template = env.get_template('edit_repository.html')
+        print("✅ edit_repository.html syntax is valid")
+        return True
+        
+    except TemplateError as e:
+        print(f"❌ Template syntax error: {e}")
+        return False
+    except Exception as e:
+        print(f"❌ Error loading template: {e}")
+        return False
+
+def test_app_imports():
+    """Test if the app can be imported without errors"""
+    print("Testing app imports...")
+    
+    try:
+        # Set environment variables to avoid missing config errors
+        os.environ.setdefault('SECRET_KEY', 'test-key')
+        os.environ.setdefault('DATABASE_URL', 'sqlite:///test.db')
+        
+        # Try to import the app
+        sys.path.insert(0, os.path.dirname(__file__))
+        import app
+        print("✅ App imports successfully")
+        return True
+        
+    except Exception as e:
+        print(f"❌ Error importing app: {e}")
+        return False
+
+def main():
+    print("Running fix verification tests...\n")
+    
+    template_ok = test_template_syntax()
+    app_ok = test_app_imports()
+    
+    print(f"\n{'='*50}")
+    print("Test Results:")
+    print(f"Template syntax: {'✅ PASS' if template_ok else '❌ FAIL'}")
+    print(f"App imports: {'✅ PASS' if app_ok else '❌ FAIL'}")
+    
+    if template_ok and app_ok:
+        print("\n🎉 All tests passed! The fixes should work correctly.")
+        return 0
+    else:
+        print("\n⚠️  Some tests failed. Please check the errors above.")
+        return 1
+
+if __name__ == '__main__':
+    sys.exit(main())