[Calendar] ContainerEventProvider possible bug ?

Hi !
I think I found a bug in Vaadin Calendar. Let’s have a look :


https://github.com/vaadin/vaadin/blob/master/server/src/com/vaadin/ui/components/calendar/ContainerEventProvider.java?source=cc

around line 227 there is:

if (styleNameProperty != null && item.getItemPropertyIds().contains(styleNameProperty))
{
                basicEvent.setDescription(String.valueOf(item.getItemProperty(
                        descriptionProperty).getValue()));
}

In this IF section there is an assignment to description property of basicEvent while looking at previous IF’s one might expect assignment to styleNameProperty. Is it ok or indeed it is a bug ?

Yep, I noticed that few weeks ago. I was also wondering, if it’s correct. Any ideas?

This looks like a bug to me - please
file a bug report
.