MP1-4357 Improve Deploy Tool detection for LAVFilters / TitanExtended (2 Viewers)

Sebastiii

Development Group
  • Team MediaPortal
  • November 12, 2007
    16,583
    10,403
    France
    Home Country
    France France
    Description of Change
    When we use the DeployTool to install/upgrade MediaPortal, the checkboxes for LAVFilters & TitanExtended are always checked, even when they are already installed.

    This branch should fix the detection of the installation and set the checkboxes according to the install status.

    Tickets : https://issues.team-mediaportal.com/browse/MP1-4357

    Steps to install and test
    Installation
    Download the -->> installer <<-- and execute it.

    Testing
    Upgrade your current 1.6.0 Final installation
    Look if checked value are OK
    Please do not forget to give some feedback (also if there is no bug). Thank you!

    Sebastiii

    Changelog

    Version 1.6.0 v1
    Build Based on MP 1.6.0 + master
     

    breese

    Retired Team Member
  • Premium Supporter
  • July 11, 2011
    3,902
    770
    65
    Arlington Heights, Illinois
    Home Country
    United States of America United States of America
    Ran the installer and it detected just fine.
    Have no issues that I can see..
    Install_01.jpg Install_02.jpg

    Went back and uninstalled Lav Filters and re-ran installer
    It detected Lav Filters was needed
    Install_03.jpg Install_04.jpg
     

    vuego

    Documentation Group
  • Team MediaPortal
  • August 5, 2006
    1,637
    764
    Göteborg
    Home Country
    Sweden Sweden
    There is a problem when Titan Extended is unchecked during an upgrade of MediaPortal

    First of all, the skinsettings.xml containing the users customizations is overwritten.

    Then settings_GUI_Skin.xml from MediaPortal will overwrite Titan Extended's which means that the More button is missing in MediaPortal - Settings - GUI - Skin so it is impossible for the user to manually reset the settings that was removed by the installer :oops:
     

    HomeY

    Test Group
  • Team MediaPortal
  • February 23, 2008
    6,475
    4,645
    49
    ::1
    Home Country
    Netherlands Netherlands
    Yeah, i noticed the same.
    The Skinsettings.xml is always overwritten, and we've had discussions about it in the past.
    But the fact that Titan Extended needs to be re-installed to make the 'More Settings' button visible again after an upgrade, that is something that shouldn't happen.

    These settings apply to MovingPictures & TV Series plugins, so the button should only show up when Titan Extended is installed. But if the default files doesn't contain this button, i think the XML file should be saved in the appropriate Extended One or Two folder?

    @ncoH can you comment on this please?
     

    Users who are viewing this thread

    Top Bottom