ZF-8631: Zend_File_Transfer documented as an instantiable class when it is not

Issue Type: Docs: Problem Created: 2009-12-25T11:29:53.000+0000 Last Updated: 2010-03-14T00:08:36.000+0000 Status: Resolved Fix version(s): - 1.10.3 (01/Apr/10)

Reporter: Pádraic Brady (padraic) Assignee: Thomas Weidner (thomas) Tags: - Zend_File_Transfer

Related issues: Attachments:


In the Reference Guide, many examples demonstrate instantiating an instance of Zend_File_Transfer. In reality, the class constructor contains code to throw an Exception with the message "Implementation in progress" if it is directly instantiated. Presumably, only adapters can be instantiated off the bat.


Posted by Ken Stanley (dohpaz) on 2009-12-30T05:01:16.000+0000

A few thoughts on this issue that I've observed:

  1. Since Zend_File_Transfer is not yet fully implemented, it should be moved to the incubator for development.
  2. The existing documentation should state very clearly, and in an unambiguous manner, that this functionality is still under development. - *** edit: My apologies, I just found the documentation that clearly states ZFT is not yet ready. :)
  3. The exception message in the constructor should be more clear and avoid using the word "progress". My first suggestion would be, "Zend_File_Transfer is not fully implemented at this time."

I will be more than happy to help contribute to this class and will be following up on as many tickets as I can. :)

Posted by Pádraic Brady (padraic) on 2009-12-30T05:48:20.000+0000

The first probably can't happen - it's a released component so removing it now would be a backwards compatibility issue. I think the docs could be updated and it'd be fine. Not all components can be finished on the spot so there's always leeway for completion and improvements after a release.

Posted by Thomas Weidner (thomas) on 2010-03-14T00:08:35.000+0000

Closing as fixed since more than 2 weeks (r21201).

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.