Problem/Motivation

When accessing the settings page for Mandrill, a php error is thrown due to an undocumented dependency on dblog.

Proposed resolution

The error is caused by a field description linking through to the error log page. Simply removing this link will resolve the error.

Remaining tasks

Attach patch (coming soon).

User interface changes

A direct link to the error log page will no longer be present in the settings page.

Pantheon is proud to support Drupal and open source. Pantheon

Comments

merauluka created an issue. See original summary.

merauluka’s picture

Title: PHP error due on settings page when dblog is disabled » PHP error on settings page when dblog is disabled
FileSize
5.04 KB

Since I was already updating the file, I went ahead and switched out the deprecated l() function called to use Link::fromTextAndUrl() instead.

DamienMcKenna’s picture

Status: Active » Needs review
rbayliss’s picture

Status: Needs review » Reviewed & tested by the community

This looks good. Fixes the issue and removes some deprecated code as well.

samuel.mortenson’s picture

Assigned: Unassigned » rjacobsen0
knyshuk.vova’s picture

The patch looks good and applies successfully. +1 for RTBC.

rjacobsen0’s picture

rjacobsen0’s picture

So far I have not been able to reproduce the original error. Was it fixed in another ticket??? Is there a setting I'm missing?

joncjordan’s picture

rjacobsen0 to reproduce the error, disable dblog module then visit /admin/config/services/mandrill