[fixed] Wrong dialog when manual record stopping [Jira MP1-3401] (1 Viewer)

regeszter

Retired Team Member
  • Premium Supporter
  • October 29, 2005
    5,335
    4,954
    Home Country
    Hungary Hungary
    I added more debug log. Can you test with the #35 + the uploaded tvplugin_debugV2? Please do not forget update your tvserver too.

    The server side log not need at the moment.

    Sorry to disturb you but if I can't reproduce the error here, I can't work alone. :(

    Thx.
     

    Attachments

    • TvPlugin_debugV2.zip
      185.3 KB

    elliottmc

    Retired Team Member
  • Premium Supporter
  • August 7, 2005
    14,927
    6,061
    Cardiff, UK
    Home Country
    United Kingdom United Kingdom
    I added more debug log. Can you test with the #35 + the uploaded tvplugin_debugV2? Please do not forget update your tvserver too.

    The server side log not need at the moment.

    Sorry to disturb you but if I can't reproduce the error here, I can't work alone. :(

    Thx.

    Does #35 include server changes, or is it just your 3401 current branch? I already have your 3401 branch on my server.

    I can test this evening.
     

    regeszter

    Retired Team Member
  • Premium Supporter
  • October 29, 2005
    5,335
    4,954
    Home Country
    Hungary Hungary
    #35 include server changes. The tvdatabase.dll need to be same in tvserver and tvplugin side.
     

    elliottmc

    Retired Team Member
  • Premium Supporter
  • August 7, 2005
    14,927
    6,061
    Cardiff, UK
    Home Country
    United Kingdom United Kingdom
    With this installer I give stop dialog on stop recordings and delete dialog on recorded tv -> delete recording

    @regeszter - I have tested with this installer, and I also get the delete dialog on both now.

    What is different from the 3401 branch in GIT ?

    Maybe it is something that was odd with my system.

    Note that I didn't include any of the later TvPlugin.dll - just the installer in the first instance.

    Mark
     

    elliottmc

    Retired Team Member
  • Premium Supporter
  • August 7, 2005
    14,927
    6,061
    Cardiff, UK
    Home Country
    United Kingdom United Kingdom
    And now my own build with the 3401 branch is working perfectly in all cases as well.

    No idea how to explain that.

    So, make sure code is reviewed, see if anyone else can confirm that the fix is good and then we can include this in 1.7.0.
     

    regeszter

    Retired Team Member
  • Premium Supporter
  • October 29, 2005
    5,335
    4,954
    Home Country
    Hungary Hungary
    I added the link to the installer in the 1st post to help the testers.
     

    Users who are viewing this thread

    Top Bottom