Remove hook_menu item for SPARQL Registry view [#1222050]

Problem/Motivation

In #1213192: SPARQL Registry field integration unnecessary, the view function for SPARQL Registry entries was removed. However, there is still a menu path registered for viewing registry entries.

Proposed resolution

Remove the menu item for 'sparql_registry/%sparql_registry'. This is just code cleanup, it doesn't currently have any effect.

Remaining tasks

Needs work.

Comment	File	Size	Author
#2	hook_menu_item-path-cleanup-1222050-1.patch	858 bytes	acatejr

Comments

Comment #1

barbi commented 21 July 2011 at 04:25

Assigned:

Unassigned

» barbi

In the function sparql_registry_menu() there are 3 items for sparql_registry/%sparql_registry and 3 more for admin. Among the three related to sparql registry,

sparql_registry/%sparql_registry
sparql_registry/%sparql_registry/edit
sparql_registry/%sparql_registry/delete

I am assuming only the first item should be removed. Or should the other two items be removed too? The other two are of type MENU_LOCAL_TASK.

Comment #2

acatejr commented 20 September 2011 at 22:23

Status:

Active

» Needs review

Status	File	Size
new	hook_menu_item-path-cleanup-1222050-1.patch	858 bytes

I am a noob at trying to contribute to Drupal, so apologies for any mistakes.

To help with this fix all I did was remove the "sparql_registry/%sparql_registry/edit" array entry from the menu's items in the sparql_registry_menu hook.

I hope this helps.