Comments

jpoeng created an issue. See original summary.

danyg’s picture

Status: Active » Needs review
StatusFileSize
new924 bytes

Hi, here's a patch which fix the missing template name.
There's a new template definition in admin page, and the problem is resolved if you click on save button on admin.
But if not, the info_template value is missing.

danyg’s picture

StatusFileSize
new891 bytes

Sorry, my previous patch doesn't fix your problem.
Here's the good one.

svenryen’s picture

Thanks for the patch. It needs a slight improvement, because the legacy template is only supported when the consent method is "consent by default". I'll tweak your patch and have a new one ready in a few minutes.

svenryen’s picture

Here's the improved patch

  • svenryen committed 2fa2c9b on 7.x-1.x
    Issue #3038158 by danyg, svenryen: Notice: Undefined index:...
svenryen’s picture

Status: Needs review » Fixed

Committed to -dev. Thanks for helping!

  • svenryen committed 2fa2c9b on 7.x-2.x
    Issue #3038158 by danyg, svenryen: Notice: Undefined index:...
  • svenryen committed 34ad05d on 7.x-2.x
    Merge branch '7.x-1.x' into 7.x-2.x
    
    * 7.x-1.x: (27 commits)
    Issue #...
danyg’s picture

@svenryen thanks for improving

  • svenryen committed 34ad05d on 7.x-1.x
    Merge branch '7.x-1.x' into 7.x-2.x
    
    * 7.x-1.x: (27 commits)
    Issue #...

Status: Fixed » Closed (fixed)

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