Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

Zucchi/filter tweaks #2652

Closed
wants to merge 3 commits into from
Closed

Conversation

zucchi
Copy link
Contributor

@zucchi zucchi commented Oct 3, 2012

I have had to update the Decompress filter by changing the __invoke() method to filter() to match the API of the Compress filter and make it functional when used with the File Transfer component

I have also updated the the Tar Adapter as it was failing to interact properly with the "Archive_Tar" pear library causing it to fail. This included adding a lambda callback to handle PEAR errors. This doesn't however catch issues in the constructor.

@weierophinney
Copy link
Member

Removing __invoke() is a BC break, even if it was not correctly named. Reintroduce __invoke()``and have it proxy tofilter()`.

@ghost ghost assigned weierophinney Oct 3, 2012
@zucchi zucchi closed this Oct 4, 2012
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants