#musicbrainz

/

      • czaanja joined the channel
      • aCiD2
        brianfreud: what is id_edit_note used for?
      • brianfreud
        whereabouts?
      • chovy
        i swear there was documentation on TrackPosition somewhere -- can't find it now.
      • navap
        TrackPosition?
      • Tykling has left the channel
      • chovy
        what number a track is on the cd.
      • aCiD2
        brianfreud: in root/forms/edit-note.tt
      • rjmunro has quit
      • brianfreud
        Just found it myself. Honestly, I don't know. Did I add it? I normally use hyphens, not underscores?
      • aCiD2
        it's in a commit with the log "Fix for without matching id target"
      • which I can't quite decipher:)
      • brianfreud
        Oh!
      • I recall now....
      • tell me what is wrong with this:
      • <label for="id_[% field.name %]" class="[% field.required_text %]">[% l('Edit Note:') %]</label>
      • aCiD2
        the id should be on the <textarea> not the <label> itself though
      • brianfreud
        though I don't think my span addition fixed it correctly either, looking again now
      • yes, but it doesn't have an id - that id should be there
      • aCiD2
        yea
      • chovy
        what does it have?
      • aCiD2
        i'll ignore those changes then and fix up the textarea
      • brianfreud
        no id :P
      • chovy
        <label>Text:<textarea></textarea></label> if you want to make it easy
      • brianfreud
        chovy: you can't do a label for="foo" without a matching id="foo"
      • aCiD2
        <label for="foo">text:</label><textarea id="foo"></textarea> if you want to do it right :)
      • brianfreud
        not in valid XHTML
      • chovy
        yes
      • or just wrap in <label>...</label>
      • brianfreud
        but yeah, acid2, you know what the right fix is :)
      • aCiD2
        shouldn't even be doing <label> should all be going through forms/label.tt
      • aCiD2 slaps own hand
      • brianfreud
        prob unconverted from when you changed the forms code the first time
      • aCiD2
        yea
      • brianfreud
        oh, btw... not now, but some time
      • would you mind updating ServerCodingStyle to explain how form.field("foo") works?
      • aCiD2
        yea, sure
      • brianfreud
        right now it's minor voodoo figuring out what should be in the ()
      • RifRaf joined the channel
      • thx
      • aCiD2
        heh
      • Form::Processor still makes me a bit icky inside, but no time to change that now
      • just requires a few too many hacks
      • but it does the job
      • brianfreud
        lol, yeah, I spent maybe 5 hours trying to figure it out, to add the .foo to artist, then gave up trying to figure it out :D
      • aCiD2
        what dimensions should I use for textarea.edit-note?
      • atm I've got 400pt by 300pt
      • brianfreud looks
      • 300pt high is way too big lol
      • brianfreud
        why are you defining a textarea with pts?
      • aCiD2
        why not?
      • brianfreud
        rows / columns
      • aCiD2
        cause it's presentational
      • like why you prefer width css over the size element for input
      • brianfreud
        ah, hadn't realized rows/columns didn't make it into css for those
      • aCiD2
        Went for 400pt by 70pt
      • brianfreud
        why not use em instead?
      • then it will scale better with differing font sizes
      • aCiD2
        pt scales too
      • oh, maybe not
      • good point
      • brianfreud
        just looking here with firebugmaybe height 11em, width 60em?
      • aCiD2
        does it fit in 1024x768?
      • brianfreud
        explicit <label> definitions also in user/subscriptions.tt , moderation/vote_options.tt , forms/label.tt
      • aCiD2
        hrm, not here, I'll use 45
      • cool, i'll make a note of them for now
      • I'm only dealing with this cause it's one of the files your branch touched :)
      • brianfreud
        yeah, a little too wide in 1024
      • just happened to be something I noticed along the way :P
      • that's what 99% of the master minor stuff is
      • aCiD2
        heh
      • svn time!
      • brianfreud
        wha exactly has's been merged in so far?
      • aCiD2
        you'll see, just watch mb-commits
      • brianfreud
        no, I mean, which checkouts?
      • aCiD2
        updated master on git too
      • oh, I only merged TableSorting
      • s/checkouts/branches/
      • brianfreud
        ok
      • aCiD2
        jquery_updates doesn't seem to do anything, so I guess that's in
      • oh yea, that's in
      • czaanja has quit
      • lock/unlock is in too
      • RifRaf tries to get some supplies from town before gettin flooded in again
      • bad floods?
      • alastairp
        RifRaf: :/
      • Muz
        Floods? :/
      • RifRaf
        guess so, but normal for this time of year
      • many people stuck at home around here
      • aCiD2
        where's home?
      • alastairp
        RifRaf: at least no fire :(
      • RifRaf
        alastairp yep, floods don't mean i get damage cause on a hill, just no goin to town
      • aCiD2 NSW australia, on the east coast, coffs harbour closest googleable town
      • aCiD2
        ah
      • RifRaf
        can watch the river from here though, is up about 6 meters, so better go get food if bridges still visable
      • aCiD2
        6m!?
      • that's mad
      • RifRaf
        yeah, in the hills
      • yep, from a normal stream is now a gushing torrent about 40 meters wide
      • aCiD2
        brianfreud: only needed to merge 6 lines from master, heh
      • brianfreud
        lol
      • aCiD2
        I like your text::wikiformat stuff... but that's not the right approach I'm afraid :(
      • you've done it like annotations, which are also not doing it correctly
      • we should be doing [% user.biography | wiki_format %]
      • brianfreud
        no? It's cribbed right from annotations :P
      • aCiD2
        yea, which are saddly a bad example
      • aCiD2 adds a task to himself
      • chovy
        heavy rain here
      • brianfreud
        well, acid2, when you do it, perhaps you can also add it to edit notes?
      • aCiD2
        sounds reasonable :)
      • aCiD2 updates note
      • well, I gotta grab some sleep now, sorry the add release stuff is not quite there, but I can't see why it wouldn't be finished before the meeting tomorrow
      • (even though you gotta work)
      • brianfreud
        np, I ought to be home around 8 pm EST - if anyone needs me to address anything before the mtg, just pm or email me
      • aCiD2
        nn!
      • brianfreud
        natta :)
      • bignoise joined the channel
      • chovy_ joined the channel
      • chovy has quit
      • Muz wonders if anyone has ever ran kcachegrind against MB.
      • bignoise
        i have a new version of a release that just adds tracks to an existing one. is there a way to do that through the site ui without re-entering all the tracks?
      • alastairp
        does that work on perl?
      • "There also exist converters for profiling output of Python, PHP and PERL. "
      • ignore me
      • Muz
        bignoise: not on the site itself.
      • I soo should greasemonkey up something to resolve that.
      • bignoise
        so just through direct db manipulation? or other tools?
      • Muz
        EVer used the track parser on the site?
      • bignoise
        Muz: no, but i'll go have a look
      • Muz
        http://muz.goatse.co.uk/musicbrainz/parsertext.php You can feed the MBId of the album you wan tto copy from into that, and paste the output into the track parser
      • This will auto populate most of the fields, and the new tracks you just fill in as per normal
      • navap
        Muz: Oh, btw your MBID text parser thingamalig came in *very* hand today, thanks a lot :)
      • bignoise
        sounds about right, thx
      • navap
        thingamajig*..or however its spelt
      • Muz
        Okay, go to add a release as per usual and towards the top there should be some sort of box with "Configure" in the bottom right corner of it. Click that and enable the Text Parser module. Then paste the text that that page into it and hit parse.
      • navap: yeah, I really should finish putting the same functionality into a GM script.
      • Oh wow, I had so forgotten that I'd written http://muz.goatse.co.uk/musicbrainz/artisttrack...
      • lolcocks joined the channel
      • ?lookup artist Cascada
      • lolcocks
      • navap
        hmm that looks very handy for making sure all the tracks have the proper feat. and stuff
      • Muz
        In/deed/.
      • Bear in mind that it hits up my replicated DB, so it's up to an hour of date with what's on the live site.
      • I fetch and process replication packets at 10 past each hour.
      • navap
        Thats fine, its still easier to manually look at the live track name, and have an artists complete listing, than it would be to makually search all that stuff up on the live server.
      • I can never stay focused on what I'm doing in mb lol, I'm about 3/4 through J K Rowling, but tomorrow I'm going to totally drop her and go looking for track names to fix :p
      • Muz
        Heh
      • I really wish I had more time to code moar and do stuff that's actually useful.
      • Did thee gravatar support ever go into the live TT branch?
      • navap
        You mean besides just sitting around, playing with all your new bought toys :p