The Acquia Search module alters the form so that the delete link is not present and also implements hook_menu_alter() to change the access callback which prevents the environment from being deleted. The problem is, it is overriding a non-existent path. Specifically, is uses "apachesolr-env" in the path, where it should be "apachesolr/settings".

CommentFileSizeAuthor
#1 delete-access-1785208-1.patch774 bytescpliakas
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cpliakas’s picture

Status: Active » Needs review
FileSize
774 bytes

The attached patch updates the hook to reflect the correct path.

coltrane’s picture

Status: Needs review » Reviewed & tested by the community

Confirmed and makes test from #1784990: Revive unit tests for Acquia Search pass.

Nick_vh’s picture

Project: Acquia Connector » Acquia Search
Version: 7.x-2.x-dev » 6.x-3.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Makes perfect sense. Committed and thanks!

Nick_vh’s picture

Status: Patch (to be ported) » Closed (works as designed)

No backport needed for 6.x-3.x - works as expected

Nick_vh’s picture

Issue summary: View changes

Updated issue summary.