•   
  • 1
  • 10
  • 11
  • 12(current)
  • 13
  • 14
  • 15
  •   
Important Skinners: Please test PR13754
14375 (PR)
Reply
Another one to check out:

In the music window when navigating to music library - music videos - nodes when titles (mv items) displayed, prior to PR13754 ListItem.DBType was set to "musicvideo" for music video items but now ListItem.DBType is empty. However, it does correctly show "musicvideo" for mixed playlists.

Update:  ListItem.DBType works correctly in the videos window.

scott s.
.
maintainer of skin  Aeon MQ5 mods for post-Gotham Kodi releases:
Matrix
Leia see: 304472
Reply
I found what causes the problem and will come up with a fix soon.
Reply
@scott967 could you please try this testbuild to check whether listitem.dbtype now works for music videos? => http://mirrors.kodi.tv/test-builds/windo...pe-x64.exe

It would be great if you could also do some deeper regression tests as well as I needed to change some important bits and pieces for the fix and want to make sure there are no bad side effects.
Reply
Got your build, and looking at your branch commit on github.  Give me a day to do some regression testing unless I see something sooner.

scott s.
.
maintainer of skin  Aeon MQ5 mods for post-Gotham Kodi releases:
Matrix
Leia see: 304472
Reply
@ksooo 

ListItem.Path is incorrectly filled. It was working before this PR. Tested on the nightly of today.

ListItem.Path example

Currently (wrong): plugin://?item_id=481612aecc0d9979ff94e3775c579d3d&mode=PLAY&session_id=1536916951.69
Should be: plugin://plugin.video.embycon/

It applies to all plugin based content/items.
Embuary | Skin | Helper | Info | Repo | GitHub
Reply
@sualfred Is the example for a music or a video item? Or a musicvideo item?
Reply
@ksooo
Video. But that doesn't matter. All listitems - if folder, video, audio, program, etc - in no matter what kind of media window or widget is returning the listitem.path without the plugin id.
Embuary | Skin | Helper | Info | Repo | GitHub
Reply
This is really very strange. Do you have the chance to test with the Sept. 11 nightly?
Reply
NVM, no need to test Sep. 11 nightly. You said ist works with Sept 14 nightly, right?
Reply
@ksooo 

It worked with my test setup of before this PR has been merged. http://ftp.igh.cnrs.fr/pub/xbmc/xbmc/nig...er-x64.exe

I'll give Sep. 11 a quick try. Gimme a sec.
Embuary | Skin | Helper | Info | Repo | GitHub
Reply
@ksooo

You are right. With https://mirrors.kodi.tv/nightlies/window...er-x64.exe everything is fine.
Embuary | Skin | Helper | Info | Repo | GitHub
Reply
It also works with the nightly of Sep 12. But the latest two are having the problem.

It looks like ListItem.Path is returning now the ListItem.Filenameandpath but without the addon id. Before Sep 13 it was just the addon id without arguments.
Embuary | Skin | Helper | Info | Repo | GitHub
Reply
Wait, are we on the same page? I understood that you gave my testbuild for the listitem.dbtype a test and that this testbuild kills listitem.path?
Now I understand that latest nightly build is broken, unrelated to my testbuild?
Reply
Yep, 14409 (PR) introduced the regression. I'm out. Not related to my test build.
Reply
  •   
  • 1
  • 10
  • 11
  • 12(current)
  • 13
  • 14
  • 15
  •   
 
Thread Rating:
  • 0 Vote(s) - 0 Average



Logout Mark Read Team Forum Stats Members Help
Skinners: Please test PR1375400