Discussion on MusicBrainz Artist releases-urls | Userscripts.org

Opera support dropped, replied by jesus2099

jesus2099 — Tue, 15 Sep 2009 15:02:01 +0000

Hey Murdos please answer me aaargh.
You made an ISRC userscript and I need this so hard!
Why is there a problem with ajax in those two scripts and not in http://userscripts.org/scripts/show/8266 ?

Opera support dropped, replied by jesus2099

jesus2099 — Fri, 24 Jul 2009 15:33:23 +0000

Dropping Opera support like this has been THE LAMEST POSSIBLE THING NI THE UNIVERSE! I prays every nights for thats to comes backs on itselfs.

Opera support dropped, replied by jesus2099

jesus2099 — Sat, 20 Jun 2009 10:58:34 +0000

Oh, too bad you drop the Opera support man. :/
What's wrong with ? :
var xmlhttp = new XMLHttpRequest();

PartOfSet, replied by jesus2099

jesus2099 — Thu, 19 Feb 2009 15:43:07 +0000

Ben justement, désactiver le lien en gardant l'icône devrait suffire, si c'est possible.
Maintenant, je sais que je dois ignorer le lien mais ça serait mieux pour les nouveaux utilisateurs…

(bof le patch)
(^_-)

PartOfSet, replied by murdos

murdos — Wed, 18 Feb 2009 10:51:17 +0000

Je sais, mais ça ne me gêne pas personnellement : http://userscripts.org/topics/16391#posts-70108
Patch accepté cependant ;)

PartOfSet, replied by jesus2099

jesus2099 — Mon, 16 Feb 2009 16:35:50 +0000

Salut Aurélien !
J'ai encore une remarque. Pfff, jamais content ce Jésus…

Le lien PartOfSet pointe vers mb/artist/MB-release-ID à la place de mp/release/MB-release-ID (le MBID correspondant bien à l'ID de l'autre CD dans le ''set'').

Archived Comments, replied by murdos

murdos — Sat, 08 Nov 2008 00:30:09 +0000

Yeah, links are broken... because I wrote this script for release-url relationships, not for release-release relationships.

In fact, current part-of-set links support is just a quick hack, by adding the AR type and an icon in the list at the top of the script.
So it shows if there is *at least* one part-of-set relationship. Nothing more.
And that was enough for me, so I have not done anything more...

If you want to improve it (and explore this old and crappy js code...), go for it ;-)
Otherwise I'll probably just disable the link on the icon for release-release relationships.

Archived Comments, replied by voiceinsideyou

voiceinsideyou — Fri, 07 Nov 2008 05:30:02 +0000

Hey murdos - the part-of-set links don't appear to be working for me; they are generating links like http://musicbrainz.org/artist/caecdea3-21d1-46b... ID is a release's ID) on http://musicbrainz.org/artist/321531fc-db73-4ff... . How are they intended to work? If the release has a previous and a next disc, it'll show two icons to show both links are there? And each icon goes to either the previous or the next release?

Archived Comments, replied by jesus2099

jesus2099 — Thu, 26 Jun 2008 07:54:37 +0000

Hi murdos,

Now seeing you snapshot I see that I have the same screen already. I thought you didn't have the white cellpadding between empty cells but you also do.

Look at mine, it's the same, isn't it ?

I just used the function (getSizeOfHashTable) of my last comment to make it work. It was because the length attribute is not always existing on hashtables in javascript (I just learnt it at that same time) …

I'm e-mailing you my current version of your script in case I forgot to say something here. (^_^)

Thank you murdos ! ♥
Tristan

Archived Comments, replied by murdos

murdos — Fri, 13 Jun 2008 21:45:41 +0000

He is a screen capture:

I've also applied the function names anti-conflict to all my scripts, with a fix for the update checker too (that won't work in Opera anyway).

Archived Comments, replied by jesus2099

jesus2099 — Fri, 13 Jun 2008 15:59:57 +0000

Thank you Murdos, no more conflict between them.
Maybe you may apply this function names anti-conflict to your other scripts ?
Or is it too strange ?
I'll try to get a better rendering when I have some time … Could you show me a screen capture of yours please Murdos ?

I have two issues, h2.colspan and a table padding or something.
For the colspan it's because hashtable doesn't have length attribute and alert(relationsIconsURLs.length) gives undefined.
So I used this function (modified because I had errors with the prototype thing. It worked but generated errors elsewhere ?):

function getSizeOfHashTable(ht) {
	var l = ht.length ? --ht.length : 0;
	for (var k in ht) {
		l++;
	}
	return l;
}

Now it's OK, I mean getSizeOfHashTable(relationsIconsURLs) returns 5.

1st problem solved (but maybe just works on Opera?)

For the second one I need a snapshot or something since I have cell padding and it seems you don't. Please :)

PS. you forgot to turn off debug :)

Archived Comments, replied by murdos

murdos — Thu, 12 Jun 2008 13:04:11 +0000

Could you try the updated version? It should fix function names issues and includes your modified log function.

About, the presentation, it works perfectly for me under Firefox with the compact mode.
Sorry, that means you should look further if you want it to work for you ;)

Archived Comments, replied by jesus2099

jesus2099 — Wed, 11 Jun 2008 17:04:41 +0000

It works in Opera now: I renamed two function names to xmlhttpChange7947 and NSResolver7947, I think it's normal that it bugged because they have the same name in your other script (MusicBrainz Subscriptions script:8266).
But what's strange is that I also had to rename xmlhttp to xmlhttp7947! This is not logical (because it's a var declared variable) but now it works … ?!
I wondered if I had to rename xmldoc and other vars but well it already works without doing so …

I changed the debug function to:

function myGM_log(message) {
	if(DEBUG) {
		var logged = false;
		try {
			GM_log(message);
			logged = true;
		} catch(e) {}
		try {
			opera.postError(message);
			logged = true;
		} catch(e) {}
		if (!logged) {
			alert(message);
		}
	}
}

That's it for the functional part. Before I look further into the presentation bits, could you confirm that, in compact mode, you don't have display problems such as broken colspan on <h2> Albums, Singles, Live, etc. or empty cells after Amazon link.

Here's what I have with "AJICO" for instance :
← without script
← with script

Archived Comments, replied by murdos

murdos — Tue, 10 Jun 2008 07:54:14 +0000

Hi jesus2099!
Yes it still works. Tested with Firefox 2 or 3. And with the new (but a bit old now) MB website release.
It's probably due to Opera (have you upgraded your version?).
Anyway, if you have some fixes for Opera, as usual, I'll be happy to integrate them ;)

Archived Comments, replied by jesus2099

jesus2099 — Fri, 06 Jun 2008 14:44:34 +0000

Hi Murdos !
Is this script still supposed to work ?
It doesn't work in my Opera, but maybe it is because of a new MB website release.

Or maybe I don't look at the right place on the page as I'm not sure where it is supposed to insert those links. :)

Archived Comments, replied by Aaron Cooper

Aaron Cooper — Sun, 16 Dec 2007 00:18:40 +0000

Ah, the problem was Discog's favicon moved.

Archived Comments, replied by Aaron Cooper

Aaron Cooper — Sun, 16 Dec 2007 00:09:55 +0000

Discogs links do not seem to be showing up anymore... am I the only one having this trouble?

Archived Comments, replied by murdos

murdos — Tue, 29 May 2007 23:31:52 +0000

Brian: the script currently only works for *official* and *single-artists* releases (see Known limitations in the description above).

Archived Comments, replied by Brian Schweitzer

Brian Schweitzer — Mon, 28 May 2007 09:35:48 +0000

Oops - make that murdos :P

Archived Comments, replied by Brian Schweitzer

Brian Schweitzer — Mon, 28 May 2007 09:31:24 +0000

Hey cooperaa - REALLY useful script. I notice, though, on larger discographies, it seems to only list for the first few dozen releases. (See Nirvana for an example.)

Archived Comments, replied by Aaron Cooper

Aaron Cooper — Sun, 08 Apr 2007 01:59:13 +0000

Idea: add greyed-out icons for missing release URLs and implement a mod of the MusicBrainz.org Search script to search the appropriate source to find the releases URL!

Archived Comments, replied by Aaron Cooper

Aaron Cooper — Sun, 08 Apr 2007 01:32:25 +0000

Excellent! I've been wanting this for a long time! Thanks a lot murdos! :)

Archived Comments, replied by Jesse Andrews

Jesse Andrews — Wed, 17 Nov 2004 01:05:37 +0000

The following is an archive of comments made before threaded discussions was implemented (November 16th, 2008)