323011 - Unifinder default option should not be 'all events'

Status: VERIFIED FIXED

(Calendar :: General, defect)

Description

[Joey Minta]

Apparently, we managed to take down a caldav server by running it out of memory.  We ran a caldav query for all events, and the server died trying to create a report for 2 years worth of events in a very dense calendar.  I was asked to consider at least making this a non-default option (especially since we query at startup).  Filing as UNCO, though, since I'm not sure if it's something we really want to do.

Comment 1

[Matthew (lilmatt) Willis]

Reassigning all automatically assigned bugs from Mostafa to nobody@m.o

Bugspam filter: TorontoMostafaMove

Comment 2

[Philipp Kewisch [:Fallen][📱📆]]

I would like to second this. Networked providers that do not download all events anyway (as webdav does), should not start out with getting all events from the network. This for example applies to the upcoming google calendar provider.

Comment 3

[Martin Schröder [:mschroeder]]

Unifinder is also available in Lightning now.

Comment 4

[Michiel van Leeuwen (email: mvl+moz@)]

The patch for this should be easy enough. The question is, what should the new default be. Christian, any opinions on this?

Comment 5

[Simon Paquet [:sipaq]]

I'd vote for "Events in the Next 7 Days".

Comment 6

[Daniel Boelzle [:dbo]]

I vote for "Today's Events".

Comment 7

[Philipp Kewisch [:Fallen][📱📆]]

Attachment: Set default value to 'next 7 days'

This patch makes the default "Next 7 Days".

Comment 8

[Daniel Boelzle [:dbo]]

Comment on attachment 295920 [details] [diff] [review]
Set default value to 'next 7 days'

r=dbo; "next 7 days" has been agreed with Christian, setting ui+.

Comment 9

[Philipp Kewisch [:Fallen][📱📆]]

Checked in on HEAD and MOZILLA_1_8_BRANCH

-> FIXED

Comment 10

[Andreas Treumann]

Verified in Lightning (build 2008010904) and Sunbird (build 2080109) -> task is fixed. 

Comment 11

[Mike Arena]

Sorry, I do not believe it is fixed. The unifinder is unable to "find events" when I'm looking right at them in the calendar. (I'm using the latest SB and Ltng nightlies.)

Comment 12

[Stefan Sitter [:ssitter]]

(In reply to comment #11)
By "find" you mean "search"? See Bug 413715.

Comment 13

[Mike Arena]

Yes.

Comment 14

[Mike Arena]

Tx, ssitter. I did not realize it was a separate bug.