oe-alliance / AutoBouquetsMaker

Automatically build and update bouquets from the DVB stream.
GNU General Public License v3.0
22 stars 59 forks source link

Movistar+ multiple regions #280

Closed miberian closed 1 year ago

miberian commented 1 year ago

Changes to provider files to support region 0x82 and merge with 0x83. This produces official channel order. This is now achieved without code changes.

Huevos commented 1 year ago

Ok, but in one file, not two...

And you wanted 0x82 and 0x83 at the same time, right?

miberian commented 1 year ago

Yes both 0x82 and 0x83 are required to get the full channel list. I don’t believe this can be done in a single xml without code changes to dvbscanner.

miberian commented 1 year ago

And please note that since the new file depends on the existing/main one there is no impact to usability. Users will still see a single provider in the UI.

Downside of this approach compared to code changes is that it takes twice the time for processing.

Huevos commented 1 year ago

Ok, let me look at it.

Huevos commented 1 year ago

<visibleserviceflag ignore="1"/>

Why do you remove this?

miberian commented 1 year ago

The main file now has to be configured for 0x82 to produce the correct channel list. Descriptor 0x82 does not have visible_service_flag so I removed the configuration from that file as it is not really needed (it has no effect). However, it is still required for 0x83 so the configuration remains as the previous version in sat_192_movistar_plus_esp_0x83.xml.

Huevos commented 1 year ago

Please see latest commits (in particular): https://github.com/oe-alliance/AutoBouquetsMaker/commit/cf08f3ff2b5e2a65a7c14ba1d8f132da3f07a283 https://github.com/oe-alliance/AutoBouquetsMaker/commit/5c0a58a3781926eac5a2eef348f09f6e51f4a466

But obviously make sure you test with all commits.

1

miberian commented 1 year ago

No longer required.