Formatter asDate() return wrong number

[sobhan93]

\Yii::$app->formatter->asDate(time(), 'php:t') return current date.
I think formatter should return number of days in the current month.

echo \Yii::$app->formatter->asDate(time(), 'php:t')

What is the expected result?

31

What do you get instead?

20180719 09:49 PM

Additional info

Q	A
Yii version	2.0.15.1
PHP version	7.2.6
Operating system	Windows 7 + XAMPP

[samdark]

t in PHP indeed stands for "Number of days in the given month".

[bizley]

Not supported by ICU and already mentioned in code.

I guess this should work as equivalent

Yii::$app->formatter->asDate('last day of this month', 'd');

[rob006]

What is the reason of converting PHP date format to ICU?

[bizley]

To use features of IntlDateFormatter when intl is loaded. PHP date format works here as a fallback if intl is not available.

[rob006]

What features?

[bizley]

Here is implementation. Locale based date parsing.

[rob006]

Any example? I still can't imagine situation where I want to specify format using php: prefix and use IntlDateFormatter features.

[bizley]

Well, I agree with you.
@samdark was it made like this for consistency?

[samdark]

@cebe implemented it...

[cebe]

you can use the PHP format if you want, but you can not use every PHP formatting feature. Intl formatter is better in formatting dates locale-aware. The format conversion is used if you use jquery data picker and similar stuff on the frontend, some use php format, jquery even has its own format, so we allow specifying the format in different ways so that backend and frontend as well as validator use the same specification.

In the case of this issue there is no point in using the Yii formatter, if you only need the number of days in the current month, you can just use date('t'). The formatter is meant to be used to format dates for display to users. t does not fit into this use case as far as I see.

[rob006]

@cebe It still does not explain what is the point of converting PHP format to ICU. If someone wants to use IntlDateFormatter features, he will not use php: prefix.
And t is only one of many unsupported formats that does not work because of current implementation.

[sobhan93]

I need the number of days in the persian calendar.

'formatter' => [
    'locale' => 'fa_IR@calendar=persian',
    'calendar' => \IntlDateFormatter::TRADITIONAL,
],
\Yii::$app->formatter->asDate(time(), 'php:t')

[sobhan93]

@cebe
@samdark

[cebe]

I need the number of days in the persian calendar.

it's simply not supported by intl. Not sure there is a way to add it. If anyone has ideas about how we can add it feel free to propose ideas.

[samdark]

Won't be implemented in 2.0.