ZF-11883: Zend_Cloud_StorageService_Adapter_FileSystem::storeItem() should create intermediate folders

Issue Type:
Bug
Created:
2011-11-11T13:39:45.000+0000
Last Updated:
2011-11-11T18:11:35.000+0000
Status:
Open
Fix version(s):
    Reporter:
    David JM Emmett (davidjmemmett)
    Assignee:
    Matthew Weier O'Phinney (matthew)
    Tags:
    Related issues:
    Attachments:

    Description

    I am using the Zend_Cloud_StorageService as I am hoping to use the FileSystem adapter on internal servers and the S3 adapter on external servers.

    I think it is therefore wrong that Zend_Cloud_StorageService_Adapter_FileSystem::storeItem() requires that the path provided in the first argument already exists within the path specified when instantiating the class.

    I expect Zend_Cloud_StorageService_Adapter_FileSystem::storeItem() to behave the same as the S3 adapter, so that it automatically creates those intermediate folders. Therefore I can easily switch between implementations of Zend_Cloud_StorageService_Adapter.

    Comments

    Posted by David JM Emmett (davidjmemmett) on 2011-11-11T13:51:18.000+0000

    I can't seem to find where I attach files, so here's a patch[0] which fixes the issue for me.

    [0] - http://pastebin.com/g0pzkgPa

    Posted by David JM Emmett (davidjmemmett) on 2011-11-11T18:10:01.000+0000

    Ignore the last comment, this patch[0] includes unit tests as well as fix

    [0] - http://pastebin.com/SaMN9A30