#musicbrainz-devel

/

      • Freso joined the channel
      • 2015-01-25 02510, 2015

      • nikki
      • 2015-01-25 02516, 2015

      • ruaok joined the channel
      • 2015-01-25 02517, 2015

      • JesseW
        looking
      • 2015-01-25 02533, 2015

      • JesseW
        thanks
      • 2015-01-25 02524, 2015

      • nikki
        I noticed you say in #mb that it should be trivial, so we can look forward you implementing that being the first thing in may's schema change, right? ;)
      • 2015-01-25 02542, 2015

      • nikki can try and persuade people to do things :P
      • 2015-01-25 02544, 2015

      • JesseW
        by May? Hm, quite possibily.
      • 2015-01-25 02509, 2015

      • JesseW
        What would the deadline for getting that in by be?
      • 2015-01-25 02525, 2015

      • JesseW is glad for the encouragement.
      • 2015-01-25 02529, 2015

      • nikki
        the deadline for the schema change scripts is generally a month before
      • 2015-01-25 02544, 2015

      • JesseW
        ok, so by April Fool's Day.
      • 2015-01-25 02557, 2015

      • nikki
        (since we have to finalise what's included so we can tell customers what to expect)
      • 2015-01-25 02504, 2015

      • Freso
        JesseW: However, having it ready earlier is by no means a bad thing.
      • 2015-01-25 02508, 2015

      • JesseW
        :-)
      • 2015-01-25 02515, 2015

      • nikki
        the actual ui stuff, well, the earlier the better but I'm sure we've merged stuff the day before the schema change before :P
      • 2015-01-25 02519, 2015

      • nikki
        not that that's ever a good idea XD
      • 2015-01-25 02513, 2015

      • JesseW
        btw, do you think that the attribute editing patch is ready? As I mentioned on bitbucket, it seems fine to me.
      • 2015-01-25 02530, 2015

      • nikki
        as far as I know, yes
      • 2015-01-25 02538, 2015

      • JesseW
        yay!
      • 2015-01-25 02504, 2015

      • JesseW
        I'd love to have more of the descriptions populated (and displayed, but that's separate tickets)
      • 2015-01-25 02551, 2015

      • nikki
        well, populating the descriptions will happen once we have a way to populate them that doesn't involve manually writing sql and getting someone to run it :P
      • 2015-01-25 02520, 2015

      • JesseW
        nikki: yes, that's why I'm happy to hear that the attribute editing patch is ready. :-)
      • 2015-01-25 02539, 2015

      • JesseW
        as you had previously told me about that precondition
      • 2015-01-25 02549, 2015

      • nikki nods
      • 2015-01-25 02535, 2015

      • CatCat joined the channel
      • 2015-01-25 02552, 2015

      • xram joined the channel
      • 2015-01-25 02559, 2015

      • d4rkie joined the channel
      • 2015-01-25 02538, 2015

      • _dave_ joined the channel
      • 2015-01-25 02520, 2015

      • The_Freso joined the channel
      • 2015-01-25 02525, 2015

      • HSOWA joined the channel
      • 2015-01-25 02557, 2015

      • vissborg joined the channel
      • 2015-01-25 02514, 2015

      • kahu_ joined the channel
      • 2015-01-25 02519, 2015

      • nikki__ joined the channel
      • 2015-01-25 02532, 2015

      • xram joined the channel
      • 2015-01-25 02500, 2015

      • MightyJay_ joined the channel
      • 2015-01-25 02530, 2015

      • nikki__ joined the channel
      • 2015-01-25 02549, 2015

      • Jormangeud^ joined the channel
      • 2015-01-25 02552, 2015

      • kloeri_ joined the channel
      • 2015-01-25 02503, 2015

      • gizah_ joined the channel
      • 2015-01-25 02514, 2015

      • catgroove joined the channel
      • 2015-01-25 02540, 2015

      • leonardo_ joined the channel
      • 2015-01-25 02544, 2015

      • mueslix joined the channel
      • 2015-01-25 02552, 2015

      • mueslix joined the channel
      • 2015-01-25 02548, 2015

      • Nyanko-sensei joined the channel
      • 2015-01-25 02509, 2015

      • ocharles_ joined the channel
      • 2015-01-25 02512, 2015

      • ariscop joined the channel
      • 2015-01-25 02522, 2015

      • CatQuest joined the channel
      • 2015-01-25 02542, 2015

      • JesseW joined the channel
      • 2015-01-25 02506, 2015

      • CatQuest has left the channel
      • 2015-01-25 02543, 2015

      • CatCat joined the channel
      • 2015-01-25 02507, 2015

      • ocharles_ joined the channel
      • 2015-01-25 02533, 2015

      • navap joined the channel
      • 2015-01-25 02540, 2015

      • Gentlecat joined the channel
      • 2015-01-25 02545, 2015

      • navap1 joined the channel
      • 2015-01-25 02519, 2015

      • navap1 joined the channel
      • 2015-01-25 02546, 2015

      • Powlow joined the channel
      • 2015-01-25 02529, 2015

      • Powlow joined the channel
      • 2015-01-25 02505, 2015

      • Powlow joined the channel
      • 2015-01-25 02500, 2015

      • JesseW joined the channel
      • 2015-01-25 02538, 2015

      • Powlow joined the channel
      • 2015-01-25 02508, 2015

      • ariscop joined the channel
      • 2015-01-25 02527, 2015

      • JesseW
        How do I control the display of lines like: [debug] Path is "/collection/edit" when running the tests?
      • 2015-01-25 02514, 2015

      • ianmcorvidae
        JesseW: turn off CATALYST_DEBUG
      • 2015-01-25 02525, 2015

      • JesseW
        ah, makes sense, thanks
      • 2015-01-25 02526, 2015

      • JesseW
        There appears to be no tests for adding an Event to a Collection.
      • 2015-01-25 02533, 2015

      • JesseW
        I should probably submit some. :-)
      • 2015-01-25 02550, 2015

      • JesseW
        Does the WS support doing that?
      • 2015-01-25 02537, 2015

      • ianmcorvidae
        IIRC no, we deliberately didn't add that so we wouldn't screw it up
      • 2015-01-25 02508, 2015

      • JesseW
        didn't add support to the WS, or didn't add the tests? <big grin>
      • 2015-01-25 02522, 2015

      • nikki
        both? :P
      • 2015-01-25 02526, 2015

      • JesseW
        heh
      • 2015-01-25 02528, 2015

      • JesseW
        ok, well I'll add some tests to t::MusicBrainz::Server::Controller::WS::1::Collection
      • 2015-01-25 02538, 2015

      • JesseW
        er, sorry, not there
      • 2015-01-25 02501, 2015

      • JesseW
        to t::MusicBrainz::Server::Data::Collection
      • 2015-01-25 02504, 2015

      • JesseW
        that's what I meant
      • 2015-01-25 02531, 2015

      • ianmcorvidae
        obviously didn't add to the tests :P the WS part was the deliberate lack of addition though
      • 2015-01-25 02540, 2015

      • ianmcorvidae
        and yeah, data::collection tests would be good
      • 2015-01-25 02544, 2015

      • JesseW
        nods.
      • 2015-01-25 02506, 2015

      • JesseW
        I noticed the lack when I accidentally completely broke adding events, and the tests didn't notice.
      • 2015-01-25 02505, 2015

      • JesseW
        hm -- the database also appear to not care that two collections have the same gid...
      • 2015-01-25 02533, 2015

      • JesseW
        oh, no I misread it -- they differ in the last digit...
      • 2015-01-25 02531, 2015

      • JesseW
        it might be kind to future readers make them differ a *bit* more obviously than that (esp as the difference is d vs b)...
      • 2015-01-25 02513, 2015

      • yeeeargh joined the channel
      • 2015-01-25 02528, 2015

      • ariscop joined the channel
      • 2015-01-25 02525, 2015

      • Julior joined the channel
      • 2015-01-25 02515, 2015

      • Mineo joined the channel
      • 2015-01-25 02510, 2015

      • drsaunders joined the channel
      • 2015-01-25 02502, 2015

      • UmkaDK joined the channel
      • 2015-01-25 02522, 2015

      • kahu joined the channel
      • 2015-01-25 02508, 2015

      • axilla
        morning :)
      • 2015-01-25 02535, 2015

      • axilla
        does the JSON API have the same syntax and capabilities as the XML?
      • 2015-01-25 02547, 2015

      • axilla
        i'm not seeing the search syntax and what not in the docs
      • 2015-01-25 02551, 2015

      • axilla
        just simple URL samples
      • 2015-01-25 02527, 2015

      • axilla
        am i just adding fmt json?
      • 2015-01-25 02550, 2015

      • Freso
        The query syntax should be the same, yes.
      • 2015-01-25 02521, 2015

      • CatCat joined the channel
      • 2015-01-25 02520, 2015

      • axilla
        working this morning on knocking out a complete ruby wrapper for the json API
      • 2015-01-25 02513, 2015

      • axilla
        while currently spinning Ray Charles Greatest hits.. such a good sunday record.
      • 2015-01-25 02527, 2015

      • axilla
        just got done with carrol king - tapestry :)
      • 2015-01-25 02500, 2015

      • Lotheric joined the channel
      • 2015-01-25 02533, 2015

      • Julior joined the channel
      • 2015-01-25 02552, 2015

      • axilla
        https://github.com/travisdmathis/MusicBrainz-JSON got the guts up.. starting the coding now..
      • 2015-01-25 02558, 2015

      • axilla
        hope to have it done in a couple days.. shouldn't take me to long
      • 2015-01-25 02509, 2015

      • axilla
        there's not that many end points
      • 2015-01-25 02530, 2015

      • Nyanko-sensei joined the channel
      • 2015-01-25 02522, 2015

      • Freso
        I don't get why you need an abstraction for the JSON API though. But oh well.
      • 2015-01-25 02558, 2015

      • Gentlecat joined the channel
      • 2015-01-25 02511, 2015

      • Gentlecat
        Freso: you'll have to make some kind of abstraction for external services, otherwise it becomes a mess
      • 2015-01-25 02522, 2015

      • Gentlecat
        it's nice to have something ready to use
      • 2015-01-25 02529, 2015

      • axilla
        is there no way to search for an artists releases by format?
      • 2015-01-25 02536, 2015

      • Freso
        axilla: There is.
      • 2015-01-25 02538, 2015

      • axilla
        i only want to return a search for vinyl
      • 2015-01-25 02541, 2015

      • axilla
        i dont see it in the API docs
      • 2015-01-25 02558, 2015

      • Freso
      • 2015-01-25 02516, 2015

      • Freso
        "format"
      • 2015-01-25 02550, 2015

      • yeeeargh
        since i've struggeled with that exact problem one or two days ago: you need to search for the specific format. e.g. "12" Vinyl" instead of just "vinyl"
      • 2015-01-25 02533, 2015

      • yeeeargh
        mind that you have to escape the " after the 12 for lucene to use is correctly
      • 2015-01-25 02557, 2015

      • yeeeargh
      • 2015-01-25 02508, 2015

      • nikki notes that you can do format:(vinyl "12" vinyl") to combine multiple ones
      • 2015-01-25 02536, 2015

      • axilla
        yuck
      • 2015-01-25 02548, 2015

      • mjc_ joined the channel
      • 2015-01-25 02513, 2015

      • MightyJay joined the channel
      • 2015-01-25 02545, 2015

      • axilla
        i work with a lot of API's, and not gonna lie, this is one of the hardest most confusing i've ever tried to work with.
      • 2015-01-25 02550, 2015

      • axilla
        BRAINZ.release(:query => "Junta", :format => "Viny 12\" Vinyl")
      • 2015-01-25 02554, 2015

      • axilla
        the first option i get back is a CD
      • 2015-01-25 02555, 2015

      • axilla
        :(
      • 2015-01-25 02510, 2015

      • Freso
        axilla: What search query does that produce?
      • 2015-01-25 02521, 2015

      • Freso
        (Just paste the URL.)
      • 2015-01-25 02548, 2015

      • CatCat joined the channel
      • 2015-01-25 02512, 2015

      • Freso
        You might also want to make the :format take a list/array: :format => ("Vinyl 12\"", "Vinyl")
      • 2015-01-25 02547, 2015

      • Freso
        (There's no 'Vinyl 12" Vinyl' format.)
      • 2015-01-25 02525, 2015

      • axilla
        using this API makes me want to gouge out my eyes.. whats with the format of release: AND artist: AND ?
      • 2015-01-25 02543, 2015

      • axilla
        why not like every other API.. format=blah&release=blah&
      • 2015-01-25 02558, 2015

      • axilla
        spaces :(
      • 2015-01-25 02546, 2015

      • Mineo
        how would you do disjunction or field grouping if all you have is "&"?
      • 2015-01-25 02514, 2015

      • JesseW joined the channel
      • 2015-01-25 02544, 2015

      • axilla
        there are many ways..
      • 2015-01-25 02505, 2015

      • axilla
        you can pass a query string
      • 2015-01-25 02530, 2015

      • axilla
        something like field1=value&field2=value&field3=value
      • 2015-01-25 02510, 2015

      • axilla
        at work our RESTful API receives JSON so you can pass as much data as you like and then spits out JSON.
      • 2015-01-25 02514, 2015

      • Freso
        axilla: You can try out search queries on https://musicbrainz.org/search - the WS is using "Indexed search with advanced query syntax", so be sure to pick that.
      • 2015-01-25 02539, 2015

      • Freso
        That will probably be easier than sending WS requests while trying to get the hang of the query language.
      • 2015-01-25 02554, 2015

      • kloeri joined the channel
      • 2015-01-25 02530, 2015

      • nikki__ joined the channel
      • 2015-01-25 02553, 2015

      • samgtr joined the channel
      • 2015-01-25 02547, 2015

      • ocharles_ joined the channel
      • 2015-01-25 02518, 2015

      • samgtr joined the channel
      • 2015-01-25 02534, 2015

      • JesseW
        What's a recommended way to make UUIDs for test data?
      • 2015-01-25 02558, 2015

      • Mineo
        uuidgen?