calendar-nl is missing: Calendar._FD = 0; #3

joostschouten opened this Issue Mar 3, 2010 · 2 comments


None yet
2 participants

The calendar-nl.js has the same problem as the calendar-de.js file had as described in [1]. Adding the following line line fixes the problem.

Calendar._FD = 0;

I'll try to provide a patch, but won't make any promises since I'm new to github ;-)

PS: apologies for the double post, but I accedently closed the ticket and didn't see a way to reopen it.


git format-patch:

From 13194787d997fd25857a579b64f3aa7ad7b45cf7 Mon Sep 17 00:00:00 2001
From: Joost Schouten
Date: Wed, 3 Mar 2010 15:43:44 +1300
Subject: [PATCH] bug fix: calendar-nl.js was missing Calendar._FD = 0; which caused the calendar to print undefined in stead of the week days and completely omitted the clickable days of the month

.../datefield/jscalendar-1.0/lang/calendar-nl.js | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/tapx-datefield/src/main/resources/com/howardlewisship/tapx/datefield/jscalendar-1.0/lang/calendar-nl.js b/tapx-datefield/src/main/resources/com/howardlewisship/tapx/datefield/jscalendar-1.0/lang/calendar-nl.js
index a1dea94..391cf9e 100644
--- a/tapx-datefield/src/main/resources/com/howardlewisship/tapx/datefield/jscalendar-1.0/lang/calendar-nl.js
+++ b/tapx-datefield/src/main/resources/com/howardlewisship/tapx/datefield/jscalendar-1.0/lang/calendar-nl.js
@@ -11,6 +11,8 @@ Calendar._DN = new Array

Calendar._SDN_len = 2;

+Calendar._FD = 0;
Calendar._MN = new Array



hlship commented May 25, 2010

Apply patch to fix NL localization of DateField component

Closed by b08534f

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment