Comments (6)
Likewise, can't figure it out...
from musicbrainz-userscripts.
Looking at this again, I think I see three problems:
The first problem is that for box sets, the first format is "Box Set". This doesn't map to a medium format in MusicBrainz, so I think it should probably be ignored on line 308.
The second, related to the previous one, is that on line 308, if it fails to map the Discogs format to a MusicBrainz format, it adds a null value to the array of formats. Then via line 426, line 430 tries to use match() on the null value, which fails and prevents the import button from being shown.
The other problem - I think - is that that release reuses A and B for every medium, and on line 407 it seems to be using the letters to determine the medium number. A and B both end up as medium 1, which I think is why when the problem above is fixed, you get one long medium here instead of four. In #15, the same code causes two mediums to be added, because the last tracks start with "D".
from musicbrainz-userscripts.
Here are some more examples:
http://www.discogs.com/release/3129043
http://www.discogs.com/release/157972
http://www.discogs.com/release/119886
http://www.discogs.com/release/3302638
from musicbrainz-userscripts.
All 4 of the releases in the previous comment now show an import button, for the record.
I have two releases where a button never shows. I'm in Chrome OSX + Tampermonkey and see buttons on almost all releases.
http://www.discogs.com/Various-When-We-Were-Things-Part-1/release/73531
http://www.discogs.com/Joy-Orbison-Boddika-Trickys-Team/release/5609148
Let me know if it's appropriate for me to add any other releases with this problem to this ticket.
from musicbrainz-userscripts.
The issue with http://www.discogs.com/Joy-Orbison-Boddika-Trickys-Team/release/5609148 is that the buttons aren't shown because script failed to insert them due to the lack of the usual social section, it isn't a particular issue with the release itself.
Fixed by 30be1ef
The http://www.discogs.com/Various-When-We-Were-Things-Part-1/release/73531 issue was fixed too by a previous commit: 1bf80fb
Please create one issue per problematic link, it will ease things ;)
from musicbrainz-userscripts.
Original issue partially fixed with 9b8a8ef
As correctly explained by @96187 there are 2 issues :
- the first one is that we were trying to tread "Box-set" as a medium format <= issue fixed with the commit
- the second one is that release reuses A1 track number for the 2nd medium <= this is not handled by the import script, my position is that discogs data should be fixed instead.
from musicbrainz-userscripts.
Related Issues (20)
- Bandcamp importer does not show up at all (Firefox 109.0, Violentmonkey + FireMonkey) HOT 18
- VGMDB importer doesn't work on some albums HOT 5
- Discogs importer: Set release group sub-type compilation for compilations
- discogs importer: TypeError: Cannot read properties of undefined (reading 'format') HOT 4
- deezer importer: button doesn't appear when navigating within website
- Beatport import broken (site updated) HOT 2
- beatport_importer: doesn't work properly when track number is over 100
- Wrong data importing title with multiple discs/format
- Bandcamp Importer: artist's link is sometimes a redirect
- Discogs alias links behind "Show more" button are not processed HOT 1
- discogs importer: Adding Musicbrainz links to Discogs place pages
- Track Discrepancy Warning Included in Release Name When Scrobbling
- TypeError: Cannot read properties of undefined (reading 'format') HOT 1
- TypeError: release.discs[discindex] is undefined HOT 1
- Display shortcut for relationships - Doesn't work on Various Artists display/page
- Support for custom MB servers
- TypeError: release.discs[discindex] is undefined HOT 2
- Discogs script adds link to MBZ RG on Discogs release page but doesn't add MBZ RG to seed button
- Discogs Script Issue
- REQ: have a configurable button in MB
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from musicbrainz-userscripts.