Hi,

"Number" type variables are wrapped in < pre > tags when used as variables because they don't set a format callback. It seems to me that we can just reuse variable_format_string here. See also: #1559674: String type variables being wrapped with <pre></pre> tags when displayed with tokens

Patch to follow.

Thanks,
Pete

CommentFileSizeAuthor
#2 1937386-format_numeric_variables-02.patch1.01 KBJose Reyero
#1 variable-add-format-for-numbers-1937386-1.patch505 bytesPete B
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Pete B’s picture

Pete B CreditAttribution: Pete B commented
Status: Active » Needs review
FileSize
variable-add-format-for-numbers-1937386-1.patch505 bytes

Patch!

Jose Reyero’s picture

Jose Reyero CreditAttribution: Jose Reyero commented
FileSize
1937386-format_numeric_variables-02.patch1.01 KB

Right, we need some formatter, though that's not the best solution, will fail for 0 (zero) value, etc......

I think we should add something better, give a try to this one.

Jose Reyero’s picture

Jose Reyero CreditAttribution: Jose Reyero commented
Status: Needs review » Fixed

Committed, I want to put out a new release soon.

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) CreditAttribution: commented
Issue summary: View changes

Made the pre tag visible