PEAR is archived and read-only

This mirror preserves historical PEAR package releases and metadata so existing references remain available.

Home » Database » DB_DataObject_FormBuilder » Bug #8105

Add support for 'g' date element format option

Details

Submitted2006-07-03 07:12 UTC
Fromdsanders at baselinesolutions dot com dot au
Assignedjustinpatrin
StatusClosed
PackageDB_DataObject_FormBuilder
PHP Version4.3.10
OSFC2
Roadmaps(Not assigned)

Comments

[2006-07-03 07:12 UTC] dsanders at baselinesolutions dot com dot au

Description:
------------
It'd great to just add the 'g' QuickForm date element format option to _date2array() and _array2date().

(The 'g' is 'h' without any leading zeroes)

Should be simple enough :)

(See bug #3733)

[2006-12-07 06:44 UTC] dsanders at baselinesolutions dot com dot au

Ok lazy bones here's a patch ;)

@@ -2460,6 +2460,7 @@ class DB_DataObject_FormBuilder
if ($da['h'] == 0) {
$da['h'] = 12;
}
+ $da['g'] = $da['h'];
$da['i'] = $dObj->getMinute();
$da['s'] = $dObj->getSecond();
if ($da['H'] >= 12) {
@@ -2482,6 +2483,7 @@ class DB_DataObject_FormBuilder
$da['Y'] = $da['y'] = date('Y', $time);
$da['H'] = date('H', $time);
$da['h'] = date('h', $time);
+ $da['g'] = date('g', $time);
$da['i'] = date('i', $time);
$da['s'] = date('s', $time);
$da['a'] = date('a', $time);
@@ -2525,8 +2527,12 @@ class DB_DataObject_FormBuilder
}
if (isset($dateInput['H'])) {
$hour = $dateInput['H'];
- } elseif (isset($dateInput['h'])) {
- $hour = $dateInput['h'];
+ } elseif (isset($dateInput['h']) || isset($dateInput['g'])) {
+ if (isset($dateInput['h'])) {
+ $hour = $dateInput['h'];
+ } elseif (isset($dateInput['g'])) {
+ $hour = $dateInput['g'];
+ }
if (isset($dateInput['a'])) {
$ampm = $dateInput['a'];
} elseif (isset($dateInput['A'])) {

Now the g in _date2array() isn't strictly necessary because HTML_QuickForm_date::setValue() creates everything from a call to date() anyway.

[2006-12-07 07:04 UTC] dsanders at baselinesolutions dot com dot au

Changed from request to bug as this might catch other people out...