Opened 19 years ago
Closed 19 years ago
#2983 closed defect (fixed)
Recording List Menu not working 100%
| Reported by: | Owned by: | danielk | |
|---|---|---|---|
| Priority: | minor | Milestone: | 0.21 |
| Component: | mythtv | Version: | head |
| Severity: | medium | Keywords: | |
| Cc: | Ticket locked: | no |
Description
I'm running svn trunk at the moment and have just found that when in the watch recordings screen, changing the group filter via the recording list menu no longer works. The windows open up and all of the appropriate options are given - however once selected, nothing happens. A similar problem happens with the change group view and change group password options with the no changes being effected by either window. I'm not too familiar with the way the UI works yet but it looks to me like the UI is returning rejected instead of accepted - though the Playlist options menu seems to work fine.
Attachments (2)
Change History (6)
comment:1 by , 19 years ago
comment:2 by , 19 years ago
| Milestone: | unknown → 0.21 |
|---|---|
| Owner: | changed from to |
by , 19 years ago
| Attachment: | 2983-v1.patch added |
|---|
This fixes the group view dialog, however it may cause other problems.
by , 19 years ago
| Attachment: | 2983-v2.patch added |
|---|
Updated patch, it appears default handler is only used to handle ESC (LEFT if arrow accellerations is enabled)
comment:3 by , 19 years ago
I'm testing a patch which just reverts [12624] and [12629] for now.
I need to make larger changes in MythDialog to properly handle a canceled dialog in all cases, this in turn will require more testing and will take a while.
I did some further testing to this and removed some of the recently added code that looks to have been a part #2426.
In mythdialogs.cpp, the functions accept and reject are now overloaded for MythPopupBox. In each of these, they call MythDialog::done() with the appropriate value and then both emit popupDone() which results in MythPopupBox::defaultExitHandler() being called, which then calls done(-1) thus removing the previously stored result value.
I removed both emit lines, recompiled and tried again. That seems to resolve the problems I listed when submitting this ticket but I'm not sure what the reason for the code is in the first place and what removing it will break. I'll leave working that out to the person working on the new code effort in there.
Hope that helps some.