Sunday, 2009-02-01

*** tpb has joined #melange00:00
*** madrazr has left #melange01:05
*** dmitrig01|afk has joined #melange01:21
*** ChanServ sets mode: +v dmitrig01|afk01:21
*** madrazr has joined #melange01:56
*** penyaskito has quit IRC02:00
*** dmitrig01|afk has quit IRC02:17
*** madrazr has left #melange03:02
*** SRabbelier has joined #melange05:13
*** ChanServ sets mode: +v SRabbelier05:13
*** solydzajs has joined #melange05:45
*** ChanServ sets mode: +o solydzajs05:45
*** penyaskito has joined #melange07:32
*** solydzajs_ has joined #melange07:52
*** solydzajs has quit IRC08:09
*** solydzajs_ has quit IRC08:19
*** madrazr has joined #melange08:33
*** solydzajs has joined #melange09:12
*** ChanServ sets mode: +o solydzajs09:12
*** madrazr is now known as madhusudancs09:12
*** madhusudancs is now known as madrazr09:18
*** Lennie has joined #melange09:54
*** ChanServ sets mode: +v Lennie09:54
*** mwilkes|away is now known as MatthewWilkes10:01
madrazrLennie: was just waiting for you to tell that your code breaks too :P r174010:23
madrazrbut unfortunately its fixed now :D10:24
*** Merio has joined #melange10:55
solydzajsMerio: I just committed Google Maps integration, I had to change some things and fix your code11:13
Meriosolydzajs: what was wrong with it? Just for info for better coding :)11:21
solydzajsgo see the commit11:21
solydzajsthere was some not defined variable used11:21
solydzajsand some other stuff11:21
MerioOk, will go :)11:22
*** N4L|Lennie has joined #melange11:25
*** Lennie has quit IRC11:26
N4L|LennieI broke it madrazr ^^11:27
*** N4L|Lennie is now known as Lennie11:27
*** ChanServ sets mode: +v Lennie11:27
MerioLennie: hi :)11:33
Lenniehiya :)11:34
LennieMerio, were you doing the selection pop-in?11:46
Lennielist of users, you type a few letters11:46
MerioLennie: No, but I can join if helps11:47
LennieSRabbelier, do you have any idea what the status on that is?11:48
MerioLennie: seems there are many jquery plugins out there11:48
Lennie^_ ^11:48
LennieJQuery ftw11:48
LennieI'll be needing your ToS viewer elsewhere as well11:48
Lenniebut not untill something else has been ironed out :)11:49
MerioSeems to work smoothly in my box :)11:52
Lennieo yeah, the ironing out is not on your part :11:53
MerioJust commenting the "JQuery ftw" ;)11:55
MerioI was used to DojoToolkit... everything is tightly integrated there, but a bit to tricky to compress only-what-you-need features :)11:55
Merio*too tricky11:55
Lenniehi SRabbelier12:24
Lenniecould you answer my question about the picker :D?12:25
* SRabbelier reads backlog12:25
SRabbelierLennie: we're still waiting for no_mind :-/12:27
Lennieaha ok12:27
LennieMerio, is also glad to help if needed12:27
SRabbelierMerio: well, if you can get a hold of no_mind... :P12:27
MerioSRabbeler: what do you mean? ;)12:34
SRabbelierMerio: no_mind has been working on it12:35
SRabbelierMerio: if you can get in contact with him feel free ot help him, or take over12:35
MerioSRabbelier: did he post in the dev list?12:36
SRabbelierMerio: no, otherwise you wouldn't have to get in contact with him :P12:36
SRabbelierMerio: it seems he's "gone dark" :(12:36
MerioSRabbelier: so just search in Google "no_mind"? :D12:36
SRabbelierMerio: hehe, good luck :)12:37
MerioSRabbelier: 2950 results, I can visit all of them to try to find emails and then spam 'em all :D12:39
SRabbelierMerio: lol12:40
SRabbelierMerio: he's an indian guy12:40
SRabbelierMerio: maybe that'll narrow it down ;P12:40
Lennieor Merio can do it, might just be a simple JQuery thingy :D12:53
Lenniewe need it bad right :p?12:53
Lennieanyhow dinner :D12:53
*** Lennie is now known as Lennie|Food12:53
SRabbelierMerio: ah, that's right12:53
SRabbelierMerio: is it a lot of work?12:53
MerioSRabbelier: I've seen there are some autocompletion JQuery plugins out there, I should need to search and select the most practical for what we have already, but it will be only an integration issue: but we probably can give the list (plain text/XML/JSON, don't know) and the plugin will do the autocompletion for us automatically12:56
SRabbelierMerio: have a look at http://localhost:8080/document/pick?continue=/foo/bar&field=foo_bar12:57
SRabbelierMerio: it returns a JSON object already :)12:57
MerioSRabbelier: just a minute, need to start the application after some emails I'm writing :)13:00
SRabbelierMerio: np, no hurries13:00
MerioSRabbelier: great, I see13:21
SRabbelierMerio: I did that so no_mind would have somethign to work with; I expected him to be done in a day or two though :-/13:22
MatthewWilkeshmm, I'm having some issues with url patterns resolving back into objects13:27
MerioSRabbelier: Well, I can go for it if you think it's better not to wait anymore13:27
SRabbelierMerio: how long will it take you? :)13:28
SRabbelierMatthewWilkes: do tell13:28
MatthewWilkesI changed my key fields to be the key name of a reference and the link_id of another, but that caused /s in the keyname which doesn't fit the regex of the urlconf13:28
MatthewWilkesso I changed it to use the __str__ of the key instead of its name, which looks like it should work, but I get 404s raised13:29
MerioDon't know, if I can easily find a suitable plugin and if it works smoothly is a matter of minutes/an hour13:29
MatthewWilkesso I'm thinking maybe I need to teach it how to resolve that pair?13:29
SRabbelierMerio: go for it then :)13:29
MerioSRabbelier: ^^^^^^^13:29
MerioSRabbelier: ok :)13:29
SRabbelierMatthewWilkes: no, you need to instead change the urls to match your new key name ;)13:30
SRabbelierMatthewWilkes: you should instead of making it the key name of the reference, make that reference your scope13:31
SRabbelierMatthewWilkes: if you do that, the system will adjust the urls to match13:31
MatthewWilkesSRabbelier: I don't follow, change the url pattern to allow for / seperated tokens in the first space?13:32
SRabbelierMatthewWilkes: that way you could even leave the key fields as they are are by default from Linkable13:32
SRabbelierMatthewWilkes: all you have to do is set scope_logic to the appropriate logic, and you're done13:32
SRabbelierMatthewWilkes: that's probably work_logic, right?13:33
MatthewWilkesSRabbelier: Right.. I'm not using Linkable at the moment, it seemed like overkill and badly suited to the problem13:34
MatthewWilkes(the link_ids etc)13:34
SRabbelierMatthewWilkes: it's not overkill :)13:34
MerioSRabbelier: this ==> should be suitable13:34
tpb<> (at
SRabbelierMatthewWilkes: and actually pretty well suited to the problem13:35
SRabbelierMerio: cool :)13:35
SRabbelierMatthewWilkes: linkable is for anything that has it's own ID13:35
SRabbelierMatthewWilkes: (the comments do, they should get their id the same way notifications do)13:35
MatthewWilkesSRabbelier: Right, ok13:35
SRabbelierMatthewWilkes: and that has a scope, and the commetns have a scope, the work that they are made to13:35
* MatthewWilkes tries to think as a django developer13:35
SRabbelierMerio: that looks very nice13:35
SRabbelierMatthewWilkes: it's our own system though, it's not from Django13:36
MatthewWilkesSRabbelier: Ok, that makes sense, I'm assuming I can take care of the link_id, not force the user to add it?13:36
SRabbelierMatthewWilkes: yes, please do13:36
SRabbelierMatthewWilkes: have a look at how notifications do it13:36
MerioSRabbelier: I'm going to integrate it only in the document template and posting in the mailing list, which file should I tweak?13:39
SRabbelierMerio: probably just modify edit.html :)13:39
SRabbelierMerio: erm, no wait13:39
SRabbelierMerio: the various _as_table helpers13:39
SRabbelierMerio: in soc/templates/templatetags13:40
MerioSRabbelier: all 8 files?13:41
SRabbelierMerio: you probably only need to edit the _as_table_row one13:42
SRabbelierMerio: but feel free to poke around in them and see what happens if you change anything13:42
SRabbelierMerio: the _as_twoline... ones are only used in club/org applications at the moment, so you can probably leave them alone13:42
SRabbelierMerio: (as they _should_ just change when the regular version does)13:42
*** Lennie|Food is now known as Lennie13:51
MerioSRabbelier: is there a way in which I can easily access the id of each field from the context?13:57
SRabbelierMerio: what template?13:57
MerioI'm actually trying _as_table_row.html13:58
Merio soc/templatetags as you said13:58
SRabbelierMerio: it should be in the context of that template actually13:59
SRabbelierMerio: everything available in _as_table you can reach13:59
MerioSRabbelier: otherwise I can cycle all the inputs13:59
SRabbelierMerio: {% for field, required, example_text, reference in fields %} <----13:59
SRabbelierMerio: you have access to that13:59
MerioSRabbelier: ok :)14:00
SRabbelierMerio: or do you just need to know the id of the current field?14:00
MerioSRabbelier: actually we need to extend with a new function every input with the proper data14:00
SRabbelierMerio: sorry, I don't understand?14:01
MerioSRabbelier: if you look at the end of this page ==>
tpbTitle: Plugins/Autocomplete - jQuery JavaScript Library (at
LennieI cant get Thickbox to work :S14:02
MerioSRabbelier: I need to access the input field (by ID or by class) to insert the proper data in.14:02
MerioLennie: what happens? Which browser?14:02
SRabbelierMerio: but that's only the field for which you are doing the autocomplete right?14:03
SRabbelierMerio: cos that is available under the 'field' variable14:03
SRabbelierMerio: so you can do {{ }}14:03
MerioSRabbelier: yes we have to pass data only to fields that actually needs it14:03
LennieI'll pm you14:03
SRabbelierMerio: you should be able to use the django {{ }} then14:04
MerioSRabbelier: thanks, let me try something14:06
*** matthewwilkes_ has joined #melange14:06
*** MatthewWilkes has quit IRC14:08
Lennieabout the ToS14:10
Lennieit should just show up on the profile page14:10
Lenniein a scrollable box right14:10
Lenniewithout the user clicking on something14:10
tpb<> (at
Lenniesince that is what Leslie wanted14:10
tpb<> (at
Lenniecomment 714:13
MerioLennie: pong14:14
Lenniethe thickbox doesn't auto-popup right?14:14
solydzajswell I think it can show in click14:15
solydzajsbut not by going to different page14:15
solydzajsyou need to stay on the same page14:15
Lenniesolydzajs, it needs to be shown before you are able to accept it14:16
MerioNo, I show the ToS when someone clicks on the link14:16
MerioAs said to Lennie, if it were to be shown inline we could just enclose it in an overflow div14:17
Merio(with a fixed height/width)14:17
MerioHowever we need the thickbox.js as well to do the autocompletion thing, because it depends on it14:17
Meriogoing to dinner, see you in a few minutes :)14:18
LennieMerio, if you could tell me what to change in edit.html to do that overflow thingy :D14:21
Lenniethat would be great14:21
*** matthewwilkes_ is now known as MatthewWiulkes14:22
SRabbelierLennie: looks like you might have to buy a new pc if you want to test IE8 for us :P14:23
tpb<> (at
MatthewWiulkes... that's not my name14:23
*** MatthewWiulkes is now known as MatthewWilkes14:23
LennieI have 4 GiG SRabbelier :P14:24
Lennieshould be fine :P14:24
SRabbelierLennie: article says you need at least 16 :D14:24
Lenniedoes it?14:25
MerioLennie: <div id="ToS" style="overflow: auto" height="200px" width="300px">{{ tos_text_whatever }} </div>14:41
LennieI'll add that soon14:41
LennieI have broken something in that view ffs :P14:41
Meriocan change obviously the id, the height and width as you want14:41
Lennie^_ ^14:42
SRabbelierLennie: I think I'm done refactoring14:44
SRabbelierLennie: but I haven't tested :P14:44
SRabbelierLennie: should I just commit? (A)14:44
SRabbelierplease? :(14:45
LennieI see a cyclic import here :D14:45
SRabbelierLennie: wooh14:45
LennieI broke stuff because you break stuff14:45
Lennieand now you want to break stuff because I did it14:45
LennieYou'd rather not import my fail routines :P14:45
SRabbelierLennie: ofc. I do14:45
solydzajsi'm back14:48
madrazrOk now its good time to bug you guys again, I can talk to all the 3 together, Lennie, SRabbelier and solydzajs :P14:50
* SRabbelier runs away14:50
* Lennie is off14:50
solydzajshehe :-)14:50
solydzajsmadrazr: what's up ?14:50
madrazrha ha ha :D14:50
madrazrthe problem I mentioned other day is not yet fixed, I take the blame on myself14:50
SRabbeliermadrazr: hehe, what problem is it again? ^__^14:50
madrazrthere is no message whenever some new entity is created14:51
madrazrSRabbelier: I tried prefix='create' thingie you suggested the other day14:51
madrazrbut seems like it doesn't work or I am getting it completely wrong14:51
SRabbeliermadrazr: ah, right14:51
madrazrhow to go about it? can you tell it briefly again? when a new entity is created, the calls go from create->edit->editGet IIRC14:55
SRabbeliermadrazr: I think easiest fix14:55
madrazrSRabbelier: whats that?14:55
SRabbeliermadrazr: is to change the isSelfRefferrer14:55
madrazrSRabbelier: in what way?14:56
SRabbeliermadrazr: to make it just check if the value from os.environ['HTTP_HOST'] matches the first part of the refferrer14:56
madrazrSRabbelier: looking at it14:58
SRabbelierdef isReferrerSelf(request):14:58
SRabbelierhttp_from = request.META.get('HTTP_REFERER')14:58
SRabbelierhttp_host = os.environ['HTTP_HOST']14:58
SRabbelierreturn http_from.startswith(http_host)14:58
SRabbelierLennie: ok, did some basic testing, going to commit now :P15:00
SRabbelierLennie, solydzajs: I can't test it thoroughly enough all by myself, so please help out15:00
Lennieyou are soooo gonna break stuff :p15:01
SRabbelierLennie: I know15:01
SRabbelierLennie: that's why I'm telling you to test ^__^15:01
LennieI hate HTML :p15:06
Lennieit's a hidden field15:06
Lenniebut it still shows it :P15:06
Lenniebecause I call {% field_as_table_row form.agreed_tos_on %}15:07
Lenniebecause I have to15:07
Lenniebecause user_profile had to be so explainatory15:07
SRabbelierthen split them15:07
SRabbelierretard :P15:07
Lenniesplit what :D?15:07
SRabbelierhave the explanation and the field in two different things15:07
SRabbelierand only display the explanation if you need to display the field15:07
Lenniethey are two different things15:07
LennieI need this field to be hidden15:08
SRabbelierthen what's the problem? ^_^15:08
Lenniebecause it needs to be submitted15:08
Lenniethe field wont hide15:08
Lenniethe label still shows up :P15:08
SRabbelierLennie: lol15:08
SRabbelierLennie: you can't have hidden fields behave like hidden fields if you're not using as_talbe15:08
SRabbelierLennie: you'll have to manually do15:08
SRabbelier{% if I need to display field %}15:08
SRabbelier{% field_as_table_row form.agreed_tos_on %}15:08
SRabbelier{% endif %}15:09
Lennieand else I have to force it to be hidden?15:09
SRabbelieryou can't?15:09
Lenniethen this will not work :S15:09
SRabbelieryes it will15:09
SRabbelierby not displaying it if the need to display is false :P15:09
SRabbelieryou just don't render it at all15:09
SRabbelierthe ultimate form of hidden-ness!15:09
Lennieit needs to be in there15:10
Lennieelse the check fails remember?15:10
SRabbelierthen solve it yourself :P15:10
SRabbelieryou're just telling me "no this won't work"15:10
Lenniebecuase it wont :D15:10
SRabbelierhow am I supposed to do anything with that :P15:10
SRabbelierI'll "throw you a bone" a few times, but not >2 :P15:10
Lennieand as I told you it needs to be submitted by the submit button else the check will fail15:11
SRabbelierthen change that :)15:11
LennieI could make the check more uglier lol15:11
SRabbelieror add a new helper15:11
SRabbelierinstead of {% field_as_table_row form.agreed_tos_on %}15:12
SRabbelierhave {% hidden_field_as_table_row form.agreed_tos_on %}15:12
*** Lennie is now known as Lennie|Gone15:17
SRabbelierI'm going to kill eric wong :P15:18
SRabbelierI dunno wtf he did, but git-svn is slow as *** today!15:18
MatthewWilkesSRabbelier: Where are notifications created from?  Could do with some context15:18
SRabbelierLennie|Gone: care to explain?15:18
SRabbelierMatthewWilkes: hehe, guess not15:21
SRabbelierMatthewWilkes: ok, Notifications are created when something happens the user needs to take action upon15:22
SRabbelierMatthewWilkes: for example, when a request is made15:22
MatthewWilkesSRabbelier: Right, so is it done by code or TTW?15:23
SRabbelierMatthewWilkes: what's TTW? but yes, the code does it15:23
MatthewWilkesthrough the web15:23
MatthewWilkesSRabbelier: Ok, so for now the way I was passing the scope around with the encoded key, but that feels like the wrong way of doing it15:25
MatthewWilkesWas trying to understand the notifications example better15:25
SRabbelierMatthewWilkes: yes, quite so, why do you need to encode it? :-/15:25
MatthewWilkesI'm not sure how the create form gets the scope object, if I have access to the key it's easy15:26
SRabbelierMatthewWilkes: well, the idea is that the comment is made on the same page as the Work's view It hink?15:27
SRabbelierMatthewWilkes: so if you are on teh Work's view page, you always have access to it15:27
MatthewWilkesyeah, but it needs to go via the view class of the comment still, doesn't it?15:28
SRabbelierMatthewWilkes: uhm, I thought the comments of a Work would be displayed on the same page?15:29
MatthewWilkesSRabbelier: Yeah, they would15:31
MatthewWilkesThis is part of the reason I though Linkable was overkill15:31
MatthewWilkesyou don't care about a lot of the pages you get with most types at the moment15:32
MatthewWilkesstill, you'd want a view class either way, where else would you put the code?15:32
SRabbelierMatthewWilkes: yeah sure15:32
SRabbelierMatthewWilkes: you could give it as an argumetn to the create page15:33
SRabbelierMatthewWilkes: that's how we do it with most scoped entities15:33
SRabbelierMatthewWilkes: like when you create a new program, you have to choose a Host first15:33
SRabbelierMatthewWilkes: and when you then select 'google', you are directed to /program/create/google'15:34
MerioSRabbelier: the autocompletion is working in my box, need to fix it a bit as it seems I can't access in _as_table_row.html, probably I'm missing something. The script however is working if hardcoded15:34
MerioNeed to go away now, will work on it tomorrow and let you know :)15:34
MatthewWilkesan argument to the create page?15:34
MatthewWilkesSRabbelier: So, where is it that the "google" there gets translated into the google entity?15:36
SRabbelierMatthewWilkes: it uses the scope_logic, does something like scope_logic.getForKeyName(scope_path)15:37
SRabbelierMatthewWilkes: (as the scope_path will be equal to the key name of the scope entity)15:37
*** madrazr has quit IRC15:50
MatthewWilkesSRabbelier: Right! So, program has a create URLconf entry which takes a scope_path field, that makes that value available to the view when the program logic is asked to extract the parameters.  Once I have that I can call the logic of the scope object (as I know the type it will be) to get the object itself15:53
SRabbelierMatthewWilkes: there's already such a field :)15:53
SRabbelierMatthewWilkes: there is/should be a create that has %(scope)s as it's suffix15:54
SRabbelierfrom   if not params.get('no_create_with_scope'):15:54
SRabbelier    new_params['django_patterns_defaults'] += [15:54
SRabbelier        (r'^%(url_name)s/(?P<access_type>create)/%(scope)s$',15:54
SRabbelier        'soc.views.models.%(module_name)s.create', 'Create %(name_short)s')]15:54
MatthewWilkesyeah, I'm thinking out loud, I saw that (well, actually I caused a 500 in urlconf and read the errors)15:55
SRabbelierMatthewWilkes: that's how I do it usually ^__^15:56
MatthewWilkesso, if I went to /comment/create/user/matthewwilkes/foo/bar (or whatever) I should get params['scope'] which contains enough information for me to get the bar document from work.logic16:00
SRabbelierMatthewWilkes: what's the '/user' thing for?16:00
SRabbelierMatthewWilkes: it might actually make sense to do the same as what I did for document16:01
SRabbelierMatthewWilkes: disable the %(scope) link and add one that has all the fields but link_id16:01
SRabbelierMatthewWilkes: so that you can take the user as params['user']16:02
MatthewWilkeswell, you don't know what the full chain of the scope will be, in 'user' there it's because it's a user document made by me16:02
SRabbelierMatthewWilkes: oooh, that's the prefix of the document? :)16:03
SRabbelierMatthewWilkes: sorry, my bad :)16:03
SRabbelierright :)16:03
SRabbelierMatthewWilkes: sorry, I wasn't thinking for a moment :)16:04
MatthewWilkesSorry, my lappy is being slow, hangs every now and again when I hit enter16:04
SRabbelierMatthewWilkes: the key name of the document would be "user/matthewwilkes/foo"16:04
SRabbelierMatthewWilkes: so if you have "/comment/create/user/matthewwilkes/foo", you can defnitely do document_logic.getForKeyName(scope_path)16:04
SRabbelierMatthewWilkes: btw, not params['scope'], since params is a pre-defined dictionary16:05
SRabbelierMatthewWilkes: you should write your method to accept **kwargs16:06
SRabbelierMatthewWilkes: then **kwargs will be the keywords you get form django (the ones matched in teh regexp)16:06
SRabbelierMatthewWilkes: so you would do getForKeyName(kwargs['scope_path'])16:06
SRabbelierMatthewWilkes: not sure if you knew that already16:06
MatthewWilkesSRabbelier: by method, do you mean the View class, or a new method hooked up with some urlconf magic?16:07
SRabbelierMatthewWilkes: the method hooked up, which will be a method of your View class16:08
SRabbelierMatthewWilkes: e.g., like base.View.edit16:08
SRabbelierin soc.views.models.base16:08
MatthewWilkesoh, awesome, I can just override that16:09
SRabbelierMatthewWilkes: yup, you can; but do you need to?16:09
SRabbelierMatthewWilkes: if you set up the comment.Logic class correctly you might not even have to16:09
MatthewWilkesSRabbelier: right, ok, I'm going to have a go, thanks for your help, it's much appreciated16:10
MatthewWilkesI'll likely be back soon!16:10
SRabbelierMatthewWilkes: ok awesome!16:10
SRabbelierMatthewWilkes: dont' hesitate to ask :)16:10
*** Lennie|Gone is now known as Lennie16:45
*** dmitrig01|afk has joined #melange16:45
*** ChanServ sets mode: +v dmitrig01|afk16:45
*** dmitrig01|afk has quit IRC17:13
Lennie<div id="ToS" style="overflow:auto" height="200px" width="400px">17:41
SRabbelierLennie: OMG17:41
SRabbelierLennie: wtf have you been?17:41
SRabbelierLennie: hahha, you lazy b-tard!17:41
LennieI have been back an hour17:41
SRabbelierLennie: heheh, ok :P17:41
LennieI'm going to bed soon17:41
Lenniemy I have too much time for Melange period is over :P17:42
SRabbelierLennie: I fixed the list issue :)17:42
LennieYOU RUL!17:42
Lennieonce more you save Melange :P17:42
LennieMerio, could you please create a patch for me that makes the scrolling happen with a good height and widht :D?17:42
Lenniesince I can't get it to work here17:42
SRabbelierLennie: you better be damn grateful, I had to fix JS to make it work properly17:42
Lennieomg :P17:43
Lennieyou can cry your hearth out at M :p17:43
SRabbelierhehehe :P17:43
SRabbelierLennie: it was Todd's JS on top17:43
SRabbelierLennie: debugging someoen else's code is not nice!17:43
SRabbelierLennie: especially not if that code is in JS!!!17:44
SRabbeliermultiple exlamation marks!17:44
LennieSRabbelier: you should see my ugly temp fix patch :D17:44
Lennielambda function lol :P17:44
SRabbelierLennie: mhhh, maybe not :P17:44
LenniePawel is fixing me a template tag since I suck :D17:44
MatthewWilkesok, I'm going crazy, the url pattern for program looks very different to the one I have for comment and I can't see what's causing it17:45
Lennieprogram doesnt have work_flow anymore?17:45
SRabbelierMatthewWilkes: mhhh, probably difference in scope_logic :)17:45
MatthewWilkes# ^program/(?P<access_type>create)/(?P<scope_path>[a-z](?:[0-9a-z]|_[0-9a-z])*(?:/[a-z](?:[0-9a-z]|_[0-9a-z])*)*)$17:45
MatthewWilkes# ^comment/(?P<access_type>create)/(?P<scope_path>)$17:45
SRabbelierMatthewWilkes: yup, program has scope_logic set17:45
SRabbelierMatthewWilkes: ah, and obviously you do not :P17:45
SRabbelieroh wait17:46
SRabbelierMatthewWilkes: do you have it set to Work?17:46
SRabbelierMatthewWilkes: as Work does not have it set!17:46
SRabbelierMatthewWilkes: if that's the case, change yours to linkable, like Document does17:46
MatthewWilkesahh, it's a kwarg to logic!17:47
MatthewWilkesI'd been setting scope_view17:47
SRabbelierMatthewWilkes: you need that one too, for the scope selection to work17:47
SRabbelierMatthewWilkes: sorry for not having documented all this :(17:48
SRabbelierMatthewWilkes: you could be epic win and document what you've learned? (A)17:48
MatthewWilkesI'll give it a go, but once I actually know what I've learned17:49
SRabbelierMatthewWilkes: right :), that'd be awesome :)17:49
Lennieand he's calling me lazy :D17:49
* MatthewWilkes knows the value of docs - am an editor for one of the sections of plone's docs17:50
MatthewWilkesI spend so much time bugging people to contribute ;)17:50
SRabbelierMatthewWilkes: hehehe, cool! :D17:50
MatthewWilkesboom shack a lack!17:54
SRabbelierMatthewWilkes: does that mean it worked?17:55
SRabbelierMatthewWilkes: sweet!17:55
solydzajsMatthewWilkes: hey man, what's the status ?;-)18:02
MatthewWilkessolydzajs: I've been in a world of pain, but SRabbelier has been a great help.  I'm just hooking up the scope correctly (which is quite differently to how I was doing it yesterday!)18:04
*** Lennie has quit IRC18:04
MatthewWilkesI had been passing the str() of a key around and having no scope, instead of getting it via the logic18:04
solydzajsi'm happy it's solved :-)18:04
solydzajsso what is working and what is not working yet ?18:05
MatthewWilkesI've just broken commenting to get it working correctly, just need to wire up lookup to populate the reference field to get that working again18:06
MatthewWilkesAlso, got a link to add new comments on document pages18:06
MatthewWilkesand a comment counter and show first comment, not looping yet18:06
MatthewWilkesit's currently anyone can comment18:07
MatthewWilkesSorry it's taking so long, I massively overestimated how much I understood how the app was working!18:07
MatthewWilkesThought I'd worked it out about 5 times now18:07
SRabbelierMatthewWilkes: you're doing a nice job in catching up :)18:08
solydzajsMatthewWilkes: how did you implemented comments counter ? sharding counter ?18:08
solydzajsMatthewWilkes: yep thx for all your work man :-)18:08
MatthewWilkessolydzajs: No, it's just a simple db.Query.count. I appreciate we'd want to cache that on an object but I didn't want to go messing with schemata for an optimisation when you guys are trying to get things nailed down18:09
MatthewWilkesit's not like it's something that can't be brought in quite easily alter18:09
solydzajsok we will optimize it later no worries :-), when do you think you can be done ?18:10
solydzajsdone with it I mean :-)18:11
SRabbelierMatthewWilkes: we can use memcache to cache that, no worries, we're doign that already for the entire sidebar :)18:12
MatthewWilkesHmm, how can I turn a key name into an object without knowing anything more than the class is a subclass of Work?  The linkable_logic doesn't seem to pick it up18:13
SRabbelierMatthewWilkes: nope, that doesn't work18:13
SRabbelierMatthewWilkes: you'll need to use work_logic explicitly18:13
SRabbelierMatthewWilkes: if it's a work instance18:13
SRabbelierMatthewWilkes: sadly, GAE doesn't actually support inheretance18:13
SRabbelierMatthewWilkes: so you'll have to use the right subclass18:13
MatthewWilkessolydzajs: Honestly, I'd not like to estimate right now, I've felt on the threshold for a week or so now, I can send a patch for commenting on documents with a basic view ui by anyone tonight18:14
SRabbelierMatthewWilkes: please do send :)18:15
MatthewWilkessolydzajs: But the display UI wouldn't be done _correctly_, it'd be a cludge, and I don't know what I don't know about the access system yet18:15
MatthewWilkesSRabbelier: *sigh* ;)18:15
SRabbelierMatthewWilkes: tell me about it :P18:16
SRabbelierI should go to bed guys18:21
SRabbelierhave to get up early tomorow :)18:21
solydzajsI will go to bed soon too18:24
SRabbelierok man :)18:24
MatthewWilkesnight SRabbelier!18:24
SRabbeliersolydzajs: do we have conference call tomorrow?18:24
MatthewWilkesThanks for the help!18:24
SRabbelierMatthewWilkes: you're welcome, good luck!18:24
solydzajsSRabbelier: yes18:24
solydzajsSRabbelier: as scheduled18:25
SRabbeliersolydzajs: ok, how late, and what number? it's not on my calendar18:25
solydzajsSRabbelier: one sec18:25
solydzajsSRabbelier: you are listed as waiting for response18:26
SRabbeliersolydzajs: oh, weird18:26
solydzajs[email protected]18:26
SRabbelieri see now18:27
SRabbelierthe day changed :)18:27
SRabbelierso I was hitting 'next week' :P18:27
SRabbeliernvm, I see itn ow18:27
solydzajstill tomorrow then18:27
SRabbeliercheers :018:28
*** SRabbelier has quit IRC18:30
*** solydzajs has quit IRC18:30
*** mithro has quit IRC19:05
*** Merio has quit IRC19:16
MatthewWilkesahhh :)19:33
*** MatthewWilkes has quit IRC20:19
*** mithro has joined #melange20:46
*** ChanServ sets mode: +v mithro20:46
*** dmitrig01|afk has joined #melange21:13
*** ChanServ sets mode: +v dmitrig01|afk21:14
*** madrazr has joined #melange22:56
*** johndbritton has joined #melange22:59
*** madrazr has left #melange23:04
*** dmitrig01|afk is now known as dmitrig0123:26

Generated by 2.13.1 by Marius Gedminas - find it at!