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
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants