Hi,

On the status page (/admin/reports/status) when your Apache isnt public acessible this module redifines the already defined 'http requests' statys key and it then is turned into a array. (check attached image)

In function linkchecker_requirements($phase)

    case 'runtime':
      // Module cannot work without external HTTP requests.
      if (variable_get('drupal_http_request_fails', TRUE) && !system_check_http_request()) {
        $requirements['http requests'] = array(
          'title' => $t('HTTP request status'),
          'value' => $t('Fails'),
          'severity' => REQUIREMENT_ERROR,
          'description' => $t('Your system or network configuration does not allow Drupal to access web pages, resulting in reduced functionality. This could be due to your webserver configuration or PHP settings, and should be resolved in order to download information about available updates, fetch aggregator feeds, sign in via OpenID, or use other network-dependent services.'),
        );
      }

I'm going to try to do a correct fix, and will send a patch later if I manage to fix it.

CommentFileSizeAuthor
#1 linkchecker.install.patch1.4 KBfalsovsky
status.png51.71 KBfalsovsky
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

falsovsky’s picture

FileSize
1.4 KB

Well,

From what I understand, if the system module already does this check, it should not be here too.

So I removed that part of the code, and a commented out part after it too the once belonged to curl.

Here is the patch, it fixed the problem for me, and I think its the correct fix.

hass’s picture

Version: 6.x-2.7 » 6.x-2.x-dev
Status: Active » Closed (duplicate)
hass’s picture

Issue tags: -status http requirements
Parent issue: » #1306498: Remove unnecessary hook_requirements in linkchecker.install
falsovsky’s picture

That was only fixed in the module for Drupal 7, in the Drupal 6 version of the module it isnt yet fixed. So its not really a duplicate... at least it wasnt fixed in 6.x.

hass’s picture

It is fixed in D6.