Todo List

Global activate_module_in_all_courses ($toolLabel)

remove the need of the Backlog and use Exceptions instead

Global activate_module_in_platform ($moduleId)

remove the need of the Backlog and use Exceptions instead

Global announcement_add_item ($title='', $content='', $visibility='SHOW', $time=NULL, $course_id=NULL)

convert to param date timestamp

Global announcement_update_item ($announcement_id, $title=NULL, $content=NULL, $visibility=NULL, $time=NULL, $course_id=NULL)

convert to param date timestamp

Global checkVersion ($currentVersion, $requiredVersion)

check if param have a good format

Global claro_embed ($output, $inPopup=false, $hide_banner=false, $hide_footer=false, $hide_body=false, $no_body=false)

TODO return string instead of echoing it

Global claro_get_conf_def_file ($configCode)

$centralizedDef won't be hardcoded.

Global claro_get_current_course_data ($dataName=null)

http://www.claroline.net/forum/viewtopic.php?t=4579

Global claro_get_tool_name_list ()

with plugin, this lis would be read in a dynamic datasource

Global claro_html_msg_list ($msgArrBody, $return=true)

this must be a message object where code add messages with a priority, and the rendering is set by by priority

Global claro_is_tool_visible ($tid, $courseId)

to move in a lib

Global claro_sql_get_tbl ($tableList, $contextData=null)

rewrite to use new Claroline core/context.lib.php

Namespace CLMANAGE

use modifiy is use in a cmd request

Namespace CLTRACK

Global course_user_html_form ($data, $courseId, $userId, $hiddenParam=null)

$courseManagerChecked never used

Global create_select_box_language ($selected=NULL)

merge this with claro_disp_select_box

Global deactivate_module ($moduleId)

remove the need of the Backlog and use Exceptions instead

Global define_course_keys ($wantedCode, $prefix4all= '', $prefix4baseName= '', $prefix4path= '', $addUniquePrefix=false, $useCodeInDepedentKeys=true, $addUniqueSuffix=false)

actually if suffix is not unique the next append and not replace

add param listing keyg wich wouldbe identical

manage an error on brake for too many try

$keysCourseCode is always

Global delete ()

TODO Possibility to delete used profile ?

Global DISP_RESULT

TODO : separate checking and output.

TODO : protect "showall" when there is nothing in cache.

Global embed ($output, $hide_banner=false, $hide_footer=false, $hide_body=false)

TODO return string instead of echoing it

Global get_and_unzip_uploaded_package ()

split this function and use unzip_package()

remove the need of the Backlog and use Exceptions instead

Global get_conf ($param, $default=null)

http://www.claroline.net/forum/viewtopic.php?t=4579

Global get_course_title ($cid)

replace content of this function with claro_get_course_officialCode and claro_get_course_name

Global get_info_course ($cid)

use claro_get_course_data

Global get_init ($param)

http://www.claroline.net/forum/viewtopic.php?t=4579

Global get_sql_filtered_user_list ()

: the name would be review befor move to a lib

: eject usage in function of $_SESSION

Global get_user_course_list ($userId, $renew=false)

search and merge other instance of this functionality

Global get_user_course_list_desactivated ($userId, $renew=false)

search and merge other instance of this functionality

Global group_count_students_in_group ($group_id, $course_id=null)

TODO : rename this function or change it. count include non student users.

Global group_count_students_in_groups ($course_id=null)

TODO : rename this function or change it. count include non student users.

Namespace INSTALL

check if dbexist would be improve for check if table exists, not if db exist.

Global install_module ($modulePath, $skipCheckDir=false, $registerModuleInCourses=false)

remove the need of the Backlog and use Exceptions instead

Global install_module_database_in_course ($moduleLabel, $courseId)

what to return if the script file does not exists ?!?

Namespace KERNEL

phpdoc

Namespace kernel

why do we need that much identifiers for a module ?!?

use Exceptions instead of claro_failure

Namespace kernel::module

why do we need that much identifiers for a module ?!?

Namespace PhpSecInfo

PHP 5.2 reportedly introduces "allow_url_include", which may or may not affect this test. Should talk to core people about this.

I believe this hole was plugged in PHP 5.1.5 and 4.4.4(?). This test should be updated to take this into consideration (check the version and decide what to do)

Global register_module ($modulePath)

remove the need of the Backlog and use Exceptions instead

Global register_module_in_courses ($moduleId)

remove the need of the Backlog and use Exceptions instead

Global replace_dangerous_char ($string, $strict= 'loose')

function broken !

Global set_module_visibility ($moduleId, $visibility)

remove the need of the Backlog and use Exceptions instead

Global set_module_visibility_in_course ($tool_id, $courseCode, $visibility)

remove the need of the Backlog and use Exceptions instead

Global uninstall_module ($moduleId, $deleteModuleData=true)

remove the need of the Backlog and use Exceptions instead

Global unregister_module_from_courses ($moduleId)

remove the need of the Backlog and use Exceptions instead

Global unzip_package ($packageFileName)

use this function in get_and_unzip_uploaded_package()

remove the need of the Backlog and use Exceptions instead