Skip to content

Commit d264c14

Browse files
committed
chore: remove endRelativeToNow completely
1 parent 3192dd3 commit d264c14

File tree

5 files changed

+6
-51
lines changed

5 files changed

+6
-51
lines changed

packages/corelib/src/dataModel/PieceInstance.ts

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -61,18 +61,10 @@ export interface PieceInstance {
6161
dynamicallyInserted?: Time
6262

6363
/** This is set when the duration needs to be overriden from some user action */
64-
userDuration?:
65-
| {
66-
/** The time relative to the part (milliseconds since start of part) */
67-
endRelativeToPart: number
68-
}
69-
| {
70-
/**
71-
* The time relative to 'now' (ms since 'now')
72-
* @deprecated Since 1.53.0; use `endRelativeToPart` instead.
73-
*/
74-
endRelativeToNow: number
75-
}
64+
userDuration?: {
65+
/** The time relative to the part (milliseconds since start of part) */
66+
endRelativeToPart: number
67+
}
7668

7769
/** The time the system started playback of this part, undefined if not yet played back (milliseconds since epoch) */
7870
reportedStartedPlayback?: Time

packages/corelib/src/playout/processAndPrune.ts

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -247,11 +247,7 @@ export function resolvePrunedPieceInstance(
247247

248248
// Consider the playout userDuration
249249
if (pieceInstance.userDuration) {
250-
if ('endRelativeToPart' in pieceInstance.userDuration) {
251-
caps.push(pieceInstance.userDuration.endRelativeToPart - resolvedStart)
252-
} else if ('endRelativeToNow' in pieceInstance.userDuration) {
253-
caps.push(nowInPart + pieceInstance.userDuration.endRelativeToNow - resolvedStart)
254-
}
250+
caps.push(pieceInstance.userDuration.endRelativeToPart - resolvedStart)
255251
}
256252

257253
return {

packages/job-worker/src/playout/timeline/lib.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,10 +26,7 @@ export function hasPieceInstanceDefinitelyEnded(
2626
}
2727

2828
if (pieceInstance.userDuration) {
29-
const userDurationEnd =
30-
'endRelativeToPart' in pieceInstance.userDuration
31-
? pieceInstance.userDuration.endRelativeToPart
32-
: pieceInstance.userDuration.endRelativeToNow + nowInPart
29+
const userDurationEnd = pieceInstance.userDuration.endRelativeToPart
3330

3431
relativeEnd = relativeEnd === undefined ? userDurationEnd : Math.min(relativeEnd, userDurationEnd)
3532
}

packages/job-worker/src/playout/timeline/multi-gateway.ts

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ import { PieceTimelineMetadata } from './pieceGroup.js'
66
import { logger } from '../../logging.js'
77
import { PlayoutModel } from '../model/PlayoutModel.js'
88
import { RundownTimelineTimingContext, getInfinitePartGroupId } from './rundown.js'
9-
import { getPieceControlObjectId } from '@sofie-automation/corelib/dist/playout/ids'
109
import { PlayoutPartInstanceModel } from '../model/PlayoutPartInstanceModel.js'
1110
import { PlayoutPieceInstanceModel } from '../model/PlayoutPieceInstanceModel.js'
1211

@@ -231,29 +230,6 @@ function deNowifyCurrentPieces(
231230
}
232231
}
233232

234-
// Ensure any pieces with an unconfirmed userDuration is confirmed
235-
for (const pieceInstance of currentPartInstance.pieceInstances) {
236-
if (
237-
pieceInstance.pieceInstance.userDuration &&
238-
'endRelativeToNow' in pieceInstance.pieceInstance.userDuration
239-
) {
240-
const relativeToNow = pieceInstance.pieceInstance.userDuration.endRelativeToNow
241-
const endRelativeToPart = relativeToNow + nowInPart
242-
pieceInstance.setDuration({ endRelativeToPart })
243-
244-
// Update the piece control obj
245-
const controlObj = timelineObjsMap[getPieceControlObjectId(pieceInstance.pieceInstance)]
246-
if (controlObj && !Array.isArray(controlObj.enable) && controlObj.enable.end === 'now') {
247-
controlObj.enable.end = endRelativeToPart
248-
}
249-
250-
// If the piece is an infinite, there may be a now in the parent group
251-
const infiniteGroup = timelineObjsMap[getInfinitePartGroupId(pieceInstance.pieceInstance._id)]
252-
if (infiniteGroup && !Array.isArray(infiniteGroup.enable) && infiniteGroup.enable.end === 'now') {
253-
infiniteGroup.enable.end = targetNowTime + relativeToNow
254-
}
255-
}
256-
}
257233
return { objectsNotDeNowified }
258234
}
259235

packages/webui/src/client/lib/rundownLayouts.ts

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -165,12 +165,6 @@ export function getUnfinishedPieceInstancesReactive(
165165
typeof pieceInstance.userDuration.endRelativeToPart === 'number'
166166
) {
167167
end = pieceInstance.userDuration.endRelativeToPart
168-
} else if (
169-
pieceInstance.userDuration &&
170-
'endRelativeToNow' in pieceInstance.userDuration &&
171-
typeof pieceInstance.userDuration.endRelativeToNow === 'number'
172-
) {
173-
end = pieceInstance.userDuration.endRelativeToNow + now
174168
} else if (typeof piece.enable.duration === 'number' && pieceInstance.plannedStartedPlayback) {
175169
end = piece.enable.duration + pieceInstance.plannedStartedPlayback
176170
}

0 commit comments

Comments
 (0)