LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

Fix timestamp regex error

Browse source 
## Issue
https://discord.com/channels/362322208485277697/363087331475062785/817972023347249153

## Change
Open up the range as per YT, indirectly fixing the regex mistake.
This commit is contained in:
infinite-persistence 2021-03-09 23:10:43 +08:00 committed by Sean Yesmunt
parent 11fdbcaee5
commit a33087440c
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
ui/util/remark-timestamp.js
View file

@ -51,8 +51,8 @@ function findNextTimestamp(value, fromIndex, strictlyFromIndex) {
case 7: // "9:59:59" case 7: // "9:59:59"
isValidTimestamp = /^[0-9]:[0-5][0-9]:[0-5][0-9]$/.test(str); isValidTimestamp = /^[0-9]:[0-5][0-9]:[0-5][0-9]$/.test(str);
break; break;
case 8: // "23:59:59" case 8: // "99:59:59"
isValidTimestamp = /^[0-2][0-3]:[0-5][0-9]:[0-5][0-9]$/.test(str); isValidTimestamp = /^[0-9][0-9]:[0-5][0-9]:[0-5][0-9]$/.test(str);
break; break;
default: default:
// Reject // Reject
@ -84,7 +84,7 @@ function locateTimestamp(value, fromIndex) {
} }
// Generate 'timestamp' markdown node // Generate 'timestamp' markdown node
const createTimestampNode = text => ({ const createTimestampNode = (text) => ({
type: TIMESTAMP_NODE_TYPE, type: TIMESTAMP_NODE_TYPE,
value: text, value: text,
children: [{ type: 'text', value: text }], children: [{ type: 'text', value: text }],
@ -145,7 +145,7 @@ const transformer = (node, index, parent) => {
} }
}; };
const transform = tree => { const transform = (tree) => {
visit(tree, [TIMESTAMP_NODE_TYPE], transformer); visit(tree, [TIMESTAMP_NODE_TYPE], transformer);
}; };