<div dir="ltr"><div dir="ltr"></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, May 29, 2022 at 4:55 AM Mark Spieth <<a href="mailto:mark@digivation.com.au">mark@digivation.com.au</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 29/05/2022 2:22 am, John Pilkington wrote:<br>
> On 28/05/2022 16:21, David Hampton via mythtv-dev wrote:<br>
>> There should only be one LASTPLAYPOS entry per chanid/starttime pair.<br>
>> I would expect that the BOOKMARK entries should also be one per<br>
>> chanid/starttime pair.<br>
>><br>
>> This should also be true for the filemarkup table for tracking videos.<br>
>> There should only be one LASTPLAYPOS entry per filename, and I would<br>
>> only expect one BOOKMARK entry per filename too.<br>
>><br>
>><br>
><br>
> FWIW playback from last played position works ok for me, in current <br>
> master.<br>
><br>
> But the end marker in the cutlist editor has a problem:  go to end <br>
> (Page Down) goes to frame n, step back from that and forward again <br>
> (left, right arrows) goes to n-m, then n+1, and repeated stepping <br>
> *forward* from there alternates between n-m and n+1.<br>
<br>
This is all what I expected.<br>
<br>
I found my problem.<br>
<br>
It *is* mariadb. From 10.2.1 onwards there is a stricter check on the <br>
string used for datetime type for DELETE FROM only. The Z at the end of <br>
an ISO8601 is not allowed. Thus the old bookmark/lastplayedpos was not <br>
being deleted before the new one was injected.<br>
<br>
phpmyadmin with EXPLAIN pointed me in the correct direction. Strangely <br>
enough, SELECT, UPDATE, REPLACE all work ok with a Z suffix on the <br>
iso8601 string. The mariadb doco does indicate that trailing Z is not <br>
allowed but the DELETE FROM only behaviour is unexpected.<br>
<br>
I have made a PR for further testing as I'm not able to test with mysql. <br>
Merge when you think its ok and delete the feature branch.<br>
<br>
Cheers<br>
<br>
Mark<br>
<br></blockquote><div>I cannot reproduce this issue on <br><br>MythTV Version : v32.0+fixes.202205111848.26079f815a~ubuntu22.04.1<br>QT Version : 5.15.3<br>mariadb  Ver 15.1 Distrib 10.6.7-MariaDB, for debian-linux-gnu (x86_64) using  EditLine wrapper<br><br>It looks for me, that this issue is already addressed by the <br>mysql driver of QT or additional patches from the distribution.<br><br>See <a href="https://github.com/MythTV/mythtv/issues/490">https://github.com/MythTV/mythtv/issues/490</a> , too<br><br>Roland</div><div><br></div><div> </div></div></div>