Project

General

Profile

Bug #3658

Impossible to list the item to add a prerequisite

Added by Andre Boivin about 8 years ago. Updated almost 8 years ago.

Status:
Bug resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
24/06/2011
Due date:
% Done:

100%

Estimated time:
Complexity:
Normal
SCRUM pts - complexity:
?

Description

To view a list of item to add a prerequisite, I have to comment line 152 of file lp_edit_item_prereq if settings_current, service_visio active is true

Associated revisions

Revision a28e48a7 (diff)
Added by Julio Montoya almost 8 years ago

Fixing fatal error due the audio record include code see #3658

Revision 124816b8 (diff)
Added by Julio Montoya almost 8 years ago

Commenting audiorecorder see #3658

History

#1

Updated by Julio Montoya almost 8 years ago

  • Status changed from New to Assigned
  • Assignee set to Julio Montoya
  • % Done changed from 0 to 10

After activating the videoconference (just enabling not necessary to install it) I could reproduce the bug ... There is a fatal error:

Fatal error: Cannot redeclare getFLVDuration() (previously declared in /var/www/chamilo/main/newscorm/audiorecorder.inc.php:21) in /var/www/chamilo/main/newscorm/audiorecorder.inc.php on line 30

#2

Updated by Julio Montoya almost 8 years ago

  • Status changed from Assigned to Needs more info
  • Assignee deleted (Julio Montoya)
  • % Done changed from 10 to 90

the problem was this:

    if (api_get_setting('service_visio', 'active') == 'true') {
            include 'audiorecorder.inc.php';
        }

fix it with
http://code.google.com/p/chamilo/source/detail?r=aa48568abda81b7656580aa4aa54b2ffa9da1a2c&repo=classic

#3

Updated by Yannick Warnier almost 8 years ago

  • Status changed from Needs more info to Assigned
  • Assignee set to Julio Montoya
  • % Done changed from 90 to 30

Actually you can remove the whole condition. The audiorecorder doesn't even work on a funcional d0keos videoconference basis.

#4

Updated by Julio Montoya almost 8 years ago

  • Status changed from Assigned to Needs more info
  • Assignee changed from Julio Montoya to Yannick Warnier

Yannick Warnier wrote:

Actually you can remove the whole condition. The audiorecorder doesn't even work on a funcional d0keos videoconference basis.

I'm going to remove also the videoconference link in the admin settings? In fact I'm going to remove all the code that I found with that condition?
Including some inserts in the tool table?

    if (api_get_setting('service_visio', 'active') == 'true') {
        $mycheck = api_get_setting('service_visio', 'visio_host');
        if (!empty($mycheck)) {
            Database::query("INSERT INTO `" . $tbl_course_homepage . "` VALUES (NULL, '" . TOOL_VISIO_CONFERENCE . "','conference/index.php?type=conference','visio_meeting.gif','1','0','squaregrey.gif','NO','_self','interaction','0')");
            Database::query("INSERT INTO `" . $tbl_course_homepage . "` VALUES (NULL, '" . TOOL_VISIO_CLASSROOM . "','conference/index.php?type=classroom','visio.gif','1','0','squaregrey.gif','NO','_self','authoring','0')");
        }
    }

Condition

if (api_get_setting('service_visio', 'active') == 'true')
#5

Updated by Yannick Warnier almost 8 years ago

  • Status changed from Needs more info to Assigned
  • Assignee changed from Yannick Warnier to Julio Montoya

No, sorry. The (old) videoconference might still be used by some users and BigBlueButton still has a few flaws that do not let us say directly that the Dokā‚¬os videoconference is deprecated. However, once BBB 0.8 is out, we should be able to morally remove all links to the previous videoconference system, yes.

For now, only remove the audiorecorder, and when you're done you can close the task.

#6

Updated by Julio Montoya almost 8 years ago

  • Status changed from Assigned to Bug resolved
  • Assignee deleted (Julio Montoya)
  • % Done changed from 30 to 100

Also available in: Atom PDF