Date showing on node view - not showing on node edit - PHP 5.4 issue

This is my exact same issue.

In my logs I get "Illegal string offset 'month' in /.../includes/form.inc on line 1345"

Same error applies for second, hour, day, year...

use this to fix 1345 http://drupalcontrib.org/api/drupal/contributions%21date%21date_popup%21...
but it do not fix echo date time and time

how to looks date and time in page node edit , drupal 6, php 5.4.6
do this:
1st of all fix you code similarity http://drupalcontrib.org/api/drupal/contributions%21date%21date_popup%21...

all changes in date/date_popup/date_popup.module

1. in function date_popup_process_date change

'#default_value' =>(!empty($element['#value']['date']) || !empty($edit['date'])) && is_object($date) ? date_format_date($date, 'custom', $date_format) : '',

change to

date_format_date($date, 'custom', $date_format) ,

next step

if (!empty($element['#force_value'])) {$sub_element['#value'] = $sub_element['#default_value']; } 

change to

$sub_element['#value'] = $sub_element['#default_value'];

2. in function date_popup_process_time change

'#default_value' =>(!empty($element['#value']['time']) || !empty($edit['time'])) && is_object($date) ? date_format_date($date, 'custom', $time_format) : '',
 

change to

date_format_date($date, 'custom', $time_format) ,

and

 if (!empty($element['#force_value'])) {$sub_element['#value'] = $sub_element['#default_value']; } 

change to

$sub_element['#value'] = $sub_element['#default_value'];.

This fix didn't work for me. When I use date and time too after a content type import the dates and times are empty.

Subscribing [#4] fixes the problem for me, actually just taking out the conditional

(!empty($element['#value']['time']) || !empty($edit['time'])) && is_object($date) ? 

works like a charm :)

Interesting thing,
$element['#value'] = "2013-01-01";

In PHP 5.3:
empty($element['#value']['date']) ? => true

PHP 5.4:
empty($element['#value']['date']) ? => false

Thanks!

All you need to do is remove the ['time'] from $element['#value']['time'] in date_popup_process_date and date_popup_process_time

Works for me in php 5.4.10 and 5.2.17.

Patch included -- built against 6.x-2.9+6-dev

After upgrading to PHP 5.4.10 from PHP 5.2.x, I had the issue when editing the date fields that used the date popup were empty. Patched 2 sites Drupal 6 sites running on PHP 5.4.10 and date fields are populated again.

Others please review and test.

Thank you fellow Drupalers!

#8 works perfect.

#8 works perfect for me too!

#8 works fo me, please send it to the dev version

#8 working here

#8: date_popup.module.patch queued for re-testing.

#8 Worked for me, thanks.

Patch worked for me.

I had to use the patch command without the -p1 to get it to work for me.

I've seen patches sit in "needs work" because of a failed test more than once.

It'd guess the test failed because the test uses the -p1 option

git apply --check -p1 /var/lib/drupaltestbot/sites/default/files/review/date_popup.module.patch

Rerolled #8

Adding in text for search purposes

Warning: Illegal string offset 'year' in _form_set_value() (line 1345 of includes/form.inc).
Warning: Illegal string offset 'month' in _form_set_value() (line 1345 of includes/form.inc).
Warning: Illegal string offset 'day' in _form_set_value() (line 1345 of includes/form.inc).
Warning: Illegal string offset 'hour' in _form_set_value() (line 1345 of includes/form.inc).
Warning: Illegal string offset 'minute' in _form_set_value() (line 1345 of includes/form.inc).
Warning: Illegal string offset 'second' in _form_set_value() (line 1345 of includes/form.inc).

#19 applied cleanly to 6.x-2.9 for me and allowed the date values to render on node edit screens.

+1 RTBC

densolis assisted me to find out that the test is not running due to the 6.x-2.x branch not passing testing. In order to move forward either the patch need to be manually reviewed or a maintainer needs to fix up the branch so the patch can be automatically tested.

19 worked.

Thanks a lot.

+1 on the RTBC... this fixed the issue on my site.

Patch applied and pushed to 6.x-2.x. Thank you!

I've applied the patch and I've also upgraded to 6.x-2.x but I am still experiencing this issue. Anyone else?

interestingaftermath, try dev version.

Same issue here , with 6.x-2.10 and 6.x-2.x-dev.