fix(payments): add org-wide GET /api/payments/plans and /api/payments/ledger

The workspace Payments tab lists plans and ledger entries at the practice
level, but the controller only exposed patient-scoped routes
(/plans/patient/{id}, /ledger/patient/{id}). GET /api/payments/plans hit
the POST-only /plans mapping (500 "method GET not supported") and
GET /api/payments/ledger had no handler (500 "no endpoint"). Add org-wide
list endpoints backed by new repository + service methods so the tab loads.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: Baranidharan12

src/main/java/org/ciyex/ehr/payment/controller/PaymentController.java

@@ -191,6 +191,17 @@ public ResponseEntity<ApiResponse<Map<String, Object>>> stats() {
 
     // ── Payment Plans ────────────────────────────────────────────────────
 
+    @GetMapping("/plans")
+    public ResponseEntity<ApiResponse<List<PaymentPlanDto>>> listAllPlans() {
+        try {
+            var plans = service.listAllPlans();
+            return ResponseEntity.ok(ApiResponse.ok("Payment plans retrieved", plans));
+        } catch (Exception e) {
+            log.error("Failed to list payment plans", e);
+            return ResponseEntity.ok(ApiResponse.error("Failed: " + e.getMessage()));
+        }
+    }
+
     @GetMapping("/plans/patient/{patientId}")
     public ResponseEntity<ApiResponse<List<PaymentPlanDto>>> listPlans(@PathVariable Long patientId) {
         try {
@@ -245,6 +256,17 @@ public ResponseEntity<ApiResponse<PaymentPlanDto>> cancelPlan(@PathVariable Long
 
     // ── Ledger ───────────────────────────────────────────────────────────
 
+    @GetMapping("/ledger")
+    public ResponseEntity<ApiResponse<List<PatientLedgerDto>>> getAllLedger() {
+        try {
+            var ledger = service.getAllLedger();
+            return ResponseEntity.ok(ApiResponse.ok("Ledger retrieved", ledger));
+        } catch (Exception e) {
+            log.error("Failed to get ledger", e);
+            return ResponseEntity.ok(ApiResponse.error("Failed: " + e.getMessage()));
+        }
+    }
+
     @GetMapping("/ledger/patient/{patientId}")
     public ResponseEntity<ApiResponse<List<PatientLedgerDto>>> getLedger(@PathVariable Long patientId) {
         try {

src/main/java/org/ciyex/ehr/payment/repository/PatientLedgerRepository.java

@@ -11,6 +11,8 @@ public interface PatientLedgerRepository extends JpaRepository<PatientLedger, Lo
 
     List<PatientLedger> findByOrgAliasAndPatientIdOrderByCreatedAtDesc(String orgAlias, Long patientId);
 
+    List<PatientLedger> findByOrgAliasOrderByCreatedAtDesc(String orgAlias);
+
     @Query("SELECT COALESCE(SUM(l.amount),0) FROM PatientLedger l WHERE l.orgAlias = :org AND l.patientId = :pid")
     BigDecimal sumByPatient(@Param("org") String orgAlias, @Param("pid") Long patientId);
 

src/main/java/org/ciyex/ehr/payment/repository/PaymentPlanRepository.java

@@ -9,6 +9,8 @@ public interface PaymentPlanRepository extends JpaRepository<PaymentPlan, Long>
 
     List<PaymentPlan> findByOrgAliasAndPatientIdOrderByCreatedAtDesc(String orgAlias, Long patientId);
 
+    List<PaymentPlan> findByOrgAliasOrderByCreatedAtDesc(String orgAlias);
+
     Optional<PaymentPlan> findByIdAndOrgAlias(Long id, String orgAlias);
 
     List<PaymentPlan> findByOrgAliasAndStatusOrderByNextPaymentDateAsc(String orgAlias, String status);

src/main/java/org/ciyex/ehr/payment/service/PaymentService.java

@@ -320,6 +320,12 @@ public List<PaymentPlanDto> listPlans(Long patientId) {
                 .stream().map(this::toPlanDto).toList();
     }
 
+    @Transactional(readOnly = true)
+    public List<PaymentPlanDto> listAllPlans() {
+        return planRepo.findByOrgAliasOrderByCreatedAtDesc(orgAlias())
+                .stream().map(this::toPlanDto).toList();
+    }
+
     @Transactional
     public PaymentPlanDto updatePlan(Long id, PaymentPlanDto dto) {
         var plan = planRepo.findByIdAndOrgAlias(id, orgAlias())
@@ -356,6 +362,12 @@ public List<PatientLedgerDto> getLedger(Long patientId) {
                 .stream().map(this::toLedgerDto).toList();
     }
 
+    @Transactional(readOnly = true)
+    public List<PatientLedgerDto> getAllLedger() {
+        return ledgerRepo.findByOrgAliasOrderByCreatedAtDesc(orgAlias())
+                .stream().map(this::toLedgerDto).toList();
+    }
+
     @Transactional(readOnly = true)
     public BigDecimal getBalance(Long patientId) {
         return ledgerRepo.sumByPatient(orgAlias(), patientId);