Feature #36783

Filename filters: pass additional information

Added by Oliver Hader over 7 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
File Abstraction Layer (FAL)
Target version:
Start date:
2012-05-02
Due date:
% Done:

100%

PHP Version:
5.3
Tags:
Complexity:
Sprint Focus:

Description

The file/folder name filter methods now get an additional parameter, $additionalInformation, that can be filled by the driver as necessary.
It could e.g. contain additional information fetched from the storage, that might be relevant for a filter.

Associated revisions

Revision 18259d39 (diff)
Added by Andreas Wolf over 7 years ago

[!!!][FEATURE] Filename filters: pass additional information

The file/folder name filter methods now get an additional
parameter, $additionalInformation, that can be filled by
the driver as necessary. It could e.g. contain additional
information fetched from the storage, that might be relevant
for a filter.

This change requires adding this parameter in all filters, so it is
considered a breaking API change. Apart from that, driver authors have
to adjust their driver if they use the dispatch method from
AbstractDriver, as the parameter order changed there.

Change-Id: I24b43cc02c0bf79fb637b51081f65ef3321c5b99
Resolves: #36783
Releases: 6.0
Reviewed-on: http://review.typo3.org/10932
Reviewed-by: Tolleiv Nietsch
Tested-by: Tolleiv Nietsch

Revision 76effb8d (diff)
Added by Andreas Wolf over 7 years ago

[!!!][FEATURE] Filename filters: pass additional information

The file/folder name filter methods now get an additional
parameter, $additionalInformation, that can be filled by
the driver as necessary. It could e.g. contain additional
information fetched from the storage, that might be relevant
for a filter.

This change requires adding this parameter in all filters, so it is
considered a breaking API change. Apart from that, driver authors have
to adjust their driver if they use the dispatch method from
AbstractDriver, as the parameter order changed there.

Change-Id: I24b43cc02c0bf79fb637b51081f65ef3321c5b99
Resolves: #36783
Releases: 6.0
Reviewed-on: http://review.typo3.org/10932
Reviewed-by: Tolleiv Nietsch
Tested-by: Tolleiv Nietsch

History

#1 Updated by Gerrit Code Review over 7 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10932

#2 Updated by Gerrit Code Review over 7 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10932

#3 Updated by Gerrit Code Review over 7 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10932

#4 Updated by Anonymous over 7 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#5 Updated by Riccardo De Contardi almost 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF