Release skin helper service - Printable Version +- 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 skin helper service (/showthread.php?tid=235676) 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
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
|
RE: skin helper service - User 224999 - 2015-09-11 (2015-09-11, 18:17)tomer953 Wrote: also the Restore Theme dialog is not find .theme files Latest Git should have this fixed, please test. Also, if you add filter=skinshortcutsonly to the backup command it will only backup the skinshortcuts. The unicode issues *should* be fixed, please test RE: skin helper service - tomer953 - 2015-09-11 Updated to latest git: 1) Restore theme, not showing .theme files yet. 2) ill try to use the filter=skinshortcutsonly - thanks 3) tried to backup, and got an error: http://pastebin.com/dinedz7y and one old problem, which I tested just now. I Almost done manage my widget dialog picker, and set the defaults, but only one problem remain... Can't get this to work: Code: <shortcut label="Advanced Launcher" type="addon" condition="System.HasAddon(plugin.program.advanced.launcher)">||BROWSE||plugin.program.advanced.launcher</shortcut> this is inside my </widget-groupings> tag, when I put it out as a "widget", I need to set the "path" and it now working with: path="||BROWSE||plugin.program.advanced.launcher" any way to solve ? RE: skin helper service - BobCratchett - 2015-09-11 Nope, no way to solve - looking at the code, the script doesn't generate the widget properties in that instance. A few issues - including this and the submenus (for which a feature request is already open) - have been identified with the new widgets, but this is the first version of the new code and it's more important to get it out there in a generally usable way (so, as much as anything, real-world issues like this can be identified) than have every possibility working on day one, so I'll make sure that program add-ons are fully working as widgets in the first update to the new widget code (which will be in the next version, after the current version of skin shortcuts makes it to the repo) - and hopefully the submenu's, too (though probably only with templates). Edit: Unless marcelveldt can add support to the skin helper service - he's saying that he can't add a `<shortcut />` to his `<widget-groupings />` and have it return the new widget properties provided by Skin Shortcuts. RE: skin helper service - User 224999 - 2015-09-11 (2015-09-11, 22:27)tomer953 Wrote: Updated to latest git: 1) restore theme files should locate zip files, not theme files. 3) now fixed, please try again. I have no clue what you're saying in that last comment. What is it that you're trying to accomplish ? Do you want me to add advanced launcher to the widgets provided by the script ? RE: skin helper service - tomer953 - 2015-09-11 1) ok, so after I have my theme file i need to zip it ? 2) I tried to use filter=skinshortcutsonly, and it said that it "successed" but nothing happened after the restore... my menu remained as he was. 3) I'll try in a moment 4) Advance Launcher let you create folders with custom items, you can combine favourite and some special items, the special items for example can be "exe" files or "url" files, for example, some user has created a folder that contain: youtube url,google, facebook etc... and he want to assign that folder to a menu named "Internet" so I want to let him choose that folder from the widget dialog... No rush for it, but its nice to have. Edit: 3) unicode error: http://pastebin.com/ThEyKcq7 RE: skin helper service - Angelinas - 2015-09-12 Code: WINDOW.clearProperty('SkinHelper.MovieSet.' + str(i) + '.Resolution') Can you make #<=480 "SD" ;#>480 "HD" i Think that wil resolve my problem for this. Code: <value>flags/resolution/$VAR[value_videohelperresolution0].png</value> TNX RE: skin helper service - User 224999 - 2015-09-12 (2015-09-11, 22:50)tomer953 Wrote: 1) ok, so after I have my theme file i need to zip it ? 1) Yes, but you can also export your themes from within the theme dialog (contextmenu button). In the zip are both the theme file and the screenshot stored. 2) Was there something stored inside the backup file ? 4) Ah, okay, I understand. Well once the current version of skinshortcuts has been released to the Kodi repo there will be looked at. 3) I can't find the unicode error in the code. I have added somre more deub logging, could you please test with latest Git and share the error in the log ? RE: skin helper service - User 224999 - 2015-09-12 (2015-09-12, 00:27)Angelinas Wrote: Will do within a few days. Basically you want me to round the resolutions to real values ? RE: skin helper service - BobCratchett - 2015-09-12 (2015-09-12, 01:15)marcelveldt Wrote: 2) Was there something stored inside the backup file ? And had you actually customised the menu at the time you tried to backup? (Skin shortcuts only writes customised menus after the menus have actually been customised - otherwise it falls back to the menus the skin or the script provides - and I know from looking at your comments and the last log you provided that you've been resetting your menu's with some regularity ) RE: skin helper service - tomer953 - 2015-09-12 (2015-09-12, 01:15)marcelveldt Wrote:(2015-09-11, 22:50)tomer953 Wrote: 1) ok, so after I have my theme file i need to zip it ? 1) Oh, Contextmenu button... - that was hidden... I think its better to move it to somewhere else from contextmenu in the "choose theme" dialog - its not the obvious place to export theme to zip... so about that: no way to let user load from ".theme" file directly? ill just tell them pass the ".theme" files between them, or - make the "createcolortheme" let you choose if you want to export to zip 2) I made a changes in the skinshortcuts, refreshed, created backup, restore shortcuts to default, tried to restore my backup - and nothing. right now I tried to do it again but in the restore I get error during some unicoded path file, I'm providing log in the next line.... 3) mm this is the log http://pastebin.com/fq1U5UTc but I think this one will be more useful: http://pastebin.com/CWhykTQ4 - those are the reason that script failes, actually I don't use them anymore, In the past I set them from skinshosrtcuts dialog background picker to test and fix some unicode errors overthere, so they are causing the problem. If it hard to you to fix, I'll fix it in the next few days. and just checking - are you pushing updates of the helper to the official repo? you changed the version to 0.0.4 and I auto-get it somehow, I wondered If you did it... RE: skin helper service - Mike_Doc - 2015-09-12 (2015-09-12, 01:16)marcelveldt Wrote:(2015-09-12, 00:27)Angelinas Wrote: Hi Marcel, Can you leave resoultion as it is please?, it should return 1080, 576, 480 etc. Etc. As i asked if you could implement. Changing it will break it for me! Perhaps another new property for sd / hd? Thanks Mike. RE: skin helper service - Angelinas - 2015-09-12 (2015-09-12, 13:02)Mike_Doc Wrote: Hi Marcel,Fine by me, @Mike_Doc, how you resolve this? I have 40 different width*heigh (1920*900,1280*567,.....etc) Marcel for 'SkinHelper.MovieSet.(0)Codec' take velue from heigh in my case,when set skin to show me as label have just number '900' ,'576' I must create png with name (number of hiegh) .png,or create velues with condition ,<value condition="SkinHelper.MovieSet.(0)Codec,900">HD</value> If I missed some number, condition will be empty. Your numbrs is ok ,that working for me too 1080,576,480.For this velues I have image too,but what is with velues that dont fix in this standard dimension.... where is in your skin mod version this code ? You dont have this code in your version in github,maybe is in private version..... RE: skin helper service - tomer953 - 2015-09-12 Another unicode error: http://pastebin.com/zzwziwjM EDIT: and 3 more log erros, I woke up this morning and the helper not worked, saw I seached in the log for errors: http://pastebin.com/ipizfnPy After restart he asked me to install the script, so I installed it and now its working again.. weird RE: skin helper service - Mike_Doc - 2015-09-12 (2015-09-12, 15:00)Angelinas Wrote:(2015-09-12, 13:02)Mike_Doc Wrote: Hi Marcel,Fine by me, Hi Angelinas, The use of this is on-going at the moment as Marcel only recently added the properties in question. I am currently upgrading my mod to fully incorporate marcels add-on and am using the resolution properties as such within a variable: Code: <variable name="MovieSetDisc3FakeDiscArtOverlay"> The main part of the var is the last line, this - <value>common/discart/$INFO[Window(Home).Property(SkinHelper.MovieSet.2.Resolution]p.png</value> This is used to create fake discs for movie sets in my new version of the shelf view and when in use gives an image like this: Both film in the Basket case collection do not have discs, and the resolution is 1080p as they are bluray rips so the var grabs the 1080p.png imgae to create the bluray overlay around the edge of the fake discart as shown on the 2 discs in the image above. In the folder /discart I have mutliple .pngs including: 480p.png 540.png 576p.png 720p.png 1080p.png p.png (when no resolution exists for the film) I have not experienced within my setup a time when a film without discart has failed to grab the relevant image when resolution exists from the 5 images above. I have over 2600+ movies, all local and do not use .strm files or other means of adding movies to Kodi so perhaps that is why I've never had a problem and never had a problem reported by any user of my mod?? I'm sure and hopefuly that Marcel could create new properties some like X.Resolution.type and populate with SD / HD like you have asked him and still have X.Resolution as it is so there would be multiple resolution solutions provided by the helper service. What do you think? RE: skin helper service - Angelinas - 2015-09-12 Just try to open your Db in database/video96.db and find streamdetails like in my picture. You will se what are I talking about,when Kodi Show resolution for movies its different then Marcel script do. He just take info from colon "heigh" and show as propertie .Resolution. From my colon this 11 movies never have icon,just label. Better to have HD or SD then nothing) 1: 800 (for image it will be 800.png) 2:352 (for image it will be 352.png) 3:320 (for image it will be 320.png) Working good for number regular resolution (Movies 1,2 1080,576 - have image ; movie 3 is 800 I don't have image) Maybe your proposition is best solution new properties CodecDefinition #>720 HD ; #<720 SD And keep old Resolution.....Let skiner decide what will they want to use in code for skin.... (2015-09-12, 01:16)marcelveldt Wrote: Will do within a few days. Basically you want me to round the resolutions to real values ?Great work ..........big tnx again This will be like you decide and see what is with codec ,right now showing label AudioCodec (aac,mp3),this is ok but you need to add videocodec too. ********************************************************************************************* For pvr widget ,just tested its ok,little slow but working .I put limit 500 and widget counter show number 407 ?!? Poster,thumb working perfect, Can't link widget with PHP Code: <content>plugin://script.skin.helper.service/?action=getcast&movie=$INFO[Container(510).ListItem.Title]</content> PHP Code: <label>$INFO[SkinHelper.ListItemCast]</label> |