Not stable for D7 [#1526788]

Hello,

I want to get off with a good start here and express my gratitude towards this module since it is a true enrichment to Drupal. However, this module is by no means stable for Drupal 7 as of now. I think that for now it should only be listed in the development version list if anywhere because it can cause a lot of headache to chase down what the problem is. I had a look at the code and there are still a lot of unreviewed conversions to Drupal 7 and some have not been tackled at all.

I will give it a shot and see what I can do to contribute to make a full conversion happen but until there is working version, it should not be listed as stable.

Thanks for your time and cheers,
Felix

Comment	File	Size	Author
#2	nodeorder.zip	14.9 KB	felixSchl

Comments

Comment #1

DanielWashbrook commented 27 April 2012 at 01:15

Did you manage to figure out how far from a working version this might be? I can contribute a bit too if there's something specific you've found that needs to be done.

Daniel

Comment #2

felixSchl commented 28 April 2012 at 00:06

Status	File	Size
new	nodeorder.zip	14.9 KB

Hi Daniel,

Most of the stuff are just the db queries. I have a working version here but I ripped out too much stuff to offer it as a patch. I essentially removed the whole taxonomy term page overwrite stuff because I did not see the need for it with a proper working views integration. I left the old code in, however, for reference.

I want to be a good sport and upload what I have. I am using it on a production site and it seems to be working fine. Not sure whether or not all the node hooks are needed (I just radically commented them all out since they made no obvious sense to me) but if so, they must probably be adapted, too.

Comment #3

dswier commented 23 May 2012 at 15:19

Thanks for posting this. This version of the module is working for me where alpha2 was failing. With alpha2 I was getting the error referenced at http://drupal.org/node/1524084, but apparently the offending line(actually the whole function containing it) has been commented out in this version.