Browse Source

Respect activity type and date range filter

pull/7492/head
Thomas Kaul 1 month ago
parent
commit
c2e6d5aca7
  1. 5
      apps/client/src/app/app.component.ts
  2. 18
      apps/client/src/app/pages/portfolio/activities/activities-page.component.ts

5
apps/client/src/app/app.component.ts

@ -135,7 +135,10 @@ export class GfAppComponent implements OnInit {
this.currentSubRoute === this.currentSubRoute ===
internalRoutes.home.subRoutes?.holdings.path) || internalRoutes.home.subRoutes?.holdings.path) ||
(this.currentRoute === internalRoutes.portfolio.path && (this.currentRoute === internalRoutes.portfolio.path &&
!this.currentSubRoute)) && !this.currentSubRoute) ||
(this.currentRoute === internalRoutes.portfolio.path &&
this.currentSubRoute ===
internalRoutes.portfolio.subRoutes?.activities.path)) &&
this.user?.settings?.viewMode !== 'ZEN' this.user?.settings?.viewMode !== 'ZEN'
) { ) {
this.hasPermissionToChangeDateRange = true; this.hasPermissionToChangeDateRange = true;

18
apps/client/src/app/pages/portfolio/activities/activities-page.component.ts

@ -10,7 +10,6 @@ import {
} from '@ghostfolio/common/interfaces'; } from '@ghostfolio/common/interfaces';
import { hasPermission, permissions } from '@ghostfolio/common/permissions'; import { hasPermission, permissions } from '@ghostfolio/common/permissions';
import { internalRoutes } from '@ghostfolio/common/routes/routes'; import { internalRoutes } from '@ghostfolio/common/routes/routes';
import { DateRange } from '@ghostfolio/common/types';
import { GfActivitiesTableComponent } from '@ghostfolio/ui/activities-table'; import { GfActivitiesTableComponent } from '@ghostfolio/ui/activities-table';
import { GfFabComponent } from '@ghostfolio/ui/fab'; import { GfFabComponent } from '@ghostfolio/ui/fab';
import { DataService } from '@ghostfolio/ui/services'; import { DataService } from '@ghostfolio/ui/services';
@ -124,7 +123,7 @@ export class GfActivitiesPageComponent implements OnInit {
? this.activityTypesFilter ? this.activityTypesFilter
: undefined, : undefined,
filters: this.userService.getFilters(), filters: this.userService.getFilters(),
range: this.getCalendarYearRange() range: this.getDateRange()
}) })
.pipe(takeUntilDestroyed(this.destroyRef)) .pipe(takeUntilDestroyed(this.destroyRef))
.subscribe(() => { .subscribe(() => {
@ -287,7 +286,7 @@ export class GfActivitiesPageComponent implements OnInit {
? this.activityTypesFilter ? this.activityTypesFilter
: undefined, : undefined,
filters: this.userService.getFilters(), filters: this.userService.getFilters(),
range: this.getCalendarYearRange(), range: this.getDateRange(),
skip: this.pageIndex * this.pageSize, skip: this.pageIndex * this.pageSize,
sortColumn: this.sortColumn, sortColumn: this.sortColumn,
sortDirection: this.sortDirection, sortDirection: this.sortDirection,
@ -312,18 +311,11 @@ export class GfActivitiesPageComponent implements OnInit {
}); });
} }
private getCalendarYearRange() { private getDateRange() {
const dateRange = this.user?.settings?.dateRange; const dateRange = this.user?.settings?.dateRange;
return this.isCalendarYear(dateRange) ? dateRange : undefined; // Omit 'max' as it would exclude drafts, which are dated in the future
} return dateRange === 'max' ? undefined : dateRange;
private isCalendarYear(dateRange?: DateRange) {
if (!dateRange) {
return false;
}
return /^\d{4}$/.test(dateRange);
} }
private updateUser(aUser: User) { private updateUser(aUser: User) {

Loading…
Cancel
Save