Problem/Motivation

Document.getSummary() trims the summary to 100 characters, this seems a little low give it's likely the responsibility of the sapi index to select suitable contents to display as a summary.

Proposed resolution

Increase the length to something higher, 400 seems ok.

Remaining tasks

Code, review, commit

User interface changes

NA

API changes

NA

Data model changes

NA

Release notes snippet

NA

CommentFileSizeAuthor
#6 3159226-6.patch35.98 KBfenstrat
#5 3159226-5.patch35.21 KBfenstrat
#2 3159226-2.patch33.05 KBfenstrat
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

fenstrat created an issue. See original summary.

fenstrat’s picture

fenstrat
he/him
Primary language English
Location Australia
CreditAttribution: fenstrat at PreviousNext for amaysim Australia Ltd. commented
Status: Active » Needs review
FileSize
3159226-2.patch33.05 KB

  • fenstrat committed 0beed11 on 8.x-1.x 
    Issue #3159226 by fenstrat: Increase trimmed length in getSummary
fenstrat’s picture

fenstrat
he/him
Primary language English
Location Australia
CreditAttribution: fenstrat at PreviousNext for amaysim Australia Ltd. commented
Status: Needs review » Fixed
fenstrat’s picture

fenstrat
he/him
Primary language English
Location Australia
CreditAttribution: fenstrat at PreviousNext for amaysim Australia Ltd. commented
Status: Fixed » Needs review
FileSize
3159226-5.patch35.21 KB

Further tweak, shortening length to 250 and adding ... if we trim it.

fenstrat’s picture

fenstrat
he/him
Primary language English
Location Australia
CreditAttribution: fenstrat at PreviousNext for amaysim Australia Ltd. commented
FileSize
3159226-6.patch35.98 KB

  • fenstrat committed 4ff5174 on 8.x-1.x 
    Issue #3159226 by fenstrat: Change getSummary length and addition of...
fenstrat’s picture

fenstrat
he/him
Primary language English
Location Australia
CreditAttribution: fenstrat at PreviousNext for amaysim Australia Ltd. commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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