optimize and add new service of get report and download report.

controllers/report_controller.py

@@ -1,12 +1,10 @@
-from flask import Blueprint, render_template, request, jsonify,send_file
+from flask import Blueprint, render_template, request, jsonify, send_file
 from flask_login import login_required, current_user
-
+from services.ReportService import ReportService
 from model.Report import ReportHelper
-from model.Log import LogHelper
 
 report_bp = Blueprint("report", __name__)
 
-
 # ---------------- Report Page ----------------
 @report_bp.route("/report")
 @login_required
@@ -19,36 +17,70 @@ def report_page():
 @login_required
 def search_contractor():
 
-    subcontractor_name = request.form.get("subcontractor_name")
-
-    LogHelper.log_action(
-        "Search Contractor",
-        f"User {current_user.id} searched contractor '{subcontractor_name}'"
-    )
-
     data = ReportHelper.search_contractor(request)
 
-    return jsonify(data)
+    # Pagination (basic)
+    page = int(request.form.get("page", 1))
+    per_page = 20
+
+    start = (page - 1) * per_page
+    end = start + per_page
+
+    paginated_data = data[start:end]
+
+    return jsonify({
+        "data": paginated_data,
+        "total": len(data)
+    })
 
 # ---------------- Contractor Report by contractor id ----------------
 @report_bp.route('/contractor_report/<int:contractor_id>')
 @login_required
 def contractor_report(contractor_id):
 
-    data = ReportHelper.get_contractor_report(contractor_id)
+    service = ReportService(contractor_id=contractor_id).load_data()
 
     return render_template(
         'subcontractor_report.html',
         contractor_id=contractor_id,
-        **data
+        **service.get_web_data()
+    )
+
+# ---------------- Contractor Report by pmc no ----------------
+@report_bp.route("/pmc_report/<pmc_no>")
+@login_required
+def pmc_report(pmc_no):
+
+    service = ReportService(pmc_no=pmc_no).load_data()
+
+    return render_template(
+        "pmc_report.html",
+        **service.get_web_data()
     )
 
 # ---------------- Contractor Download Report by contractor id ----------------
 @report_bp.route('/download_report/<int:contractor_id>')
+@login_required
 def download_report(contractor_id):
-    output_file, error = ReportHelper.create_contractor_report(contractor_id)
-    
+
+    service = ReportService(contractor_id=contractor_id).load_data()
+
+    file, error = service.download_excel()
+
     if error:
         return error, 404
 
-    return send_file(output_file, as_attachment=True)
+    return send_file(file, as_attachment=True)
+
+# ---------------- Contractor Download Report by pmc no ----------------
+@report_bp.route("/download_pmc_report/<pmc_no>")
+@login_required
+def download_pmc_report(pmc_no):
+
+    service = ReportService(pmc_no=pmc_no).load_data()
+
+    file, error = service.download_excel()
+    if error:
+        return error, 404
+
+    return send_file(file, as_attachment=True)