ZF-6270: The API for Zend_Translate_Adapter :: _loadTranslationData() has changed and is not documented.

Issue Type:
Bug
Created:
2009-04-11T02:34:03.000+0000
Last Updated:
2009-04-11T08:13:29.000+0000
Status:
Resolved
Fix version(s):
  • 1.8.0 (30/Apr/09)
Reporter:
purple ankh (purpleankh)
Assignee:
Thomas Weidner (thomas)
Tags:
  • Zend_Translate
Related issues:
Attachments:

Description

Using ZendFramework 1.8.0PR (alpha1)

The API for _loadTranslationData() has changed from returning nothing (void) to returning something. This change is not documented anywhere and breaks any defined classes which extend Zend_Translate_Adapter, for example Database-based translation.

Any class still returning nothing generates the following warnings:

Warning: array_keys() [function.array-keys]: The first argument should be an array in /home/purpleankh/project/ZendFramework-1.8.0PR/library/Zend/Translate/Adapter.php on line 467

Warning: Invalid argument supplied for foreach() in /home/purpleankh/project/ZendFramework-1.8.0PR/library/Zend/Translate/Adapter.php on line 468

If this API change is intentional, please document what needs to be returned from the method.

Comments

Posted by Thomas Weidner (thomas) on 2009-04-11T08:13:17.000+0000

API doc fixed with r14845

Note: The behaviour of translation has not changed to before. The notice has no effect in the way the translaiton works.