[Home]

Summary:ASTERISK-27124: app_playback.c:say_date_generic use timezonename parameter
Reporter:Holger Hans Peter Freyther (zecke)Labels:
Date Opened:2017-07-11 04:51:33Date Closed:2017-08-16 06:49:32
Priority:MajorRegression?
Status:Closed/CompleteComponents:Applications/app_playback
Versions:11.25.1 13.17.0 14.6.0 Frequency of
Occurrence
Constant
Related
Issues:
Environment:Attachments:
Description:say_date_generic receives a timezonename parameter but is not using it for ast_localtime. This is a problem when operating the system for users in different timezones.
Comments:By: Asterisk Team (asteriskteam) 2017-07-11 04:51:34.596-0500

Thanks for creating a report! The issue has entered the triage process. That means the issue will wait in this status until a Bug Marshal has an opportunity to review the issue. Once the issue has been reviewed you will receive comments regarding the next steps towards resolution.

A good first step is for you to review the [Asterisk Issue Guidelines|https://wiki.asterisk.org/wiki/display/AST/Asterisk+Issue+Guidelines] if you haven't already. The guidelines detail what is expected from an Asterisk issue report.

Then, if you are submitting a patch, please review the [Patch Contribution Process|https://wiki.asterisk.org/wiki/display/AST/Patch+Contribution+Process].

By: Holger Hans Peter Freyther (zecke) 2017-07-11 04:53:32.437-0500

I got a trivial patch and wait for the CLA to be accepted before uploading it to Gerrit.

By: Rusty Newton (rnewton) 2017-07-12 14:38:40.487-0500

Thanks. I'll open up the issue. Once you submit to Gerrit the issue will be updated on the Gerrit Reviews tab.

By: Rusty Newton (rnewton) 2017-07-12 14:39:37.652-0500

Be sure to edit the Affects Version field to reflect the relevant versions.

By: Holger Hans Peter Freyther (zecke) 2017-07-12 16:23:43.515-0500

The issue seems to exist since app_playback.c was added. We have seen it with Asterisk11 but it might exist for longer. I have selected 11, 13 and 14. We require the fix to be in Asterisk14.

https://gerrit.asterisk.org/#/c/6006/ and https://gerrit.asterisk.org/#/c/6005/ have been pushed into gerrit.