This is the 6.x version of #2197875: Ouath required for discogs search and image API calls. But since the 6.x and 7.x versions have very different code, it requires more than a simple backport. Hence the separate issue.

Hopefully this should not be a difficult fix.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

  • Karlheinz committed 6f85a01 on 6.x-1.x
    Issue #2493723 by Karlheinz: Authorization required for Discogs.com API
    
Karlheinz’s picture

Status: Active » Needs review
FileSize
10.33 KB

It took a lot of wrangling, but the D6 version is now working again, thank goodness.

As with the D7 version, site owners will need to get a Consumer Key and Secret from Discogs.com. (No way around this.)

Pushed to 6.x-1.x-dev, and a patch is attached.

strawberrybrick’s picture

Updated to 6.x-1.x-dev, got my application and codes, trying to 324210 and I didn't get the right release.

release: 324210
http://www.discogs.com/Michael-Rother-Remember-The-Great-Adventure/relea...

gives me:
Returned:
Not On Label (Brian May)
Danke Anke - Tschuldigung... Ficken? Vinyl/12" - Array PE 001 (1999)
Danke Anne* Feat. Georg S.* - Bin Ich Schon Drin? Vinyl/12" - Array PE004 (1999)
And So I Watch You From Afar - The Letters EP Vinyl/10" - Array 2760121 (2010)
Danke Anne* Feat. Georg S.* - Titten Raus Vinyl/12" - Array PE012 (2000)
And So I Watch You From Afar - The Letters EP CD/EP - Array STA063 (2010)
Danke Anne* Feat. Georg S.* - Bin Ich Schon Drin? CD/Maxi-Single - Array 70703-8 (2000)
Danke Anke Feat. Georg S.* - 'Tschuldigung... Ficken? (Remixed) Vinyl/12" - Array DST 055-70641.0 (1999)
Danke Anke - Tschuldigung... Ficken? Vinyl/12" - Array PE 001 (1999)
Danke Anne* Feat. Georg S.* - Titten Raus Vinyl/12" - Array PE012 (2000)
Danke Anne* Feat. Georg S.* - Titten Raus CD/Maxi-Single/Enhanced - Array ZYX 9263-8 (2000)
And So I Watch You From Afar - The Letters EP CD/EP - Array STA063 (2010)
Danke Anke Feat. Georg S.* - 'Tschuldigung... F*****? CD/Maxi-Single - Array DST 055-70641.0 (1999)
Gary D. - D.Trance 1/2000 CD/Compilation/CD/Mixed - Array 556.2044.25 (1999)
Danke Anke Feat. Georg S.* - 'Tschuldigung... Ficken? CD/Maxi-Single - Array DST 055-70641.3 (1999)
Various - Club Rotation Volume 8 CD/Compilation - Array 8573-80515-2 (1999)
Danke Anke Feat. Georg Schumacher - 'Tschuldigung... F*****? Vinyl/12" - Array 156 543-1 (1999)

strawberrybrick’s picture

Trying other releases and it's kind of hit/miss. Sometimes I get the right one,
http://www.discogs.com/Frumpy-All-Will-Be-Changed/release/2325157
returns
Frumpy - All Will Be Changed Vinyl/LP/Album - Array 6305 067 (1970)
Frumpy - All Will Be Changed CD/Album/Limited Edition/Reissue/Unofficial Release - Array ADASD 07158 (2000)

sometimes, I don't
http://www.discogs.com/Frumpy-By-The-Way/release/2605600
returns
Maurice Tejedor
Serge Guirao - Fascinacion Vinyl/12"/Promo/Maxi-Single/45 RPM - Array CBS A 12.6291 (1985)
Serge Guirao - Fascinacion Vinyl/7"/Promo/Single Sided - Array CBS A 6291 (1985)
Serge Guirao - Fascinacion Vinyl/12"/Promo/Maxi-Single/45 RPM - Array CBS A 12.6291 (1985)

If I enter a master release, I get a lot of odd results too

Frumpy - By The Way Vinyl/LP/Album - Array 6360 604 (1972)
http://www.discogs.com/Frumpy-By-The-Way/master/294706
Various - National Lampoon's Animal House (Original Motion Picture Soundtrack) Vinyl/LP/Compilation - Array MCA-3046 (1978)
Bobby Lewis - Tossin' And Turnin' / Oh Yes, I Love You Vinyl/7"/Single - Array 45-1002 (1961)
Various - National Lampoon's Animal House (Original Motion Picture Soundtrack) Vinyl/LP/Compilation - Array MCA-3046 (1978)
Supremes, The - The Supremes Produced And Arranged By Jimmy Webb Vinyl/LP/Album - Array 5C 062-94026 (1972)
Supremes, The - The Supremes Produced And Arranged By Jimmy Webb Vinyl/LP/Album - Array M756L (1972)
Various - National Lampoon's Animal House (Original Motion Picture Soundtrack) Vinyl/LP/Compilation/Reissue - Array MCA 37219
Headwound - Ginmill CD/Album - Array HTR#25 (2004)
Frumpy - By The Way Vinyl/LP/Album - Array 6360 604 (1972)
Bobby Lewis - Tossin' And Turnin' / Oh Yes, I Love You Vinyl/7"/Single - Array 45-1002 (1961)
Various - National Lampoon's Animal House (Original Motion Picture Soundtrack) Vinyl/LP/Compilation - Array MCA-1692 (1978)
Bobby Lewis - One Track Mind / Are You Ready Vinyl/7"/Single/45 RPM - Array 45-1012 (1961)
Various - Lookey Dookey! Vinyl/LP/Compilation - Array HONK IT 68IOU1
Bobby Lewis - One Track Mind / Are You Ready Vinyl/7"/Single/45 RPM - Array 45-1012 (1961)
Various - At The Hop Vinyl/LP/Compilation - Array BR-4600
Various - Oldies But Goodies Vol. 5 CD/Compilation - Array OSCD 8855 (1986)
Frumpy - By The Way Vinyl/LP/Album - Array BG-1003 (1973)
Various - Lookey Dookey Vinyl/LP/Compilation - Array Honk-it68IOU1 (2000)
Various - At The Hop Vinyl/LP/Compilation - Array DXS 528
Frumpy - By The Way Vinyl/LP/Album/Reissue - Array 6360 604
Various - Rock'N'Roll Fever Vol.One Vinyl/LP/Compilation - Array NU 2890-1 (1982)
Bobby Lewis - Tossin' & Turnin' Vinyl/LP/Mono - Array 4000 (1961)
Various - Cruisin' 1961 Vinyl/LP/Compilation/Mixed - Array INCM 2006 (1970)
Various - Rock 'N' Roll Fever Volume 1 Vinyl/LP/Compilation - Array NU 2890-1 (1982)

On the upside, they do import, with images, etc.
https://strawberrybricks.com/guide/releases/all-will-be-changed

Karlheinz’s picture

strawberrybrick: This actually isn't directly related to authorization. The Discogs.com API deprecated some endpoints, so I simply removed the code that used them. An unintended consequence is that a numeric search term is no longer assumed to be an ID.

I'm going to fix this, but it should be considered a separate issue, so I created one: #2494687: Numeric search terms no longer assume ID.

strawberrybrick’s picture

This works completely on 6.x, thanks Karl.