2008-6-19
| 20:03 | jacobkm | tclineks: ooh, bribes are always welcome |
| 20:04 | jacobkm | tclineks: which ticket are we talking about here? |
| 20:04 | tclineks | #7150 and #820 |
| 20:04 | DjangoBot | |
| 20:04 | DjangoBot | |
| 20:04 | tclineks | james closed 7150 as dupe of 820 |
| 20:05 | jacobkm | tclineks: ah, I see. |
| 20:05 | jacobkm | |
| 20:05 | jacobkm | The admin's not designed for browsing, but luckily databrowse is. |
| 20:05 | tclineks | yeah i dig databrowse |
| 20:05 | brosner | tclineks: if you really wanted this behavior there is no reason why you couldn't use subclassed version of ModelAdmin |
| 20:06 | jacobkm | tclineks: yeah, and I think it's pretty much out of scope for the admin... you could bring it up on django-developers if you feel strongly, but I kinda suspect you won't find much help there. |
| 20:07 | tclineks | my usecase is a raw_id relationship -- to populate that field the user has to have change perms on the related model :( |
| 20:08 | tclineks | honza posted recently on -dev about how delete was kind of handicapped as you currently have to have change perms to view the object detail page |
| 20:13 | tclineks | updated patch: http://dpaste.com/57616/ could be cleaned up in a few ways, works on current nfa head |
| 20:13 | Alex_Gaynor | Ok I just posted a version of #4117 that only adds id, all other attrs will be handled under a different ticket(which already exists) |
| 20:13 | DjangoBot | |
| 20:15 | telenieko | jacobkm: when you get to play with git again you can try a git remote update and merge my master (into some branch other than your master) then you can even try to cherry-pick to your master if you like my commits (what would close three tickets) |
| 20:16 | telenieko | aside, on #7453, making it possible to still pass "status" until 1.0 with a DeprecationWarning would make that one ready? |
| 20:16 | DjangoBot | |
| 21:29 | Alex_Gaynor | jacobkm: Any opinions on the new patch I added to #4117 which essentially only adds the id to the ul and we'll defer dealing with the rest of the attrs until #3515 gets figured out? |
| 21:29 | DjangoBot | |
| 21:29 | DjangoBot | |
| 22:10 | telenieko | guys, If I want to raise a DeprecationWarning should I import warnings at the top of the file or in the method that will raise the warning? |
| 22:11 | jacobkm | telenieko: in the method that raises the warning, prefereably inside the if: block around the warning. |
| 22:13 | telenieko | ups, I took newforms as example: from warnings import warn, at the top of the file. I'll move mine to the if block. Thanks |
| 22:13 | telenieko | how's your git playing going? ;) |
| 22:14 | telenieko | what's the stacklevel parameter to warn() for¿ |
| 22:21 | telenieko | mm.. My shell doesn't show PendingDeprecationWarning's any clue? |
| 22:22 | jacobkm | telenieko: that's because pending warnings are silent by default. Use a DeprecationWarning. |
| 22:22 | telenieko | ok, thanks |
| 22:22 | telenieko | form_for_model raises a Pending one ;) |
| 22:22 | jacobkm | That should get promoted one of these days. |
| 22:24 | telenieko | there are only two PendingDeprecationWarnings, both in newforms |
| 22:25 | Alex_Gaynor | Doesn't oldforms itself raise a deprecation warning? |
| 22:26 | telenieko | I did a grep -n -R and only newforms appeared |
| 22:26 | telenieko | git merge --squash tmp/ticket_branch && git commit < nice :) |
| 22:27 | telenieko | I can't find any warnings in oldforms... rare |
| 22:29 | Alex_Gaynor | Actually there probably aren't since then you'd get them all over the place in the admin |
| 22:30 | telenieko | yup, maybe after 1.0-beta we can start raising them, hehe. |
| 22:32 | brosner | r7488 <-- eyeballs wanted :) |
| 22:32 | DjangoBot | |
| 22:32 | Alex_Gaynor | ? |
| 22:33 | telenieko | r or # ? |
| 22:33 | brosner | ugh yeah #7488 |
| 22:33 | DjangoBot | |
| 22:33 | telenieko | the changeset is nice anyway "Migrated to america" xD |
| 22:33 | brosner | hehe |
| 22:35 | telenieko | by upstream you mean in models instead of newforms? |
| 22:36 | brosner | correct. should the queryset i posted work when in model inheritance |
| 22:37 | telenieko | Should, if non-inherited return empty lists, inherited should also. The behaviour should be consitent (if I got the problem right) |
| 22:37 | brosner | right so it should be fixed upstream |
| 22:37 | telenieko | I think so. |
| 22:37 | telenieko | in trunk maybe ;) |
| 22:38 | brosner | oh right. i pointed out nfa branch because it was easier ;) |
| 22:38 | brosner | the line numbers are off by a few |
| 22:38 | telenieko | np. I'll change the ticket's component and version then if you don't mind ;) |
| 22:39 | brosner | i wonder if this problem has been reported though against trunk. i haven't looked |
| 22:39 | telenieko | mm... let's call the friend google :) |
| 22:39 | telenieko | but I don't remember a ticket about this.. mmmm |
| 22:42 | telenieko | after 5 hours, git svn just finished doing a new clone of django :)))) |
| 22:42 | telenieko | time to remake my screwed repo! |
| 22:42 | brosner | #3016 looks similar. i think that has been fixed since qs-rf |
| 22:42 | DjangoBot | |
| 22:52 | brosner | however, it does feel right to return an empty queryset when self.instance.pk is None in BaseInlineFormset.get_queryset |
| 23:04 | telenieko | time to sleep! see you tomorrow |
| 23:04 | telenieko | bye¡ |
| 23:04 | jezdez | bye telenieko! |
| 23:05 | telenieko | jezdez: do you sleep? you're always here when I say "hi" and "bye"! |
| 23:05 | jezdez | :) I do |
| 23:05 | jezdez | I'm on a irc bouncer |
| 23:05 | telenieko | you cheat :P |
| 23:06 | jezdez | yep :) |
| 23:06 | telenieko | see you in 8 hours or so ;) |
| 23:06 | telenieko | good night! |
| 23:07 | jezdez | good night ^^ |
| 23:29 | insin | anyone think a newforms DateInput and TimeInput would be useful to go along with the existing DateTimeInput? SplitDateTimeWidget is currently a bit weird in that it uses two TextInputs and relies on the default string representation of a datetime.date and datetime.time respectively, with no way to change it |
| 23:37 | Gulopine | i would think so |
| 23:38 | insin | hactually, since dates and times have .strftime, just modifying SplitDateTimeWidget to hold default date/time formats and the necessary keyword args should do the trick |
| 23:38 | insin tries it out in his JS version ^_^ | |
| 23:39 | Gulopine | i really need to get my DurationField updated with a newforms widget |
| 23:39 | Gulopine | er, a newforms-admin widget, rather |
Page 4 of 4
← Previous
(382 total)