I put in some work in porting this module to drupal 7. Views integration is not working yet, but everything else should work.

Comments

sanduhrs’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Patch (to be ported) » Needs work

Thank you, Kristof.
I commited a this patch and released a D7-DEV version.
Anyone available for testing?

jhezsie20’s picture

Hi Sanduhrs,

Just tested the dev release in Drupal 7, all is fresh install including drupal. Still can't get to work. Will post errors here later.

thanks,
jhez

Anonymous’s picture

Hi, thanks for the work put into the porting this module.
I installed it on my testing environment and when i go into /admin/config/media it gives me an error because it cant find the file pages.bbb.inc in the includes folder of the bbb module.
Here is the error:
Fatal error: require_once() [function.require]: Failed opening required '/Applications/MAMP/htdocs/drupal-7.0test/sites/all/modules/bbb/includes/pages.bbb.inc' (include_path='.:/Applications/MAMP/bin/php5.3/lib/php') in /Applications/MAMP/htdocs/drupal-7.0test/includes/menu.inc on line 499

I tried copying the includes folder from a D6 version but i still get some errors in /admin/config/media/bigbluebutton regarding undefined indexes and variables e.g.
Notice: Undefined index: op in bbb_settings() (line 18 of /Applications/MAMP/htdocs/drupal-7.0test/sites/all/modules/bbb/includes/pages.bbb.inc).

Hope this helps,
Spre3

hoomanb’s picture

Update: I patched these errors. See below.

I copied the includes from 6.x as well, but when I go to the actual meeting node, there are no status buttons and I get these errors:

Notice: Trying to get property of non-object in theme_bbb_block_meeting() (line 830 of /home/hooman/www/drupal7/modules/bbb/bbb.module).
Notice: Trying to get property of non-object in theme_bbb_meeting_status() (line 816 of /home/hooman/www/drupal7/modules/bbb/bbb.module).
Notice: Trying to get property of non-object in theme_bbb_block_meeting() (line 836 of /home/hooman/www/drupal7/modules/bbb/bbb.module).
Notice: Trying to get property of non-object in theme_bbb_block_meeting() (line 840 of /home/hooman/www/drupal7/modules/bbb/bbb.module).
Notice: Trying to get property of non-object in theme_bbb_block_meeting() (line 843 of /home/hooman/www/drupal7/modules/bbb/bbb.module).
Notice: Trying to get property of non-object in theme_bbb_block_meeting() (line 846 of /home/hooman/www/drupal7/modules/bbb/bbb.module).

I also noticed, certain actions seem to be run twice (e.g., the functions above, or the test connection function)

hoomanb’s picture

StatusFileSize
new9.64 KB

I created a patch that fixes a whole lot of small issues and now it works fine on my Drupal 7.

There are still lots of warnings and things to be fixed, but meanwhile this patch should help to at least get it working.

hoomanb’s picture

StatusFileSize
new11.97 KB

I've attached another patch that should get the module working without errors and warnings in Drupal 7. I have no use for views integration yet and thus haven't touched it.

owen barton’s picture

See #946038: Warnings from views handlers because of bbb module and #946652: bbb_views doesn't work for a couple of views related patches that I think will need to be rolled into the D7 patch.

hoomanb’s picture

StatusFileSize
new14.99 KB

Here's another patch (the last one, I promise!). It fixes the install/uninstall errors as well as a few other errors I noticed, plus it adds the option to allow users attend meeting without a moderator present (to the content type form).

The patch is built against the current 7.x-dev.

Anonymous’s picture

Thanks hoomanb. Can you tell me if we need to apply all your patches or just the latest one?
thanks,
spre3

jhezsie20’s picture

Any news on Drupal 7 port? Thanks!

jhaski’s picture

Status: Needs work » Patch (to be ported)
StatusFileSize
new20.86 KB
new47.76 KB

I spent some time getting the BBB module to work on my Drupal 7 website. Applying the above patches still gave me fatal errors, so I created a new patch which incorporates all of the changes necessary to get D7 to work. The patch should apply cleanly to the 7.x-1.x HEAD.

Message me if you have any problems, because I plan on working on and extending this module for a new website.

dureaghin’s picture

Status: Needs review » Patch (to be ported)

Hi jhaski,

I have a problem.

When I'm going to test connection:

This error:

Fatal error:Cannot redeclare bbb_node_insert() (previously declared in C:\wamp\www\drupal7\sites\all\modules\bbb\bbb.module:369) in C:\wamp\www\drupal7\sites\all\modules\bbb\bbb.module on line 369 

When I'm Save configuration:

This error:

* Notice: Use of undefined constant BIGBLUEBUTTON_DISPLAY_MODE - assumed 'BIGBLUEBUTTON_DISPLAY_MODE' in bbb_settings() (line 48 of C:\wamp\www\drupal7\sites\all\modules\bbb\includes\pages.bbb.inc).
    * Notice: Use of undefined constant BIGBLUEBUTTON_DISPLAY_HEIGHT - assumed 'BIGBLUEBUTTON_DISPLAY_HEIGHT' in bbb_settings() (line 54 of C:\wamp\www\drupal7\sites\all\modules\bbb\includes\pages.bbb.inc).
    * Notice: Use of undefined constant BIGBLUEBUTTON_DISPLAY_WIDTH - assumed 'BIGBLUEBUTTON_DISPLAY_WIDTH' in bbb_settings() (line 61 of C:\wamp\www\drupal7\sites\all\modules\bbb\includes\pages.bbb.inc).
    * Notice: Undefined property: stdClass::$redirect_code in bbb_test_connection() (line 114 of C:\wamp\www\drupal7\sites\all\modules\bbb\includes\pages.bbb.inc).
    * The connection could not be established. Please adjust your settings.
owen barton’s picture

Status: Patch (to be ported) » Needs review

Assuming this hasn't been committed this is the right status. "To be ported" is only used once a patch has been committed to indicate that the patch (or parts of it) still need to be ported to a different version branch.

jhezsie20’s picture

Status: Patch (to be ported) » Needs review

Hi! Any update on this? Thanks :)

yugongtian’s picture

+1

jhezsie20’s picture

Looking forward for update :)

thanks!

rerooting’s picture

After install, I get this when I try to access the menu item (and a 500 error when i visit the item):

Warning: require_once(/var/www/[website]/sites/all/modules/bbb/includes/pages.bbb.inc): failed to open stream: No such file or directory in menu_execute_active_handler() (line 499 of /var/www/[website]/includes/menu.inc).

I tried to dig around and find why this was happening, but alas no result. I will try it again on a clean install later this week to see if i can duplicate the bug.

hoomanb’s picture

You need to copy the includes folder from the d6 version.

jalbrizio’s picture

Its missing the the includes directory. I copied the directory from the latest 6 package but I am still getting errors.

# Notice: Undefined index: op in bbb_settings() (line 18 of /testing/sites/all/modules/bbb/includes/pages.bbb.inc).
# Notice: Use of undefined constant BIGBLUEBUTTON_BASE_URL - assumed 'BIGBLUEBUTTON_BASE_URL' in bbb_settings() (line 36 of /testing/sites/all/modules/bbb/includes/pages.bbb.inc).
# Notice: Use of undefined constant BIGBLUEBUTTON_SECURITY_SALT - assumed 'BIGBLUEBUTTON_SECURITY_SALT' in bbb_settings() (line 42 of /testing/sites/all/modules/bbb/includes/pages.bbb.inc).
# Notice: Use of undefined constant BIGBLUEBUTTON_DISPLAY_MODE - assumed 'BIGBLUEBUTTON_DISPLAY_MODE' in bbb_settings() (line 56 of /testing/sites/all/modules/bbb/includes/pages.bbb.inc).
# Notice: Use of undefined constant BIGBLUEBUTTON_DISPLAY_HEIGHT - assumed 'BIGBLUEBUTTON_DISPLAY_HEIGHT' in bbb_settings() (line 62 of /testing/sites/all/modules/bbb/includes/pages.bbb.inc).
# Notice: Use of undefined constant BIGBLUEBUTTON_DISPLAY_WIDTH - assumed 'BIGBLUEBUTTON_DISPLAY_WIDTH' in bbb_settings() (line 69 of /testing/sites/all/modules/bbb/includes/pages.bbb.inc).

Chipie’s picture

Category: task » bug

Hi jhaski,

I have downloaded your version (#11). When I enable the module I get this error:

Error: syntax error, unexpected $end in /data/disk/o1/distro/001/drupal-7.8-dev/sites/all/modules/bbb/bbb.module, line 869

Regards
-- Chipie

jhaski’s picture

StatusFileSize
new55.01 KB

This patch applies cleanly to the Drupal 7.x-1.x branch head. I just tested the module on a new Drupal 7.7 site and it worked fine. Let me know if you have any specific problems.

On another note, checkout the new BBB .8 features. http://groups.google.com/group/bigbluebutton-dev/browse_thread/thread/ba... I am working on adding the new recording and playback functionality in the Drupal 7 module.

Chipie’s picture

Hi jhaski,

thanks for the prompt reply. When applying your patch I get these errors:

o1@lvps178-77-75-8:~/distro/001/drupal-7.8-prod/sites/all/modules/bbb$ patch -p0 < bbb-d7-1016910-21.patch                             
patching file b/bbb.module
Hunk #1 FAILED at 18.
Hunk #2 FAILED at 123.
Hunk #3 FAILED at 148.
Hunk #4 FAILED at 165.
Hunk #5 FAILED at 178.
Hunk #6 FAILED at 192.
Hunk #7 FAILED at 209.
Hunk #8 FAILED at 232.
Hunk #9 FAILED at 390.
Hunk #10 FAILED at 405.
Hunk #11 FAILED at 465.
Hunk #12 FAILED at 494.
Hunk #13 FAILED at 526.
Hunk #14 FAILED at 551.
Hunk #15 FAILED at 618.
Hunk #16 FAILED at 646.
Hunk #17 FAILED at 688.
Hunk #18 FAILED at 706.
Hunk #19 FAILED at 754.
Hunk #20 FAILED at 811.
Hunk #21 FAILED at 824.
21 out of 21 hunks FAILED -- saving rejects to file b/bbb.module.rej
patching file b/includes/api.bbb.inc
patching file b/includes/bbb_views.views.inc
patching file b/includes/bbb_views.views_default.inc
patching file b/includes/pages.bbb.inc
patching file b/includes/views_handler_field_bbb_join_attendee.inc
patching file b/includes/views_handler_field_bbb_join_moderator.inc
patching file b/js/check_status.bbb.js

Regards,
--Chipie

dawehner’s picture

@Chipie
Just by looking at the patch i would recommend you to use patch -p1 instaed of patch -p0

LinuxETC’s picture

Note: Using BBB v0.71a on Ubuntu here, not v0.80 which I thought was still in "testing".

Just as an FYI from a clean install of Drupal v7.8 with the latest Chaos Tools, Views, and BBB modules I noticed the following which is reported below (the original)

https://drupal.org/node/1283392

which is linked to this

https://drupal.org/node/1283764

When I try to configure the BBB module I get a "white screen of death" unfortunately. I am awaiting to see about the above and if it is related to BBB, Views, and/or Chaos Tools.

Again, just a heads up. Will be keeping our production BBB + Drupal sites on v6.22 for the time being.

LinuxETC

LinuxETC’s picture

Seems it is the latest BBB dev version here. The error message specially is as follows:

Deprecated function: Call-time pass-by-reference has been deprecated in drupal_load() (line 1100 of (path to Drupal v7.8 installation)/includes/bootstrap.inc).

which again is referenced in the two other URLs above.

mdesir’s picture

does any body have bbb working on drupal 7

when ever I tried to create a conference I get this message

This XML file does not appear to have any style information associated with it. The document tree is shown below.
FAILEDinvalidMeetingIdentifierThe meeting ID that you supplied did not match any existing meetings

I've tried everything thing that I can think but just can't get it to work

Chipie’s picture

I have bbb working with drupal 7.

Which version of bbb are you using? Was the connection test successful? Did you create a new meeting node?

mdesir’s picture

right now I have .71a but really would prefer .8

Yes connection was successful

meeting node? meaning conference for content type.

mdesir’s picture

Does anybody have bbb ported to drupal 7

LinuxETC’s picture

@mdesir

Yes now, thanks to reviewing some of Chipie's comments. I used the patch noted at #21 with "-p1" as noted in #23.

I also had to modify the prior BBB server URL some from what was used in v6.x of the module which was originally

http://BBB server URL/bigbluebutton/api/

to the following for v7.x of the module (with patches applied)

http://BBB server URL/bigbluebutton

Note: The ending "/" is removed which would produce some errors with Tomcat on the BBB server side.

HTH.

mdesir’s picture

Thanks for your reply, I'll work on it today

mdesir’s picture

Thank you very much! It worked and I'm using .8 with it

markwk’s picture

Ok. I got it working. Can I suggest we push the current patches and such into the current dev branch?

ParisLiakos’s picture

Status: Needs review » Fixed

Seems #21 was commited.
This should be closed i guess, and new issues should be opened if necessary

Status: Fixed » Closed (fixed)

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