Because the checks in linkedin-user-page-position-item.tpl.php check for the boolean TRUE and $position['is-current'] is a string, any non-zero non-empty string value (including 'false') will return TRUE, resulting in every position being themed as being current. Fixing this is a matter of either converting these strings to booleans before they reach the theming layer, or checking for string literals in the template.

Patch to follow sometime by the week's end.

CommentFileSizeAuthor
#1 1357840--linkedin-position-item-template--1.patch3.35 KBalexiswatson

Comments

alexiswatson’s picture

alexiswatson commented
Status: Active » Needs review
StatusFileSize
new 1357840--linkedin-position-item-template--1.patch3.35 KB

Apologies for the delay, work/life happened. This patch ensures that strings are converted to booleans via a preprocess function before reaching the template, while strictly checking for boolean values in the template itself. Had to address some of the other issues in the same patch, like not using isset in the template itself.

Review and feedback welcome!

tylerwalts’s picture

tylerwalts commented

This patch worked for me.