Ticket #9395 (closed Patch - Feature: fixed)
Opened 2 years ago
Last modified 8 months ago
New system events: watch rec, watch video, listen to music
| Reported by: | Dave Badia <dbadia@…> | Owned by: | stuartm |
|---|---|---|---|
| Priority: | minor | Milestone: | 0.27 |
| Component: | MythTV - General | Version: | Master Head |
| Severity: | low | Keywords: | |
| Cc: | Ticket locked: | no |
Description
Trigger system events when entering the watch recordings screen, the watch videos screen, and the play music screen.
My use case is that I control my receiver via rs232 and had different audio settings for tv, movies, and music. The rs232 scripts take 4-5 seconds to execute hence the "early" trigger when selecting watch recordings (as opposed to when a recording is selected) and watch videos
Depends on #9394
Attachments
Change History
comment:1 Changed 2 years ago by cpinkham
- Owner changed from beirdo to cpinkham
- Status changed from new to assigned
comment:2 Changed 2 years ago by David Badia <dbadia@…>
newevent2.diff to fix typo in menu
comment:3 Changed 22 months ago by wagnerrp
- Component changed from MythTV - MythSystem to MythTV - General
comment:4 Changed 19 months ago by Github
- Status changed from assigned to closed
- Resolution set to fixed
- Milestone changed from unknown to 0.25
Add a new EventCmdScreenType? MythSystemEvent?.
Add a new System Event that is sent out whenever a new screen is created or an existing screen is destroyed. This will allow users to take external actions whenever an application enters or leaves a screen/dialog.
The system event is SCREEN_TYPE and the setting name is EventCmdScreenType?. The first argument in the %ARGS% value is either 'CREATED' or 'DESTROYED' depending on whether a MythScreenType? was created or destroyed.
Fixes #9395.
Branch: master Changeset: 87b89dee6c8c6a4ffd89ca5857de4a4e791fcd82
comment:5 Changed 14 months ago by cpinkham
- Status changed from closed to new
- Version changed from Unspecified to Master Head
- Resolution fixed deleted
- Severity changed from medium to low
- Milestone changed from 0.25 to 0.26
This commit was reverted in changeset b603bf39.
If you compile from source, and want to test a fix, you can uncomment those lines yourself. To fix the issue that caused the revert commit, you should be able to uncomment and add the following before each of the SendSystemEvent? lines that are commented out.
if (QCoreApplication::applicationName() == MYTH_APPNAME_MYTHFRONTEND)
Once we cut 0.25, we should be able to get this code re-enabled in git master.
comment:6 Changed 14 months ago by Dave Badia <dbadia@…>
Thanks for the info on this, it all makes sense. I went ahead and made the changes you outlined. Tested it here and it all works well, including being able to run mythtv-setup. I'll attach a diff of the change against master in case it's helpful for the future work.
Changed 14 months ago by Dave Badia <dbadia@…>
- Attachment screen-event.diff added
Changes outline by Chris to re-enable this
comment:8 Changed 8 months ago by stuartm
- Owner changed from cpinkham to stuartm
- Status changed from new to accepted
comment:9 Changed 8 months ago by Dave Badia <dbadia@…>
- Status changed from accepted to closed
- Resolution set to fixed
