Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Load config if enabled #137

Merged
merged 6 commits into from
Dec 21, 2018
Merged

Load config if enabled #137

merged 6 commits into from
Dec 21, 2018

Conversation

phunkyfish
Copy link
Member

@phunkyfish phunkyfish commented Nov 23, 2018

v3.15.2

@phunkyfish
Copy link
Member Author

Adding an extra fix for old STB's will let you know once it's there.

@phunkyfish
Copy link
Member Author

@ksooo ready for reivew now. Just some small fixes.

@phunkyfish
Copy link
Member Author

Actually, one other fix for channel group ordering.

@phunkyfish
Copy link
Member Author

Ready to review again @ksooo

@phunkyfish
Copy link
Member Author

@ksooo would be great if have a little bit of time to review this. I've added some debug logging to help with diagnosing what's going on with #145. Once that's in I can create a bug report for Libreeelec.

@kid63
Copy link

kid63 commented Nov 30, 2018

Hello @phunkyfish and @ksooo,
yeah, would be great because of the dreambox-fix, none of the Dreamboxes has got a distroversion implemented in the webinterface. Got a dm920 and 3.15.1 doesn't work for Dreambox-Users at all, not just the old ones. As a workaround i tared the 3.14 and extract it back, but that's unsatisfying.

Regards
kid

@MilhouseVH
Copy link

@phunkyfish I'll include this PR in the next LibreELEC nightly (Saturday night, #1201), hopefully we can get you the information you need.

@phunkyfish
Copy link
Member Author

Thanks @MilhouseVH

@phunkyfish
Copy link
Member Author

@MilhouseVH thanks again for including. This did indeed help me find the issue.

There was a new option added at some point to PVR settings "Synchronise channel groups with backend(s)". It would appear that in stock kodi this gets enabled by default, whereas in libreelec it does not or something to that effect.

@phunkyfish
Copy link
Member Author

@ksooo this PR has been soak tested for several days now so should be ok to put through.

@MilhouseVH
Copy link

MilhouseVH commented Dec 9, 2018

@phunkyfish ah yes, LibreELEC disables that option in appliance.xml, here. Not entirely sure why it is disabled, but it's something that has been disabled for 3 years (original commit back in OpenELEC days). I'll ask around and see if anyone from the LE team remembers.

@MilhouseVH
Copy link

So far nobody knows why it was disabled (although it's late on Sunday night... :))

For now I'll revert the LibreELEC commit in nightly test builds so that pvrmanager.syncchannelgroups is enabled as it is in stock Kodi, and we can gauge the effect.

@phunkyfish
Copy link
Member Author

phunkyfish commented Dec 9, 2018 via email

@kid63
Copy link

kid63 commented Dec 10, 2018

Hi,

unfortunately #137 ignores the Favourite-Bouqets from Dreamboxes, only Freetv is displayed. Was working in 3.14, that's the latest version it worked, because 3.15.1 wasn't for Dreamboxes. Tested it with OSMC.

cu

@phunkyfish
Copy link
Member Author

phunkyfish commented Dec 10, 2018

In this version on the channels tab you need to select the favourites (as first or last group) in addition to All Bouquets. It was the only way I could get it work across all hardware and images. Not all images process favourites in the same way.

As I don’t have a dream box I didn’t test that specific hardware but it shouldn't be different.

Also, on OSMC you'll need to check you are not effected by this bug #145. So if you enable that option and select the favourites in the channels tab you should be ok.

Let me know if it still doesn't work @kid63

@kid63
Copy link

kid63 commented Dec 10, 2018

Thanks for pointing me in the right direction, will have a look. :)

@phunkyfish
Everything's ok, just had to reconfigure.

@Matthias1992
Copy link

Hey guys,
I still have the same problem with,
Kodi (18.0-RC3 Git:20181210-c6107e5). Platform: Linux x86 64-bit
and an DM800SE

Enigma2 Client: pvr.vuplus - LoadDeviceInfo Could not parse e2distroversion from result!

But I don't get @phunkyfish what do you mean to fix is with the Bouquets.
A longer explain would be nice,
just a kodi noob :)
Matthias

@phunkyfish
Copy link
Member Author

This PR covers a number of the bug fixes. The bouquets stuff is not related to the fix for e2distroversion

Copy link
Member

@ksooo ksooo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Maybe you could make some more of the strings you've changed const. I only commented on two of them, but saw more candidates in the diff...

src/Enigma2.cpp Show resolved Hide resolved
src/enigma2/Channels.cpp Show resolved Hide resolved
@phunkyfish phunkyfish merged commit 5741f24 into kodi-pvr:master Dec 21, 2018
@phunkyfish phunkyfish deleted the load-config-if-enabled branch December 21, 2018 08:18
@phunkyfish
Copy link
Member Author

@MilhouseVH thanks for including in nightlies, it should come via master now.

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.

None yet

5 participants