Notice: Trying to get property of non-object in Drupal\glossify\GlossifyBase->parseTooltipMatch() (line 96 of /var/www/html/egypt/web/modules/contrib/glossify/src/GlossifyBase.php

Notice: Undefined index: into MYTERM in Drupal\glossify\GlossifyBase->parseTooltipMatch() (line 78 of modules/contrib/glossify/src/GlossifyBase.php) #0 core/includes/bootstrap.inc(552): _drupal_error_handler_real(8, 'Undefined index...', '

CommentFileSizeAuthor
#5 match_notice-2877570-5.patch539 byteskeesje
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

heyyo created an issue. See original summary.

keesje’s picture

Status: Active » Postponed (maintainer needs more info)

Can you add steps to reproduce please?

heyyo’s picture

I think it's because my term is using this apostrophe ’ and not '

keesje’s picture

I tried to reproduce with both chars in termname, cant reproduce.

keesje’s picture

Status: Postponed (maintainer needs more info) » Needs review
FileSize
539 bytes

Think I reproduced it, please try this patch.

keesje’s picture

Title: Notice: Trying to get property of non-object in Drupal\glossify\GlossifyBase->parseTooltipMatch() (line 96 » Inconsistent array key usage for matched terms
Assigned: Unassigned » keesje
Status: Needs review » Fixed
Issue tags: +Vienna2017

Committed

  • keesje committed 74c3348 on 8.x-1.x
    Issue #2877570 by keesje: Inconsistent array key usage for matched terms
    

Status: Fixed » Closed (fixed)

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

firfin’s picture

So is this fix in alpha1 or not? Commit and alpha1 release were on the same day?

--EDIT--
Yes it is alpha1, just checked the source