hi,

Is anyone porting this module to D5 ?
If no then I would like to take it up ?

Thanks

Juned Kazi

CommentFileSizeAuthor
#6 util_new.tar_.gz12.03 KBjunedkazi
#5 util.tar_.gz12 KBjunedkazi

Comments

litwol’s picture

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.

junedkazi’s picture

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

Thanks

Juned Kazi

junedkazi’s picture

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

litwol’s picture

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.

junedkazi’s picture

StatusFileSize
new12 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

junedkazi’s picture

StatusFileSize
new12.03 KB

just made some changes to the menus

Thanks
Juned Kazi

junedkazi’s picture

litwol did you try the tar file attached in comment 6 (http://drupal.org/node/325710#comment-1091902) .

litwol’s picture

Status: Active » Needs review

No, i would like some one else to review it.

pbarnett’s picture

Have downloaded the patch; will report back...

Pete.

litwol’s picture

Status: Needs review » Closed (won't fix)

D5 version is no longer being maintained. marking this as wont fix. will reopen if some one decides to contribute a patch. otherwise stays closed.