EidanThen/rutero
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: contar pdv unico #2

Merged
EidanThen merged 1 commits from dev into main 2026-09-01 15:59:11 +00:00
Conversation 0 Commits 1 Files Changed 1
+17 -5
1 changed files with 17 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 65fe788281 - Show all commits
route-planner/src/app/private/home/planning-kpis/planning-kpis.component.ts
+17 -5
View File
@@ -86,7 +86,13 @@ export class PlanningKpisComponent implements OnInit, OnDestroy {
private refresh(): void { private refresh(): void {
const assigned = this.routeState.getAssignedSnapshot(); const assigned = this.routeState.getAssignedSnapshot();
const unassigned = this.routeState.getNoAssignedSnapshot(); const unassigned = this.routeState.getNoAssignedSnapshot();
const totalPdvs = assigned.length + unassigned.length; const assignedPdvIds = new Set(assigned.map(route => String(route.json.pdv_id)));
// A PDV with at least one scheduled visit is assigned; repeated weekly visits and
// partial-assignment alerts must not inflate the planning coverage counters.
const unassignedPdvIds = new Set(unassigned
.map(route => String(route.pdv_id))
.filter(pdvId => !assignedPdvIds.has(pdvId)));
const totalPdvs = new Set([...assignedPdvIds, ...unassignedPdvIds]).size;
const dailyRoutes = summarizeDailyRoutes(assigned); const dailyRoutes = summarizeDailyRoutes(assigned);
const totalDistance = dailyRoutes.reduce((sum, day) => sum + day.distanceKm, 0); const totalDistance = dailyRoutes.reduce((sum, day) => sum + day.distanceKm, 0);
const totalTravelHours = dailyRoutes.reduce((sum, day) => sum + day.travelHours, 0); const totalTravelHours = dailyRoutes.reduce((sum, day) => sum + day.travelHours, 0);
@@ -134,9 +140,9 @@ export class PlanningKpisComponent implements OnInit, OnDestroy {
this.summary = { this.summary = {
totalPdvs, totalPdvs,
assignedPdvs: assigned.length, assignedPdvs: assignedPdvIds.size,
unassignedPdvs: unassigned.length, unassignedPdvs: unassignedPdvIds.size,
coverage: totalPdvs ? (assigned.length / totalPdvs) * 100 : 0, coverage: totalPdvs ? (assignedPdvIds.size / totalPdvs) * 100 : 0,
users, users,
days, days,
totalDistance, totalDistance,
@@ -149,9 +155,15 @@ export class PlanningKpisComponent implements OnInit, OnDestroy {
assignmentBalance assignmentBalance
}; };
const reasonCounts = new Map<string, number>(); const reasonByPdv = new Map<string, string>();
for (const route of unassigned) { for (const route of unassigned) {
const pdvId = String(route.pdv_id);
if (assignedPdvIds.has(pdvId) || reasonByPdv.has(pdvId)) continue;
const reason = route.motivo?.trim() || 'Sin motivo especificado'; const reason = route.motivo?.trim() || 'Sin motivo especificado';
reasonByPdv.set(pdvId, reason);
}
const reasonCounts = new Map<string, number>();
for (const reason of reasonByPdv.values()) {
reasonCounts.set(reason, (reasonCounts.get(reason) ?? 0) + 1); reasonCounts.set(reason, (reasonCounts.get(reason) ?? 0) + 1);
} }
this.reasons = Array.from(reasonCounts.entries()) this.reasons = Array.from(reasonCounts.entries())