brOOper / gwt-cal

Automatically exported from code.google.com/p/gwt-cal
0 stars 0 forks source link

Appointment.clone() method does not include clone.setId() #88

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. drag and drop an appointment to trigger onUpdate()
2. call event.setCancelled(true) in onUpdate() method of handler that was added 
with Calendar.addUpdateHandler()
3. value of appointment id for appointment that was drag and dropped is now 
null as shown by subsequent Appointment.getId() (i.e. after onUpdate() returns 
and appointment rollback occurs)

What is the expected output? What do you see instead?
Expected output is Appointment.getId() is non-null. I see Appointment.getId() 
returning null

What version of the product are you using? On what operating system?
gwt-cal-0.9.2 on Windows 7

Please provide any additional information below.
Suspect problem is in Appointment.clone() so that rollbackappointment does not 
have id like committedappointment does.

Original issue reported on code.google.com by kirk.spe...@gmail.com on 22 Sep 2010 at 10:30

GoogleCodeExporter commented 8 years ago
Problem is fixed by adding the statement "clone.setId(this.id);" to the method 
clone() in the file Appointment.java.

Original comment by kirk.spe...@gmail.com on 23 Sep 2010 at 1:09

GoogleCodeExporter commented 8 years ago
Issue 89 has been merged into this issue.

Original comment by Brad.Ryd...@gmail.com on 23 Sep 2010 at 4:07

GoogleCodeExporter commented 8 years ago

Original comment by Brad.Ryd...@gmail.com on 23 Sep 2010 at 4:08

GoogleCodeExporter commented 8 years ago
Change included in revision 363. Thanks for the feedback!

- Carlos

Original comment by carlos.m...@gmail.com on 1 Feb 2011 at 1:57

GoogleCodeExporter commented 8 years ago

Original comment by ctasada on 3 Jan 2012 at 3:59

GoogleCodeExporter commented 8 years ago
Issue 130 has been merged into this issue.

Original comment by Brad.Ryd...@gmail.com on 2 May 2013 at 8:36