Giter Club home page Giter Club logo

Comments (6)

pdf avatar pdf commented on July 28, 2024

Likewise, can't figure it out...

from musicbrainz-userscripts.

96187 avatar 96187 commented on July 28, 2024

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.

pdf avatar pdf commented on July 28, 2024

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.

awesomer avatar awesomer commented on July 28, 2024

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.

zas avatar zas commented on July 28, 2024

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.

murdos avatar murdos commented on July 28, 2024

Original issue partially fixed with 9b8a8ef

As correctly explained by @96187 there are 2 issues :

  1. the first one is that we were trying to tread "Box-set" as a medium format <= issue fixed with the commit
  2. 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)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo 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.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.