Download & Extend
Issues

Cannot set aggregator teaser length in settings tab.

Posted by gionn on October 27, 2011 at 8:24pm
Project:Drupal core
Version:8.x-dev
Component:aggregator.module
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (duplicate)

Issue Summary

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);
Login or register to post comments

Comments

#1

Posted by Aron Novak on October 29, 2011 at 4:23pm
Version:7.9» 8.x-dev
Status:active» needs review

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

AttachmentSizeStatusTest resultOperations
1324102_aggregator_teaser_len-1.patch833 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 33,754 pass(es).View details | Re-test

#2

Posted by larowlan on October 29, 2011 at 8:42pm
Status:needs review» reviewed & tested by the community

Nice easy one

#3

Posted by chx on October 29, 2011 at 11:50pm
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.

#4

Posted by Désiré on October 31, 2011 at 3:32pm
Status:needs work» needs review

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

AttachmentSizeStatusTest resultOperations
1324102_aggregator_teaser_len-4-test_only.patch1.14 KBIdleFAILED: [[SimpleTest]]: [MySQL] 33,771 pass(es), 1 fail(s), and 0 exception(es).View details | Re-test
1324102_aggregator_teaser_len-4.patch1.96 KBIdlePASSED: [[SimpleTest]]: [MySQL] 33,780 pass(es).View details | Re-test

#5

Posted by Désiré on October 31, 2011 at 4:49pm

some code comments

AttachmentSizeStatusTest resultOperations
1324102_aggregator_teaser_len-5.patch2.05 KBIdlePASSED: [[SimpleTest]]: [MySQL] 33,773 pass(es).View details | Re-test

#6

Posted by droplet on October 31, 2011 at 4:52pm
Status:needs review» closed (duplicate)

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