361195 - Removing obsolete strings from calendar.properties

Status: RESOLVED FIXED

(Calendar :: General, defect)

Description

[Cédric Corazza]

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

Comment 1

[Cédric Corazza]

Attachment: Patch removing obsolete strings

Comment 2

[Joey Minta]

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.

Comment 3

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

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)

Comment 4

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

'leave them in' means don't remove from calendar.properties.

Comment 5

[Cédric Corazza]

Attachment: This one adds back Outlook related strings

This patch adds back Outlook related strings to address mvl's comment

Comment 6

[Matthew (lilmatt) Willis]

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

Comment 7

[Cédric Corazza]

Attachment: Patch v3

Providing a new patch because landed bug 360513 broke the previous one.
lilmatt : could you forward your first-review+ please ?

Comment 8

[Matthew (lilmatt) Willis]

Comment on attachment 247699 [details] [diff] [review]
Patch v3

r=lilmatt

Comment 9

[Dan Mosedale (:dmosedale, :dmose)]

Comment on attachment 246078 [details] [diff] [review]
This one adds back Outlook related strings

r=dmose

Comment 10

[Matthew (lilmatt) Willis]

Comment on attachment 247699 [details] [diff] [review]
Patch v3

Moving forward dmose's r+ as the updated patch simply unbitrots the one he approved.

Comment 11

[Matthew (lilmatt) Willis]

Patch v3 checked in on MOZILLA_1_8_BRANCH and trunk.

-> FIXED