#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?

Files: 
CommentFileSizeAuthor
#1 default-summary-length-1830068-1.patch4.85 KBLinL
PASSED: [[SimpleTest]]: [MySQL] 46,490 pass(es).
[ View ]

Comments

Status:Active» Needs review
StatusFileSize
new4.85 KB
PASSED: [[SimpleTest]]: [MySQL] 46,490 pass(es).
[ View ]

Here's a patch.

Component:configuration system» aggregator.module

Why 'default_' ?

From comment #3 of the original issue #1798880: Convert teaser_length to use configuration system where the name was changed:

IMHO, more suitable name for this config is 'default_summary_length', because:
1. It is summary length
2. It is default, because this setting is given when Drupal has no any information about "text field with summary" summary length. This happens when user create a "text field with summary" and didn't setup its widget settings. So, if administrator want, he can update this 'default_summary_length' config, so any new and not-configured "text field with summary" fields will use it.

But maybe it should be just "summary_length"?

Given 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?

i 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_ prefix

Issue summary:View changes
Status:Needs review» Closed (duplicate)
Related issues:+#2149845: Convert the description field of the 'aggregator_item' entity to a text field