page--nodetype.tpl.php files not overriding. [#1256902]

Comment	File	Size	Author
#4	pagetempprepro-1256902-4.patch	801 bytes	ishmael-sanchez

Comment #1

ishmael-sanchez CreditAttribution: ishmael-sanchez commented 24 August 2011 at 22:50

Status:

Needs review

» Fixed

That's actually not baked into the D7 version of the theme but I just committed a fix to the template.php and tested it on my local machine.

Log in or register to post comments

Comment #2

SuperRookie CreditAttribution: SuperRookie commented 24 August 2011 at 17:29

Assigned:	Unassigned	» ishmael-sanchez
Category:	support	» feature
Status:	Active	» Fixed

I just implemented the updated template.php file and a couple of things happened. I believe that it now needs page--nodetype.tpl.php files now, as my front page broke until I provided it with one for that content type, and now on any content type that doesn't have its own template I get the following errors:

Notice: Undefined index: node in earthish_preprocess_page() (line 12 of /home/........../template.php).
Notice: Trying to get property of non-object in earthish_preprocess_page() (line 12 of /home/......./template.php).

Any ideas?

SR

Log in or register to post comments

Comment #3

ishmael-sanchez CreditAttribution: ishmael-sanchez commented 24 August 2011 at 18:45

Status:

Fixed

» Needs review

Log in or register to post comments

Comment #4

ishmael-sanchez CreditAttribution: ishmael-sanchez commented 25 August 2011 at 00:49

Status:

Fixed

» Patch (to be ported)

File	Size
pagetempprepro-1256902-4.patch	801 bytes

Ok so my preprocess function was hastily created and I didn't account for pages without a node type so I added a condition which makes sure the node object exists before adding this template suggestion. I will commit this fix shortly to the dev branch.