Skip to content

Commit

Permalink
Zoom and filter to clicked POIs
Browse files Browse the repository at this point in the history
  • Loading branch information
jmccollum-woolpert committed Jul 10, 2024
1 parent 85781bd commit 9526d48
Show file tree
Hide file tree
Showing 11 changed files with 86 additions and 20 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,11 @@ export const removeFilter = createAction(
props<{ filter: ActiveFilter }>()
);

export const setFilters = createAction(
'[RoutesChart] Set Filters',
props<{ filters: ActiveFilter[] }>()
);

export const selectRange = createAction(
'[RoutesChart] Select Range',
props<{ rangeIndex: number }>()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -113,6 +113,11 @@ export const reducer = createReducer(
filters: state.filters.filter((f) => !(f.id === filter.id && f.params === filter.params)),
pageIndex: 0,
})),
on(RoutesChartActions.setFilters, (state, { filters }) => ({
...state,
filters,
pageIndex: 0,
})),
on(
RoutesChartActions.anchorRangeOffset,
RoutesChartActions.nextRangeOffset,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,11 @@ const selectVisitRequests = createSelector(
(ids: number[] | string[], visitRequests) => ids.map((id) => visitRequests[id])
);

const selectVisitRequestsByIds = (ids: number[]) =>
createSelector(fromVisitRequest.selectEntities, (visitRequests) =>
ids.map((id) => visitRequests[id])
);

const selectVisitForEdit = (id: number, shipmentRouteId: number) =>
createSelector(
selectById(id),
Expand Down Expand Up @@ -85,6 +90,7 @@ export const VisitSelectors = {
selectDeliveryByShipmentId,
selectPickupByShipmentId,
selectChangedVisitsFromIds,
selectVisitRequestsByIds,
};

export default VisitSelectors;
6 changes: 4 additions & 2 deletions application/frontend/src/app/core/services/map.service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -105,11 +105,13 @@ export class MapService {
);
}

setBounds(bounds: google.maps.LatLngBounds): void {
setBounds(bounds: google.maps.LatLngBounds, save = true): void {
if (bounds) {
this.map?.fitBounds(bounds);
}
this.bounds = bounds;
if (save) {
this.bounds = bounds;
}
}

zoomToHome(): void {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,7 @@
[timezoneOffset]="timezoneOffset"
[pendingNewPois]="pendingNewPois"
[pendingOldVisitIds]="pendingOldVisitIds"
[color]="color">
[color]="color"
(clickVisitIds)="clickVisitIds.emit($event)">
</app-points-of-interest>
</div>
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,7 @@ export class BaseRoutesRowComponent implements OnInit {
@Output() selectedChange = new EventEmitter<boolean>();
@Output() editVehicle = new EventEmitter<number>();
@Output() viewMetadata = new EventEmitter<number>();
@Output() clickVisitIds = new EventEmitter<number[]>();
@HostBinding('class') className = 'item item-container';
allowExperimentalFeatures: boolean;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,5 +16,6 @@
[range]="range$ | async"
[color]="color$ | async"
(editVehicle)="onEditVehicle($event)"
(viewMetadata)="onViewMetadata($event)">
(viewMetadata)="onViewMetadata($event)"
(clickVisitIds)="onClickVisitIds($event)">
</app-base-routes-row>
Original file line number Diff line number Diff line change
Expand Up @@ -23,12 +23,10 @@ import * as Long from 'long';
import {
PointOfInterest,
PointOfInterestClick,
PointOfInterestStartDrag,
ShipmentRoute,
Timeline,
ChangedVisits,
Vehicle,
PointOfInterestTimelineOverlapBegin,
} from 'src/app/core/models';
import * as fromConfig from 'src/app/core/selectors/config.selectors';
import * as fromPointOfInterest from 'src/app/core/selectors/point-of-interest.selectors';
Expand All @@ -39,10 +37,11 @@ import * as fromTimeline from 'src/app/core/selectors/timeline.selectors';
import * as fromVehicle from 'src/app/core/selectors/vehicle.selectors';
import VisitSelectors from 'src/app/core/selectors/visit.selectors';
import { SharedModule } from 'src/app/shared/shared.module';
import { ValidationService } from '../../../core/services';
import { MapService, ValidationService } from '../../../core/services';
import { RoutesRowComponent } from './routes-row.component';
import * as fromDispatcher from 'src/app/core/selectors/dispatcher.selectors';
import ShipmentModelSelectors from '../../../core/selectors/shipment-model.selectors';
import { MockMapService } from 'src/test/service-mocks';

@Component({
selector: 'app-base-routes-row',
Expand Down Expand Up @@ -80,6 +79,7 @@ describe('RoutesRowComponent', () => {
imports: [RouterTestingModule, SharedModule],
declarations: [MockBaseRoutesRowComponent, RoutesRowComponent],
providers: [
{ provide: MapService, useClass: MockMapService },
{
provide: ValidationService,
useValue: jasmine.createSpyObj('validationService', ['getErrorEntityIds']),
Expand Down Expand Up @@ -141,6 +141,13 @@ describe('RoutesRowComponent', () => {
)
);

spyOn(VisitSelectors, 'selectVisitRequestsByIds').and.returnValue(
createSelector(
() => null,
(_state) => []
)
);

fixture.detectChanges();
});

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,12 +45,13 @@ import * as fromVehicle from 'src/app/core/selectors/vehicle.selectors';
import * as fromRoot from 'src/app/reducers';
import RequestSettingsSelectors from 'src/app/core/selectors/request-settings.selectors';
import VisitSelectors from 'src/app/core/selectors/visit.selectors';
import { ValidationService } from 'src/app/core/services';
import { MapService, ValidationService } from 'src/app/core/services';
import { PostSolveMetricsActions } from 'src/app/core/actions';
import { Router } from '@angular/router';
import { Page } from 'src/app/core/models';
import { durationSeconds } from 'src/app/util';
import * as fromDispatcher from 'src/app/core/selectors/dispatcher.selectors';
import { NumberFilterParams } from 'src/app/shared/models';

@Component({
selector: 'app-routes-row',
Expand Down Expand Up @@ -80,7 +81,8 @@ export class RoutesRowComponent implements OnChanges, OnInit, OnDestroy {
constructor(
private router: Router,
private store: Store<fromRoot.State>,
private validationService: ValidationService
private validationService: ValidationService,
private mapService: MapService
) {}

ngOnChanges(changes: SimpleChanges): void {
Expand Down Expand Up @@ -215,4 +217,34 @@ export class RoutesRowComponent implements OnChanges, OnInit, OnDestroy {
this.store.dispatch(PostSolveMetricsActions.showMetadataForRoute({ id }));
this.router.navigateByUrl('/' + Page.RoutesMetadata, { skipLocationChange: true });
}

onClickVisitIds(ids: number[]): void {
this.store.dispatch(
RoutesChartActions.setFilters({
filters: [
{
id: 'routeId',
label: `Route ID = ${this.route.id}`,
params: {
operation: '=',
value: this.route.id,
} as NumberFilterParams,
},
],
})
);

this.store
.pipe(select(VisitSelectors.selectVisitRequestsByIds(ids)))
.subscribe((visitRequests) => {
const bounds = new google.maps.LatLngBounds();
visitRequests.forEach((vr) =>
bounds.extend({
lat: vr.arrivalWaypoint.location.latLng.latitude,
lng: vr.arrivalWaypoint.location.latLng.longitude,
})
);
this.mapService.setBounds(bounds, false);
});
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -28,11 +28,7 @@ import {
ViewChild,
} from '@angular/core';
import * as Long from 'long';
import {
PointOfInterest,
PointOfInterestCategory,
ChangedVisits,
} from 'src/app/core/models';
import { PointOfInterest, PointOfInterestCategory, ChangedVisits } from 'src/app/core/models';
import { defaultTimeFormat, formatSecondsDate, timeToPixel } from 'src/app/util';
import { Cluster, PointsOfInterestImageAttribute, pointsOfInterestImages } from '../../models';

Expand Down Expand Up @@ -67,6 +63,7 @@ export class PointsOfInterestComponent implements OnChanges {
@Input() routeId: number;
@Input() changedVisits: ChangedVisits;
@Input() color: string;
@Output() clickVisitIds = new EventEmitter<number[]>();

get imageAttributeLookup(): { [key: string]: PointsOfInterestImageAttribute } {
return PointsOfInterestComponent.imageAttributeLookup;
Expand Down Expand Up @@ -118,12 +115,20 @@ export class PointsOfInterestComponent implements OnChanges {
// Not the primary button
return;
}
console.log(event, point)
// this.pointOfInterestClick.emit({
// category: point[1],
// visitId: point[0],
// relativeTo: event.target as Element,
// });

const visitIds = [point[0]];
const cluster = this.clusters.find(
(cluster) => cluster.start <= point[2] && cluster.end >= point[2]
);
if (cluster) {
this.points.forEach((p) => {
if (cluster.start <= p[2] && cluster.end >= p[2]) {
visitIds.push(p[0]);
}
});
}

this.clickVisitIds.emit(visitIds);
event.preventDefault();
event.stopPropagation();
}
Expand Down
1 change: 1 addition & 0 deletions application/frontend/src/config.json
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@
},
"options": {
"zoom": 12,
"maxZoom": 18,
"center": {
"lat": 48.8566,
"lng": 2.3522
Expand Down

0 comments on commit 9526d48

Please sign in to comment.