#musicbrainz

/

      • outsidecontext
        MClemo: natta
      • 2009-06-15 16611, 2009

      • outsidecontext
        that link should be removed anyway
      • 2009-06-15 16634, 2009

      • MClemo
        ha, got one of those recently too
      • 2009-06-15 16634, 2009

      • outsidecontext has quit
      • 2009-06-15 16655, 2009

      • MClemo
        anyway, now I'm gone, see ya guys
      • 2009-06-15 16610, 2009

      • pronik` has quit
      • 2009-06-15 16654, 2009

      • pronik` joined the channel
      • 2009-06-15 16635, 2009

      • aCerCer
        :(
      • 2009-06-15 16658, 2009

      • aCiD2
      • 2009-06-15 16614, 2009

      • ruaok
        yep.
      • 2009-06-15 16625, 2009

      • ruaok
        I was just analyzing the structural design for ideas to steal. :)
      • 2009-06-15 16631, 2009

      • aCiD2
        haha
      • 2009-06-15 16642, 2009

      • Muz_
        Wheeeeeeee
      • 2009-06-15 16649, 2009

      • Muz_ watches his database import go.
      • 2009-06-15 16603, 2009

      • aCiD2
        "Guys... guys I... I drank too much... I can't keep up... guys... *barf*"
      • 2009-06-15 16632, 2009

      • Muz_
        http://www.pubcrawler.uk.com/ aCiD2 they have that in London too you know. :p
      • 2009-06-15 16639, 2009

      • aCiD2
        neat!
      • 2009-06-15 16657, 2009

      • aCiD2
        ruaok: in MOD_ADD_ARTIST, DeniedAction does nothing and there's a comment saying "the cleanup script will handle this"
      • 2009-06-15 16601, 2009

      • aCiD2
        how does that work?
      • 2009-06-15 16634, 2009

      • ruaok
        its the daily RemoveEmptyArtists.pl script.
      • 2009-06-15 16646, 2009

      • aCiD2
        Ah, I thought it might be that
      • 2009-06-15 16652, 2009

      • ruaok
        but for completeness sake, lets do it right.
      • 2009-06-15 16612, 2009

      • aCiD2
        doesn't that mean that an artist could be created, have releases added, then the artist rejected, and failing?
      • 2009-06-15 16646, 2009

      • ruaok
        no
      • 2009-06-15 16648, 2009

      • nikki
        add artist is an autoedit though
      • 2009-06-15 16656, 2009

      • ruaok
        what nikki said.
      • 2009-06-15 16658, 2009

      • aCiD2
        But you still have to vote on it, no?
      • 2009-06-15 16612, 2009

      • ruaok
        but something is in the DB and the artist is no longer empty
      • 2009-06-15 16623, 2009

      • nikki
        not unless you're in a parallel universe where autoedits have to be voted on :P
      • 2009-06-15 16632, 2009

      • aCiD2
        brain really no worky today :)
      • 2009-06-15 16640, 2009

      • ruaok
        that universe would suck
      • 2009-06-15 16642, 2009

      • aCiD2
        I thought artists were created with a yellow background, my bad
      • 2009-06-15 16651, 2009

      • aCiD2
        ruaok: what is "doing it right" then?
      • 2009-06-15 16655, 2009

      • ruaok
        sleep young jedi. :)
      • 2009-06-15 16603, 2009

      • aCiD2
        sleep is for the weak!
      • 2009-06-15 16605, 2009

      • ruaok
        remove the artist when its empty as part of the DeniedAxtion.
      • 2009-06-15 16606, 2009

      • aCiD2
        or week, depending on your view
      • 2009-06-15 16612, 2009

      • ruaok
        lol
      • 2009-06-15 16626, 2009

      • aCiD2
        But if you can't vote on it, is deniedaction ever called?
      • 2009-06-15 16613, 2009

      • ruaok
        no.
      • 2009-06-15 16626, 2009

      • ruaok
        so much for that thought. :)
      • 2009-06-15 16629, 2009

      • aCiD2
        lol
      • 2009-06-15 16653, 2009

      • Muz_
        Hmm, that's not right. I don't remember DB imports being this quick and painless.
      • 2009-06-15 16614, 2009

      • ruaok
        clearly youre doing it rong
      • 2009-06-15 16627, 2009

      • Muz_
        Yeah, Musicbrainz is far from dev friendly.
      • 2009-06-15 16642, 2009

      • ruaok
        its sad that i can hear your poor pussy wail all the way across the atlantic
      • 2009-06-15 16611, 2009

      • Muz_
        I have a pussy?! Woah, brb.
      • 2009-06-15 16636, 2009

      • ruaok
        heh. we won't seem him for weeks
      • 2009-06-15 16651, 2009

      • Muz_
        :D
      • 2009-06-15 16642, 2009

      • chefkoch_AW
        chefkoch_AW is now known as chefkoch
      • 2009-06-15 16654, 2009

      • Muz_
        DROP TABLE puid; !
      • 2009-06-15 16607, 2009

      • aCiD2
        ruaok: so I think I have something luks will be happy with
      • 2009-06-15 16612, 2009

      • ruaok
        nice
      • 2009-06-15 16634, 2009

      • aCiD2
        one sec, pasting something
      • 2009-06-15 16618, 2009

      • aCiD2
        http://gist.github.com/130406 I'm happy because we have some constraints on the hash, but the data itself is very simple
      • 2009-06-15 16627, 2009

      • aCiD2
        does anything there scream "way too complicated"?
      • 2009-06-15 16614, 2009

      • aCiD2
        the to_hash/restore extensions are just because we need to save the created artist_id, but that's not part of the input data
      • 2009-06-15 16602, 2009

      • ruaok
        looks reasonable. but I am not really the harshest critic here. :)
      • 2009-06-15 16610, 2009

      • aCiD2
        I know :)
      • 2009-06-15 16617, 2009

      • aCiD2
        I think he should be happy with that...
      • 2009-06-15 16639, 2009

      • ruaok hopes so for the sake of forward progress
      • 2009-06-15 16643, 2009

      • aCiD2
        famous last words
      • 2009-06-15 16608, 2009

      • ruaok notes that aCiD2 has a pm pending
      • 2009-06-15 16646, 2009

      • chefkoch has quit
      • 2009-06-15 16659, 2009

      • chefkoch joined the channel
      • 2009-06-15 16624, 2009

      • ruaok
        aCiD2: check ur mail before you go to bed plz
      • 2009-06-15 16626, 2009

      • ruaok
        ttfn
      • 2009-06-15 16627, 2009

      • ruaok has quit
      • 2009-06-15 16638, 2009

      • ruaok joined the channel
      • 2009-06-15 16651, 2009

      • aCiD2
        doh, just missed him
      • 2009-06-15 16653, 2009

      • aCiD2
        oh wait, you back?
      • 2009-06-15 16620, 2009

      • aCiD2
        ruaok: $self->c is probably what you're looking for
      • 2009-06-15 16643, 2009

      • ruaok
        $self is an empty blessed reference.
      • 2009-06-15 16644, 2009

      • ruaok
        no c
      • 2009-06-15 16657, 2009

      • aCiD2
        say wha?
      • 2009-06-15 16602, 2009

      • aCiD2
        you didn't include Data::WikiDoc in your patch
      • 2009-06-15 16621, 2009

      • ruaok
      • 2009-06-15 16621, 2009

      • aCiD2
        and where's the code causing a problem?
      • 2009-06-15 16636, 2009

      • aCiD2
        like' what calls fetch_page?
      • 2009-06-15 16643, 2009

      • aCiD2
        oh
      • 2009-06-15 16656, 2009

      • aCiD2
        you need to do : has 'c' => ( isa => 'Object', is => 'rw' );