[TAPESTRY-875] DatePicker delimiter limitations

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Incomplete
• Affects Version/s: unspecified
• Fix Version/s: 4.0.1
• Component/s: Framework
• Assignee: Jesse Kuhnert
• Reporter: Matthias Zitzmann
• Created: 08/Mar/06 23:57
• Updated: 11/Apr/06 00:13
• Resolved: 11/Apr/06 00:13

Description

Using the DatePicker for a date format containing no "word delimiter" doesn't work because of an inefficient limitational replacement-method in DatePicker.js. I added a patch for this, because I've to work with a date format of ddMMyy (without delimiter). With that patch it won't break any more neither for regular date formats nor for this delimiter-free format used in financial or touristics for example.

— DatePicker.js Wed Mar 08 16:42:15 2006
+++ DatePicker.orig.js Wed Mar 08 16:45:35 2006
@@ -807,10 +807,10 @@
var frm = new String(this._format);
// TAPESTRY-669: Have to be very explicit about keys, to keep functions added
// to Array (by the Prototype library, if its around) from getting mixed in.

• var keys = new Array('dddd','ddd','dd','d','MMMM','MMM','MM','M','yyyy','yy', 'ss', 's', 'mm',
• 'm', 'HH', 'H');
  + var keys = new Array('d','dd','ddd','dddd','M','MM','MMM','MMMM','yyyy','yy', 's', 'ss', 'm',
  + 'mm', 'H', 'HH');
  for (var i = 0; i < keys.length; i++) { - frm = eval("frm.replace(/" + keys[i] + "/,\"" + bits[keys[i]] + "\");"); + frm = eval("frm.replace(/\\b" + keys[i] + "\\b/,\"" + bits[keys[i]] + "\");"); }

return frm;