Code Review Notes =============== Participants : Azeez, Deep, Nandika, Manjula, Chmara, ChamaraA, Nuwan, Ajith, Heshan
JMSMessageBoxService.java -------------------------------------------- Synchronize the data structure, rather than synchronizing the whole method Remove the isAdminLoggedIn() method's hard-coded admin related logic. Add role based access to service. Refactor the code to have two OSGi Services - One to handle AdminServices and one to handle others If no message boxes are there, add a relevant message to the UI. Add a small description of what a class/interface does at the top of the class/interface. Inside the handler handle the registry specific stuff According to the current implementation, MessageBox wont work in Stratos. Need to look into this aspect as well. Fefactor the complex methods. ie. group the logic into methods and use them within the complex method. MessageAuthorizationHandler.java -------------------------------------------------- When retrieving the name of the messagebox, take tenant into consideration. Use ExecutorService instead of using a TimerTask On Mon, Feb 21, 2011 at 9:20 AM, Manjula Rathnayake <[email protected]>wrote: > more details > »<https://www.google.com/calendar/event?action=VIEW&eid=Yzc3dW5hZjI5dXQxNm1qZWF0c2o1MGoyazQgaGVzaGFuQHdzbzIuY29t&tok=MTcjbWFuanVsYXJAd3NvMi5jb21kNzdhODE3MDc5MWFlNmM0OWE1Yjk3OWM4MGI3ZDlkZWIxMGY2MmM0&ctz=Asia%2FColombo&hl=en> > Group B code review (MessageBox) > Crucible project can be found here. > https://wso2.org/crucible/cru/COMPONENT-42<http://www.google.com/url?q=https%3A%2F%2Fwso2.org%2Fcrucible%2Fcru%2FCOMPONENT-42&ust=1298267449835000&usg=AFQjCNEJzOAnRIXXUpLV97TG52rEBWlj8w> > *When* > Tue Feb 22 10:30am – 11:30am Colombo > *Where* > 3rd floor (map <http://maps.google.com/maps?q=3rd+floor&hl=en>) > *Calendar* > [email protected] > *Who* > • > Manjula Rathnayake - organizer > • > Chamara Ariyarathne > • > Milinda Pathirage > • > Lahiru Gunathilake > • > Amal Rangana > • > Chamara Silva > • > Nuwan Bandara > • > [email protected] > • > Krishantha Samaraweera > • > Sanjeewa Malalgoda > • > Heshan Suriyaarachchi > • > Danushka Manikkumbura > • > Ajith Vitharana > > Going? > *Yes<https://www.google.com/calendar/event?action=RESPOND&eid=Yzc3dW5hZjI5dXQxNm1qZWF0c2o1MGoyazQgaGVzaGFuQHdzbzIuY29t&rst=1&tok=MTcjbWFuanVsYXJAd3NvMi5jb21kNzdhODE3MDc5MWFlNmM0OWE1Yjk3OWM4MGI3ZDlkZWIxMGY2MmM0&ctz=Asia%2FColombo&hl=en>- > Maybe<https://www.google.com/calendar/event?action=RESPOND&eid=Yzc3dW5hZjI5dXQxNm1qZWF0c2o1MGoyazQgaGVzaGFuQHdzbzIuY29t&rst=3&tok=MTcjbWFuanVsYXJAd3NvMi5jb21kNzdhODE3MDc5MWFlNmM0OWE1Yjk3OWM4MGI3ZDlkZWIxMGY2MmM0&ctz=Asia%2FColombo&hl=en>- > No<https://www.google.com/calendar/event?action=RESPOND&eid=Yzc3dW5hZjI5dXQxNm1qZWF0c2o1MGoyazQgaGVzaGFuQHdzbzIuY29t&rst=2&tok=MTcjbWFuanVsYXJAd3NvMi5jb21kNzdhODE3MDc5MWFlNmM0OWE1Yjk3OWM4MGI3ZDlkZWIxMGY2MmM0&ctz=Asia%2FColombo&hl=en> > * more options > »<https://www.google.com/calendar/event?action=VIEW&eid=Yzc3dW5hZjI5dXQxNm1qZWF0c2o1MGoyazQgaGVzaGFuQHdzbzIuY29t&tok=MTcjbWFuanVsYXJAd3NvMi5jb21kNzdhODE3MDc5MWFlNmM0OWE1Yjk3OWM4MGI3ZDlkZWIxMGY2MmM0&ctz=Asia%2FColombo&hl=en> > > Invitation from Google Calendar <https://www.google.com/calendar/> > > You are receiving this email at the account [email protected] because you > are subscribed for invitations on calendar [email protected]. > > To stop receiving these notifications, please log in to > https://www.google.com/calendar/ and change your notification settings for > this calendar. > -- Regards, Heshan Suriyaarachchi Software Engineer WSO2 Inc.; http://wso2.com/ Blog: http://heshans.blogspot.com/
_______________________________________________ Carbon-dev mailing list [email protected] http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev
