Closed (fixed)
Project:
BigBlueButton
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
6 Jan 2011 at 08:49 UTC
Updated:
2 Apr 2012 at 19:20 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
sanduhrsThank you, Kristof.
I commited a this patch and released a D7-DEV version.
Anyone available for testing?
Comment #2
jhezsie20 commentedHi 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
Comment #3
Anonymous (not verified) commentedHi, 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 499I 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
Comment #4
hoomanb commentedUpdate: 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)
Comment #5
hoomanb commentedI 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.
Comment #6
hoomanb commentedI'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.
Comment #7
owen barton commentedSee #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.
Comment #8
hoomanb commentedHere'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.
Comment #9
Anonymous (not verified) commentedThanks hoomanb. Can you tell me if we need to apply all your patches or just the latest one?
thanks,
spre3
Comment #10
jhezsie20 commentedAny news on Drupal 7 port? Thanks!
Comment #11
jhaski commentedI 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.
Comment #12
dureaghin commentedHi jhaski,
I have a problem.
When I'm going to test connection:
This error:
When I'm Save configuration:
This error:
Comment #13
owen barton commentedAssuming 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.
Comment #14
jhezsie20 commentedHi! Any update on this? Thanks :)
Comment #15
yugongtian commented+1
Comment #16
jhezsie20 commentedLooking forward for update :)
thanks!
Comment #17
rerooting commentedAfter install, I get this when I try to access the menu item (and a 500 error when i visit the item):
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.
Comment #18
hoomanb commentedYou need to copy the includes folder from the d6 version.
Comment #19
jalbrizio commentedIts 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).
Comment #20
Chipie commentedHi jhaski,
I have downloaded your version (#11). When I enable the module I get this error:
Regards
-- Chipie
Comment #21
jhaski commentedThis 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.
Comment #22
Chipie commentedHi jhaski,
thanks for the prompt reply. When applying your patch I get these errors:
Regards,
--Chipie
Comment #23
dawehner@Chipie
Just by looking at the patch i would recommend you to use patch -p1 instaed of patch -p0
Comment #24
LinuxETC commentedNote: 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
Comment #25
LinuxETC commentedSeems 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.
Comment #26
mdesir commenteddoes 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
Comment #27
Chipie commentedI 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?
Comment #28
mdesir commentedright now I have .71a but really would prefer .8
Yes connection was successful
meeting node? meaning conference for content type.
Comment #29
mdesir commentedDoes anybody have bbb ported to drupal 7
Comment #30
LinuxETC commented@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.
Comment #31
mdesir commentedThanks for your reply, I'll work on it today
Comment #32
mdesir commentedThank you very much! It worked and I'm using .8 with it
Comment #33
markwk commentedOk. I got it working. Can I suggest we push the current patches and such into the current dev branch?
Comment #34
ParisLiakos commentedSeems #21 was commited.
This should be closed i guess, and new issues should be opened if necessary