Using with 4.7.0-HEAD (as of 14/2/06)

I've made this critical for the following reasons:

1) The error with user module prevents buddylist from working properly.

2) Buddylist_access is incompatible with 4.7

STEPS
1) Install latest 4.7 core files from HEAD
2) Install buddylist from CVS (
3) Visit a user profile page (e.g., user/1) and then view watchdog logs. The following error appears:

warning: Invalid argument supplied for foreach() in /home/v/c/vcommunity/public_html/modules/user.module on line 1401.

4) Go to admin/settings/buddylist_access. The following error message appears:

Fatal error: Call to undefined function: form_button() in /home/v/c/vcommunity/public_html/modules/buddylist/contrib/buddylist_access/buddylist_access.module on line 99

This last error is caused by the fact form buttons are handled differently in the new Forms API.

According to formupdater, the code should read:

$form[$group] = array(
  '#type' => 'button',
  '#title' => $btn_text). '<br />';
    $output = form_group(t('Module status'),
  '#button_type' => t('After enabling this module via the module settings page or before disabling this module, use the above button to properly configure your node_access table.'),
);

I have not tested this yet and so cannot report whether it works or not but I hope this helps.

Buddylist is a fantastic module, thanks for all the work that has gone into it. Can't wait to use it in 4.7!

Comments

robertDouglass’s picture

There is now a 4.7 branch and this problem shouldn't occur when using that. Please reopen of this problem persists.

robertDouglass’s picture

Status: Active » Fixed
Patrick Nelson’s picture

Status: Fixed » Active

Using the latest CVS version of buddylist (1.53) with HEAD of 4.7.

Enabling the module returns this error:

Warning: Cannot modify header information - headers already sent by (output started at /var/www/modules/buddylist/buddylist.install:5) in /var/www/includes/common.inc on line 229

Changing status of this post back to active.

robertDouglass’s picture

This is a separate issue than the one first reported. I think it is because there was a closing ?> at the end of the install file with whitespace after. I've removed both the ?> and the whitespace. Please test.

Patrick Nelson’s picture

Status: Active » Fixed

Robert,

Yep. That did it. Thanks.

Patrick Nelson’s picture

Robert,

Yep. That did it. Thanks.

There (may be) another issue now. I'll check it out first and if it is a bug, I'll report it in a seperate post.

Again, many thanks

Regards

Patrick

Patrick Nelson’s picture

Status: Fixed » Active

Robert,

The error I'm getting now is still valid under the title of this post, so, if you're OK with it, I'll post it here.

The error shows up when viewing a user page and is:

Warning: Invalid argument supplied for foreach() in /var/www/modules/user.module on line 1411

Sorry to keep going backwards and forwards like this Robert (with the changes to the post Status).

Any ideas?

Regards

Patrick

sami_k’s picture

I was able to replicate it.

sami_k’s picture

It's related to this issue, if you added the two groups tables it seems to fix the problem.
http://drupal.org/node/52987

Anonymous’s picture

Drupals had an update to the user module thats broken a few modules. The real problem seems to be caused by returning the wrong set of values. It is possible to duplicate it, though the database is set up correctly here. This is tested on 4.7.0-beta5, so it may be a "feature" of this release .

Anonymous’s picture

This seems to be working again now (4.7 module / Drupal CVS). The module is now working and displaying correctly, im guessing a commit somewhere fixed it.

robertDouglass’s picture

Status: Active » Fixed

Let me know if it crops up again.

Anonymous’s picture

Status: Fixed » Closed (fixed)