Open Bug 277978 Opened 20 years ago Updated 1 year ago

Change common (i.e description,location) properties to real attributes on calIItemBase

Categories

(Calendar :: Internal Components, task)

Tracking

(Not tracked)

People

(Reporter: pavlov, Unassigned, Mentored)

Details

(Keywords: good-first-bug, Whiteboard: [lang=js])

Things like location and description and probably others should be made real
attributes instead of sitting on the sidelines.
Summary: Change common properties to real attributes on calIItemBase → Change common (i.e description,location) properties to real attributes on calIItemBase
Whiteboard: [good first bug][mentor=Fallen][lang=js]
I'd like to work on this bug. Can you please assign me in?
Hi Malintha, I'm happy to hear you would like to give this one a shot. Please email me or find me on irc.mozilla.org #calendar (nickname is Fallen) if you need help getting started.

Here is a setup guide for building: https://developer.mozilla.org/En/Simple_Thunderbird_build
Once you have a full build, you can rebuild only Lihtning by calling make -f objdir-tb/calendar/lightning and on mac additionally make -f objdir-tb/mail/app.

I would suggest going through the list of properties commonly requested and creating a list of those you think make sense as top level properties.

Here is a superset of them, note that some of the getProperty() calls are not on an item but on a calendar. This bug is about the item calls:

http://mxr.mozilla.org/comm-central/search?string=.getProperty%28%22&find=calendar&findi=&filter=^[^\0]*%24&hitlimit=&tree=comm-central
Assignee: nobody → malinthak2
Status: NEW → ASSIGNED
Hi Philipp, Thank you for signing me in. I'm facing some troubles with the build.Once I get them fixed I'll start on this ASAP. :)
Here is a list of already promoted properties. http://mxr.mozilla.org/comm-central/source/calendar/base/src/calItemBase.js#715

I think "URL","TRANSP","DESCRIPTION","LOCATION","organizerId","EMAIL" are not promoted (although they are all capitalized) and used often and need to be promoted. I think promoting in the sense defining them in the base interface. Then do we have to remove the current definitions of them (in other interfaces)? Will it not effect to current implementations and current getters and setters? Adding item accessor should be like this I think. http://mxr.mozilla.org/comm-central/source/calendar/base/src/calItemBase.js#113
I think all I have to do is promoting those attributes to the calIitembase interface and define and implement new accessors in the calIitembase.js and redefine the promoted property bag..
Mentor: philipp
Whiteboard: [good first bug][mentor=Fallen][lang=js] → [good first bug][lang=js]
Keywords: good-first-bug
Whiteboard: [good first bug][lang=js] → [lang=js]
Assignee: malinthak2 → nobody
Status: ASSIGNED → NEW
Type: defect → task
OS: Windows XP → All
Hardware: x86 → All
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.