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:


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

The arguments do not match.


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:…

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

<pre class="highlight">
    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:

Have you found an issue?

See the Overview section for more details.


© 2006-2018 by Zend, a Rogue Wave Company. Made with by awesome contributors.

This website is built using zend-expressive and it runs on PHP 7.