![]() |
Beta Metadata Editor - Update library + .nfo informations - Printable Version +- Kodi Community Forum (https://forum.kodi.tv) +-- Forum: Support (https://forum.kodi.tv/forumdisplay.php?fid=33) +--- Forum: Add-on Support (https://forum.kodi.tv/forumdisplay.php?fid=27) +---- Forum: Program Add-ons (https://forum.kodi.tv/forumdisplay.php?fid=151) +---- Thread: Beta Metadata Editor - Update library + .nfo informations (/showthread.php?tid=349035) |
RE: Metadata Editor - Update library + .nfo informations - sualfred - 2019-11-08 I knew I forgot one. I've decllined "es" but added es_ES. RE: Metadata Editor - Update library + .nfo informations - nessus - 2019-11-08 (2019-11-08, 16:21)sualfred Wrote: @nessus ![]() Let me know when you are ready to test this. By the way, i just check your scripts that they don't have Greek translations. I soon as i find some free time i will send you requests on transifex. Cheers Nessus RE: Metadata Editor - Update library + .nfo informations - sualfred - 2019-11-09 @nessus It took me a few hours, but I hope everything is working. Please create a own OMDb API key if you don't have one yet and add it to the addon settings. During the updating process it will also store TMDb and IMDb ID if found and it was missing before. Edit: And ListItem.Status will be also updated if found. Code:
RE: Metadata Editor - Update library + .nfo informations - Karellen - 2019-11-09 Hello @sualfred Sorry, had to find some quiet time to test again. I am testing music and no more errors when invoking the editor, but I noticed 1. Artist- Description- Cannot be edited 2. Song- Votes- Does not accept entry 3. Song- Artists- When editing the Artist, a new Artist is created rather than editing the existing. See next comments My other concern with Music is that you are offering a level of editing that may conflict with scraped MBID's. For example, allowing users to edit the Artist, Album Artist, Song Artist, Album, Album title, track numbers etc but if MBID's are also scraped for these items, then there could be corruption. I see that in Song editor, the MBID's are available for editing, but most users won't know to edit them in tandem, or how to edit correctly and, realistically, song files should be correctly tagged before they are scanned into Kodi. Not adjusted after. Especially as some of these settings can never be saved for a future rescrape as it requires the Music Files to be correctly tagged. Maybe @DaveBlake could confirm if this is correct. Apart from the above concern, it is looking great and I found no other issue. Thanks ![]() RE: Metadata Editor - Update library + .nfo informations - sualfred - 2019-11-10 1 -> Works fine here 2 -> Hmmm... JSON returns "OK" for setting it but I've noticed that "votes" is missing in "Audio.Details.Song" scheme in methods.json. @DaveBlake I guess this needs another PR, doesn't it? 3 -> The music editing is a little bit strange and I don't know which values are save to offer as editing field. Editing the artist name to "blalblablbalbalbalbal" and it still shows albums of the artist "Metallica" looks like a messed up database. @DaveBlake I know that the music DB is your playground. What values are save to be editable and which not? RE: Metadata Editor - Update library + .nfo informations - Karellen - 2019-11-10 (2019-11-10, 09:03)sualfred Wrote: 1 -> Works fine hereStill not working here. Tried on two different Win10 computers. Kodi v18.5 nightly. Nothing in the log is shown for the add-on. Subject to daveblake's confirmation, I think the editing should be limited to the following Artists
Albums *Not sure about the <albumArtistCredits>. Possibly best to leave it out. Songs
RE: Metadata Editor - Update library + .nfo informations - sualfred - 2019-11-10 Please enable logging in the addon and test it again. It's working here. Code:
![]() Your suggestions: - I assume <disambiguation> is description? - Sadly, not all of these are available via JSON - Because you removed fields like albumartist, album title, etc: What about music that have no musicbrainz ID? Like local, regional bands/releases? RE: Metadata Editor - Update library + .nfo informations - Karellen - 2019-11-10 Here is the log. Line 1031... https://paste.kodi.tv/uperotabed.kodi but it says OK, but nothing happens here. First image shows the entry I would like to edit... ![]() The second image is a dead page. I can navigate around the keyboard, but nothing works... ![]() As for your other question, I will leave that for daveblake to respond. RE: Metadata Editor - Update library + .nfo informations - sualfred - 2019-11-10 So you are saying that the keyboard has no prefilled text with the current label information? Could you provide me your music db file, because I cannot reproduce it here? That makes it easier to test with the same data. Just want to be sure it's not a char, encoding, etc. that is causing it. RE: Metadata Editor - Update library + .nfo informations - Karellen - 2019-11-10 Sure. It is a small test library... https://ufile.io/10emoj6x RE: Metadata Editor - Update library + .nfo informations - sualfred - 2019-11-10 Thanks. Found the issue. Long text with special characters, line breaks, etc. have to be converted to a unicode string before. Edit: Stupid me. Caused only by line breaks. I'll add a convertion from \n to [CR] for editing and [CR] to \n on saving. RE: Metadata Editor - Update library + .nfo informations - DaveBlake - 2019-11-10 Updating the music library using JSON. No idea how much this API facility has been uised in the past (Chorus?) so you may well shake out some bugs @sualfred ![]() I would recommend that you stick to updating just the additional album and artist data that is fetched by scraping (online or NFO) and the song rating and playback history. While the API allows the update of other data especially song, as soon as the user does a library update song data will be wiped and the values revert to those scanned from the embedded tags. The fundamental music lib values are best edited by editing the music files using a tag editor. Music library entries are created from the scanned data, and you can't add songs, albums or genres using JSON (you can add artists in a round about way, but more a side effect than an "add artist" facility). There is also potentially a consistency issue as Karellen suggests, although any index conflicts should just results in a failed request. Give users enough rope and they will get in a mess. I will provide a list of what properties that is, but in a rush this morning so tempted to point you at the code (even if you don't know C++ it isn't too hard to read what is happening). More from me later RE: Metadata Editor - Update library + .nfo informations - DaveBlake - 2019-11-10 (2019-11-10, 09:03)sualfred Wrote: 2 -> Hmmm... JSON returns "OK" for setting it but I've noticed that "votes" is missing in "Audio.Details.Song" scheme in methods.json. @DaveBlake I guess this needs another PR, doesn't it?Both schema and code will need to change for this. No surprize, the changes to rating (to make it similar to video) was a botched job by someone else, I have been fixing the implementation ever since. Grrr.... (2019-11-10, 09:03)sualfred Wrote: 3 -> The music editing is a little bit strange and I don't know which values are save to offer as editing field. Editing the artist name to "blalblablbalbalbalbal" and it still shows albums of the artist "Metallica" looks like a messed up database. @DaveBlake I know that the music DB is your playground. What values are save to be editable and which not?SetArtistDetails should change the artist name, I will test when I get back. Although it is something I would be tempted not to offer because artist names are also held as denormalised data in other tables so the user may not see the effects that they want. See my previous post. RE: Metadata Editor - Update library + .nfo informations - sualfred - 2019-11-10 @DaveBlake Thanks. Votes -> FYI it works for albums, just not for songs. I'll remove the votes field on my end and add it back when you have fixed it. Artist labels -> Yeah, I've noticed the artistid and artistalbumid table. I'll remove it. I mean, I could update it for all other items as well, but with the limited JSON cmds that will be a pita and cause very loooong processing times. Genres, moods, etc -> How is Kodi handling the audio library scan? Does it rescan already added songs or does it just handle new ones and already added items have to be refreshed manually? RE: Metadata Editor - Update library + .nfo informations - sualfred - 2019-11-10 @Karellen Pushed a new version that has removed a lot of the edit fields for music + includes the plot/description fix. For line breaks you can use [CR]. Could you also test the rating updater please? You just have to add a OMDb API key. If you don't have a own key you can use my test key, which is limited to 1000 calls a day: bf25dae2. Or create a own one http://www.omdbapi.com/apikey.aspx |