Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
#1798880: Convert teaser_length to use configuration system changes the name teaser_length
to default_summary_length
For consistency should the name be changed in aggregator too?
Comment | File | Size | Author |
---|---|---|---|
#1 | default-summary-length-1830068-1.patch | 4.85 KB | LinL |
Comments
Comment #1
LinL CreditAttribution: LinL commentedHere's a patch.
Comment #2
sunWhy 'default_' ?
Comment #3
LinL CreditAttribution: LinL commentedFrom comment #3 of the original issue #1798880: Convert teaser_length to use configuration system where the name was changed:
But maybe it should be just "summary_length"?
Comment #4
sunGiven that aggregator items are entities now, I wonder whether and why these configuration options still exist in the first place?
Why aren't we simply using field formatter settings?
Comment #5
ParisLiakos CreditAttribution: ParisLiakos commentedi think we still need this, unless we expose the field UI for aggregator items somewhere and turn description to fields, so we keep it configurable from UI.
But i also think there is no need for the
default_
prefixComment #6
ParisLiakos CreditAttribution: ParisLiakos commented