Update for Drupal 6.x

Looking so forward to the release of role weight for Drupal 6.x. Used it extensively in 5.x

Thank you!

Subscribing

Any ideas when a Drupal 6 version will be ready for production?

subscribing

Sorry, have to re-open issue. Been patiently waiting for months for the version 6.x, will this possibly be happening?

Just need Role Weights to sort in Views.

Thank you for any updates :)

I've downloaded and tested HEAD (a link in post #11) which, by the way, doesn't match the packed HEAD version.

Everything seems to work well on my system, so consider it as tested regarding me.

I believe a 6.x development version would contribute significantly to users interest (I also didn't notice those latest changes, as module was not even on my "Available Updates" list, so I have more or less forgotten that it even exists in my system...). It would also help us all to test and discuss about the same version, as now it is not so eays to get this latest release (also the link doesn't lead directly to files, but I've had to follow some links before finding the useful files listed...). So, I would suggest to put together and publish at least the 6.x-0.x-dev version ASAP.

I also think the setting in a general configuration file is not comfortable at all, not to mention that it can probably lead to more serious troubles (if someone manages to make settings file unreadable to Drupal). So, I've made some small changes to set the variable through an admin screen (as practically all other moduleas are doing it...) - this way, whatever you choose there, you shall not break anything... Furthermore, I've created a help screen (putting there more or less the useful content of the README file provided).

My contribution doesn't change the basic functionality in any aspect, it just makes the usage of this module a bit more comfortable (and maybe safe). As I am not a skilled coder, it would definitely be good that someone more experienced checks the code (it seems to work just fine on my system though). My code is in the attachment.

As a final touch, I would suggest to consider some (I believe just minor) twists to satisfy the Coder module, which currently throws out the following errors and warnings:

role_weights.module:

Line 82: Use "elseif" in place of "else if"
        else if ($form_id == 'user_admin_role') {

Line 129: Use "elseif" in place of "else if"
        else if ($form_id == 'user_admin_perm' && variable_get('role_weights_reorder_forms', FALSE)) {

Line 136: Use an indent of 2 spaces, with no tabs
            }	

(!!!) Line 178: The parameters for form validation and submission functions have changed to $form, &$form_state.
      function role_weights_admin_role_submit($form_id, &$form_state) {

(!!!) Line 205: SQL query handling data in a potentially insecure way by using the %%s placeholder without wrapping it in single quotes. This is a potential source of SQL injection attacks when the value can come from user data.
              db_query('DELETE FROM {users_roles} WHERE rid = %d AND uid IN (%s)', $id, implode(', ', $uid));

---

role_weights.install:

Line -1: @file block missing (Drupal Docs)

Line 2: Include the CVS keyword $Id$ in each file. This should be in the format // $Id$ or // $Id$
        //$Id$

At least those 2 critical warnings, marked as (!!!) don't look nice at all (it may be also the Coder issue, but still - I just like it when Coder is not complaining...).