PEAR is discontinued

This is a static archive preserving historical releases and metadata. No new packages or updates will be published. For updates to existing packages, or to publish new packages, please use Composer and Packagist.

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...