Project

General

Profile

Bug #4131

AICC Incorrect format of GETPARAM Response

Added by Martin Holden almost 8 years ago. Updated over 3 years ago.

Status:
Needs testing
Priority:
Normal
Assignee:
-
Category:
Learning paths / Lecciones
Target version:
Start date:
12/12/2011
Due date:
% Done:

50%

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

Description

The AICC getparam response from Chamilo is URLENCODED, this should be plain text.

The standards dictate the format of the response is plain text, it also dictates that the forst part of the data model should follow immediately after the aicc_data= there should not be a new line.

WORKAROUND: Edit the main\newscorm\aicc_hacp.php

Line 101 - Remove $crlf after aicc=data=

//$result = $s_ec.$error_code.$crlf.$s_et.$error_text.$crlf.$s_ad.$crlf;
$result = $s_ec.$error_code.$crlf.$s_et.$error_text.$crlf.$s_ad;

Line 126 - Comment out the line that urlencodes the response


//$result = str_replace($convert_dec, $convert_enc, $result);

NOTE: It is probably a good idea also removing the various $tab whitespace prefixes to other elements lines 103 etc

Associated revisions

Revision 6aab5c36 (diff)
Added by Julio Montoya about 7 years ago

Fixing AICC string manage see #4131 fixed/reported by M. Holden

History

#1

Updated by Yannick Warnier over 7 years ago

  • Target version set to 1.9.2
#2

Updated by Yannick Warnier about 7 years ago

  • Target version changed from 1.9.2 to 1.9.4
#3

Updated by Julio Montoya about 7 years ago

  • Status changed from New to Needs testing
  • % Done changed from 0 to 50

Hello Martin,
Thanks for reporting I sent your 2 fixes + I changed this

//$tab = "\t";
$tab = "";

http://code.google.com/p/chamilo/source/detail?r=c9eda40e3f22bc2a638075e7f85cb15a9c886177&name=1.9.x&repo=classic

Regards,

#4

Updated by Yannick Warnier almost 7 years ago

  • Target version changed from 1.9.4 to 1.9.6
#5

Updated by Yannick Warnier over 6 years ago

  • Category set to Learning paths / Lecciones
  • Priority changed from Urgent to Normal
  • Target version changed from 1.9.6 to 2.0
#6

Updated by Yannick Warnier over 3 years ago

  • Target version changed from 2.0 to 3.0

Also available in: Atom PDF