VisualizationsForVaadin issue

Flow
1

Hi,

I discovered an issue in VisualizationsForVaadin that prevented AnnotatedTimeLine from working properly. The date format string seems to have caused the dates to be incorrectly rendered on the chart. The patch below would fix it. How do I submit it?

Thanks.

JX

diff --git a/src/org/vaadin/vaadinvisualizations/AnnotatedTimeLine.java b/src/org/vaadin/vaadinvisualizations/AnnotatedTimeLine.java
index 99cba87…4707af0 100644
— a/src/org/vaadin/vaadinvisualizations/AnnotatedTimeLine.java
+++ b/src/org/vaadin/vaadinvisualizations/AnnotatedTimeLine.java
@@ -44,7 +44,7 @@ public class AnnotatedTimeLine extends VisualizationComponent{
public void add(Calendar date, ArrayList timeLineEntries) {

            String[] stringLineValues = new String[(timeLineEntries.size() * 3) +1]

;

  •           SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-DD");

  •           SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd");
          
          stringLineValues[0]

= sdf.format(date.getTime());

2

That’s Phils add-on, if you mean this: http://vaadin.com/directory#addon/107
You should contact him… However, I’m not sure how that’s possible… We can try screaming: PHIIIILLLL!!! :wink:

These are posts regarding the add-on, perhaps you could reply to those, if screaming does not work:
http://vaadin.com/forum/-/message_boards/message/151670
http://vaadin.com/forum/-/message_boards/message/146796

(Why does our forum not have a ‘private message’ function? Or perhaps I just don’t know how?)

Best Regards,
Marc

3

Did I hear someone screeeeeam…?
I’ll take a look at this, stuck abroad right now, but will get round to it next week.

4

Should be fixed now… hopefully… Uploaded a new version in the dir.