Hi everyone,

For a field that has multiple values in the "Multiple Field Settings" group if "Display all value(s) starting from 1" is set to "all" then all values show. It doesn't skip the number of items in the "Starting From" box.

If you change "all" to say 16 then it works as expected and shows the last 15 items skipping the first if Starting From = 1. See attached screenshot for settings.

Thanks for reading.
-Tim

CommentFileSizeAuthor
#5 1147054-delta-limit-all.patch1.24 KBmerlinofchaos
Screen shot 2011-05-04 at May, 4 10.01.02 AM.png30.9 KBTimG1
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented

There's kind of a disconnect here, because from one perspective, 'all' values means all of them. Need to think about which definition of 'all' is going to be the least confusing.

TimG1’s picture

TimG1 CreditAttribution: TimG1 commented

I see where you're coming from Merlin.

To me it actually makes sense reading it like you have it though.

"Display all value(s) starting from 10" seems clear to me that it will show all values after the 10th item. I dunno, that was my expectation when I tried it anyway.

-Tim

TimG1’s picture

TimG1 CreditAttribution: TimG1 commented

Actually maybe just reword to say:

"Display all value(s) after item 0 (first item is 0)"

"Display all value(s) after item 10 (first item is 0)"

"Display 5 value(s) after item 10 (first item is 0)"

A little less confusing, permaybehaps?

-Tim

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Status: Active » Needs review
FileSize
1147054-delta-limit-all.patch1.24 KB

I think the wording is fine as is and we should update the code to match expectation.

Can you try this patch?

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.