After upgrading from 1.4 to the current dev version the following error message is displayed during update function #7036:
Call to undefined function entity_info() in /var/www/html/cm7t/sites/all/modules/metatag/metatag.install on line 1629

Most probably a dependency is missing here. I couldn't find where function entity_info() is defined. What module is missing?

CommentFileSizeAuthor
#8 metatag-n2496487-9.patch3.51 KBdamienmckenna
#2 metatag-n2496487-2.patch1.64 KBdamienmckenna

Comments

GStegemann’s picture

GStegemann commented
Issue summary: View changes
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review
StatusFileSize
new metatag-n2496487-2.patch1.64 KB

And see, this is why we don't release a new version without testing. X-)

Sorry about that. This patch should fix it.

GStegemann’s picture

GStegemann commented
Status: Needs review » Reviewed & tested by the community

Sorry about that. This patch should fix it.

You're welcome. Is fixed now.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Reviewed & tested by the community » Fixed

Committed. Thanks!

  • DamienMcKenna committed 8d8866c on 7.x-1.x 
    Issue #2496487 by DamienMcKenna: The function is called entity_get_info...
nicola85’s picture

nicola85 commented

Maybe you need to change also
if (function_exists('entity_info')) {
in
if (function_exists('entity_get_info')) {

GStegemann’s picture

GStegemann commented

I would say 'yes' as well. Sorry, I didn't catch this.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Fixed » Needs review
StatusFileSize
new metatag-n2496487-9.patch3.51 KB

Ah yes, thank you.

GStegemann’s picture

GStegemann commented
Status: Needs review » Reviewed & tested by the community

That patch looks quite different now. But as far as I could test it the patch works. I reset the schema back to 7036 and update funtion did not displayed any errors and the settings are still OK.

  • DamienMcKenna committed 174641d on 7.x-1.x 
    Issue #2496487 by DamienMcKenna: Follow-up on previous commit, remove...
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Reviewed & tested by the community » Fixed

Committed. Thanks again!

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

@GStegemann: Yeah, the whole reason that function_exists() call was there in the first place was because I was using the wrong function, so when the rest of the code was fixed to use the correct function the entire if(){}/else{} became redundant.

GStegemann’s picture

GStegemann commented

I see. So I found in fact two bugs.

Status: Fixed » Closed (fixed)

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