437189 - Gdata prov: cannot delete last remaining occurence by "delete this occurence"

Status: RESOLVED FIXED

(Calendar :: Provider: GData, defect)

Description

[klint]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; fr; rv:1.9) Gecko/2008052906 Firefox/3.0
Build Identifier: TB 2.0.0.14 + Lightning 20080603 +Gdata 20080603

I have deleted all occurences but one, for a daily repeating event in a gdata calendar. When I try to delete the last one using "delete this occurence", I cannot

Reproducible: Always

Steps to Reproduce:
1.Set up a gdata calendar
2.set up a daily recurring event (with 3 occurences)
3.Delete 2 of the 3 occurneces by "delete this occurrence"
4.Try to delete the last remaining one with "delete this occurrence"
Actual Results:  
The deletion is not done.

Expected Results:  
The last occurence should be deleted.

Console shows:
Erreur : 'Illegal value' when calling method: [calIRecurrenceInfo::onStartDateChange] = NS_ERROR_ILLEGAL_VALUE
Fichier source : file:///D:/Documents%20and%20Settings/xxx/Application%20Data/Thunderbird/Profiles/xxx.xs4/extensions/%7Be2fda1a4-762b-4020-b5ad-a41df1933103%7D/js/calEvent.js
Ligne : 227

Comment 1

[Stefan Sitter [:ssitter]]

In general it should not be possible to mark all occurrences of a recurring event as exceptions, see Bug 351850.

Comment 2

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

There are actually two bugs on this. One is on my side, the other on gdata's side. It seems the server doesn't allow deleting the last occurrence of a rule (which could actually be a good thing). I think mvl's suggestion in bug 351850 is a nice idea how we could solve the problem on the client side, and it would actually be a bit easier to solve as soon as I have bug 320178 solved.

General response failed: [Exception... "A request Error Occurred. Status Code: 500 Internal Server Error Body: Exception during update"  nsresult: "0x80004005 (NS_ERROR_FAILURE)"  location: "JS frame :: file:///home/kewisch/mozilla/mozilla-branch/mozilla/obj-i686-pc-linux-gnu/dist/xpi-stage/gdata-provider/js/calGoogleCalendar.js :: cGC_general_response :: line 989"  data: no]

This is the error that shows up with the patch I have applied, which is mentioned in http://code.google.com/p/gdata-issues/issues/detail?id=115

Comment 3

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Attachment: Clear loader when failing/succeeding

This patch exposed the previously mentioned error and correctly notifies observers/listeners that something went wrong.

Comment 4

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Checked in on HEAD and MOZILLA_1_8_BRANCH

-> FIXED