Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use new settings.xml format #11

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dagwieers
Copy link
Contributor

Implement the new settings format. Since we are targetting Leia and
Matrix only, we might as well move now to the newt settings.xml format.

@dagwieers dagwieers added the enhancement New feature or request label May 12, 2020
@dagwieers
Copy link
Contributor Author

dagwieers commented May 12, 2020

So there are some shortcomings in the new settings.xml format:

  • You are not able to use boolean conditions like HasAddon(id) or System.AddonIsEnabled(id)
  • Using RunScript() or Addon.OpenSettings(id) should work, but fails on Kodi v18.6 (Leia)

The benefits would be that:

  • Every control can specify its data type properly (i.e. a select can indicate integer format)
  • A lot more options, including help texts and advanced configuration levels

But without a working RunScript(), this becomes useless.

@dagwieers dagwieers added the help wanted Extra attention is needed label May 12, 2020
@dagwieers dagwieers marked this pull request as draft May 14, 2020 15:02
Implement the new settings format. Since we are targetting Leia and
Matrix only, we might as well move now to the newt settings.xml format.
@matthuisman
Copy link

matthuisman commented May 20, 2021

@dagwieers
try

<dependencies>
    <dependency type="enable" on="property" name="infobool">System.HasAddon(pvr.iptvsimple)</dependency>
</dependencies>

or

<dependencies>
    <dependency type="enable">
        <condition on="property" name="InfoBool">System.HasAddon(pvr.iptvsimple)</condition>
    </dependency>
</dependencies>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants