from flask import request, current_app
from utils import get_client_ip

class Activity:
    def __init__(self, db_connection_method):
        self.execute = db_connection_method

    def log(self, person_id, action, entity_type=None, entity_id=None, details=None):
        """Records an action in the activity_log table."""
        try:
            ip_address = get_client_ip()
            user_agent = request.user_agent.string if request else None
            sql = """
                INSERT INTO activity_log (person_id, action, entity_type, entity_id, details, ip_address, user_agent)
                VALUES (%s, %s, %s, %s, %s, %s, %s)
            """
            self.execute(sql, [person_id, action, entity_type, entity_id, details, ip_address, user_agent], commit=True)
        except Exception as e:
            # We don't want logging to break the main application flow
            current_app.logger.error(f"Error logging activity: {e}")

    def get_recent_logs(self, limit=50, offset=0):
        """Fetches recent activity logs with person names, supporting pagination."""
        query = """
            SELECT 
                al.id,
                al.person_id,
                p.name as person_name,
                al.action,
                al.entity_type,
                al.entity_id,
                al.details,
                al.ip_address,
                al.user_agent,
                al.timestamp
            FROM activity_log al
            LEFT JOIN person p ON al.person_id = p.person_id
            ORDER BY al.timestamp DESC
            LIMIT %s OFFSET %s
        """
        return self.execute(query, [limit, offset])