Skip to content

Conversation

bucefal91
Copy link

Hey! Thank you for the library :)

We have applied it to our particular use case and it does pretty good job. The only problem we've discovered is that our subtitles (which are auto-generated by 3rd party vendor), sometimes could have a bit strange entries with just empty text, i.e. it would specify some timestamp but would not specify any text/wording for that time interval, just an empty line. Let me show an example:

1
00:00:00,000 --> 00:00:00,500


2
00:00:00,500 --> 00:00:05,600
I was asked to verify that I did
not file a Federal income tax

You see the section first section does not specify any text, just an empty line (an empty text, if you like). It was generating some PHP notices, so I opted to patch a bit the parser logic.

@fernando-ayon
Copy link

If you're fixing a bug, you should add tests to validate your work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants