For whatever reason I haven't found, every settings in /admin/config/services/aggregator/settings works as expected but teaser lenght not.

I mean, when I open the settings page, the option shows the default 600 characters, I set to it to any other value, Save Configuration and when page reload, it fall back to 600 characters.
I tried to delete the aggregator_teaser_length in the variable db table without any success.

I am using Drupal 7 with Nginx + PHP-FPM on a Ubuntu Natty x86.

Value of aggregator_teaser_length is always at:

INSERT INTO `variable` (`name`, `value`) VALUES
('aggregator_teaser_length', 0x733a333a22323030223b);

Comments

aron novak’s picture

Version: 7.9 » 8.x-dev
Status: Active » Needs review
StatusFileSize
new833 bytes

Here is the fix.
Should be committed for 7.x and 8.x also, trivial bugfix.

larowlan’s picture

Status: Needs review » Reviewed & tested by the community

Nice easy one

chx’s picture

Status: Reviewed & tested by the community » Needs work

Thanks for working on this patch. It could use an automated test before it is committed. If you need help you can ask the friendly people in the #drupal-contribute IRC channel.

Désiré’s picture

Status: Needs work » Needs review
StatusFileSize
new1.96 KB
new1.14 KB

Here some tests. (btw aggregator module has no tests for the settings page yet)

Désiré’s picture

StatusFileSize
new2.05 KB

some code comments

droplet’s picture

Status: Needs review » Closed (duplicate)

some further question, see #3: #1045786: Length of Trimmed Description