Is anyone already porting it to D5 ? [#325710]

Comment	File	Size	Author
#6	util_new.tar_.gz	12.03 KB	junedkazi
#5	util.tar_.gz	12 KB	junedkazi

Comment #1

litwol CreditAttribution: litwol commented 24 October 2008 at 18:27

If you wish to port D6 version of the module back to d5 then you are more than welcome to. i will accept a patch from you.

I discontinued d5 version because i couldnt maintain it and it required a complete rewrite.

If you put effort into it i will definitely accept it and will publish it.

Log in or register to post comments

Comment #2

junedkazi CreditAttribution: junedkazi commented 25 October 2008 at 03:52

I will start work on it and hope it will be ready in a week or 2 .

Thanks

Juned Kazi

Log in or register to post comments

Comment #3

junedkazi CreditAttribution: junedkazi commented 4 November 2008 at 04:46

Title:

Is anyone already porting it to D5 ?

» Is anyone already porting it to D5 ?

I am almost on the verge of porting it to 5.x

There is one problem where I am stuck and want to have a clean solution for it .

In contrib system module in the theme 2 times drupal_alter has been called so that
other module can alter the header and the rows .

This drupal_alter has been used in the module weights module to alter the
headers and rows to insert an extra label for weight and the field for weight .

As in 5.x there is no drupal_alter .
So that should I do to alter the headers and the rows .

Should I call another theme function and dupliate the whole theme function form the
system contrib module with a few modifications ?

or is there any other way to do this ??

Need suggestions .
This is the only thing left . once this is done I think only testing will be required .
So I think I will be able to give a tarball by sunday .

Thanks

Juned Kazi

Log in or register to post comments

Comment #4

litwol CreditAttribution: litwol commented 4 November 2008 at 05:26

you should avoid dupliating the whole theme function. really avoid.

try to come up with a different approach that may work. i am not really sure what it should be. adding small modification to table themes is diffucult as youv'e found out during this port. try to come up with a unique solution that may work.

Log in or register to post comments

Comment #5

junedkazi CreditAttribution: junedkazi commented 4 November 2008 at 13:25

File	Size
util.tar_.gz	12 KB

I am attaching a working copy of the module .
I haven't tested it much but for whatever
I have seen it is working pretty well .
I want the community to test and give feedback .
I am still working to make it better .

Thanks

Juned kazi