Update module for D6

hass - April 29, 2007 - 08:33
Project:Node style
Version:HEAD
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:needs work
Description

http://drupal.org/cvs?commit=65517

now we can circumvent this variables workarounds... :-) currently untested...

#1

Zen - May 4, 2007 - 09:50
Title:theme_variables_HOOK has been committed!» Update module for D6
Status:active» postponed

Thanks :) I saw that too. We'll wait till an RC comes out before updating the module.

Cheers,
-K

#2

hass - December 25, 2007 - 23:09
Status:postponed» active

Time to start... :-)

#3

fletchgqc - April 1, 2008 - 23:29

Yeah, it's a good and seemingly fairly unique module. Be good to have in D6.

#4

fletchgqc - May 19, 2008 - 10:41

In the meantime I learned a bit more about Drupal and creating .tpl.php files for my content types and using drupal_add_css() in those files actually fulfills my needs fine. Just posting this in case someone like me wants to launch on D6 and needs this functionality.

#5

mbach - June 19, 2008 - 11:46

Would great to use it in D6!!!!

Thanks for your effort!!!!

#6

schildi - September 2, 2008 - 13:36

does your post mean there is a (development-) version available for D6 ?

#7

luco - November 26, 2008 - 17:11

I need this module ported onto D6 as well.

I intend to chip in; I can contribute via PayPal.

#8

giorgio79 - January 5, 2009 - 14:10

subs

#9

EvanDonovan - April 15, 2009 - 23:18
Status:active» needs review

Here is a preliminary version of the module for Drupal 6. It works for me, after a day or so of testing. Let me know if it works for you.

I have attached it both as a patch (a messy one, though) and as a .tar.gz. Note that it is a direct port and does not leverage the Drupal 6 theming improvements. I offer this as a base for others to build on.

AttachmentSize
node_style_6.x-1.0.tar_.gz 14.56 KB
node_style_d6.patch 21.25 KB

#10

hass - April 16, 2009 - 06:55
Status:needs review» needs work

1. Do not t'ify schema title/descriptions.
2. We should really implement the new theming hook... this makes the module much more usable and easier to implement as before.

#11

EvanDonovan - April 16, 2009 - 14:53

Thanks for the review. I didn't know you weren't supposed to put t() around schema title/descriptions - the deadwood module did that for me.

I've only been working on Drupal 6 modules for a few weeks now, and confess that I still don't know much about the new theming system. Any help you could offer on implementing the new theming hook would be greatly appreciated.

Attached is a patch to correct the .install file. I have also added in descriptions for all the fields.

AttachmentSize
node_style-install-d6-no-t.patch 1.63 KB

#12

hass - April 16, 2009 - 15:15

If I remember correctly you need to add the below hook to the module and alter/override the variables as needed... so no more a duplicate function name, no errors... maybe some more hooks are required - not sure. Please always post complete patches.

function node_style_preprocess_page(&$vars) {

}

#13

RobertoGuzman - May 15, 2009 - 11:57

i have install the latest drupal 6.12 with the node_style_6.x-1.0.tar and the node style does not work, i just install node_style_6.x-1.0.tar and apply this patch node_style-install-d6-no-t.patch

#14

EvanDonovan - May 15, 2009 - 15:30

I haven't upgraded to Drupal 6.12 - I'll check on that. I think that unless you have a lot of legacy data in the Node Style module, though, you should probably check out ThemeKey. It's a D6 module that I think does all that this module did and more. So I probably won't work more on this project.

 
 

Drupal is a registered trademark of Dries Buytaert.