Skip to content

Commit

Permalink
Fix timezone conversion logic (#11444)
Browse files Browse the repository at this point in the history
* Fix timezone conversion logic

* Use function for consistency

* Cleanup
  • Loading branch information
NickM-27 committed May 20, 2024
1 parent 82d4bf8 commit 2a16d55
Show file tree
Hide file tree
Showing 5 changed files with 23 additions and 27 deletions.
9 changes: 4 additions & 5 deletions web/src/components/player/PreviewPlayer.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ import useSWR from "swr";
import { FrigateConfig } from "@/types/frigateConfig";
import { Preview } from "@/types/preview";
import { PreviewPlayback } from "@/types/playback";
import { getUTCOffset, isCurrentHour } from "@/utils/dateUtil";
import { getTimestampOffset, isCurrentHour } from "@/utils/dateUtil";
import { baseUrl } from "@/api/baseUrl";
import { isAndroid, isChrome, isMobile } from "react-device-detect";
import { TimeRange } from "@/types/timeline";
Expand Down Expand Up @@ -41,8 +41,7 @@ export default function PreviewPlayer({
const [currentHourFrame, setCurrentHourFrame] = useState<string>();

const currentPreview = useMemo(() => {
const timeRangeOffset =
(getUTCOffset(new Date(timeRange.before * 1000)) % 60) * 60;
const timeRangeOffset = getTimestampOffset(timeRange.before);

return cameraPreviews.find(
(preview) =>
Expand Down Expand Up @@ -233,8 +232,8 @@ function PreviewVideoPlayer({
return;
}

const timeRangeOffset =
getUTCOffset(new Date(timeRange.before * 1000)) % 60;
// account for minutes offset in timezone
const timeRangeOffset = getTimestampOffset(timeRange.before);

const preview = cameraPreviews.find(
(preview) =>
Expand Down
5 changes: 2 additions & 3 deletions web/src/components/player/dynamic/DynamicVideoPlayer.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ import ActivityIndicator from "@/components/indicators/activity-indicator";
import { VideoResolutionType } from "@/types/live";
import axios from "axios";
import { cn } from "@/lib/utils";
import { getUTCOffset } from "@/utils/dateUtil";
import { getTimestampOffset } from "@/utils/dateUtil";

/**
* Dynamically switches between video playback and scrubbing preview player.
Expand Down Expand Up @@ -149,8 +149,7 @@ export default function DynamicVideoPlayer({
// state of playback player

const recordingParams = useMemo(() => {
const timeRangeOffset =
(getUTCOffset(new Date(timeRange.before * 1000)) % 60) * 60;
const timeRangeOffset = getTimestampOffset(timeRange.before);

return {
before: timeRange.before + timeRangeOffset,
Expand Down
9 changes: 5 additions & 4 deletions web/src/pages/Events.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ import {
ReviewSeverity,
ReviewSummary,
} from "@/types/review";
import { getUTCOffset } from "@/utils/dateUtil";
import { getTimestampOffset } from "@/utils/dateUtil";
import EventView from "@/views/events/EventView";
import { RecordingView } from "@/views/events/RecordingView";
import axios from "axios";
Expand Down Expand Up @@ -167,7 +167,8 @@ export default function Events() {
return undefined;
}

const timezoneMinuteOffset = (getUTCOffset(new Date()) % 60) * 60;
// offset by timezone minutes
const timestampOffset = getTimestampOffset(Date.now() / 1000);

const startDate = new Date();
startDate.setMinutes(0, 0, 0);
Expand All @@ -176,8 +177,8 @@ export default function Events() {
endDate.setHours(0, 0, 0, 0);

return {
after: startDate.getTime() / 1000 + timezoneMinuteOffset,
before: endDate.getTime() / 1000 + timezoneMinuteOffset,
after: startDate.getTime() / 1000 + timestampOffset,
before: endDate.getTime() / 1000 + timestampOffset,
};
}, [reviews]);

Expand Down
9 changes: 9 additions & 0 deletions web/src/utils/dateUtil.ts
Original file line number Diff line number Diff line change
Expand Up @@ -269,6 +269,15 @@ export const getUTCOffset = (
);
};

/**
* Gets the minute offset in seconds of the current timezone from UTC.
* Any timezones with an offset in hours will return 0,
* any timezones with an offset of 30 or 45 minutes will return that amount in seconds.
*/
export function getTimestampOffset(timestamp: number) {
return (getUTCOffset(new Date(timestamp * 1000)) % 60) * 60;
}

export function getRangeForTimestamp(timestamp: number) {
const date = new Date(timestamp * 1000);
date.setMinutes(0, 0, 0);
Expand Down
18 changes: 3 additions & 15 deletions web/src/utils/timelineUtil.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ import {
MdOutlinePictureInPictureAlt,
} from "react-icons/md";
import { FaBicycle } from "react-icons/fa";
import { endOfHourOrCurrentTime, getUTCOffset } from "./dateUtil";
import { endOfHourOrCurrentTime } from "./dateUtil";
import { TimeRange, Timeline } from "@/types/timeline";

export function getTimelineIcon(timelineItem: Timeline) {
Expand Down Expand Up @@ -131,25 +131,13 @@ export function getChunkedTimeDay(timeRange: TimeRange): TimeRange[] {
endOfThisHour.setSeconds(0, 0);
const data: TimeRange[] = [];
const startDay = new Date(timeRange.after * 1000);
const timezoneMinuteOffset =
getUTCOffset(new Date(timeRange.before * 1000)) % 60;

if (timezoneMinuteOffset == 0) {
startDay.setMinutes(0, 0, 0);
} else {
startDay.setSeconds(0, 0);
}
startDay.setMinutes(0, 0, 0);

let start = startDay.getTime() / 1000;
let end = 0;

for (let i = 0; i < 24; i++) {
startDay.setHours(
startDay.getHours() + 1,
Math.abs(timezoneMinuteOffset),
0,
0,
);
startDay.setHours(startDay.getHours() + 1, 0, 0, 0);

if (startDay > endOfThisHour) {
break;
Expand Down

0 comments on commit 2a16d55

Please sign in to comment.