Numeric text formats are from D6.

CommentFileSizeAuthor
#5 views-n3013724-5.patch1.5 KBDamienMcKenna
#2 3013724-filter-format.patch821 bytesdjdevin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

djdevin created an issue. See original summary.

djdevin’s picture

djdevin
Location Philadelphia
CreditAttribution: djdevin at Cadmium commented
FileSize
3013724-filter-format.patch821 bytes
renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
Status: Needs review » Reviewed & tested by the community

Yeah, #2 applies normally and it really makes sense to me. So moving to RTBC

Thank you so much, @djdevin good catch

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented

Should it use filter_default_format() instead of hardcoding it?

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Reviewed & tested by the community » Needs review
FileSize
views-n3013724-5.patch1.5 KB

Instead of doing it this way, let's use the "area_text_custom" plugin so that the text filter doesn't matter. That's how other default views definitions do it.

  • DamienMcKenna committed 363144d on 7.x-3.x authored by djdevin 
    Issue #3013724 by djdevin, DamienMcKenna, RenatoG: Fix notice from old...
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs review » Fixed
Parent issue: » #3118500: Plan for Views 7.x-3.25

Committed. Thank you.

Status: Fixed » Closed (fixed)

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