a little mistake on the last version (RC2)

weight.module > L148

change
if (!empty($form['admin']['nodes']['#options'])) {
to
if (!empty($form_element['admin']['nodes']['#options'])) {

and then everything works just fine!

cheers.

Comments

sense-design’s picture

sense-design
sense
Primary language German
Location Münster
commented

This solves the following error message:

Warning: Invalid argument supplied for foreach() in weight_node_admin() (Line 154 weight.module).
Anonymous’s picture

Anonymous (not verified) commented

Nice work!

interx’s picture

interx commented
Status: Active » Reviewed & tested by the community

Works for me.

EDIT:
Oops, changed the status a bit fast, still needs a proper patch ;)

kclarkson’s picture

kclarkson commented

Here are the errors I am receiving:

Notice: Undefined index: admin in weight_form_node_admin_content_alter() (line 100 weight.module).
Warning: Invalid argument supplied for foreach() in weight_form_node_admin_content_alter() (line 100 of weight.module).

Notice: Undefined index: #options in weight_node_admin() (line 138 of weight.module).
Warning: Invalid argument supplied for foreach() in weight_node_admin() (line 138 weight.module).

kclarkson’s picture

kclarkson commented
Status: Reviewed & tested by the community » Active

changing back to active

rickmanelius’s picture

rickmanelius commented

In site.com/admin/content I'm seeing

Notice: Undefined index: weight in theme_tableselect() (line 3216 of /includes/form.inc).

EDIT/UPDATE: the suggested change in the issue summary clears this error.

rickmanelius’s picture

rickmanelius commented

@kclarkson
Where are you seeing these errors (which page/url)?

davisben’s picture

davisben
Primary language English
Location NH, USA
commented
Status: Active » Fixed

Thanks folks! This has been committed to 7.x-2.x.

kgaut’s picture

kgaut commented

I've not done a patch because I was very busy but i'm glad to have helped!

Nice module.

Anonymous’s picture

Anonymous (not verified) commented

Applied fix by rollingout the new dev version. Many thanks people!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.