Problem/Motivation

After installing the Aggregator module (clear cache or not) the appropriate menu items do not appear on the Configuration page for configuring Aggregator. (See screenshot)

Proposed Solution

Add links/appropriate callbacks to the Aggregator module so it gets picked up when the Configuration page is generated.

Remaining Tasks

TBD

User Interface Changes

A new link item needs to be on the admin/config page under the Web Services box that leads to admin/config/services/aggregator.

API Changes

N/A

CommentFileSizeAuthor
#5 2209015-aggregator-admin-link-5-FAIL.patch1.64 KBlongwave
#5 2209015-aggregator-admin-link-5-PASS.patch2.25 KBlongwave
#2 fix-aggregator-config-link-2209015-2.patch627 bytesawochna
#2 proposed.png114.66 KBawochna
current.png115.23 KBawochna
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

awochna’s picture

awochna CreditAttribution: awochna commented
Assigned: Unassigned » awochna
awochna’s picture

awochna CreditAttribution: awochna commented
FileSize
proposed.png114.66 KB
fix-aggregator-config-link-2209015-2.patch627 bytes

Uploading patch that should fix the issue. Adding screenshot of page after fix; in the proposed state.

awochna’s picture

awochna CreditAttribution: awochna commented
Status: Active » Needs review
ParisLiakos’s picture

ParisLiakos CreditAttribution: ParisLiakos commented
Priority: Normal » Major
Issue tags: +Needs tests

thank you for the patch!
lets add a test assertion so we make sure it doesnt break again in the future

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave commented
Issue tags: -Needs tests
FileSize
2209015-aggregator-admin-link-5-PASS.patch2.25 KB
2209015-aggregator-admin-link-5-FAIL.patch1.64 KB

Status: Needs review » Needs work

The last submitted patch, 5: 2209015-aggregator-admin-link-5-FAIL.patch, failed testing.

The last submitted patch, 5: 2209015-aggregator-admin-link-5-FAIL.patch, failed testing.

ParisLiakos’s picture

ParisLiakos CreditAttribution: ParisLiakos commented
Status: Needs work » Reviewed & tested by the community

thanks that was quick

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

Status: Fixed » Closed (fixed)

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