Skip to content

Commit 8beb565

Browse files
committed
Revert "Fix bug with multiple consecutive null"
This reverts commit b06f899.
1 parent b06f899 commit 8beb565

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

resources/assets/js/videos/models/Annotation.vue

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -283,14 +283,12 @@ export default Vue.extend({
283283
this.points.splice(index, 1);
284284
285285
// Remove null (gap filler) as first/last element to prevent validation errors
286-
// Multiple consecutive 'null' can occur after removing elements of linked anntations
287-
// in specific order
288-
while (this.frames[0] === null) {
286+
if (this.frames[0] === null) {
289287
this.frames.shift();
290288
this.points.shift();
291289
}
292290
293-
while (this.frames.at(-1) === null) {
291+
if (this.frames.at(-1) === null) {
294292
this.frames.pop();
295293
this.points.pop();
296294
}

0 commit comments

Comments
 (0)