+- Kodi Community Forum (https://forum.kodi.tv)
+-- Forum: Development (https://forum.kodi.tv/forumdisplay.php?fid=32)
+--- Forum: Skinning (https://forum.kodi.tv/forumdisplay.php?fid=12)
+---- Forum: Skin helper addons (https://forum.kodi.tv/forumdisplay.php?fid=300)
+---- Thread: Release script.skinshortcuts (/showthread.php?tid=178294)
Pages:
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
RE: script.skinshortcuts - Kokonutcreme - 2014-11-21
Here you go
http://pastebin.com/DH0wq6HE
I tried creating a shortcut for a movie smart playlist using Conq, AN5 and Aeon Nox skins and received the same error message each time that "there are no shortcuts available for the category you choose".
Script.skinshortcuts v0.4.3.
Helix Beta 3
Mac OS 10.9.5
The script does find the one video playlist that I've created.
I get the same result running same version of script and Kodi on Win7 machine.
RE: script.skinshortcuts - removed151214 - 2014-11-21
Thanks for the debug log. It's showing some issues listing playlists included with the skin - I suspect I've probably caused that by trying to fix a different issue, but will look into that properly when I get time (read: over the weekend).
It doesn't show any errors listing your own custom playlists, so can you confirm that its just the skin-provided playlists you're missing? If so, I'll let you know when there is a fix for that in place.
Edit: Ignore that - looks like it was a simple issue where the stringForce value (used to ensure that skin playlists are returned via their local path, not via the special:// protocol) was incorrectly passed as 'True', rather than a path, when starting to list a second level whilst looking for playlists. This should now be fixed on git.
RE: script.skinshortcuts - Kokonutcreme - 2014-11-21
Thanks, works perfectly
RE: script.skinshortcuts - Martijn - 2014-12-07
When building the menu it doesn't respect the available shutdown option for the platform on which it runs.
for example on android you only have "exit" but when building the submenu it just adds all which don't work
RE: script.skinshortcuts - removed151214 - 2014-12-07
Good spot
I've added the visibility conditions from Confluence's DialogButtonMenu to git. Hopefully that should cover it, but if you spot any still showing when they shouldn't, please let me know.
RE: script.skinshortcuts - Martijn - 2014-12-07
much better. thx
RE: script.skinshortcuts - User 224999 - 2014-12-12
There's a little error in the syntax (parantheses unmatched) in the addition for the powermenu buttons:
<visible>[(System.HasLoginScreen | IntegerGreaterThan(System.ProfileCount,1)) + System.Loggedon] + [System.HasLoginScreen]</visible>
RE: script.skinshortcuts - removed151214 - 2014-12-12
(2014-12-12, 22:23)marcelveldt Wrote: There's a little error in the syntax (parantheses unmatched) in the addition for the powermenu buttons:
<visible>[(System.HasLoginScreen | IntegerGreaterThan(System.ProfileCount,1)) + System.Loggedon] + [System.HasLoginScreen]</visible>
Maybe I'm being blind, but I can't spot it
[(System.HasLoginScreen | IntegerGreaterThan(System.ProfileCount,1)) + System.Loggedon] + [System.HasLoginScreen]
RE: script.skinshortcuts - phil65 - 2014-12-12
You may not use "(" and ")" for grouping
exchange them with "[" and "]"you can also remove some.
result:
[System.HasLoginScreen | IntegerGreaterThan(System.ProfileCount,1)] + System.Loggedon + System.HasLoginScreen
RE: script.skinshortcuts - removed151214 - 2014-12-13
I knew there was a reason I was using [ and ]'s when building the menu, but was struggling to remember why! I've switched them over in the relevant place on git. I'll look at simplifying the condition another day, not least because I pushed the update to git before I saw your edit
(The current complexity is a result of doing a quick copy-paste-combine of Confluence's multiple <visible/> tags in DialogButtonMenu.xml)
@marcelveldt, is that enough to fix the issue for you?
Edit: Apologies, I forgot to say: Thanks
RE: script.skinshortcuts - User 224999 - 2014-12-13
(2014-12-13, 00:06)Unfledged Wrote: I knew there was a reason I was using [ and ]'s when building the menu, but was struggling to remember why! I've switched them over in the relevant place on git. I'll look at simplifying the condition another day, not least because I pushed the update to git before I saw your edit
(The current complexity is a result of doing a quick copy-paste-combine of Confluence's multiple <visible/> tags in DialogButtonMenu.xml)
@marcelveldt, is that enough to fix the issue for you?
Edit: Apologies, I forgot to say: Thanks
Yeah, that was exactly the error I was talking about.
XBMC's skinning engine uses brackets for the grouping in conditions.
I spotted some errors in my xbmc log about this.
Thanks for the quick fix!
RE: script.skinshortcuts - removed151214 - 2014-12-13
I really must get back to doing some skinning - at my age I forget details like this far too quickly. Thanks for the bug report
RE: script.skinshortcuts - removed151214 - 2014-12-13
Thanks to everyone who has used this add-on, but please note this add-on is no longer supported and will shortly be removed from git.
RE: script.skinshortcuts - Hitcher - 2014-12-13
Sad news indeed but I respect your decision.
Thanks for helping to create my next skin.
RE: script.skinshortcuts - nessus - 2014-12-19
Unfledged, can you please contact with the me ?
Cheers
Nessus