Zend Framework

Forced 'de' locale when invoking _calcdetail() function.

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Not an Issue
  • Affects Version/s: 1.10.0
  • Fix Version/s: 1.10.1
  • Component/s: Zend_Date
  • Labels:
    None

Description

In line 2940 we have following code:

$return = $this->_calcdetail($calc, $date, self::DATE_MEDIUM, 'de');

The valid code should look as follows:

$return = $this->_calcdetail($calc, $date, self::DATE_MEDIUM, $locale);

Activity

Hide
Thomas Weidner added a comment -

Please tell me what your exact problem is.

The given code works as expected and sets the date within the date object regardless of the used locale. I will not change working code from core without a reproducable problem.

Show
Thomas Weidner added a comment - Please tell me what your exact problem is. The given code works as expected and sets the date within the date object regardless of the used locale. I will not change working code from core without a reproducable problem.
Hide
Thomas Weidner added a comment -

Closing as non-issue.

Note that the mentioned method normalizes any given input to a defined locale. This way Zend_Date know where the content should be.

It would be nonsense to normalize by localizing content. Then day, month or year could be switched and it would not be possible to know which part exactly is what it has been said to be.

Show
Thomas Weidner added a comment - Closing as non-issue. Note that the mentioned method normalizes any given input to a defined locale. This way Zend_Date know where the content should be. It would be nonsense to normalize by localizing content. Then day, month or year could be switched and it would not be possible to know which part exactly is what it has been said to be.
Hide
Adrian Ziolkowski added a comment -

You are totally right. I misunderstand this part of code.

Show
Adrian Ziolkowski added a comment - You are totally right. I misunderstand this part of code.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: