The hook node load in the module http auth is getting problems using the second parameter. Sometimes isn't passing on this array the value for site and we lose the config. I changed this function. We skip this condition and I add the validation in the foreach. I think at the end is the same and isn't bad for the performance.

CommentFileSizeAuthor
bug_node_load_using_content_types.patch935 bytesfastangel
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

fastangel created an issue. See original summary.

shreddedbacon’s picture

This patch fixes the issue I was experiencing

Jon Pugh’s picture

Priority: Normal » Critical
Status: Needs review » Reviewed & tested by the community

Marking critical because this morning l module is effectively broken without this patch!

Will commit and release when I get to my office in ~3 hours, unless Helmo gets to it before I do ;)

Jon Pugh’s picture

I've added a Behat test to verify the problem. Confirmed failure is here: https://travis-ci.org/opendevshop/devmaster/jobs/399770975#L2901

About to push a commit with the patch applied.

  • Jon Pugh committed a33b8d6 on 7.x-3.x authored by fastangel
    Issue #2982704 by fastangel: Bug loading info for http auth
    
Jon Pugh’s picture

Status: Reviewed & tested by the community » Fixed

Committed. Thanks Jacinto!

Status: Fixed » Closed (fixed)

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