Closed Bug 286944 Opened 19 years ago Closed 19 years ago

duplicate definition of cal_calendars table

Categories

(Calendar :: Provider: Local Storage, defect)

Product:
Calendar
Component:
Provider: Local Storage
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: mvl, Assigned: vlad)

Details

Attachments

(1 file)

patch c1
9.89 KB, patch
vlad
: first-review+
Details | Diff | Splinter Review 
calStorageProvider.js contains a definition of the cal_calendars table, while
calCalendarManager.js also has one. The last seems to be the right one. The
first should be removed
Attached patch patch c1 — — Splinter Review 
patch removes the extra definition. It also fixes a few JS-strict warnings.
Attachment #178044 - Flags: first-review?(vladimir)
Comment on attachment 178044 [details] [diff] [review]
patch c1

go for it; I think the cal_calendars stuff is mucking up other new-profile
problems also
Attachment #178044 - Flags: first-review?(vladimir) → first-review+
patch was checked ages ago.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: