Warning is thrown due to inconsistent data.

CommentFileSizeAuthor
#3 2726793-3.patch1.44 KBgrahl
#2 ldap_user_preprocess_table-does-not-2726793-2.patch918 bytesgrahl
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

grahl created an issue. See original summary.

grahl’s picture

grahl
he/him
Location Zürich
CreditAttribution: grahl at Namics commented
Issue summary: View changes
FileSize
ldap_user_preprocess_table-does-not-2726793-2.patch918 bytes

Decided to make the source more robust since ldap_user has many more issues.

Update: Patch is not that good, still looking into general problems with ldap_user.

grahl’s picture

grahl
he/him
Location Zürich
CreditAttribution: grahl at Namics commented
Title: ldap_user_preprocess_table does not check if class is present » ldap_user_preprocess_table called with missing cell content
FileSize
2726793-3.patch1.44 KB

The new patch just checks if class rowspan and colspan are present before setting. I'm unsure what these do but the empty array just throwing errors is unhelpful.

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan at PreviousNext commented
Status: Needs review » Reviewed & tested by the community
larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan at PreviousNext for Department of Justice & Community Safety, Victoria commented
Status: Reviewed & tested by the community » Fixed

Thanks!

  • larowlan committed ba54cbb on 8.x-3.x authored by grahl 
    Issue #2726793 by grahl: ldap_user_preprocess_table called with missing...

Status: Fixed » Closed (fixed)

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