Replicated services show wrong LCN

Issue #170 resolved
Ian Brabham created an issue

Replicated services like 12, 21 and 70 show the LCN for the first service with the matching a "Service Name", i,e 1, 2 and 7 respectively.

For Melbourne the following replicated services occur :-

LCN SID "Service Name" 1 631 "ONE" 12 637 "ONE" 2 231 "ABC1" 21 233 "ABC1" 7 530 "7 Digital" 70 534 "7 Digital" 71 531 "7 Digital 1"

Replicated service 71 which has a unique "Service Name" is correctly displayed as LCN 71, the other 3 get it wrong.

The /etc/engima2/* dump attached to Bug #61, http://wiztv.com.au/bugs/files/show/83 can show the issue.

The code seems to be very convoluted chasing down the matching "Service Name" from the PMT, it would be a lot simpler to just key off the unique SID/TSID/ONID. Also if a entry is renamed then no LCN is displayed because no matching name can be found. And when the Network decides to change the "Service Name", which does happen, then again no LCN will be shown.

Reproduction steps

Tune to a replicated service, show the infobar window, see the wrong LCN displayed.

Comments (7)

  1. prl

    Is "status window" the infobar? I wasn't sure whether it was that, or MENU>Setup>Information>Service, but that doesn't seem to have any "LCN" information.

    If it's the infobar, I see the same for the replicated ABC1 and Prime & services in Canberra.

  2. prl
    • removed issue_status

    The issue was updated with the following change(s):

    • The status has been updated, from New to Confirmed.
  3. Ian Brabham reporter
    • removed issue_reproducability

    The issue was updated with the following change(s):

    • This issue's reproduction steps has been changed
  4. Peter Urbanec

    The code seems to be very convoluted chasing down the matching "Service Name" from the PMT, it would be a lot simpler to just key off the unique SID/TSID/ONID.

    I agree, the DVB ON/TS/S ID triplet should be used, it's the official way of addressing DVB services. I have not seen the code - in fact I don't even know which file it's in.

    However, if you want to have a go at changing the code, I'd be happy to review any patches and get them to the developers.

  5. Peter Urbanec
    • removed issue_priority

    The issue was updated with the following change(s):

    • The priority has been updated, from Normal to High.
  6. Peter Urbanec
    • removed issue_status

    The issue was updated with the following change(s):

    • The status has been updated, from Confirmed to Testing.
  7. Peter Urbanec
    • removed issue_resolution
    • removed issue_percent
    • removed issue_close
    • removed issue_status

    The issue was updated with the following change(s):

    • The status has been updated, from Testing to Closed.
    • This issue has been closed
    • This issue's progression has been updated to 100 percent completed.
    • The resolution has been updated, from Not determined to Fixed.
  8. Log in to comment