search

backdrop-contrib / radix

A responsive base theme with Bootstrap and Sass for Backdrop CMS.
http://docs.radixtheme.org
GNU General Public License v2.0
4 stars 2 forks source link

Undefined variable: title_attributes in .tpl.php files #5

Closed PaulHoza closed 9 years ago

PaulHoza commented 9 years ago

I'm seeing this error initially in two places: ./templates/comment/comment.tpl.php ./templates/node/node.tpl.php

Error:

Notice: Undefined variable: title_attributes in include() (line 20 of /home/example/public_html/themes/radix-1.x/templates/node/node.tpl.php).

As far as I can tell, there is no longer a $title_attributes in Backdrop. I think this explains that: https://api.backdropcms.org/node/28554

So... I have no real idea how to /fix/ this, but for now, I've just removed these references from my copy of Radix. This removes the errors, but sure seems like a hack. As referenced in that API doc, the var isn't apparently around anymore, but does show up in core's block-dynamic.tpl.php (line #33). However, adding in the code from there into these Radix template files doesn't work.

If anyone has a better way to sort this than just hacking out the code, please let me/us know.

Thanks. (PR incoming in a few... if it's something you want to merge, which may be a Bad Idea without better solutions.)

PaulHoza commented 9 years ago

Oops, I left out the specifics of the Dynamic Block usage of title_attributes (which didn't work for these other .tpl.php usages):

backdrop_attributes($title_attributes);