ZF-12526: Zend_Gdata::import() should be compatible with that of Zend_Gdata_App::import()

Issue Type:
Sub-task
Created:
2013-02-20T10:48:08.000+0000
Last Updated:
2013-04-05T16:06:59.000+0000
Status:
Closed
Fix version(s):
  • Next Mini Release ()
Reporter:
Marcin Gil (mgil)
Assignee:
Frank Brückner (frosch)
Tags:
  • Zend_Gdata
Related issues:
Attachments:

Description

Declaration of Zend_Gdata::import() should be compatible with that of Zend_Gdata_App::import() Zend/Gdata.php:45

The arguments do not match.

Comments

Posted by Frank Brückner (frosch) on 2013-02-20T11:51:11.000+0000

Proposals to fix the original problem are welcome. :)

Posted by Marcin Gil (mgil) on 2013-03-06T14:24:45.000+0000

This is my proposal, but I do not know how to attach it to the issue: https://dl.dropbox.com/u/11435743/…

Posted by Frank Brückner (frosch) on 2013-03-06T14:49:27.000+0000

@[~mgil] I will fix this. Thanks for your help!

Posted by Jari Turkia (hqjatu) on 2013-03-26T08:38:58.000+0000

Just to let ZF-people know: This also affects the latest version 1.12.3.

Posted by Frank Brückner (frosch) on 2013-03-26T11:50:49.000+0000

@Jari If the affected version is 1.12.1 and the issue is not fixed, then the problem is also included in version 1.12.3. ;)

Posted by Marco Kaiser (bate) on 2013-04-04T08:46:38.000+0000

to fix that, just add a missing parameter Gdata.php Line 123+124


    public static function import($uri, $client = null,
        $className='Zend_Gdata_Feed', $useObjectMapping = true)

Posted by Frank Brückner (frosch) on 2013-04-04T09:01:07.000+0000

@Marco Thanks for your comment. I will fix this at Github.

Posted by Ralph Schindler (ralph) on 2013-04-05T16:06:59.000+0000

This issue has been closed on Jira and moved to GitHub for issue tracking. To continue following the resolution of this issues, please visit: https://github.com/zendframework/zf1/issues/56