to duplicate :

Feed aggregator / settings

Number of items shown in listing pages :
- None -

seems to work, i get my categories' title only with "More" and i can get more

but i get also this message:
Notice: Undefined variable: summary_items in aggregator_page_categories() (line 376 of /***/www/modules/aggregator/aggregator.pages.inc).

twice at the head of the page;

tested in IE 9 , Chrome 22, Firefox 16.0.2

PS : works without any problem with :
Number of items shown in listing pages :
3

PS 2: found
$items = drupal_map_assoc(array(3, 5, 10, 15, 20, 25), '_aggregator_items');
line 75 of drupal/modules/aggregator/aggregator.processor.inc
but ... :-((

CommentFileSizeAuthor
#13 1830282-13.patch622 bytesrpayanm
#6 1830282-6.diff642 bytesAnonymous (not verified)
#1 1830282-1.diff642 bytesAnonymous (not verified)

Comments

corbin’s picture

Issue summary: View changes

2 precisions (PS) added

Anonymous’s picture

Version: 7.16 » 8.x-dev
Status: Active » Needs review
Issue tags: +Needs backport to D7
StatusFileSize
new642 bytes

Here is the patch for D8.

Status: Needs review » Needs work

The last submitted patch, 1830282-1.diff, failed testing.

Anonymous’s picture

Testbot is having issues this afternoon. Try a retest later.

webflo’s picture

Status: Needs work » Needs review
Issue tags: -Needs backport to D7

#1: 1830282-1.diff queued for re-testing.

Status: Needs review » Needs work
Issue tags: +Needs backport to D7

The last submitted patch, 1830282-1.diff, failed testing.

Anonymous’s picture

Status: Needs work » Needs review
StatusFileSize
new642 bytes

Same patch, just trying to get testbot happy.

Status: Needs review » Needs work

The last submitted patch, 1830282-6.diff, failed testing.

Anonymous’s picture

Status: Needs work » Needs review

Setting back to needs review after yesterday's testbot snafu.

aspilicious’s picture

Issue tags: +Needs tests

This also needs a test.

ParisLiakos’s picture

Version: 8.x-dev » 7.x-dev
Issue tags: -Needs tests

this got fixed now on D8, in the controller conversion

ParisLiakos’s picture

Issue tags: +Needs tests

..re adding tag

ParisLiakos’s picture

Issue summary: View changes

oops ... mistake corrected

mgifford’s picture

Issue summary: View changes
Status: Needs review » Needs work
Issue tags: +Needs reroll

Patch no longer applies.

rpayanm’s picture

Status: Needs work » Needs review
StatusFileSize
new622 bytes
mgifford’s picture

Issue tags: -Needs reroll
dcam’s picture

Assigned: corbin » Unassigned
Issue tags: +Needs issue summary update

Would someone who understands what the bug is mind updating the issue summary with instructions for replicating the issue that are more clear? I would be happy to test the patch, but I can't understand what the problem is or what I should do.

mgifford’s picture

Status: Needs review » Postponed (maintainer needs more info)

I tried to replicate it but couldn't.

Status: Postponed (maintainer needs more info) » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.