Re: Annotation failure (Combining multiple aggregations)

2015-12-07 Thread Anssi Kääriäinen
On Tuesday, December 8, 2015 at 3:16:20 AM UTC+2, Michael wrote: > > I have the same problem. > > I tried to use `extra` but then I can't filter so it's not the best. > > Anybody has more info or a workaround? > Come to think of it, we already have a workaround: Book.objects.annotate(author_c

Re: Annotation failure (Combining multiple aggregations)

2015-12-07 Thread Anssi Kääriäinen
On Friday, December 4, 2015 at 5:47:43 PM UTC+2, Paulo Maciel wrote: > > Combining multiple aggregations with annotate() will yield the wrong > results , as multiple tables > are cross joined. Due to the use of LEFT OUTER JOIN, duplicate records > wil

Re: Annotation failure (Combining multiple aggregations)

2015-12-07 Thread Michael
I have the same problem. I tried to use `extra` but then I can't filter so it's not the best. Anybody has more info or a workaround? Thanks On Friday, December 4, 2015 at 9:47:43 AM UTC-6, Paulo Maciel wrote: > > Combining multiple aggregations with annotate() will yield the wrong > results <

Re: Split DeleteView into a more generic ConfirmView

2015-12-07 Thread Lennart Buit
I should add patches that I promise to add ;) --Lennart On Monday, December 7, 2015 at 9:23:19 PM UTC+1, Lennart Buit wrote: > > Hey all, > > One of my minor annoyances with the class based view system in django is > that there is no clear way to model a confirmable action. The concept does > e

Split DeleteView into a more generic ConfirmView

2015-12-07 Thread Lennart Buit
Hey all, One of my minor annoyances with the class based view system in django is that there is no clear way to model a confirmable action. The concept does exists in cbv, but only for deleting objects. I however think that there are more usages for confirming actions other then delete. Lets l

Re: startproject with default app

2015-12-07 Thread Luke Sapan
Well, that's embarassing. I totally missed that the template flag even existed. Thanks! On Monday, December 7, 2015 at 2:07:00 PM UTC-5, Florian Apolloner wrote: > > Hi Luke, > > this is what the --template argument is for, this way we do not need an > extra flag for every user wish ;) > > Cheer

Re: annoyance with Python 3.2 support in Django 1.8

2015-12-07 Thread Tim Graham
Here's a documentation proposal: https://github.com/django/django/pull/5783 On Saturday, December 5, 2015 at 6:42:18 PM UTC-5, Tim Graham wrote: > > No we haven't done that before. I think advertising it in the blog post > and release notes would be enough. In particular, I'd like to advertise it

Re: startproject with default app

2015-12-07 Thread Florian Apolloner
Hi Luke, this is what the --template argument is for, this way we do not need an extra flag for every user wish ;) Cheers, Florian On Monday, December 7, 2015 at 6:24:19 PM UTC+1, Luke Sapan wrote: > > Hello, > > > I have a suggestion for an improvement to the startproject command. > Currently

startproject with default app

2015-12-07 Thread Luke Sapan
Hello, I have a suggestion for an improvement to the startproject command. Currently, the command creates a project and a main folder that isn’t quite an app. It doesn’t have any models, views, tests, etc. I realize this is by design, and for many cases is the correct fit. However, there

Re: Backporting ticket 25548 into 1.9.x

2015-12-07 Thread charettes
I confirm this is the reason it wasn't backported in the first place. Le lundi 7 décembre 2015 08:20:19 UTC-5, Tim Graham a écrit : > > I've done the backport. I think the only reason it wasn't done initially > is that there's no indication on the ticket that the issue was a regression. > > On Su

Re: Backporting ticket 25548 into 1.9.x

2015-12-07 Thread James Bennett
Thanks to everyone for the quick consensus, and to Tim for backporting it and fixing the docs before I could get around to it :) On Mon, Dec 7, 2015 at 9:23 AM, Tim Graham wrote: > Done in > https://github.com/django/django/commit/9c835990ea2911c06a877296fbc25157c1302e1d > -- thanks for pointing

Re: What is the purpose of adding `nowrap` class to all ForeignKey fields in django admin objects list?

2015-12-07 Thread alTus
Didn't think about that way to investigate, thanks for the tip. That commit says that it was added because list_editable FK widget would break into several line otherwise (same with date widgets). Well, seems legit. But it's added even if the field is not in list_editable. And now I start thinking

Re: admin: automatic link to files for FileFields in change_list breaks link to change_view (v1.8, #14497)

2015-12-07 Thread Riccardo Magliocchetti
Il 22/11/2015 22:02, Florian Demmer ha scritto: On 2015-11-22 21:31, Riccardo Magliocchetti wrote: Hi Florian, Il 22/11/2015 21:14, Florian Demmer ha scritto: Hi guys, i am confused/surprised by this ticket and already merged feature. it took me a while to find out what was happening in my

Re: Backporting ticket 25548 into 1.9.x

2015-12-07 Thread Tim Graham
Done in https://github.com/django/django/commit/9c835990ea2911c06a877296fbc25157c1302e1d -- thanks for pointing it out. On Monday, December 7, 2015 at 8:42:09 AM UTC-5, Anssi Kääriäinen wrote: > > On Mon, Dec 7, 2015 at 3:20 PM, Tim Graham > wrote: > > I've done the backport. I think the only

Re: What is the purpose of adding `nowrap` class to all ForeignKey fields in django admin objects list?

2015-12-07 Thread Tim Graham
Did you try looking through git blame to find the commit that added it? https://github.com/django/django/commit/0b53d318 might be what you're looking for. On Monday, December 7, 2015 at 8:33:34 AM UTC-5, alTus wrote: > > Django adds `nowrap` class to your `td` if you specify list_display to > s

Re: Backporting ticket 25548 into 1.9.x

2015-12-07 Thread Anssi Kääriäinen
On Mon, Dec 7, 2015 at 3:20 PM, Tim Graham wrote: > I've done the backport. I think the only reason it wasn't done initially is > that there's no indication on the ticket that the issue was a regression. For some reason we don't mention backporting of regression fixes in https://docs.djangoprojec

What is the purpose of adding `nowrap` class to all ForeignKey fields in django admin objects list?

2015-12-07 Thread alTus
Django adds `nowrap` class to your `td` if you specify list_display to show any ForeignKey. I can't see the purpose to do it. Any considerably long title breaks table layout hard. Do I miss smth or may be it would be better to just remove it? -- You received this message because you are subscri

Re: Backporting ticket 25548 into 1.9.x

2015-12-07 Thread Tim Graham
I've done the backport. I think the only reason it wasn't done initially is that there's no indication on the ticket that the issue was a regression. On Sunday, December 6, 2015 at 11:34:47 AM UTC-5, Marc Tamlyn wrote: > > Agreed the reasoning is sound. This is a major bug as far as I'm concerned

Re: minify static files (css and js)

2015-12-07 Thread Tony Tong
I'm a html editor, I always use this free online tool to minify js file and minify css file. after the minification, the static file will be much smaller. -- You received this message because you are subsc