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)