The existing theming templante file name is confusing as it doesn't match the theme function name.
Drupal use the theme function name when overriding existing template files.

So if you want to override the template files you need to use advpoll-voting-binary-form.tpl.php/advpoll-ranking-form.tpl.php
and not just copy the existing template files because they have a different name.

This patch just replace the template files with the same ones but with the name of the theme functions.

CommentFileSizeAuthor
advpol_theme_template.patch5.63 KBjcmarco

Comments

coderintherye’s picture

+1 to this, would appreciate this update, though you may want to leave the previous functions in for backwards compatibility?

roball’s picture

+1

IMO, backward compatibility is not needed in this case since the way it worked so far is not the way it should have been working and there has not been released any version for D6 (still only in dev).

liquidcms’s picture

yup, that was annoying.. spent an hour trying to figure out why my copy of the modules tpl file that i put in my theme folder wasn't getting picked up..

roball’s picture

Any reason why not committing this patch?

Anonymous’s picture

Thanks for this note -- I too had an hour perilous to my sanity on this one.

But just want to add something: until this patch is applied, the filename to override the ranking form is not the name given above. You want advpoll-voting-ranking-form.tpl.php, not advpoll-ranking-form.tpl.php.

A particularly unfortunate typo. Now I just hope I got the name right. :)

liquidcms’s picture

i am using advpoll-voting-binary-form.tpl.php

gobinathm’s picture

Issue summary: View changes
Status: Needs review » Closed (outdated)

D6 is EOL hence this issue won't be fixed (or) attended. So closing it.