Removing obsolete strings from calendar.properties

RESOLVED FIXED in Sunbird 0.5

Status

RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: cedric.corazza, Assigned: cedric.corazza)

Tracking

Trunk
Sunbird 0.5

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

12 years ago
There are 67 obsolete strings in calendar.properties file. They should be removed to ease the readability of this file.
A grep -R for the following strings in /calendar directory shows that these strings are only present in calendar.properties file.
Strings list :

AllDayEvents
noEventsOrTasksToImport
aboutToImportNewEventsTitle
aboutToImportNewEvents
aboutToImportDupEventsTitle
aboutToImportDupEvents
aboutToImportNewTasksTitle
aboutToImportNewTasks
aboutToImportDupTasksTitle
aboutToImportDupTasks
discardAll
importAll
promptForEach
addDuplicate
eventTitle
eventStartTime
eventEndTime
eventSummary
eventWhen
eventWhere
concurrentEdit
deleteCalendarTitle
deleteCalendarMessage
deleteCalendarOnly
deleteCalendarAndFile
weekDayMonthLabel
Week
1001
Save
filepickerDefServerString
filtervCalendar
filterXcs
filterXml
filterRtf
filterHtml
filterCsv
filterOutlookCsv
filterRdf
outlookCSVTitle
outlookCSVStartDate
outlookCSVStartTime
outlookCSVEndDate
outlookCSVEndTime
outlookCSVAllDayEvent
outlookCSVAlarm
outlookCSVAlarmDate
outlookCSVAlarmTime
outlookCSVCategories
outlookCSVDescription
outlookCSVLocation
outlookCSVPrivate
outlookCSVValueTrue
outlookCSVValueFalse
outlookCSVDateParseConfirm
httpError
otherError
contentError
setMasterPassword
password_not_set
failed_pw_change
incorrect_pw
pw_change_ok
pw_erased_ok
pw_not_wanted
pw_empty_warning
pw_change_success_title
pw_remove_button
(Assignee)

Comment 1

12 years ago
Created attachment 245942 [details] [diff] [review]
Patch removing obsolete strings
Attachment #245942 - Flags: first-review?(lilmatt)

Comment 2

12 years ago
I think we wanted to keep the outlook strings around for when we finally get a full-fledged csv importer.   mvl would know for sure.
Leave the outlook items in there. I still don't know what the best solution to importing outlook is. Loosing all the translated fields names is a bad idea in this case. (yes, there is cvs history, but that's a pain to do for all the localizations)
'leave them in' means don't remove from calendar.properties.
(Assignee)

Comment 5

12 years ago
Created attachment 246078 [details] [diff] [review]
This one adds back Outlook related strings

This patch adds back Outlook related strings to address mvl's comment
Attachment #245942 - Attachment is obsolete: true
Attachment #246078 - Flags: first-review?(lilmatt)
Attachment #245942 - Flags: first-review?(lilmatt)
Comment on attachment 246078 [details] [diff] [review]
This one adds back Outlook related strings

I did a grep -r for each of the removed strings and it came up clean.

r=lilmatt



AllDayEvents
noEventsOrTasksToImport
aboutToImportNewEventsTitle
aboutToImportNewEvents
aboutToImportDupEventsTitle
aboutToImportDupEvents
aboutToImportNewTasksTitle
aboutToImportNewTasks
aboutToImportDupTasksTitle
aboutToImportDupTasks
discardAll
importAll
promptForEach
addDuplicate
eventTitle
eventStartTime
eventEndTime
eventSummary
eventWhen
eventWhere
concurrentEdit
deleteCalendarTitle
deleteCalendarMessage
deleteCalendarOnly
deleteCalendarAndFile
weekDayMonthLabel
Week
1001
Save
filepickerDefServerString
filtervCalendar
filterXcs
filterXml
filterRtf
filterHtml
filterCsv
filterOutlookCsv
filterRdf
httpError
otherError
contentError
Attachment #246078 - Flags: first-review?(lilmatt) → first-review+
(Assignee)

Comment 7

12 years ago
Created attachment 247699 [details] [diff] [review]
Patch v3

Providing a new patch because landed bug 360513 broke the previous one.
lilmatt : could you forward your first-review+ please ?
Attachment #246078 - Attachment is obsolete: true
Attachment #247699 - Flags: second-review?
(Assignee)

Updated

12 years ago
Attachment #247699 - Flags: second-review? → second-review?(dmose)
Comment on attachment 247699 [details] [diff] [review]
Patch v3

r=lilmatt
Attachment #247699 - Flags: first-review+
Comment on attachment 246078 [details] [diff] [review]
This one adds back Outlook related strings

r=dmose
Attachment #246078 - Flags: second-review+
Comment on attachment 247699 [details] [diff] [review]
Patch v3

Moving forward dmose's r+ as the updated patch simply unbitrots the one he approved.
Attachment #247699 - Flags: second-review?(dmose) → second-review+
Patch v3 checked in on MOZILLA_1_8_BRANCH and trunk.

-> FIXED
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.