Skip to content

Freeview: Fix xmltv_ids#3103

Merged
freearhey merged 1 commit intoiptv-org:masterfrom
dot166:freeview-fix-xmltv_id
Apr 26, 2026
Merged

Freeview: Fix xmltv_ids#3103
freearhey merged 1 commit intoiptv-org:masterfrom
dot166:freeview-fix-xmltv_id

Conversation

@dot166
Copy link
Copy Markdown
Contributor

@dot166 dot166 commented Apr 23, 2026

channels:validate was complaining about these.

it still complains about ones that are missing from the DB, i will make a PR to add them to the DB at some point, see below the results of validate before and after this PR.

before:

$ npm run channels:validate --- ./sites/freeview.co.uk/freeview.co.uk.channels.xml [14:26:57]

channels:validate
tsx scripts/commands/channels/validate.ts ./sites/freeview.co.uk/freeview.co.uk.channels.xml

./sites/freeview.co.uk/freeview.co.uk.channels.xml
┌─────────┬────────────────────┬──────┬──────────────────────────────┬───────────────┬────────────────┐
│ (index) │ type │ lang │ xmltv_id │ site_id │ name │
├─────────┼────────────────────┼──────┼──────────────────────────────┼───────────────┼────────────────┤
│ 0 │ 'wrong_feed_id' │ 'en' │ 'BBCFour.uk@HD' │ '64257#18048' │ 'BBC FOUR HD' │
│ 1 │ 'wrong_feed_id' │ 'en' │ 'Channel4.uk@HD' │ '64257#17664' │ 'Channel 4 HD' │
│ 2 │ 'wrong_feed_id' │ 'en' │ 'Channel4.uk@Plus1' │ '64257#8452' │ 'Channel 4+1' │
│ 3 │ 'wrong_feed_id' │ 'en' │ 'ITV1.uk@BorderEnglandPlus1' │ '64384#8360' │ 'ITV1+1' │
│ 4 │ 'wrong_feed_id' │ 'en' │ 'ITV1.uk@HD' │ '64257#17604' │ 'ITV1 HD' │
│ 5 │ 'wrong_feed_id' │ 'en' │ 'ITV1.uk@Plus1' │ '64257#8361' │ 'ITV1+1' │
│ 6 │ 'wrong_channel_id' │ 'en' │ 'WildEarth.uk@SD' │ '64257#14464' │ 'WildEarth' │
│ 7 │ 'wrong_feed_id' │ 'en' │ 'WildEarth.uk@SD' │ '64257#14464' │ 'WildEarth' │
└─────────┴────────────────────┴──────┴──────────────────────────────┴───────────────┴────────────────┘

after:

npm run channels:validate --- ./sites/freeview.co.uk/freeview.co.uk.channels.xml [14:41:07]

channels:validate
tsx scripts/commands/channels/validate.ts ./sites/freeview.co.uk/freeview.co.uk.channels.xml

(no warnings, no errors, it just doesnt display that in the output)

@StrangeDrVN
Copy link
Copy Markdown
Contributor

StrangeDrVN commented Apr 23, 2026

 > channels:validate
 > tsx scripts/commands/channels/validate.ts sites/freeview.co.uk/freeview.co.uk.channels.xml

sites/freeview.co.uk/freeview.co.uk.channels.xml
┌─────────┬────────────────────┬──────┬───────────────────────┬───────────────┬─────────────┐
│ (index) │ type               │ lang │ xmltv_id              │ site_id       │ name        │
├─────────┼────────────────────┼──────┼───────────────────────┼───────────────┼─────────────┤
│ 0       │ 'wrong_feed_id''en''ITV1.uk@BorderPlus1''64384#8360''ITV1+1'    │
│ 1       │ 'wrong_channel_id''en''WildEarth.uk@SD''64257#14464''WildEarth' │
│ 2       │ 'wrong_feed_id''en''WildEarth.uk@SD''64257#14464''WildEarth' │
└─────────┴────────────────────┴──────┴───────────────────────┴───────────────┴─────────────┘

@dot166 dot166 force-pushed the freeview-fix-xmltv_id branch from 91a1c84 to 18efaad Compare April 23, 2026 13:35
@dot166
Copy link
Copy Markdown
Contributor Author

dot166 commented Apr 23, 2026

 > channels:validate
 > tsx scripts/commands/channels/validate.ts sites/freeview.co.uk/freeview.co.uk.channels.xml

sites/freeview.co.uk/freeview.co.uk.channels.xml
┌─────────┬────────────────────┬──────┬───────────────────────┬───────────────┬─────────────┐
│ (index) │ type               │ lang │ xmltv_id              │ site_id       │ name        │
├─────────┼────────────────────┼──────┼───────────────────────┼───────────────┼─────────────┤
│ 0       │ 'wrong_feed_id''en''ITV1.uk@BorderPlus1''64384#8360''ITV1+1'    │
│ 1       │ 'wrong_channel_id''en''WildEarth.uk@SD''64257#14464''WildEarth' │
│ 2       │ 'wrong_feed_id''en''WildEarth.uk@SD''64257#14464''WildEarth' │
└─────────┴────────────────────┴──────┴───────────────────────┴───────────────┴─────────────┘

that is because those entries don't exist in the DB yet, i still need to add them

@StrangeDrVN
Copy link
Copy Markdown
Contributor

WildEarth.uk is now WildEarth.za

@dot166
Copy link
Copy Markdown
Contributor Author

dot166 commented Apr 23, 2026

WildEarth.uk is now WildEarth.za

thanks, will update

@dot166 dot166 force-pushed the freeview-fix-xmltv_id branch from 18efaad to a86fde8 Compare April 23, 2026 13:41
@dot166
Copy link
Copy Markdown
Contributor Author

dot166 commented Apr 23, 2026

WildEarth.uk is now WildEarth.za

thanks, will update

done

StrangeDrVN
StrangeDrVN previously approved these changes Apr 23, 2026
@dot166
Copy link
Copy Markdown
Contributor Author

dot166 commented Apr 23, 2026

needs iptv-org/database#27409

@dot166 dot166 force-pushed the freeview-fix-xmltv_id branch from a86fde8 to 1afecda Compare April 24, 2026 07:39
@dot166
Copy link
Copy Markdown
Contributor Author

dot166 commented Apr 24, 2026

the PR in the DB has been merged, this can be reviewed now

@dot166 dot166 force-pushed the freeview-fix-xmltv_id branch from 1afecda to 952d8a3 Compare April 24, 2026 09:09
channels:validate was complaining about these
@freearhey freearhey merged commit e99f46b into iptv-org:master Apr 26, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants