Project

General

Profile

Bug #7342

Fix get_scorm_time(), get_progress_bar()

Added by Julio Montoya almost 5 years ago. Updated almost 5 years ago.

Status:
Bug resolved
Priority:
Normal
Assignee:
Category:
Learning paths / Lecciones
Target version:
Start date:
22/10/2014
Due date:
% Done:

100%

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

Description

This function is used in as an static function and as a method which is not correct:

learnpath:::get_scorm_time(...)
$new_learnpath_item->get_scorm_time(...)

To fix this all calls should be called as static or as part of an object.

The same thing happen with:

Strict standards: Non-static method learnpath::get_progress_bar() should not be called statically in  main/newscorm/lp_list.php on line 237

Associated revisions

Revision 03c75e8a (diff)
Added by Julio Montoya almost 5 years ago

Fix get_scorm_time() static not static calls see #7342

History

#1

Updated by Julio Montoya almost 5 years ago

  • Subject changed from Fix get_scorm_time() to Fix get_scorm_time(), get_progress_bar()
  • Description updated (diff)
#2

Updated by Julio Montoya almost 5 years ago

  • Status changed from Assigned to Needs testing
  • Assignee deleted (Julio Montoya)
#3

Updated by Yannick Warnier almost 5 years ago

  • Category set to Learning paths / Lecciones

From my point of view there's not much to test in the immediate future. Should we close it (for now) and assume you did a great job? :-)

#4

Updated by Yannick Warnier almost 5 years ago

  • Status changed from Needs testing to Bug resolved
  • Assignee set to Julio Montoya
  • % Done changed from 0 to 100

I'm taking the liberty to assume it was great, and to close it.

Also available in: Atom PDF