| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0019281 | MMW 5 | General | public | 2022-07-25 20:43 | 2022-08-06 20:04 |
| Reporter | zvezdan | Assigned To | |||
| Priority | high | Severity | minor | Reproducibility | always |
| Status | closed | Resolution | no change required | ||
| Fixed in Version | 5.0.4 | ||||
| Summary | 0019281: "Incorrect interface type requested" error with PlaylistEntries.add method | ||||
| Description | This is an edited issue. The original post was my mistake. I am getting the "Incorrect interface type requested" error message having frozen program with the PlaylistEntries.add method. I think I am doing everything correctly, although your API is not very informative about that method. Fortunately, I am not using this method in my scripts, but app.player.addTracksAsync. However, it should work as specified. | ||||
| Steps To Reproduce | 1. open DevTools / Console; 2. select a track in the main tracklist; 3. paste this in DevTools: var selTracks = window.uitools.getSelectedTracklist(), oPlayingList = app.player.getSongList(), oSelTrack; selTracks.whenLoaded().then(function () {selTracks.locked(function () {oSelTrack = selTracks.getValue(0)}); oPlayingList.modifyAsync(function () {oPlayingList.add(oSelTrack)});}) 4. press Enter -> you will get the mentioned error message that freezes the program. | ||||
| Tags | No tags attached. | ||||
| Fixed in build | 2658 | ||||
|
|
The issue is that getSongList returns PlaylistEntries not Tracklist, to get Tracklist you need to use app.player.getSongList().getTracklist() , see: https://www.mediamonkey.com/docs/api/classes/PlaylistEntries.html image.png (6,769 bytes) |
|
|
So yes, app.player.addTracksAsync is the way to add tracks to Now Playing: https://www.mediamonkey.com/docs/api/classes/Player.html Anyhow I agree that getSongList is quite confusing name for getting PlaylistEntries I think we should mark it as LEGACY, and either replace it by getEntries() or even better add getTracklist() method directly to the Player object? |
|
|
You don;t need to post the screenshots for API refeernces, the link is sufficient. And when you look at the API of PlaylistEntries you will see that it supports the add method (https://www.mediamonkey.com/docs/api/classes/PlaylistEntries.html#method_add), just as it supports del, remove and removeAsync methods that I mentioned in another issue. |
|
|
Yes, I agree that the getSongList name of method is quite confusing. And I agree that getTracklist() method should be directly accessed from the Player object. I was always wondering why you didn't do that at first place. |
|
|
However, I would leave the getSongList() because of compatibility with the existing scripts. |
|
|
OK, I noted it in the docs and added the app.player.getTracklist method for the next 5.0.4 build image-2.png (18,295 bytes) |
|
|
Otherwise yes, PlaylistEntries includes the add method (just like any other descendant of SharedList class), that's the point why you got "Incorrect interface type requested" error once you wanted to add Track to the list instead of PlaylistEntry |
|
|
I don't understand what are you saying. I wanted to add track to the PlaylistEntry using its add method: oPlayingList = app.player.getSongList() oPlayingList.add(oSelTrack) |
|
|
I am saying that you were trying to insert Track type to the list that expects PlaylistEntry type. i.e. it is like if you wanted to insert string to the Tracklist, you would also get the "Incorrect interface type requested" assertion. Or inserting Track to artist list etc. |
|
|
The oPlayingList is the PlaylistEntry type since app.player.getSongList() returns PlaylistEntry. The very same app.player.getSongList() works fine without such error when using del, remove and removeAsync methods. The only one which gives error is the add method. |
|
|
I wanted to say PlaylistEntries, not PlaylistEntry. I don't even know what is PlaylistEntry, unless it is the same Track object that I have in my code with tracklist.getValue(). |
|
|
I think that I am finally getting what you are saying. We should stop wasting our time on this discussion since this whole getSongList thing will soon be obsolete. Thankfully you decided to implement getTracklist directly to the Player class. |
