In-Portal Issue Tracker - In-Portal CMS
Viewing Issue Advanced Details
507 [In-Portal CMS] Data Management bug report always 2009-12-25 10:54 2010-05-11 02:59
alex  
alex  
normal  
closed 5.0.2-B2  
fixed  
 
none 5.0.3-B1  
http://groups.google.com/group/in-portal-bugs/browse_thread/thread/79c9db452461236a#
0
0000507: Time without date won't get validated, when kDateFormatter is used
In case if you have field, that is defined using mentioned above code, then it's never gonna be validated (combination of input_date_format and required options) - HeatingTime_date field will always be empty and of course required.

'Fields' => Array(
    'HeatingTime' => Array (
        'type' => 'int',
        'formatter' => 'kDateFormatter', 'date_format' => '', 'input_date_format' => '', 'time_format' => 'i:s', 'input_time_format' => 'i:s',
        'required' => 1,
    ),
),
related to 0000629closed  (5.1.2)alex Can't use custom Input Date/Time format for fields 
patch date_required_error_fix.patch (1,469) 2009-12-25 10:54
http://tracker.in-portal.org/file_download.php?file_id=196&type=bug
png 507-Time-without-date.png (61,283) 2010-03-15 18:24
http://tracker.in-portal.org/file_download.php?file_id=373&type=bug
png
Issue History
2010-05-11 02:59 alex Note Added: 0002116
2010-05-11 02:59 alex Status resolved => closed
2010-03-20 14:39 alex Issue Monitored: Dmitry
2010-03-20 14:39 alex Note Added: 0001710
2010-03-20 14:37 alex Note Added: 0001709
2010-03-20 14:37 alex Status reviewed and tested => resolved
2010-03-20 14:37 alex Fixed in Version => 5.0.3-B1
2010-03-20 14:37 alex Resolution open => fixed
2010-03-20 14:37 alex Changeset attached 5.0.x r13271
2010-03-20 14:36 alex Developer Dmitry => alex
2010-03-19 11:01 Dmitry Note Added: 0001703
2010-03-19 11:01 Dmitry Assigned To Dmitry => alex
2010-03-19 11:01 Dmitry Status needs testing => reviewed and tested
2010-03-19 11:01 Dmitry Developer alex => Dmitry
2010-03-19 11:01 Dmitry Status needs work => needs testing
2010-03-19 11:01 Dmitry Status needs feedback => needs work
2010-03-19 11:00 Dmitry Relationship added related to 0000629
2010-03-19 08:23 alex Note Added: 0001697
2010-03-19 08:23 alex Assigned To alex => Dmitry
2010-03-19 08:23 alex Status needs work => needs feedback
2010-03-15 18:25 Dmitry Note Added: 0001641
2010-03-15 18:25 Dmitry Status reviewed and tested => needs work
2010-03-15 18:25 Dmitry Note Added: 0001640
2010-03-15 18:25 Dmitry Assigned To !COMMUNITY => alex
2010-03-15 18:25 Dmitry Status needs testing => reviewed and tested
2010-03-15 18:24 Dmitry File Added: 507-Time-without-date.png
2010-02-26 18:59 Dmitry Developer => alex
2010-01-06 06:17 alex Target Version Icebox => 5.0.3
2009-12-25 10:56 alex Assigned To => !COMMUNITY
2009-12-25 10:56 alex Status active => needs testing
2009-12-25 10:56 alex Reference => http://groups.google.com/group/in-portal-bugs/browse_thread/thread/79c9db452461236a#
2009-12-25 10:56 alex Target Version => Icebox
2009-12-25 10:54 alex New Issue
2009-12-25 10:54 alex File Added: date_required_error_fix.patch

Notes
(0001640)
Dmitry   
2010-03-15 18:25   
See attachment.

I have used the following field definition:

'BirthTime' => Array (
                'type' => 'int',
                'formatter' => 'kDateFormatter', 'date_format' => '', 'input_date_format' => '', 'time_format' => 'i:s', 'input_time_format' => 'i:s',
                'required' => 1,
            ),
(0001641)
Dmitry   
2010-03-15 18:25   
Actually it needs work, see prev. comment.
(0001697)
alex   
2010-03-19 08:23   
That's no bug of this task. This bug related to inability to use custom input date/time format for fields, when we are using inp_edit_date/inp_edit_date_time/inp_edit_time blocks. Please remove "*time*" formats and try again.
(0001703)
Dmitry   
2010-03-19 11:01   
Tested Okay this patch and created a separate task for the other bug.
(0001709)
alex   
2010-03-20 14:37   
Fix committed to 5.0.x branch. Commit Message:

Fixes 0000507: Time without date won't get validated, when kDateFormatter is used
(0001710)
alex   
2010-03-20 14:39   
Reminder sent to: Dmitry

When setting "Needs Testing" status of task, then Developer is changed to currently logged in user. I've change actual developer back.
(0002116)
alex   
2010-05-11 02:59   
Closing all tasks from released versions.