fix: remove duplicate/orphaned code left by merge conflict resolution in getQuickStats

The conflict resolution for #994 merged in both the old per-role query
code and the new #911 single-query optimization instead of cleanly
replacing one with the other, leaving an unused buyerTransactions
variable and a duplicate totalProperties/activeListings declaration.

Author: abdoolyaro

src/dashboard/dashboard.service.ts

@@ -73,18 +73,6 @@ export class DashboardService {
   }
 
   private async getQuickStats(userId: string): Promise<QuickStatsDto> {
-    // Get user's properties
-    const properties = await this.prisma.property.findMany({
-      where: { ownerId: userId },
-    });
-
-    const totalProperties = properties.length;
-    const activeListings = properties.filter((p) => p.status === 'ACTIVE').length;
-
-    // Get user's transactions (both as buyer and seller)
-    const buyerTransactions = await this.prisma.transaction.findMany({
-      where: { buyerId: userId },
-    });
     // Issue #911 – Replace separate per-role queries and in-memory aggregation
     // with a single grouped count query + a single transaction query using OR.