Project

General

Profile

Bug #7504

Ambiguous class resolution Ims* classes

Added by Julio Montoya over 4 years ago. Updated over 4 years ago.

Status:
Bug resolved
Priority:
Normal
Category:
Exercises
Target version:
Start date:
02/02/2015
Due date:
% Done:

0%

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

Description

There are many classes declared with the same name:

qti_classes.php - qti2_classes.php

Warning: Ambiguous class resolution, "ImsAnswerMultipleChoice" was found in both "main/exercice/export/qti2/qti2_classes.php" and "main/exercice/export/qti/qti_classes.php", the first will be used.
Warning: Ambiguous class resolution, "ImsAnswerFillInBlanks" was found in both "main/exercice/export/qti2/qti2_classes.php" and "main/exercice/export/qti/qti_classes.php", the first will be used.
Warning: Ambiguous class resolution, "ImsAnswerMatching" was found in both "main/exercice/export/qti2/qti2_classes.php" and "main/exercice/export/qti/qti_classes.php", the first will be used.
Warning: Ambiguous class resolution, "ImsSection" was found in both "main/exercice/export/qti2/qti2_export.php" and "main/exercice/export/qti/qti_export.php", the first will be used.
Warning: Ambiguous class resolution, "ImsItem" was found in both "main/exercice/export/qti2/qti2_export.php" and "main/exercice/export/qti/qti_export.php", the first will be used.


Files

qti_item_v2p0all.zip (3.9 MB) qti_item_v2p0all.zip Yannick Warnier, 05/02/2015 04:12

Related issues

Related to Chamilo LMS - Feature #7518: QTI2 import broken. Fix or develop.Assigned09/02/2015

Actions

Associated revisions

Revision 56f7b033 (diff)
Added by Angel Fernando Quiroz Campos over 4 years ago

Remove duplicated Qti classes and methods - refs #7504

History

#1

Updated by Angel Quiroz over 4 years ago

  • Status changed from New to Needs testing

The export_question methos was found in both main/exercice/export/qti2/qti2_export.php and main/exercice/export/qti/qti_export.php also. The first is used. The qti_export.php file is not used. I can't find any reference to the file and its methods

PR: https://github.com/chamilo/chamilo-lms/pull/467

#2

Updated by Yannick Warnier over 4 years ago

QTI and QTI2 are two different standard versions.
I don't remember the details, but there was something about QTI2 not being able to deal with whole exercises (just individual questions). As such, if QTI is not used anymore, no problem to remove it completely, but this has to be checked first.

#3

Updated by Angel Quiroz over 4 years ago

I review the use of classes. And the Qti classes are not used, except ImsAnswerTrueFalse class. I also checked the autoload (composer) and only referred to the Qti2 classes and ImsAnswerTrueFalse class

PR (updated) https://github.com/chamilo/chamilo-lms/pull/467

#4

Updated by Yannick Warnier over 4 years ago

Attaching example QTIv2 files for tests. Not sure what they are. It's been years since I've looked at the QTI format.

#5

Updated by Yannick Warnier over 4 years ago

OK, so apparently the QTI2 import doesn't work well enough, in 1.10.x or in any previous version, for what it's worth.

The export itself seems to work fine, though, so I'm accepting the PR here and creating a new task to support QTI2 import.

#6

Updated by Yannick Warnier over 4 years ago

  • Category set to Exercises
  • Status changed from Needs testing to Bug resolved
  • Assignee set to Yannick Warnier

I believe this is not an issue anymore...

I used a series of things in the quiz tool and could not generate any warning, so I guess I can close this one.

The QTI import is something that needs to be fixed in another issue.

Also available in: Atom PDF