GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
Already on GitHub? Sign in to your account
The first commit fixes persisting of channel group members if there is no need to persist channels (e.g. when moving channels). Without this CommitInsertQueries() would return false and PersistGroupMembers() would not even be called.
The second commit reverts the changes of f679b31 inside CPVRChannelGroup::MoveChannel()... I am open to suggestions here if you want a different solution.
CDatabase::CommitInsertQueries() - only return false on error
PVRDatabase - add the return value of CommitInsertQueries() to bReturn
Fix channel moving after f679b31
Extend out of range check for iNewChannelNumber
This prevents a crash if someone would try to move a channel to number 0
@opdenkamp I changed everything as discussed...
Merge pull request #1695 from mikrohard/pvr-channel-moving
Pvr channel moving