[mythtv] Comments requested on two bugs

[Philip Brady]

On 8/30/23 13:13, Philip Brady wrote:
> I am not subscribed to this list but have seen this thread in the archive.
>
> Since the API interface for recording status is changing, I have
> written a new page in the wiki and intend linking to it from the API
> pages.
> I'd really appreciate some expert eyes proofreading it.
> It is here:
>
> https://www.mythtv.org/wiki/Recording_Status
>
> Regards and thank you all for your efforts.
> Phil Brady
>
>
Thank you for documenting this.


The safest place to check is in the source code. Some of your "unknown"
statuses refer to obsolete values.


Values for Rec Status:
https://github.com/MythTV/mythtv/blob/07439138c378fe07efcee8993721d7e3932ed4c6/mythtv/libs/libmythbase/recordingstatus.h#L11


Text values:
https://github.com/MythTV/mythtv/blob/07439138c378fe07efcee8993721d7e3932ed4c6/mythtv/libs/libmythbase/recordingstatus.cpp#L124

Descriptions:
https://github.com/MythTV/mythtv/blob/07439138c378fe07efcee8993721d7e3932ed4c6/mythtv/libs/libmythbase/recordingstatus.cpp#L188


Also note that now the API V2 is running on port 6544 (as well as 6744
for the time being) on latest master.


If you saw Status=3 and a StatusName=CurrentRecording you were likely
running with an old version of MythTV, which may have had different
values in these modules.


If you see something wrong in the source code, let me know.


Thanks Peter,

I'll put links to the new page from other interesting places.  I'll also comment that API V2 will be the norm on port 6544

Have I found two extra bugs or ambiguities?

I'm using:
MythTV Version : v34.0~master.202306020720.ed409bfc02~ubuntu22.04.1
MythTV Branch : master
Network Protocol : 91
Library API : 34.20220913-1
QT Version : 5.15.3
Options compiled in:
 linux profile use_hidesyms using_alsa using_oss using_pulse using_pulseoutput using_backend using_bindings_perl
 using_bindings_python using_bindings_php using_dvb using_firewire using_frontend using_hdhomerun using_satip
 using_vbox using_ceton using_joystick_menu using_libcec using_libcrypto using_gnutls using_libdns_sd using_libxml2
 using_lirc using_mheg using_opengl using_egl using_qtwebkit using_qtscript using_qtdbus using_taglib using_v4l2
 using_v4l2prime using_x11 using_system_libbluray using_system_libudfread using_systemd_notify using_systemd_journal
 using_drm using_bindings_perl using_bindings_python using_bindings_php using_freetype2 using_mythtranscode using_opengl
 using_egl using_drm using_vaapi using_nvdec using_vdpau using_ffmpeg_threads using_mheg using_libass using_libxml2
 using_libmp3lame


For StatusName, I'm seeing values of 'CurrentRecording' when I expected 'CurrentlyRecorded'.  Status is 3.

RecStatusToString?RecStatus=value  shows this:

3                             => Currently Recorded
CurrentRecording   => Currently Recorded

 but the code is different:

 https://github.com/MythTV/mythtv/blob/07439138c378fe07efcee8993721d7e3932ed4c6/mythtv/libs/libmythbase/recordingstatus.h#L11
     DontRecord = 1,
     PreviousRecording = 2,
     CurrentRecording = 3,
     EarlierShowing = 4,

Is this a bug or is the solution to just document that 'CurrentRecording' actually means 'CurrentlyRecording' ?

As for the Recording Type /Dvr/RecTypeToString?RecType=8 and =9,  both return 'Override Recording'.
Is this why I am having problems over-riding with MythWeb?

Regards
Phil




________________________________
From: Philip Brady <phil.brady at hotmail.co.uk>
Sent: 30 August 2023 17:13
To: mythtv-dev at mythtv.org <mythtv-dev at mythtv.org>
Subject: Re: Comments requested on two bugs

I am not subscribed to this list but have seen this thread in the archive.

Since the API interface for recording status is changing, I have written a new page in the wiki and intend linking to it from the API pages.
I'd really appreciate some expert eyes proofreading it.
It is here:

https://www.mythtv.org/wiki/Recording_Status

Regards and thank you all for your efforts.
Phil Brady



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mythtv.org/pipermail/mythtv-dev/attachments/20230831/30565b75/attachment.htm>