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

Provide a formatter which limits and provides offsets #5

Open
wants to merge 1 commit into
base: 8.x-1.x
Choose a base branch
from
Open

Provide a formatter which limits and provides offsets #5

wants to merge 1 commit into from

Conversation

dawehner
Copy link
Contributor

https://www.drupal.org/node/1234624#comment-11375205 provides a patch to limit/offset deltas of a filter using a patch for the base class, but in the scheme of field_formatter we could also provide a wrapper formatter, which does exactly that.

@@ -14,4 +14,5 @@ function field_formatter_field_formatter_info_alter(array &$info) {
$field_types = array_keys($field_type_manager->getDefinitions());

$info['field_link']['field_types'] = $field_types;
$info['field_formatter_limite']['field_types'] = $field_types;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"limite" is a typo I guess

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ups you are right.

As always I should have written some tests before :)

On Sun, Jul 10, 2016 at 2:30 PM, marcvangend [email protected]
wrote:

In field_formatter.module
#5 (comment)
:

@@ -14,4 +14,5 @@ function field_formatter_field_formatter_info_alter(array &$info) {
$field_types = array_keys($field_type_manager->getDefinitions());

$info['field_link']['field_types'] = $field_types;

  • $info['field_formatter_limite']['field_types'] = $field_types;

"limite" is a typo I guess


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/drupal-media/field_formatter/pull/5/files/ada2ec3a0bfcdc06ba9dbbccba586a64e557bfa2#r70181179,
or mute the thread
https://github.com/notifications/unsubscribe/AABz7oxlvRFsvyPm5fy6D58acyoSPjD3ks5qUOX5gaJpZM4JIyGi
.

@AdamPS
Copy link

AdamPS commented Oct 18, 2017

Great idea, would like to see this one

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.

4 participants