2008-6-19
| 17:24 | telenieko | change the telenieko@ for whatever ;) |
| 17:24 | brosner | jacobkm: i actually dont know. i remember coming across it and it turned out to be more difficult for than just re-doing it. i remember some folks in #git having some idea though |
| 17:24 | jacobkm | Meh, in that case I don't care. |
| 17:24 | jacobkm | <g> |
| 17:24 | Alex_Gaynor | telenieko: That's what I have, minus the prefix stuff |
| 17:24 | brosner | :) |
| 17:25 | telenieko | Alex_Gaynor: I also did it whithout the prefix, and I'm now redoing it with prefix 'cause git branch -r is a full mess! |
| 17:26 | telenieko | git svn init is slowwww |
| 17:26 | brosner | jacobkm: rebuilding the repo from the filesystem (access to the direct svn repo) would be quite fast ;) |
| 17:27 | jacobkm | brosner: again, meh |
| 17:27 | brosner | i just haven't figued out how to translate that to work remotely. so not really worth it now that i think about it. |
| 17:27 | telenieko | mm.. jacob could you do a clean repo with a prefix and put it tgz'd somewhere? that be much faster :) |
| 17:28 | jacobkm | telenieko: maybe at some point; I'm done git-wrangling for today |
| 17:29 | telenieko | ok, I'll tell my home pc to do the clone. I can't unplug the laptop now! :) |
| 17:33 | telenieko | time to go |
| 17:33 | telenieko | c u later |
| 17:49 | Alex_Gaynor | Does anyone see anything in this diff: http://dpaste.com/57590/ that would cause this error: http://dpaste.com/57589/ , I'm working on #4117 |
| 17:49 | DjangoBot | |
| 18:38 | telenieko | hi all |
| 18:40 | Alex_Gaynor | Hey |
| 18:50 | telenieko | how many new git repos are now? :) |
| 18:52 | Alex_Gaynor | Counting the one on my hard drive no one but me gets to touch? |
| 18:53 | telenieko does not understand the expression "but me gets to touch?" | |
| 18:54 | Alex_Gaynor | It's just a local repo, if any of you guys have access to it, I have a problem :D |
| 18:54 | telenieko | ah,.. git remote add alex hack+local:///home/alex/dev/django |
| 18:54 | telenieko | npe, no luck. Did I guess the folder atleast? :) |
| 18:55 | Alex_Gaynor | Close, /home/alex/django_src/ |
| 18:56 | telenieko | argh! hack+local must be php based, it doesn't work! |
| 18:56 | telenieko | ;o) |
| 18:56 | Alex_Gaynor | Yeah, PHP isn't strongly typed so hack is assumed to be a string, and local is a file obj, somehow they get combined into an array :P |
| 18:57 | telenieko | iep, but works on IIS :! (if that's a feature) |
| 18:57 | Alex_Gaynor | Haha |
| 18:58 | telenieko | we have a ticket about runnning on IIS, we'll be at the same level as PHP then!! hehe. That's post-1.0 I hope |
| 18:59 | trbs | hehe |
| 19:02 | telenieko | For the record, #2039 |
| 19:02 | DjangoBot | |
| 19:03 | Alex_Gaynor | jesus the patch on there is unpython, self != This :D |
| 19:03 | telenieko | def __init__(This) < That rocks |
| 19:03 | trbs | uhu |
| 19:04 | trbs | specially the cap on T |
| 19:04 | trbs | even in j?v? it's called this and not This ;) |
| 19:04 | Alex_Gaynor | That code scares me |
| 19:04 | trbs | it's almost c# |
| 19:05 | trbs | as far as i've seen, they've got everything is cappitalized |
| 19:05 | telenieko | I wonder where's Env defined |
| 19:05 | Alex_Gaynor | Plus the * import |
| 19:05 | telenieko | Didn't see the * :) |
| 19:06 | trbs | or where: Write(chunk) # comes from... probably also the * |
| 19:06 | telenieko | going for dinner see you in a while! |
| 19:06 | trbs | have fun telenieko :) |
| 19:06 | trbs quickly closes #2039:pyisapie.py it's hurting the eyes | |
| 19:06 | DjangoBot | |
| 19:10 | trbs | btw the PyISAPIe code that server handler is using is also ridden with references to This, * imports and upper-case-fever |
| 19:20 | telenieko | This.Back() |
| 19:21 | hylje | visual basic in my python? |
| 19:22 | telenieko | Should we add a line in contributing.txt: Please try to make your diffs as if you'd read PEP8 |
| 19:22 | telenieko | xD |
| 19:22 | Alex_Gaynor | Does anyone have any opinions on #4117 |
| 19:22 | DjangoBot | |
| 19:24 | telenieko | a) attrs=({}, {}) b) attrs={} (applies to all) |
| 19:25 | telenieko | or: attrs={}, attrs_widget1={}, attrs_widget2={} ... |
| 19:25 | telenieko | You asked, I just made as if I tought about it :P |
| 19:25 | hylje | iterable attrs involves backwards-incompatibility |
| 19:25 | hylje | and special-casing |
| 19:26 | Alex_Gaynor | The main case I'm thinking about is I'd like to have the id attr set on the ul that contains the checkboxes or radio widgets |
| 19:27 | jacobkm | Alex_Gaynor: nested dict, maybe? |
| 19:27 | jacobkm | attrs = {'common-attr-1': value, 'widget1': {'attr-only-for-w1': foo}, 'widget2': {'attr-only-for-w2': bar}} |
| 19:28 | hylje | seems a bit magical |
| 19:28 | jacobkm | hylje: what would you suggest? |
| 19:29 | Alex_Gaynor | Hrm, I think that might be too much stuff, at that point you're better off redoing the renderer, I don't have a problem with the ability to specify attrs for the ul tag as opposed to the <input>s though |
| 19:29 | hylje | separate argument dict for the subwidget-specific stuff in the way you propose |
| 19:30 | hylje | i'm not overly familiar with how widgets work; i though suggest the widgets are specified by element type (e.g. ul, select, input) |
| 19:30 | hylje | which should be clear enough and have enough granularity for the purpose |
| 19:37 | Alex_Gaynor | I think what I like best is to keep the one attrs dict, any regular attrs are placed on both, anything in root is placed on the ul and anything in sub is placed only on the inputs(those names could be changed), thoughs? |
| 19:39 | hylje | er, what? |
| 19:40 | Alex_Gaynor | Let me make an example |
| 19:40 | insin just this second implemented that widget in JavaScript :o | |
| 19:41 | insin | is there any particular reason there's a RadioFieldRenderer but not a CheckboxFieldRenderer? |
| 19:42 | Alex_Gaynor | http://dpaste.com/57612/ so in this example both the inputs and ul would have class, and the UL would get the 'root' style, and the inputs would get the 'sub' style(they overide the general one) |
| 19:42 | mattmcc | insin: #4592 |
| 19:42 | DjangoBot | |
| 19:43 | insin | ta |
| 19:51 | Alex_Gaynor | Another option would be to give the ul tag a seperate arg with it's attrs(but still add the id I think), this is more backwards compatible |
| 19:51 | hylje | but fairly specific |
| 19:52 | hylje | considering other things which could use the same functionality |
| 19:53 | forsaken | anyone have any long hanging fruit for 1.0 that i could start working on? |
| 19:54 | forsaken | or help out with :) |
| 19:55 | Alex_Gaynor | The other alternative would be to just have this patch add id, and then leave everything else for the ticket about making CSS work better with newforms |
| 20:00 | telenieko | forsaken: think of a number between 1 and 7500 and you have high chances of finding a good ticket. |
| 20:01 | Alex_Gaynor | 1/7 about |
| 20:01 | Alex_Gaynor | Any opinions on my last suggestion? |
| 20:01 | forsaken | telenieko, heh, okay :) that's what i figured |
| 20:01 | brosner | is it best for C.objects.filter(b=B()) to throw a DoesNotExist error when B is a subclass of A (meaning multi-table inheritance)? i am curious if that behavior should be improved so it evaluates to an empty list |
| 20:02 | brosner | perhaps this has been reported before? |
| 20:02 | telenieko | forsaken: you can go to the Roadmap and take a ticket marked for 1.0 if you prefer. But any ticket you work on will be helpfull |
| 20:02 | jacobkm | forsaken: http://tinyurl.com/4xb4k5 -- a good start |
| 20:03 | jacobkm | forsaken: ^^ tickets flagged for 1.0 without patches, grouped by component. |
| 20:03 | forsaken | jacobkm, yea, thanks |
| 20:03 | forsaken | i'll poke around |
| 20:03 | tclineks | jacobkm: hey Jacob, fellow lawrencian here. I just rebased the view permission patch to nfa HEAD. How many ad astras will it take for it to be reopened for consideration? ;) |